Stefan Tauner has posted comments on this change. ( https://review.coreboot.org/21702 )
Change subject: nicintel_eeprom: Move variable declarations out of loops ......................................................................
Patch Set 1: -Code-Review
I actually looked in the codebase for other variable declarations in for heads and didn't see any. Perhaps there are some that you know of? In any case, if we don't want to support non-c99 then we should make that explicit in the Makefile.
I suspect that this bug/error was only triggered by a recent patch, which worked fine with Jenkins and a couple other versions of GCC. This patch is just a quick fix, which will work no matter if we force C99 compatibility or not.
Yes, you are right. My memory betrayed me and I did not check, sorry. The C90 declarations annoyed me from the beginning and I thought they were already removed but that's wrong (probably because - as I explained on IRC - forcing C99 on compilers is more problematic than one would think). So, yes, we should fix the code, but as a fixup! of the respective patch which is 9fe1fb7 (aka nicintel_eeprom: Support for I210 emulated EEprom). This also shows why a buildbot is a precious thing ;)