build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63289 )
Change subject: soc/qualcomm/common: Add strict_check flags to clock_configure() ......................................................................
Patch Set 1:
(4 comments)
File src/soc/qualcomm/sc7180/clock.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145192): https://review.coreboot.org/c/coreboot/+/63289/comment/320a196c_c80573c8 PS1, Line 218: &mdss_clk_cfg, 0, 0, false); code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145192): https://review.coreboot.org/c/coreboot/+/63289/comment/218cf625_7e7f7512 PS1, Line 218: &mdss_clk_cfg, 0, 0, false); please, no spaces at the start of a line
File src/soc/qualcomm/sc7280/clock.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145192): https://review.coreboot.org/c/coreboot/+/63289/comment/50f54c07_3c7ff175 PS1, Line 270: true); code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145192): https://review.coreboot.org/c/coreboot/+/63289/comment/27b42a87_bb3bdf19 PS1, Line 270: true); please, no spaces at the start of a line