Dear "Hiremath, Vaibhav", In message <19f8576c6e063c45be387c64729e739404370d7...@dbde02.ent.ti.com> you wrote: > ... > > > +#ifndef __ASSEMBLY__ > > > +extern struct gpmc *gpmc_cfg; > > > +extern unsigned int boot_flash_base; > > > +extern volatile unsigned int boot_flash_env_addr; > > > +extern unsigned int boot_flash_off; > > > +extern unsigned int boot_flash_sec; > > > +extern unsigned int boot_flash_type; > > > +#endif > > > > Please add prototype declarations to some appropriate header file > > instead. > > > [Hiremath, Vaibhav] Actually this code is borrowed from OMAP3EVM file, and > I think almost all devices handles this same way.
Yes, this escaped us before - but now we got aware of this problem we don't want to add more of this (and clean up the existing code ASAP). > It is bit difficult to handle this, since we are supporting multiple flash > devices and these variables are initialized run-time and so the macros below, > > CONFIG_SYS_FLASH_BASE, > CONFIG_ENV_OFFSET, > CONFIG_SYS_ENV_SECT_SIZE Um... I cannot parse this senstence. Some words seem to be missing? > I will try to dig bit more on this, but I think as of now we can go with this. Sorry, I disagree. 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 The IQ of the group is the lowest IQ of a member of the group divided by the number of people in the group. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
