Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39859 )
Change subject: nb/intel/i945: Make some cosmetic changes ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/39859/3/src/northbridge/intel/i945/... File src/northbridge/intel/i945/raminit.c:
https://review.coreboot.org/c/coreboot/+/39859/3/src/northbridge/intel/i945/... PS3, Line 2544: printk(BIOS_DEBUG, "bankaddr from bank size of rank %d\n", nonzero); Isn’t this exactly 96 characters?
https://review.coreboot.org/c/coreboot/+/39859/3/src/northbridge/intel/i945/... PS3, Line 2557: */ While you doing cosmetic changes, fix this too?
/* … … */