Attention is currently required from: Ravi kumar, Martin Roth. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52371 )
Change subject: sc7280: Add GSI FW download support ......................................................................
Patch Set 6:
(11 comments)
File src/soc/qualcomm/sc7280/include/soc/qupv3_config.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/20103772_c1d1651d PS6, Line 77: { open brace '{' following struct go on the same line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/ce1918c5_d43fe879 PS6, Line 79: uint32_t iram_dword1; /* word 1 of a single IRAM firmware entry */ please, no space before tabs
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/4fa94d80_62438b33 PS6, Line 83: { open brace '{' following struct go on the same line
File src/soc/qualcomm/sc7280/qupv3_config.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/1eb45b9a_51b52a85 PS6, Line 204: write32(®s->gsi_periph_base_lsb,0); space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/0d58d7a2_932834d0 PS6, Line 205: write32(®s->gsi_periph_base_msb,0); space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/3064244c_0677d92b PS6, Line 210: if (fwIep[i].offset < GSI_REG_BASE_SIZE) { braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/4a6e86eb_fa2849c7 PS6, Line 221: memcpy((®s->gsi_inst_ramn),(void *)fwIRam, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/94e0a1b8_baf8fc25 PS6, Line 223: setbits_le32(®s->gsi_mcs_cfg, GSI_MCS_CFG_MCS_ENABLE_BMSK ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/2cba9d84_3211f5af PS6, Line 224: setbits_le32(®s->gsi_cfg, GSI_CFG_DOUBLE_MCS_CLK_FREQ_BMSK | GSI_CFG_GSI_ENABLE_BMSK); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/c046af81_aff75cdd PS6, Line 226: write32(®s->gsi_ee_n_scratch_0_addr,0x0); space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118679): https://review.coreboot.org/c/coreboot/+/52371/comment/475cad8e_1262edaf PS6, Line 227: write32(®s->ee_n_gsi_ee_generic_cmd,0x81); space required after that ',' (ctx:VxV)
-
build bot (Jenkins) (Code Review)