Attention is currently required from: Anilkumar N, Subrata Banik, Maulik V Vaghela, Kangheui Won, Tim Wawrzynczak, Rizwan Qureshi, Sridhar Siricilla, Krishna P Bhat D.
Usha P has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59937 )
Change subject: mb/intel/adlrvp: Configure GPIOs for Alder Lake-N
......................................................................
Patch Set 11:
(1 comment)
File src/mainboard/intel/adlrvp/early_gpio_n.c:
https://review.coreboot.org/c/coreboot/+/59937/comment/893d04bd_df3f837f
PS10, Line 5: #include <commonlib/helpers.h>
> You don't need this, please look at CB:60305
Ack
--
To view, visit https://review.coreboot.org/c/coreboot/+/59937
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0c0ca52d0cc73acfd8503007d5f3d2ad9a48f8ca
Gerrit-Change-Number: 59937
Gerrit-PatchSet: 11
Gerrit-Owner: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
Gerrit-Reviewer: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Attention: Kangheui Won <khwon(a)chromium.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Attention: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Attention: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Attention: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Comment-Date: Thu, 23 Dec 2021 04:19:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subratabanik(a)google.com>
Gerrit-MessageType: comment
Attention is currently required from: Raul Rangel, Yu-hsuan Hsu.
V sujith kumar Reddy has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60072 )
Change subject: Passing EN_SPKR gpio descriptor from coreboot to kernel
......................................................................
Patch Set 3:
(5 comments)
File src/drivers/amd/i2s_machine_dev/i2s_machine_dev.c:
https://review.coreboot.org/c/coreboot/+/60072/comment/7b02751e_36767933
PS2, Line 21: dmic_select_gpio
> So this function expected the dmic_select_gpio to always have pin_count > 0 since it was checked bef […]
done
https://review.coreboot.org/c/coreboot/+/60072/comment/1d2acf79_5aa08cb2
PS2, Line 35: acpi_dp_add_gpio
> This needs a guard too
done
https://review.coreboot.org/c/coreboot/+/60072/comment/45ffdb48_b30b2a9d
PS2, Line 39: enspkr_gpio
> Should be `enspkr_gpio->pin_count`
done
https://review.coreboot.org/c/coreboot/+/60072/comment/e9e3b13a_9d563a4c
PS2, Line 40: 1
> This index will change depending if dmic-gpios is passed in.
done
File src/mainboard/google/guybrush/variants/baseboard/devicetree.cb:
PS2:
> Please split this into a separate file.
ok
--
To view, visit https://review.coreboot.org/c/coreboot/+/60072
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I784e6b58004a8a36b312b75db7739dfd6e8cd65a
Gerrit-Change-Number: 60072
Gerrit-PatchSet: 3
Gerrit-Owner: V sujith kumar Reddy <vsujithkumar.reddy(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Yu-hsuan Hsu <yuhsuan(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Yu-hsuan Hsu <yuhsuan(a)google.com>
Gerrit-Comment-Date: Thu, 23 Dec 2021 03:58:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Raul Rangel <rrangel(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Subrata Banik, Maulik V Vaghela, Nick Vaccaro, Tim Wawrzynczak.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60305 )
Change subject: mb/intel/{adlrvp,sm}: Remove unused header `helpers.h`
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/60305
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I36758089a4981bba916f4d9cf485f64fca2f81ae
Gerrit-Change-Number: 60305
Gerrit-PatchSet: 1
Gerrit-Owner: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Comment-Date: Thu, 23 Dec 2021 03:32:36 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Ariel Fang, Tim Wawrzynczak, Casper Chang.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60104 )
Change subject: mb/google/brya/var/primus:Fix some GPIO programming
......................................................................
Patch Set 14:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/60104/comment/a8390adf_57eeaf3f
PS14, Line 7: Fix
Add a space before that
https://review.coreboot.org/c/coreboot/+/60104/comment/116df745_6c5bbf6e
PS14, Line 9: -
Remove dash
https://review.coreboot.org/c/coreboot/+/60104/comment/4d9fcfc1_c8a1408d
PS14, Line 12:
Please only use ASCII characters. I would just remove these and leave it as it is.
https://review.coreboot.org/c/coreboot/+/60104/comment/3ae1e786_654bfab3
PS14, Line 14:
Remove space
Patchset:
PS14:
Thanks Ariel 👍 Last todos, then I'm good 😊
File src/mainboard/google/brya/variants/primus4es/gpio.c:
https://review.coreboot.org/c/coreboot/+/60104/comment/d2622261_7b21a663
PS14, Line 39: /* D3 : ISH_GP3 ==> M2_SSD_PLN_L */
: PAD_CFG_GPO(GPP_D3, 1, PLTRST),
Remove this one?
--
To view, visit https://review.coreboot.org/c/coreboot/+/60104
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4e269bc6fb6eda7b2de57e1a9c900864d3e86e98
Gerrit-Change-Number: 60104
Gerrit-PatchSet: 14
Gerrit-Owner: Ariel Fang <ariel_fang(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <felixsinger(a)posteo.net>
Gerrit-CC: Casper Chang <casper_chang(a)wistron.corp-partner.google.com>
Gerrit-CC: Malik Hsu <malik_hsu(a)wistron.corp-partner.google.com>
Gerrit-CC: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com>
Gerrit-CC: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-CC: Terry Chen <terry_chen(a)wistron.corp-partner.google.com>
Gerrit-CC: Zoey Wu <zoey_wu(a)wistron.corp-partner.google.com>
Gerrit-Attention: Ariel Fang <ariel_fang(a)wistron.corp-partner.google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Casper Chang <casper_chang(a)wistron.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 23 Dec 2021 03:24:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment