build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47772 )
Change subject: nb/intel/sandybridge: Use bitfields for I/O data timings ......................................................................
Patch Set 10:
(8 comments)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... File src/northbridge/intel/sandybridge/raminit_common.h:
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 104: u32 : 2; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 106: u32 : 1; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 108: u32 : 1; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 110: u32 : 5; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 118: u32 : 2; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 120: u32 : 1; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 122: u32 : 1; space prohibited before that ':' (ctx:WxW)
https://review.coreboot.org/c/coreboot/+/47772/10/src/northbridge/intel/sand... PS10, Line 124: u32 : 12; space prohibited before that ':' (ctx:WxW)