[coreboot-gerrit] Change in ...coreboot[master]: src/include/device/pci_ids.h: Added another Kabylake DT src/soc/intel...

[Alex Thiessen (Code Review)]

Alex Thiessen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32850 )

Change subject: src/include/device/pci_ids.h: Added another Kabylake DT src/soc/intelcommon/block/systemagent/systemagent.c: Added another KabyLake DT
......................................................................

Patch Set 1:

(3 comments)

https://review.coreboot.org/#/c/32850/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32850/1//COMMIT_MSG@7 
PS1, Line 7: Added
use imperative mood in the subject line

https://review.coreboot.org/#/c/32850/1//COMMIT_MSG@7 
PS1, Line 7: src/include/device/pci_ids.h: Added another Kabylake DT
           : src/soc/intelcommon/block/systemagent/systemagent.c: Added another KabyLake DT
a too long subject line

https://review.coreboot.org/#/c/32850/1/src/include/device/pci_ids.h 
File src/include/device/pci_ids.h:

https://review.coreboot.org/#/c/32850/1/src/include/device/pci_ids.h@3084 
PS1, Line 3084:  
guess this should have been a tab, not a space