Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/27168 )
Change subject: inteltool: Add PCI IDs for the C220 PCH series ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/27168/1/util/inteltool/inteltool.h File util/inteltool/inteltool.h:
https://review.coreboot.org/#/c/27168/1/util/inteltool/inteltool.h@200 PS1, Line 200: pelase use two tabs instead of space as seperator
https://review.coreboot.org/#/c/27168/1/util/inteltool/inteltool.c File util/inteltool/inteltool.c:
https://review.coreboot.org/#/c/27168/1/util/inteltool/inteltool.c@236 PS1, Line 236: "Intel(R) C8 Mobile"}, The other southbridge names doesn't have an "Intel(R)" prefix, please remove.