[U-Boot] [PATCH 2/5] port wandboards to use the generic distro configs

Wolfgang Denk wd at denx.de
Fri Dec 6 16:26:51 CET 2013


Dear Dennis Gilmore,

In message <20131206084854.0e0da0cd at adria.ausil.us> you wrote:
>
> Wolfgang Denk <wd at denx.de> escribi=F3:
> > Dear Dennis Gilmore,
> >=20
> > In message <1386296295-28658-3-git-send-email-dennis at ausil.us> you
> > wrote:
> > >
> > > -	"fdt_addr=0x11000000\0" \
> > > +	"fdt_addr=0x11100000\0" \
> > > +	"fdt_addr_r=0x11200000\0" \
> > 
> > Why would you need fdt_addr and fdt_addr_r ?  This makes no sense.
> > Two different values would only be needed if there was NOR flash
> > available where we could read the FDT from without loading it to RAM
> > first.  AFAIK ther ewill never be variants of the Wandboard with NOR
> > flash, so you will always have to load the FDT to RAM first - thus the
> > destinction makes no sense.
>
> please go and read the cmd_pxe.c file it specifies the use of the two
> variables. one is for a system provided dtb and the other is for a user
> provided dtb

But this is crap. The meaning of these variables has been wel-defined
for a long, long time.  "fdt_addr" is the FDT address in NOR flash (or
similar memory except system RAM); "fdt_addr_r" is the FDT address
when loaded to system RAM (hence the "_r" in the variable name).

Arbitariry redefining this meaning is counterproductive and confusing.

If cmd_pxe.c uses incorrect names, then please fix the bug there.

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
How does a project get to be a year late?      ... One day at a time.


More information about the U-Boot mailing list