On Jan 19, 2011, at 2:53 AM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <1295391047-10952-1-git-send-email-ga...@kernel.crashing.org> you > wrote: >> There are several users of the hwconfig APIs (8xxx DDR) before we have >> the environment properly setup. This causes issues because of the >> numerous ways the environment might be accessed because of the >> non-volatile memory it might be stored in. Additionally the access >> might be so early that memory isn't even properly setup for us. >> >> Towards resolving these issues we provide versions of all the hwconfig >> APIs that can be passed in a buffer to parse and leave it to the caller >> to determine how to allocate and populate the buffer. >> >> We use the _f naming convention for these new APIs even though they are >> perfectly useable after relocation and the environment being ready. >> >> We also now warn if the non-f APIs are called before the environment is >> ready to allow users to address the issues. >> >> Finally, we convert the 8xxx DDR code to utilize the new APIs to >> hopefully address the issue once and for all. We have the 8xxx DDR code >> create a buffer on the stack and populate it via getenv_f(). >> >> Signed-off-by: Kumar Gala <ga...@kernel.crashing.org> >> --- >> * Reworked __hwconfig to not require an additional local var per WD's >> comments >> >> arch/powerpc/cpu/mpc8xxx/ddr/options.c | 78 ++++++++++++++++++++++------- >> common/hwconfig.c | 86 ++++++++++++++++--------------- >> include/hwconfig.h | 68 +++++++++++++++++++------ >> 3 files changed, 154 insertions(+), 78 deletions(-) > > Acked-by: Wolfgang Denk <w...@denx.de> > > Please pull through 8xxx repo. > > Best regards, > > Wolfgang Denk
applied to 85xx - k _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot