build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59558 )
Change subject: mb/google/zork/var/shuboz: Add fw_config probe for ALC5682-VD & VS ......................................................................
Patch Set 1:
(3 comments)
File src/mainboard/google/zork/variants/shuboz/variant.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134075): https://review.coreboot.org/c/coreboot/+/59558/comment/1df832a9_5cb0783f PS1, Line 83: find_dev_nested_path(pci_root_bus(), machine_dev_path, ARRAY_SIZE(machine_dev_path)); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134075): https://review.coreboot.org/c/coreboot/+/59558/comment/50b908ee_a8610a5a PS1, Line 86: if (!machine_dev || (machine_dev->chip_ops != &drivers_amd_i2s_machine_dev_ops) || !machine_dev->chip_info) { line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134075): https://review.coreboot.org/c/coreboot/+/59558/comment/b8d71d5c_9674fd4a PS1, Line 96: printk(BIOS_INFO, "Configure i2s device, machine_dev_config->hid %s\n", machine_dev_config->hid); line over 96 characters