On Thu, Aug 16, 2018 at 2:17 PM, Ghannam, Yazen Yazen.Ghannam@amd.com wrote:
-----Original Message----- From: Michael Jin mikhail.jin@gmail.com Sent: Wednesday, August 15, 2018 6:41 AM To: Borislav Petkov bp@suse.de; Ghannam, Yazen Yazen.Ghannam@amd.com; Mauro Carvalho Chehab mchehab@kernel.org Cc: linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; Michael Jin mikhail.jin@gmail.com; stable@vger.kernel.org Subject: [PATCH V2] EDAC, amd64: Add Family 17h Model 10h support.
Add support for ECC error decoding on family 17h models 10h-2fh.
Can you please make this more specific? Something like "Add PCI device IDs for Fam17h Models 10h-2Fh so that amd64_edac_mod will load".
Link: https://lkml.kernel.org/r/20180810193623.24629-1- mikhail.jin@gmail.com Cc: stable@vger.kernel.org
This won't apply to all stable branches. EDAC support for Fam17h was added in v4.10.
I'm not sure if the stable tag needs to be modified, or if the stable queues automatically drop patches that don't apply.
Signed-off-by: Michael Jin mikhail.jin@gmail.com
drivers/edac/amd64_edac.c | 15 +++++++++++++++ drivers/edac/amd64_edac.h | 3 +++ 2 files changed, 18 insertions(+)
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 18aeabb1d5ee..2d7b6d37d6ec 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -3188,6 +3197,12 @@ static struct amd64_family_type *per_family_init(struct amd64_pvt *pvt) break;
case 0x17:
/* Check if CPU model is in range 10h-2fh */This comment is not needed because it's obvious from the code.
Other than those few nits this looks good to me.
Reviewed-by: Yazen Ghannam yazen.ghannam@amd.com
Thanks, Yazen
This version of the patch includes tweaks suggested by Yazen in his review for patch v2 (Link: https://lkml.kernel.org/r/BN7PR12MB2593B097267132250140FBEAF83E0@BN7PR12MB25...).
Does anyone else have any feedback for this patch I submitted a week ago?
Thanks, Michael