Attention is currently required from: Mac Chiang, leeweitse_seven, Martin Roth, Furquan Shaikh, Tim Wawrzynczak, John Hsu, David Lin, EricR Lai.
leeweitse_seven has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51971 )
Change subject: src/drivers/generic/nau8315: Add nau8315 support
......................................................................
Patch Set 4:
(5 comments)
File src/drivers/generic/nau8315/chip.h:
https://review.coreboot.org/c/coreboot/+/51971/comment/b3046021_372ee6ec
PS1, Line 8: const char *hid;
> And I think we can fixed the HID name. Since kernel driver only look up specific name.
Done
https://review.coreboot.org/c/coreboot/+/51971/comment/8cb87951_d6d2eaad
PS1, Line 13: /* GPIO used to indicate if this device is present */
> I think we can remove this. We can have FW config for present now.
Done
File src/drivers/generic/nau8315/nau8315.c:
https://review.coreboot.org/c/coreboot/+/51971/comment/6cea25a7_104ae3eb
PS1, Line 11: #if CONFIG(HAVE_ACPI_TABLES)
> use depend for it. please refer my CL. https://review.coreboot. […]
Done
https://review.coreboot.org/c/coreboot/+/51971/comment/eb46cdf7_19d3f0c7
PS1, Line 53: enable-gpio
> enable-gpios
Done
File src/drivers/generic/nau8315/nau8315.c:
https://review.coreboot.org/c/coreboot/+/51971/comment/029de2c6_f6f5a258
PS3, Line 58: return NAU8315_ACPI_NAME;
> we can return string here, remove the define.
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/51971
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie883d65ced3cd95fe318ba0914ed806ff592258d
Gerrit-Change-Number: 51971
Gerrit-PatchSet: 4
Gerrit-Owner: leeweitse_seven <scott.6986(a)hotmail.com>
Gerrit-Reviewer: David Lin <CTLIN0(a)nuvoton.com>
Gerrit-Reviewer: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: John Hsu <KCHSU0(a)nuvoton.com>
Gerrit-Reviewer: Mac Chiang <mac.chiang(a)intel.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Mac Chiang <mac.chiang(a)intel.com>
Gerrit-Attention: leeweitse_seven <scott.6986(a)hotmail.com>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: John Hsu <KCHSU0(a)nuvoton.com>
Gerrit-Attention: David Lin <CTLIN0(a)nuvoton.com>
Gerrit-Attention: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 01 Apr 2021 06:46:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: Shaoming Chen.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51432 )
Change subject: soc/mediatek: dsi: fine tune the delta time for EoTp
......................................................................
Patch Set 5: -Code-Review
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/51432/comment/dec18b16_d5a28ea2
PS1, Line 9: We seperate the EoTp packet extra data
> so which change did this? we need a reference
Shaoming, please address hungte's comment.
Commit Message:
https://review.coreboot.org/c/coreboot/+/51432/comment/c38428d6_bf6ddfba
PS2, Line 7: when EoTp enable
> Either […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/51432
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0666068cfb04b78eb706278814163f050da32b9c
Gerrit-Change-Number: 51432
Gerrit-PatchSet: 5
Gerrit-Owner: jitao shi <jitao.shi(a)mediatek.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: Chen-Tsung Hsieh <chentsung(a)chromium.org>
Gerrit-CC: Chen-Tsung Hsieh <chentsung(a)google.com>
Gerrit-CC: Jitao Shi <jitao.shi(a)mediatek.corp-partner.google.com>
Gerrit-CC: Shaoming Chen <shaoming.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Shaoming Chen <shaoming.chen(a)mediatek.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 01 Apr 2021 06:45:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Hung-Te Lin <hungte(a)chromium.org>
Comment-In-Reply-To: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Christian Walter.
Justin van Son has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51919 )
Change subject: mb/prodrive/hermes: Drop Vref configuration for older boards
......................................................................
Patch Set 1:
(2 comments)
File src/mainboard/prodrive/hermes/variants/r04/hda_verb.c:
https://review.coreboot.org/c/coreboot/+/51919/comment/0f61538b_5dbabd05
PS1, Line 42: return AZALIA_PIN_DESC(
The driver won't detect a valid pin configuration when AZALIA_PIN_DESC is used. I have no idea why, but when I change this to the hex equivalent as in case 1, it will work fine.
Also, it seems like this configuration makes the driver somehow tie this widget (and thus the output) together with NID 0x14 (green rear line out), causing windows to only have 1 output device, and thus outputting audio to the two devices at the same time. I'm not sure how we can get this separated, even though it should be possible here.
https://review.coreboot.org/c/coreboot/+/51919/comment/6fd62340_1a4f7ff9
PS1, Line 69: 0x018707e1, /* port B VREF 50% of LDO-OUT */
You don't need any of the 707 verbs, as they will get reset by realtek driver anyway
--
To view, visit https://review.coreboot.org/c/coreboot/+/51919
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I41c207f97dad6c9107c1999eb46d2d6304a6c217
Gerrit-Change-Number: 51919
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Justin van Son <justin.van.son(a)prodrive-technologies.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Comment-Date: Thu, 01 Apr 2021 06:44:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Bill XIE, Julius Werner.
Iru Cai (vimacs) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51671 )
Change subject: arch/x86: Reserve space for EC SMSC KBC1098/KBC1126
......................................................................
Patch Set 1: -Code-Review
(2 comments)
Patchset:
PS1:
Hmm, I think the code will get complicated if there are more things to add.
File src/arch/x86/bootblock.ld:
https://review.coreboot.org/c/coreboot/+/51671/comment/8b2e4286_a9d1660d
PS1, Line 35: MIN(_KBC1126_FW_LOCATIONS, MIN(_ID_SECTION, _FIT_POINTER)) - EARLYASM_SZ;
Hmm, the code will have more ``#if CONFIG(...)`` and ``MIN(...)`` to add if there're more things to add, so using one _ECFW_POINTER like in 51059 may be a better way to do this.
--
To view, visit https://review.coreboot.org/c/coreboot/+/51671
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4f0de0c4d7283e630242fbe84a46e0547783c49e
Gerrit-Change-Number: 51671
Gerrit-PatchSet: 1
Gerrit-Owner: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-Reviewer: Iru Cai (vimacs) <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Comment-Date: Thu, 01 Apr 2021 06:43:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Furquan Shaikh, Maulik V Vaghela, Subrata Banik, Ronak Kanabar, Patrick Rudolph.
Meera Ravindranath has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51849 )
Change subject: soc/intel/alderlake: Enable VT-d
......................................................................
Patch Set 5:
(1 comment)
File src/soc/intel/alderlake/romstage/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/51849/comment/4c13e225_975080d2
PS3, Line 198: /* Disable VT-d support for pre-QS platform */
> Lets first enable the VTD UPD alone and take FSP debug log to see what all FSP is doing. […]
@Furquan, we verified the working on the early silicon as well and hence skipped the check like in tigerlake and enabled by default.
--
To view, visit https://review.coreboot.org/c/coreboot/+/51849
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I96a9f3df185002a4e58faa910f867ace0b97ec2b
Gerrit-Change-Number: 51849
Gerrit-PatchSet: 5
Gerrit-Owner: Meera Ravindranath <meera.ravindranath(a)intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Furquan Shaikh <furquan(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Attention: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Attention: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Thu, 01 Apr 2021 06:38:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan(a)google.com>
Comment-In-Reply-To: Subrata Banik <subrata.banik(a)intel.com>
Comment-In-Reply-To: Meera Ravindranath <meera.ravindranath(a)intel.com>
Gerrit-MessageType: comment