Subrata Banik has posted comments on this change by Saurabh Mishra. ( https://review.coreboot.org/c/coreboot/+/81849?usp=email )
Change subject: soc/intel: Add Panther Lake PCIE device IDs
......................................................................
Patch Set 15:
(1 comment)
Patchset:
PS15:
May be some scope for improvement for in the code review. There are several wrong (several IDs are wrong, copied from LNL like GFX ID) and incomplete information (There are primarily 3 SOC and this CL only landed the IDs for one SKU w/o clarifying what is thart SKU) landed as part of this CL, which I had to fix using [CL]() and subsequent CLs.
--
To view, visit https://review.coreboot.org/c/coreboot/+/81849?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I82f47b6077e28a01f34c59b7e7697323b3d5f990
Gerrit-Change-Number: 81849
Gerrit-PatchSet: 15
Gerrit-Owner: Saurabh Mishra <mishra.saurabh(a)intel.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Ashish Kumar Mishra <ashish.k.mishra(a)intel.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com>
Gerrit-CC: Anil Kumar K <anil.kumar.k(a)intel.com>
Gerrit-CC: Appukuttan V K <appukuttan.vk(a)intel.com>
Gerrit-CC: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-CC: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-CC: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-CC: Sanju Jose Thottan <sanjujose.thottan(a)intel.com>
Gerrit-CC: Wonkyu Kim <wonkyu.kim(a)intel.com>
Gerrit-CC: sridhar siricilla <siricillasridhar(a)gmail.com>
Gerrit-Comment-Date: Wed, 17 Jul 2024 12:08:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No