build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32346 )
Change subject: [WIP]mb/fujitsu: add Esprimo E900 ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/#/c/32346/3/src/mainboard/fujitsu/esprimo_e900/d... File src/mainboard/fujitsu/esprimo_e900/devicetree.cb:
https://review.coreboot.org/#/c/32346/3/src/mainboard/fujitsu/esprimo_e900/d... PS3, Line 17: subsystemid 0x1734 0x11ba inherit trailing whitespace
https://review.coreboot.org/#/c/32346/3/src/mainboard/fujitsu/esprimo_e900/d... PS3, Line 18: device pci 00.0 on end # Host bridge trailing whitespace
https://review.coreboot.org/#/c/32346/3/src/mainboard/google/octopus/variant... File src/mainboard/google/octopus/variants/baseboard/gpio.c:
https://review.coreboot.org/#/c/32346/3/src/mainboard/google/octopus/variant... PS3, Line 252: PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 0, DEEP, NONE, Tx0RxDCRx0, DISPUPD), /* EN_PP3300_WLAN_L */ line over 80 characters