build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20472 )
Change subject: nb/intel/sandybridge/peg: Add PEG driver ......................................................................
Patch Set 7:
(10 comments)
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... File src/northbridge/intel/sandybridge/pcie.c:
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 52: static void pcie_programm_da0(device_t dev, u8 val) 'programm' may be misspelled - perhaps 'program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 97: static void pcie_programm_90x(device_t dev, u8 bundle, u8 a, u8 b) 'programm' may be misspelled - perhaps 'program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 128: /* Programm test values */ 'Programm' may be misspelled - perhaps 'Program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 129: pcie_programm_90x(dev, bundle, a[i], i); 'programm' may be misspelled - perhaps 'program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 138: /* No errors occured after 100msec ? */ 'occured' may be misspelled - perhaps 'occurred'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 177: for (bundle = 0; bundle < end_bundle; bundle ++) { space prohibited before that '++' (ctx:WxB)
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 190: /* Programm 0xda0 */ 'Programm' may be misspelled - perhaps 'Program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 191: pcie_programm_da0(dev, reg_da0[i]); 'programm' may be misspelled - perhaps 'program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 226: /* Programm 0xda0 */ 'Programm' may be misspelled - perhaps 'Program'?
https://review.coreboot.org/#/c/20472/7/src/northbridge/intel/sandybridge/pc... PS7, Line 227: pcie_programm_da0(dev, reg_da0[best_da0]); 'programm' may be misspelled - perhaps 'program'?
-
build bot (Jenkins) (Code Review)