Attention is currently required from: Doug Anderson. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52959 )
Change subject: drivers/sn65dsi86: Switch EDID reading to use "indirect mode" ......................................................................
Patch Set 1:
(2 comments)
File src/drivers/ti/sn65dsi86bridge/sn65dsi86bridge.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118554): https://review.coreboot.org/c/coreboot/+/52959/comment/3cbbdcc8_2b67bee3 PS1, Line 200: rv |= i2c_writeb(bus, chip, SN_AUX_LENGTH_REG, length); /* size of 1 Byte data */ line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118554): https://review.coreboot.org/c/coreboot/+/52959/comment/53ddb187_80274463 PS1, Line 201: rv |= i2c_writeb(bus, chip, SN_AUX_CMD_REG, AUX_CMD_SEND | (command[request] << 4)); line over 96 characters