Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45367 )
Change subject: nb/intel/ironlake: Avoid casting pointers to structs ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45367/2/src/northbridge/intel/ironl... File src/northbridge/intel/ironlake/raminit.c:
https://review.coreboot.org/c/coreboot/+/45367/2/src/northbridge/intel/ironl... PS2, Line 1750: message->dwords + (current_position >> 2),
just a thought, the compiler might be able to "see" it more clearly if array indexing is used (messa […]
It does see something more clearly, and the binary changes. I'd like to have this tested on real hardware just to make sure it works.