build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26800 )
Change subject: util/sconfig: Re-factor device structure in parse tree ......................................................................
Patch Set 9:
(12 comments)
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c File util/sconfig/main.c:
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@45 PS9, Line 45: * +------------------------+ +----------------------+ line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@46 PS9, Line 46: * | | | Mainboard | line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@47 PS9, Line 47: * +---------+ Root device (root_dev) +--------------->+ instance + line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@48 PS9, Line 48: * | | | chip_instance | (mainboard_instance)| line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@49 PS9, Line 49: * | +------------------------+ | | line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@50 PS9, Line 50: * | | +----------------------+ line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@68 PS9, Line 68: * +-------+----------+ line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@69 PS9, Line 69: * | | line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@70 PS9, Line 70: * | Mainboard chip +----------->X (new chips will be line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@71 PS9, Line 71: * | (mainboard_chip) | added here) line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@72 PS9, Line 72: * | | line over 80 characters
https://review.coreboot.org/#/c/26800/9/util/sconfig/main.c@73 PS9, Line 73: * +------------------+ line over 80 characters