coreboot-gerrit November 2021

coreboot-gerrit@coreboot.org

1 participants
6394 discussions

Change in coreboot[master]: soc/mediatek: Add PCIe support
by Jianjun Wang (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Shelley Chen, Hung-Te Lin, Yu-Ping Wu. Jianjun Wang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56791 ) Change subject: soc/mediatek: Add PCIe support ...................................................................... Patch Set 3: (1 comment) File src/include/device/pci_mmio_cfg.h: https://review.coreboot.org/c/coreboot/+/56791/comment/ca400ca7_860e3aec PS3, Line 50: pcicfg > Shelly, according to your design in CB:57861, I assume you will add this and rename it to pci_map_bu […] If it's necessary to rename it to pci_map_bus(), I think I can send a patch to do it. -- To view, visit https://review.coreboot.org/c/coreboot/+/56791 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib9b6adaafa20aeee136372ec9564273f86776da0 Gerrit-Change-Number: 56791 Gerrit-PatchSet: 3 Gerrit-Owner: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Furquan Shaikh <furquan.m.shaikh(a)gmail.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Attention: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 08:13:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Yu-Ping Wu <yupingso(a)google.com> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: soc/mediatek/mt8195: Add driver to configure PCIe
by Yu-Ping Wu (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Hung-Te Lin. Yu-Ping Wu has removed a vote from this change. ( https://review.coreboot.org/c/coreboot/+/56792 ) Change subject: soc/mediatek/mt8195: Add driver to configure PCIe ...................................................................... Removed Code-Review+2 by Yu-Ping Wu <yupingso(a)google.com> -- To view, visit https://review.coreboot.org/c/coreboot/+/56792 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If6799c53b03a33be91157ea088d829beb4272976 Gerrit-Change-Number: 56792 Gerrit-PatchSet: 3 Gerrit-Owner: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org> Gerrit-MessageType: deleteVote

1 0

Change in coreboot[master]: soc/mediatek: Add PCIe support
by Yu-Ping Wu (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Shelley Chen, Hung-Te Lin, Jianjun Wang. Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56791 ) Change subject: soc/mediatek: Add PCIe support ...................................................................... Patch Set 3: (2 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/56791/comment/f0bf6f13_c44c6af4 PS3, Line 10: support supporting File src/include/device/pci_mmio_cfg.h: https://review.coreboot.org/c/coreboot/+/56791/comment/14b297b2_0bd3a3d5 PS3, Line 50: pcicfg Shelly, according to your design in CB:57861, I assume you will add this and rename it to pci_map_bus(). Do you want MTK to wait for your patch? -- To view, visit https://review.coreboot.org/c/coreboot/+/56791 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib9b6adaafa20aeee136372ec9564273f86776da0 Gerrit-Change-Number: 56791 Gerrit-PatchSet: 3 Gerrit-Owner: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Furquan Shaikh <furquan.m.shaikh(a)gmail.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Attention: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 08:09:58 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: device/pci_device.c: Scan only one device for PCIe
by Yu-Ping Wu (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Shelley Chen, Furquan Shaikh, Paul Menzel, Jianjun Wang. Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56788 ) Change subject: device/pci_device.c: Scan only one device for PCIe ...................................................................... Patch Set 3: (3 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/56788/comment/d88b37bc_315d2945 PS3, Line 11: A PCIe downstream port normally leads to a link with : only device 0 on it. As an optimization, scan only for : device 0 in that case. Please reflow the text to match line length limit (72 chars). File src/device/pci_device.c: https://review.coreboot.org/c/coreboot/+/56788/comment/70d20428_e2123ad6 PS3, Line 1207: A PCIe Downstream Port normally leads to a Link with only Device : * 0 on it (PCIe spec r5.0, sec 7.3.1). As an optimization, scan : * only for Device 0 in that situation. Please reflow the text. https://review.coreboot.org/c/coreboot/+/56788/comment/02a04caf_3209735e PS3, Line 1260: 0x07 MIN(max_devfn, 0x07), as the passed max_devfn might be smaller than 0x07. -- To view, visit https://review.coreboot.org/c/coreboot/+/56788 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id184d03b33e1742b18efb3f11aa9b2f81fa03806 Gerrit-Change-Number: 56788 Gerrit-PatchSet: 3 Gerrit-Owner: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Reviewer: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Furquan Shaikh <furquan.m.shaikh(a)gmail.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Furquan Shaikh <furquan.m.shaikh(a)gmail.com> Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Jianjun Wang <jianjun.wang(a)mediatek.corp-partner.google.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:44:09 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: soc/medaitek/mt8186: fix wrong condition of RTC drivers
by Rex-BC Chen (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Hung-Te Lin. Rex-BC Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59653 ) Change subject: soc/medaitek/mt8186: fix wrong condition of RTC drivers ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: @hung-te, can we merge 8186 patches? Thanks! -- To view, visit https://review.coreboot.org/c/coreboot/+/59653 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I5fdf4de0383ef373dd45e8d8741aa861c9c4bdc6 Gerrit-Change-Number: 59653 Gerrit-PatchSet: 2 Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com> Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:43:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: HACK: Herobrine: Reinit TPM INT gpio after qclib executes
by build bot (Jenkins) (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Ravi kumar, Shelley Chen, Sudheer Amrabadi, mturney mturney. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57025 ) Change subject: HACK: Herobrine: Reinit TPM INT gpio after qclib executes ...................................................................... Patch Set 36: Verified-1 (2 comments) File src/soc/qualcomm/common/qclib.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134465): https://review.coreboot.org/c/coreboot/+/57025/comment/beb033bc_6b6083ad PS36, Line 254: mainboard_blob_fix(); code indent should use tabs where possible Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134465): https://review.coreboot.org/c/coreboot/+/57025/comment/cdda3bbb_2acec959 PS36, Line 254: mainboard_blob_fix(); please, no spaces at the start of a line -- To view, visit https://review.coreboot.org/c/coreboot/+/57025 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I4c43d120ebb49800c82aec263ff8c215b60c5201 Gerrit-Change-Number: 57025 Gerrit-PatchSet: 36 Gerrit-Owner: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Sudheer Amrabadi <samrabad(a)codeaurora.org> Gerrit-Attention: mturney mturney <quic_mturney(a)quicinc.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:10:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: soc/qualcomm/common/usb: Add support for common USB driver
by Sandeep Maheswaram (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Ravi kumar, Paul Menzel, mturney mturney, Julius Werner. Sandeep Maheswaram has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56091 ) Change subject: soc/qualcomm/common/usb: Add support for common USB driver ...................................................................... Patch Set 41: (4 comments) File src/soc/qualcomm/common/include/soc/usb/qusb_phy.h: https://review.coreboot.org/c/coreboot/+/56091/comment/71b015c4_9a8b5ac2 PS38, Line 95: > same here Done File src/soc/qualcomm/common/usb/snps_usb_phy.c: https://review.coreboot.org/c/coreboot/+/56091/comment/e3d75051_081cd510 PS38, Line 44: UTMI_PHY_CMN_CTRL_OVERRIDE_EN, > This can still fit on the previous line Done https://review.coreboot.org/c/coreboot/+/56091/comment/e7928fd5_d9892ab6 PS38, Line 48: POR, POR); > This too (and many more times below). Done https://review.coreboot.org/c/coreboot/+/56091/comment/690b0e70_93be0424 PS38, Line 73: PARAM_OVRD_MASK, override_data->parameter_override_x2); > What about parameter_override_x3? Done -- To view, visit https://review.coreboot.org/c/coreboot/+/56091 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1013ded22855286220cfa747cb25418070fe85a7 Gerrit-Change-Number: 56091 Gerrit-PatchSet: 41 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: mturney mturney <mturney(a)codeaurora.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Ravi Kumar Bokka <c_rbokka(a)qualcomm.corp-partner.google.com> Gerrit-CC: Sandeep Maheswaram <sanm(a)codeaurora.org> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: mturney mturney <mturney(a)codeaurora.org> Gerrit-Attention: Julius Werner <jwerner(a)chromium.org> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:10:06 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Julius Werner <jwerner(a)chromium.org> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: HACK sc7280: QSIP SPI NOR addressing mode for SKU1 and SKU2 HACK
by build bot (Jenkins) (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Shelley Chen, Ravi kumar, Sudheer Amrabadi, mturney mturney. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50586 ) Change subject: HACK sc7280: QSIP SPI NOR addressing mode for SKU1 and SKU2 HACK ...................................................................... Patch Set 98: Verified-1 (2 comments) File src/drivers/spi/spi_flash.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134463): https://review.coreboot.org/c/coreboot/+/50586/comment/bc303198_94f717af PS98, Line 533: while (1) suspect code indent for conditional statements (24, 24) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134463): https://review.coreboot.org/c/coreboot/+/50586/comment/0a735917_89eb8743 PS98, Line 554: while (1) suspect code indent for conditional statements (24, 24) -- To view, visit https://review.coreboot.org/c/coreboot/+/50586 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ied5b647d0fcc8e3effff3bb7c8680ed5a0c1f3d4 Gerrit-Change-Number: 50586 Gerrit-PatchSet: 98 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: Sajida Bhanu <quic_c_sbhanu(a)quicinc.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: Sudheer Amrabadi <samrabad(a)codeaurora.org> Gerrit-Attention: mturney mturney <quic_mturney(a)quicinc.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:10:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: sc7280: Add support for edp driver & mdp driver
by build bot (Jenkins) (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Ravi kumar. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59612 ) Change subject: sc7280: Add support for edp driver & mdp driver ...................................................................... Patch Set 3: Verified-1 (118 comments) File src/mainboard/google/herobrine/mainboard.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/6321e57a_3464477c PS3, Line 63: if (display_init_required()) { suspect code indent for conditional statements (8, 24) File src/soc/qualcomm/sc7280/display/edp_aux.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/f0ab0821_44100df0 PS3, Line 99: printk(BIOS_ERR,"%s: invalid size or buffer \n",__func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3029d194_6bc37886 PS3, Line 99: printk(BIOS_ERR,"%s: invalid size or buffer \n",__func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/f5fd23f9_be95ab56 PS3, Line 99: printk(BIOS_ERR,"%s: invalid size or buffer \n",__func__); unnecessary whitespace before a quoted newline Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/74dd3d2c_84466456 PS3, Line 106: printk(BIOS_ERR,"%s: invalid msg: size(%zu), request(%x)\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/641e6d17_ddbaeeae PS3, Line 113: printk(BIOS_ERR,"edp aux transfer tx failed\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/fa1b3fa6_2dcfedb0 PS3, Line 115: if (read) { suspect code indent for conditional statements (8, 24) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/db6b1146_7f8e7a6b PS3, Line 118: printk(BIOS_ERR,"edp aux transfer rx failed\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/5e1997d0_edccdb84 PS3, Line 138: if (err <EDID_LENGTH) { spaces required around that '<' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/42b732fc_c60e5386 PS3, Line 139: printk(BIOS_ERR, "ERROR: Failed to read EDID. :0x%x\n",err); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/9b1dc81b_28839c05 PS3, Line 148: if (err>0) spaces required around that '>' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/19ae266d_40b402a2 PS3, Line 149: err = edp_aux_transfer(EDID_I2C_ADDR, DP_AUX_I2C_READ, &edid[EDID_LENGTH], line over 96 characters Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d820e92b_b9b17aa7 PS3, Line 151: if (err<EDID_LENGTH) { spaces required around that '<' (ctx:VxV) File src/soc/qualcomm/sc7280/display/edp_ctrl.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/02354cb0_9379e7ba PS3, Line 405: static void edp_link_train_clock_recovery_delay(const uint8_t dpcd[DP_RECEIVER_CAP_SIZE]) { open brace '{' following function definitions go on the next line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/b26697d3_01fe9b22 PS3, Line 409: printk(BIOS_ERR,"AUX interval %d, out of range (max 4)\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3e87c496_3a125662 PS3, Line 426: if ((lane_status & DP_LANE_CR_DONE) == 0) that open brace { should be on the previous line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a4ce3b65_3c8e383c PS3, Line 428: printk(BIOS_ERR,"clock recovery ok failed : %x\n", lane_status); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0725755b_6b2a48e3 PS3, Line 435: static void edp_link_train_channel_eq_delay(const uint8_t dpcd[DP_RECEIVER_CAP_SIZE]) { open brace '{' following function definitions go on the next line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d6c70eb7_1a6864ae PS3, Line 440: printk(BIOS_INFO,"AUX interval %d, out of range (max 4)\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/31ea86e6_5176a469 PS3, Line 471: write32(&edp_ahbclk->interrupt_status,EDP_INTERRUPT_STATUS1_MASK); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/ef0f9bd2_4c258729 PS3, Line 490: if (dpcd[DP_EDP_CONFIGURATION_CAP] & DP_ALTERNATE_SCRAMBLER_RESET_CAP) suspect code indent for conditional statements (8, 10) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/fed1db08_8a422f8a PS3, Line 539: if (edp_aux_transfer(DP_TRAINING_LANE0_SET, DP_AUX_NATIVE_WRITE, buf, 2) <2) { spaces required around that '<' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4b019697_89ae03cb PS3, Line 540: printk(BIOS_ERR,"%s: Set sw/pe to panel failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/65dbfaa9_da2a9691 PS3, Line 551: printk(BIOS_INFO,"pattern=%x\n", p); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/bd41d518_8c3204fd PS3, Line 553: printk(BIOS_ERR,"%s: Set training pattern to panel failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/b8f2fba1_5b43f3b3 PS3, Line 570: printk(BIOS_INFO,"lane=%d req_voltage_swing=0x%x\n", i, data); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/fc0de6f1_097b167e PS3, Line 581: printk(BIOS_INFO,"lane=%d req_pre_emphasis=0x%x\n", i, data); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a9b5be74_218f6c30 PS3, Line 587: printk(BIOS_INFO,"v_level=%d, p_level=%d\n", ctrl->v_level, ctrl->p_level); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2333c05b_7fbc99fc PS3, Line 596: printk(BIOS_INFO,"train=%d", train); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/563ea136_a7ea2297 PS3, Line 606: printk(BIOS_INFO,"%s: set link_train=%u failed\n", __func__, data); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0dd57366_cd61af4f PS3, Line 611: printk(BIOS_INFO,"v=%d p=%d\n", ctrl->v_level, ctrl->p_level); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a36b4a92_a0c7e70d PS3, Line 643: printk(BIOS_ERR,"%s: read link status failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0a924e11_ef7b3388 PS3, Line 698: printk(BIOS_ERR,"%s: read link status failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a4135433_30869b99 PS3, Line 746: printk(BIOS_INFO,"new rate=0x%x\n", ctrl->link_rate_khz); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/89f5c925_901790c3 PS3, Line 809: if (edp_aux_transfer(DP_LINK_BW_SET, DP_AUX_NATIVE_WRITE, &values[0], 1)< 0) spaces required around that '<' (ctx:VxW) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a5957c0e_c3a9f265 PS3, Line 830: printk(BIOS_ERR,"%s: read link status failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/37075ff2_5d83677a PS3, Line 836: printk(BIOS_ERR,"link reconfig\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/fab615d0_205a0662 PS3, Line 839: if (edp_clock_recovery_reduced_lanes(link_status, ctrl->lane_cnt) == 0) { line over 96 characters Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d3a46917_b8594e9d PS3, Line 841: printk(BIOS_ERR,"%s: Training 1 failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0c80ec9b_ab48d240 PS3, Line 844: printk(BIOS_ERR,"link reconfig\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/dd746f8b_8c5ed441 PS3, Line 848: printk(BIOS_ERR,"%s: Training 1 failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d7734753_a206a15f PS3, Line 855: printk(BIOS_INFO,"Training 1 completed successfully\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/7f247553_51b4e2e6 PS3, Line 858: if (edp_clear_training_pattern(dpcd)) suspect code indent for conditional statements (8, 10) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2a7efbcc_9ec8f582 PS3, Line 859: return EDP_TRAIN_FAIL; Statements should start on a tabstop Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3b243cd5_7b0d02af PS3, Line 861: ret = edp_start_link_train_2(ctrl,dpcd); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/e9cefe80_75b39885 PS3, Line 866: printk(BIOS_ERR,"%s: read link status failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/527f0cb4_0c15a0ae PS3, Line 872: printk(BIOS_ERR,"link reconfig\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4da6d962_9c056b8b PS3, Line 875: printk(BIOS_ERR,"%s: Training 2 failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/1642c609_c1d33121 PS3, Line 880: printk(BIOS_ERR,"%s: Training 1 failed\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/aa4d6b8c_529e2aaf PS3, Line 883: printk(BIOS_ERR,"link reconfig\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4774862d_aa7722c6 PS3, Line 892: printk(BIOS_INFO,"Training 2 completed successfully\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/76e586d2_4a1b8f8a PS3, Line 971: printk(BIOS_ERR,"Invalid pixel mux divider\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3bce1551_a2c4154b PS3, Line 982: printk(BIOS_INFO,"num=0x%x, den=0x%x\n", *pixel_m, *pixel_n); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a276201a_a9136509 PS3, Line 997: if (DP_LINK_BW_5_4 == rate) Comparisons should place the constant on the right side of the test Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3cf89c66_9c0de921 PS3, Line 1000: if (DP_LINK_BW_8_1 == rate) Comparisons should place the constant on the right side of the test Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/a5273cb9_6f7b8b31 PS3, Line 1003: printk(BIOS_INFO,"mvid=0x%x, nvid=0x%x\n", mvid, nvid); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2a4206dd_cd76cd89 PS3, Line 1020: tu->last_partial_tu_fp =(tu->n_remainder_symbols_per_lane_fp / tu->tu_size); spaces required around that '=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0d1f19d9_30084c2a PS3, Line 1098: if (tu->new_err_fp <tu->diff_abs_fp) spaces required around that '<' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/ea5b2a85_ef3f5d35 PS3, Line 1115: int64_t temp =0; spaces required around that '=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/5ffc0dc4_d22bef92 PS3, Line 1116: int64_t temp_fp =0; spaces required around that '=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4bc4aa12_e4fc3388 PS3, Line 1136: if (((tu->lwidth % tu->nlanes) != 0) && (tu->ratio_fp < f)) that open brace { should be on the previous line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/8f8e6eb4_6f364680 PS3, Line 1142: tu->err_fp = 1000 * f ; space prohibited before semicolon Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/7deff4e3_430d1016 PS3, Line 1143: if (tu->ratio_fp > f) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/7ddb9d06_c99f7c48 PS3, Line 1147: for (tu->tu_size = 32; tu->tu_size <= 64; tu->tu_size++) that open brace { should be on the previous line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/8ade95d1_f1a19ab5 PS3, Line 1154: if(tu->new_err_fp < tu->err_fp) that open brace { should be on the previous line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/6892e7d0_9897fee7 PS3, Line 1154: if(tu->new_err_fp < tu->err_fp) space required before the open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/cad15ae3_d0a75f66 PS3, Line 1256: printk(BIOS_ERR,"TU: valid_boundary_link: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/f04b5020_6efb2086 PS3, Line 1258: printk(BIOS_ERR,"TU: delay_start_link: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2d8d8464_f822f89c PS3, Line 1260: printk(BIOS_ERR,"TU: boundary_moderation_en: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2bb9ff39_997678b0 PS3, Line 1262: printk(BIOS_ERR,"TU: valid_lower_boundary_link: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/e356a7fa_549a4e7c PS3, Line 1264: printk(BIOS_ERR,"TU: upper_boundary_count: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/25e7cf42_7e315246 PS3, Line 1266: printk(BIOS_ERR,"TU: lower_boundary_count: %d\n", space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/1fac38b9_dc058ea2 PS3, Line 1268: printk(BIOS_ERR,"TU: tu_size_minus1: %d\n", tu_table->tu_size_minus1); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d8b46f92_eb4fc38c PS3, Line 1275: edp_ctrl_calc_tu( ctrl, edid, &tu_config); space prohibited after that open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/56ce6a97_e11635f7 PS3, Line 1320: data |= (0x1) ; space prohibited before semicolon Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/5281dbff_3cfd7cd5 PS3, Line 1346: static void edp_ctrl_link_enable(struct edp_ctrl* ctrl, "foo* bar" should be "foo *bar" Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/8951ae34_9066919b PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/ffb6ae4d_c7db8b8a PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/e513d0f1_8eab3217 PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/724af12f_02de9ba7 PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3d79b117_d990f45c PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/6b49f96d_5980619d PS3, Line 1366: mdss_clock_configure(MDSS_CLK_EDP_LINK,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/74bfe322_34ab1207 PS3, Line 1368: if(ret !=0) spaces required around that '!=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/742b693f_3c201088 PS3, Line 1368: if(ret !=0) space required before the open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/83f65330_b38dfaaa PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4194931f_922ca458 PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/71a32159_4ab19255 PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/bb11b4d9_f2666de4 PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/79d37641_3bbe7894 PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/64bcfc33_fc33598e PS3, Line 1371: mdss_clock_configure(MDSS_CLK_EDP_LINK_INTF,0,1,0,0,0,0); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/13ea8c57_f581b721 PS3, Line 1373: if(ret !=0) spaces required around that '!=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/21c6c309_27198f7a PS3, Line 1373: if(ret !=0) space required before the open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/3b71eac6_f0a5c053 PS3, Line 1379: if(ret !=0) spaces required around that '!=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/131c3f27_4efd8f68 PS3, Line 1379: if(ret !=0) space required before the open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/646171a4_1ae90858 PS3, Line 1394: while(ret == EDP_TRAIN_RECONFIG) { space required before the open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/ff6e1552_3945f488 PS3, Line 1396: edp_ctrl_irq_enable( 0); space prohibited after that open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/bc2ac529_7cb63680 PS3, Line 1400: edp_ctrl_irq_enable( 1); space prohibited after that open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/4a600266_5378b89e PS3, Line 1417: printk(BIOS_ERR,"ERROR: edp native read failure\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/b9bf1b57_80b682e8 PS3, Line 1424: printk(BIOS_ERR,"ERROR: edp native read failure\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0e3f7669_5a151145 PS3, Line 1434: edp_ctrl_irq_enable( 1); space prohibited after that open parenthesis '(' Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/0286f8aa_b9575c8a PS3, Line 1439: printk(BIOS_ERR,"ERROR: edp training failure\n"); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/32df7aa4_c143884b PS3, Line 1466: printk(BIOS_ERR,"%s: AUX channel is NOT ready\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2ca4701a_fa5bc11d PS3, Line 1477: int ret =0; spaces required around that '=' (ctx:WxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/9a6190d7_55637628 PS3, Line 1485: if(edp_read_edid(edid) == 0) that open brace { should be on the previous line Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/00e9c2c5_306c0554 PS3, Line 1485: if(edp_read_edid(edid) == 0) space required before the open parenthesis '(' File src/soc/qualcomm/sc7280/display/edp_phy_7nm.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/ac9bd021_eda3598a PS3, Line 45: printk(BIOS_ERR,"%s: refgen not ready : 0x%x\n", __func__,read32(&edp_phy_pll->qserdes_com_cmn_status)); line over 96 characters Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/987e77f6_752cefbd PS3, Line 45: printk(BIOS_ERR,"%s: refgen not ready : 0x%x\n", __func__,read32(&edp_phy_pll->qserdes_com_cmn_status)); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/64c979dc_dab00c05 PS3, Line 45: printk(BIOS_ERR,"%s: refgen not ready : 0x%x\n", __func__,read32(&edp_phy_pll->qserdes_com_cmn_status)); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/dd3fcaab_4a9e08ca PS3, Line 214: printk(BIOS_ERR,"%s: Invalid link rate. rate = %u\n", __func__, space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/edb42d7a_e000ae01 PS3, Line 281: printk(BIOS_ERR,"%s: Invalid link rate. rate = %u\n", __func__, space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/86e291ca_65912608 PS3, Line 293: printk(BIOS_ERR,"%s: PLL not locked. Status\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/18c6489d_131d0c4e PS3, Line 301: printk(BIOS_ERR,"%s: PHY not ready. Status\n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2b8edbb4_dfd1efd7 PS3, Line 309: printk(BIOS_ERR,"%s: PLL not locked. Status\n", __func__); space required after that ',' (ctx:VxV) File src/soc/qualcomm/sc7280/display/mdss.c: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/7640fda8_9ae705d7 PS3, Line 55: printk(BIOS_ERR,"%s: \n", __func__); space required after that ',' (ctx:VxV) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/774955b9_adeefde3 PS3, Line 55: printk(BIOS_ERR,"%s: \n", __func__); unnecessary whitespace before a quoted newline Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/d5696e21_46f8f19a PS3, Line 213: mdss_clock_configure(MDSS_CLK_MDP, 300 * MHz, 0, 0, 0 , 0 , 0 ); space prohibited before that ',' (ctx:WxW) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/cb19bb57_cb4b43b6 PS3, Line 213: mdss_clock_configure(MDSS_CLK_MDP, 300 * MHz, 0, 0, 0 , 0 , 0 ); space prohibited before that ',' (ctx:WxW) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134461): https://review.coreboot.org/c/coreboot/+/59612/comment/2c8d51aa_7a830eb1 PS3, Line 213: mdss_clock_configure(MDSS_CLK_MDP, 300 * MHz, 0, 0, 0 , 0 , 0 ); space prohibited before that close parenthesis ')' -- To view, visit https://review.coreboot.org/c/coreboot/+/59612 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I388bce93a4d174ca51a02add1fd9b04c46899fa6 Gerrit-Change-Number: 59612 Gerrit-PatchSet: 3 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:09:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: libpayload: Parse DDR Information through coreboot tables
by build bot (Jenkins) (Code Review) 26 Nov '21

26 Nov '21

Attention is currently required from: Ravi kumar, Shelley Chen, mturney mturney, Julius Werner. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59193 ) Change subject: libpayload: Parse DDR Information through coreboot tables ...................................................................... Patch Set 6: Verified-1 (1 comment) File payloads/libpayload/include/mem_chip_info.h: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134457): https://review.coreboot.org/c/coreboot/+/59193/comment/e6a0df14_b5429e1b PS6, Line 44: }dram_info; space required after that close brace '}' -- To view, visit https://review.coreboot.org/c/coreboot/+/59193 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ieca7e9fc0e1a018fcb2e9315aebee088edac858e Gerrit-Change-Number: 59193 Gerrit-PatchSet: 6 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: mturney mturney <mturney(a)codeaurora.org> Gerrit-Reviewer: mturney mturney <quic_mturney(a)quicinc.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Ravi Kumar Bokka <c_rbokka(a)qualcomm.corp-partner.google.com> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: mturney mturney <mturney(a)codeaurora.org> Gerrit-Attention: Julius Werner <jwerner(a)chromium.org> Gerrit-Attention: mturney mturney <quic_mturney(a)quicinc.com> Gerrit-Comment-Date: Fri, 26 Nov 2021 07:08:59 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit November 2021