Hi Andy, On Fri, Nov 24, 2017 at 11:56 PM, Andy Shevchenko <[email protected]> wrote: > On Fri, 2017-11-24 at 17:42 +0200, Andy Shevchenko wrote: >> On Fri, 2017-11-24 at 16:06 +0800, Bin Meng wrote: >> > Hi, >> > >> > Intel Edison has a build warning below. >> > >> > + *env_addr = offset; >> > + ^ >> > w+../env/mmc.c: In function 'mmc_get_env_addr': >> > w+../env/mmc.c:121:12: warning: 'val' may be used uninitialized in >> > this function [-Wmaybe-uninitialized] >> > >> > I did not figure out what is wrong here. v2017.11 does not have such >> > build warning. >> > >> > Do you have any idea? > > Somewhat compiler goes crazy? > > I have a theory that instead of showing actual potential issues (which > are bogus anyway) it complains on __weak function instead. > > The real complains might be env_mmc_load() / env_mmc_save() where > offset* is uninitialized indeed. >
Thanks for the testing. I was using GCC 5.4.0 for the build testing which is the default one shipped in Ubuntu 16.04 LTS. I will try to find another version to test. Regards, Bin _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
