Attention is currently required from: YH Lin, Joey Peng.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62954 )
Change subject: mb/google/brya/var/taeko: Add new FW_CONFIG option for THERMAL
......................................................................
Patch Set 9:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62954/comment/752b892e_ccc30480
PS9, Line 7: mb/google/brya/var/taeko: Add new FW_CONFIG option for THERMAL
Mention *tarlo* somewhere?
https://review.coreboot.org/c/coreboot/+/62954/comment/1911f9a5_c5fd3c7f
PS9, Line 9: Add thermal table settings for tarlo which shares the same fw with taeko
Where is that documented, that *tarlo* and *taeko* are the same?
--
To view, visit https://review.coreboot.org/c/coreboot/+/62954
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I37f79cde502115bbf65bb97216eddb6ea22b1648
Gerrit-Change-Number: 62954
Gerrit-PatchSet: 9
Gerrit-Owner: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: YH Lin <yueherngl(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Kevin Chang <kevin.chang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Melo Chuang <melo.chuang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Rasheed Hsueh <rasheed.hsueh(a)lcfc.corp-partner.google.com>
Gerrit-CC: Sunshine Chao <sunshine.chao(a)lcfc.corp-partner.google.com>
Gerrit-Attention: YH Lin <yueherngl(a)google.com>
Gerrit-Attention: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Comment-Date: Mon, 28 Mar 2022 19:18:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: YH Lin, Joey Peng, Paul Menzel.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62954 )
Change subject: mb/google/brya/var/taeko: Add new FW_CONFIG option for THERMAL
......................................................................
Patch Set 9:
(1 comment)
File src/mainboard/google/brya/variants/taeko/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/62954/comment/61b4bd77_2fa552e7
PS9, Line 245: alias dptf_policy
Since this is unused, I'd also remove it in this CL so it doesn't get confusing
--
To view, visit https://review.coreboot.org/c/coreboot/+/62954
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I37f79cde502115bbf65bb97216eddb6ea22b1648
Gerrit-Change-Number: 62954
Gerrit-PatchSet: 9
Gerrit-Owner: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: YH Lin <yueherngl(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Kevin Chang <kevin.chang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Melo Chuang <melo.chuang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Rasheed Hsueh <rasheed.hsueh(a)lcfc.corp-partner.google.com>
Gerrit-CC: Sunshine Chao <sunshine.chao(a)lcfc.corp-partner.google.com>
Gerrit-Attention: YH Lin <yueherngl(a)google.com>
Gerrit-Attention: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Mon, 28 Mar 2022 19:11:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Terry Chen, Paul Menzel, Zhuohao Lee, YH Lin.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63022 )
Change subject: mb/google/brya/variants/crota: init overridetree for crota
......................................................................
Patch Set 8:
(2 comments)
File src/mainboard/google/brya/variants/crota/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/63022/comment/f2bb490a_bb4102d3
PS8, Line 68: register "usb2_ports[4]" = "USB2_PORT_EMPTY" # Disable USB2_C4
: register "usb2_ports[6]" = "USB2_PORT_EMPTY" # Disable USB2_C6
: register "usb2_ports[7]" = "USB2_PORT_MID(OC_SKIP)" # Type-A MLB Port
: register "usb2_ports[8]" = "USB2_PORT_EMPTY" # Disable USB2_C8
nit: use all tabs for indentation please (there are spaces at the beginning of these lines)
https://review.coreboot.org/c/coreboot/+/63022/comment/547335f5_0b46e60b
PS8, Line 74: register "tcss_ports[1]" = "TCSS_PORT_EMPTY"
same here
--
To view, visit https://review.coreboot.org/c/coreboot/+/63022
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibca9d93a81469730e472a645c607a97a624e9a1c
Gerrit-Change-Number: 63022
Gerrit-PatchSet: 8
Gerrit-Owner: Terry Chen <terry_chen(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: YH Lin <yueherngl(a)chromium.org>
Gerrit-Reviewer: Zhuohao Lee <zhuohao(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Anfernee Chen <anfernee_chen(a)wistron.corp-partner.google.com>
Gerrit-CC: Ariel Fang <ariel_fang(a)wistron.corp-partner.google.com>
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: Raihow Shi <raihow_shi(a)wistron.corp-partner.google.com>
Gerrit-CC: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-CC: Zoey Wu <zoey_wu(a)wistron.corp-partner.google.com>
Gerrit-Attention: Terry Chen <terry_chen(a)wistron.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Zhuohao Lee <zhuohao(a)google.com>
Gerrit-Attention: YH Lin <yueherngl(a)chromium.org>
Gerrit-Comment-Date: Mon, 28 Mar 2022 19:08:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/60621
to look at the new patch set (#11).
Change subject: src/lib: Remove unused <stdlib.h> and use <types.h> when appropriate
......................................................................
src/lib: Remove unused <stdlib.h> and use <types.h> when appropriate
Unused <stdlib.h> found using:
diff <(git grep -l '#include <stdlib.h>' -- src/) <(git grep -l 'memalign(\|malloc(\|calloc(\|free(' -- src/)
Change-Id: I5ad171679cbfa67d522bd1105fb58e0f84b9cb89
Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr>
---
M src/lib/bootmem.c
M src/lib/coreboot_table.c
M src/lib/fit.c
M src/lib/hardwaremain.c
M src/lib/imd.c
M src/lib/imd_cbmem.c
M src/lib/prog_loaders.c
M src/lib/rmodule.c
M src/lib/selfboot.c
M src/lib/thread.c
10 files changed, 8 insertions(+), 16 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/60621/11
--
To view, visit https://review.coreboot.org/c/coreboot/+/60621
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5ad171679cbfa67d522bd1105fb58e0f84b9cb89
Gerrit-Change-Number: 60621
Gerrit-PatchSet: 11
Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: newpatchset
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/60621
to look at the new patch set (#10).
Change subject: src/lib: Use <types.h> instead of <stdlib.h> when appropriate
......................................................................
src/lib: Use <types.h> instead of <stdlib.h> when appropriate
Unused <stdlib.h> found using:
diff <(git grep -l '#include <stdlib.h>' -- src/) <(git grep -l 'memalign(\|malloc(\|calloc(\|free(' -- src/)
Change-Id: I5ad171679cbfa67d522bd1105fb58e0f84b9cb89
Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr>
---
M src/lib/bootmem.c
M src/lib/coreboot_table.c
M src/lib/fit.c
M src/lib/hardwaremain.c
M src/lib/imd.c
M src/lib/imd_cbmem.c
M src/lib/prog_loaders.c
M src/lib/rmodule.c
M src/lib/selfboot.c
M src/lib/thread.c
10 files changed, 8 insertions(+), 16 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/60621/10
--
To view, visit https://review.coreboot.org/c/coreboot/+/60621
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5ad171679cbfa67d522bd1105fb58e0f84b9cb89
Gerrit-Change-Number: 60621
Gerrit-PatchSet: 10
Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Maulik V Vaghela, Tim Wawrzynczak, Paul Menzel, Nick Vaccaro, Eric Lai, Andrey Petrov.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62942 )
Change subject: drivers/intel/fsp2_0: Add provision to extract FSP Performance Data
......................................................................
Patch Set 5:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62942/comment/a9519068_31fb8b1a
PS2, Line 9: This patch enriches coreboot FSP2.0 driver to extract the FSP timestamp
: from FPDT (Firmware Performance Data Table) and display as part of
: ramstage boot stage if the SoC user selects the required
: `GET_FSP_TIMESTAMP` config.
> > > Extracting this data sounds great. But instead of printing it to the CBMEM console, could you please add them to the CBMEM timestamp table?
> >
> > we already have FSP TS data as part of cbmem -t table. what this CL does is to have detailed break down of the FSP timestamp which might be useful to debug or optimise the FSP per component boot time.
> >
> > >
> > > Is the FSP data theoretically also accessible from the operating sytsem?
> >
> > Not sure, if I understood your comments, but cbmem -t is able to show the FSP API wise boot time impact. where this CL is more deep dive into those FSP APIs.
> >
> > 950:calling FspMemoryInit 1,936,192 (59,275)
> > 951:returning from FspMemoryInit 81,024,105 (79,087,913)
> > 971:loading FSP-S 82,941,127 (21,318)
> > 954:calling FspSiliconInit 83,166,997 (219,188)
> > 955:returning from FspSiliconInit 102,564,474 (19,397,476)
> > 962:calling FspMultiPhaseSiInit 102,564,627 (152)
> > 963:returning from FspMultiPhaseSiInit 108,645,466 (6,080,839)
>
> @Paul, any further opens ?
@Paul, marking this resolved now, please feel free to open it if you think your question is still not answered.
--
To view, visit https://review.coreboot.org/c/coreboot/+/62942
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia1b7f6b98bafeec0afe843f0f78c99c2f34f50b3
Gerrit-Change-Number: 62942
Gerrit-PatchSet: 5
Gerrit-Owner: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com>
Gerrit-Reviewer: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
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)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Attention: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-Attention: Andrey Petrov <andrey.petrov(a)gmail.com>
Gerrit-Comment-Date: Mon, 28 Mar 2022 18:31:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subratabanik(a)google.com>
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Sridhar Siricilla, Tim Wawrzynczak, Arthur Heymans.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63141 )
Change subject: soc/intel/alderlake/pmc_descriptor.c: Don't depend on FMAP
......................................................................
Patch Set 2: Code-Review+2
(2 comments)
Patchset:
PS2:
Thanks Arthur 😊 I had forgotten about the FMAP_SECTION_*... macros,
File src/soc/intel/alderlake/bootblock/pmc_descriptor.c:
https://review.coreboot.org/c/coreboot/+/63141/comment/2fd39654_32bcb5d5
PS2, Line 62: if (cpu_get_cpuid() != CPUID_ALDERLAKE_A0)
: return;
I would probably keep this as the first early return?
--
To view, visit https://review.coreboot.org/c/coreboot/+/63141
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2d180353bfd69601ce03feb59101fb425859e779
Gerrit-Change-Number: 63141
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sridhar Siricilla <sridhar.siricilla(a)intel.corp-partner.google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Comment-Date: Mon, 28 Mar 2022 17:40:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: YH Lin, Joey Peng.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63131 )
Change subject: mb/google/brya/var/taniks: Enable Genesys L1 max entry delay
......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/google/brya/Kconfig.name:
https://review.coreboot.org/c/coreboot/+/63131/comment/ae0c15d6_9dec79fb
PS1, Line 156: if DRIVERS_GENESYSLOGIC_GL9763E
This is already selected for this board, so you can drop the `if ...` part
--
To view, visit https://review.coreboot.org/c/coreboot/+/63131
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2a5888e943c1ebf83a54f9b172f986f8b13d9b6a
Gerrit-Change-Number: 63131
Gerrit-PatchSet: 1
Gerrit-Owner: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: YH Lin <yueherngl(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Kevin Chang <kevin.chang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Sunshine Chao <sunshine.chao(a)lcfc.corp-partner.google.com>
Gerrit-Attention: YH Lin <yueherngl(a)google.com>
Gerrit-Attention: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-Comment-Date: Mon, 28 Mar 2022 17:39:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment