build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/28669 )
Change subject: mb/google/octopus: Close unused I2C bus 7 include SCL and SDA for Phaser ......................................................................
Patch Set 18:
(4 comments)
https://review.coreboot.org/#/c/28669/18/src/mainboard/google/octopus/varian... File src/mainboard/google/octopus/variants/phaser/variant.c:
https://review.coreboot.org/#/c/28669/18/src/mainboard/google/octopus/varian... PS18, Line 47: /* According to the sku id decide whether update touch screen device information: line over 80 characters
https://review.coreboot.org/#/c/28669/18/src/mainboard/google/octopus/varian... PS18, Line 49: * 2. sku id is other value then dev->enabled = 1. trailing whitespace
https://review.coreboot.org/#/c/28669/18/src/mainboard/google/octopus/varian... PS18, Line 64: /* Find all children on bus to update touchscreen enable gpio. */ line over 80 characters
https://review.coreboot.org/#/c/28669/18/src/mainboard/google/octopus/varian... PS18, Line 65: while ((child = dev_bus_each_child(children_bus, child)) != NULL) { line over 80 characters