[U-Boot-Users] [PATCH] adds loadi sub command to fpag

[eran liberty]

On 6/22/07, Wolfgang Denk <wd at denx.de> wrote:
> In message <ffc2b1d40706202325y222f1ab5ge4cae49d32162c2c at mail.gmail.com> you wrote:
> > fpga loadi <dev> <image addr>
>
> I think you should provide a bit more of a description oif what you
> are doing and why. This is a bit too cryptic.
>

this function "knows" to open an Image of an fpga design that was
bzip2/gzip/raw compressed and packages with mkimage and burn it.

> Also, I think the function you implement might be redundant.  Why  do
> you think it's necessary to add all this code?

up on closer look there is an image opener there but i need to relook
at it to tell if it can decompress bzip2 / gzip images. Anyway when i
implemented it I was unaware of the other implementation (it was not
there yet)

so we can either merge this code to the one already there adding it
whatever it lack (if it lacks)... or keep this one and trash the
other... whichever you find more eye pleasing.
>
> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> "If it ain't broke, don't fix it."                       - Bert Lantz
>