Dear Dennis Gilmore, In message <20131206084854.0e0da...@adria.ausil.us> you wrote: > > Wolfgang Denk <w...@denx.de> escribi=F3: > > Dear Dennis Gilmore, > >=20 > > In message <1386296295-28658-3-git-send-email-den...@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: w...@denx.de How does a project get to be a year late? ... One day at a time. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
