build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25382 )
Change subject: soc/cavium: Apply additional devicetree fixups ......................................................................
Patch Set 35:
(4 comments)
https://review.coreboot.org/#/c/25382/35/src/soc/cavium/cn81xx/soc.c File src/soc/cavium/cn81xx/soc.c:
https://review.coreboot.org/#/c/25382/35/src/soc/cavium/cn81xx/soc.c@83 PS35, Line 83: Remove 'phy-handle' if this option is not availble */ 'availble' may be misspelled - perhaps 'available'?
https://review.coreboot.org/#/c/25382/35/src/soc/cavium/cn81xx/soc.c@141 PS35, Line 141: /* Extract our MAC address info so we can assign them in the device tree */ line over 80 characters
https://review.coreboot.org/#/c/25382/35/src/soc/cavium/cn81xx/soc.c@173 PS35, Line 173: if (!strcmp(name, prop->prop.name)) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25382/35/src/soc/cavium/cn81xx/soc.c@269 PS35, Line 269: BDK_BGXX_CMRX_RX_DMAC_CTL(bgx_interface, bgx_index)); line over 80 characters