RE: [PATCH 1/1] EDAC/ie31200: Add two more Intel Alder Lake-S SoCs for EDAC support

[Zhuo, Qiuxu]

Hi Kyle,

> From: Kyle Manna <kyle@xxxxxxxxxxxxx>
> Sent: Thursday, July 24, 2025 2:44 PM
> To: Luck, Tony <tony.luck@xxxxxxxxx>; Borislav Petkov <bp@xxxxxxxxx>; Jason
> Baron <jbaron@xxxxxxxxxx>
> Cc: Zhuo, Qiuxu <qiuxu.zhuo@xxxxxxxxx>; James Jernigan
> <jameswestonjernigan@xxxxxxxxx>; James Morse
> <james.morse@xxxxxxx>; Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>;
> Robert Richter <rric@xxxxxxxxxx>; Lai, Yi1 <yi1.lai@xxxxxxxxx>; linux-
> edac@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Kyle Manna
> <kyle@xxxxxxxxxxxxx>
> Subject: [PATCH 1/1] EDAC/ie31200: Add two more Intel Alder Lake-S SoCs for
> EDAC support
> 
> Host Device IDs (DID0) correspond to:
> * Intel Core i7-12700K
> * Intel Core i5-12600K
> 
> See documentation:
> * 12th Generation Intel® Core™ Processors Datasheet
>     * Volume 1 of 2, Doc. No.: 655258, Rev.: 011
>     * https://edc.intel.com/output/DownloadPdfDocument?id=8297 (PDF)
> 
> Signed-off-by: Kyle Manna <kyle@xxxxxxxxxxxxx>
> ---

Do you have access to these machines to load the ie31200_edac
driver with your patch? If yes, would you take dmesg logs?

>  drivers/edac/ie31200_edac.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/edac/ie31200_edac.c b/drivers/edac/ie31200_edac.c index
> a53612be4b2f..2078c12bbed2 100644
> --- a/drivers/edac/ie31200_edac.c
> +++ b/drivers/edac/ie31200_edac.c
> @@ -94,6 +94,8 @@
> 
>  /* Alder Lake-S */
>  #define PCI_DEVICE_ID_INTEL_IE31200_ADL_S_1	0x4660
> +#define PCI_DEVICE_ID_INTEL_IE31200_ADL_S_2	0x4668
> +#define PCI_DEVICE_ID_INTEL_IE31200_ADL_S_3	0x4648
> 

I didn't find the place in your spec above that indicates these
two CPUs with these two DIDs have Out-Of-Band ECC capabilities.
Could you point it out to me?

Thanks!
-Qiuxu