[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <000901d4406c$9d04f360$d70eda20$@codeaurora.org>
Date: Thu, 30 Aug 2018 08:20:32 -0600
From: "wufan" <wufan@...eaurora.org>
To: "'Borislav Petkov'" <bp@...en8.de>
Cc: <mchehab@...nel.org>, <james.morse@....com>,
<baicar.tyler@...il.com>, <linux-edac@...r.kernel.org>,
<linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>,
"'Kani, Toshi'" <toshi.kani@....com>
Subject: RE: [PATCH] EDAC, ghes: use CPER module handles to locate DIMMs
Hi Boris,
> If we're going to do this, it needs to be tested on an x86 box which loads
> ghes_edac. Adding Toshi to Cc.
>
> Otherwise it must remain ARM-specific.
Toshi it would be great if you can help! I'll also test the change in x86 but
not sure if the firmware updates module_handle.
> > +static int ghes_edac_dimm_index(u16 handle)
>
> get_dimm_smbios_handle()
This function returns an index. So how about get_dimm_smbios_index()?
> > +{
> > + struct mem_ctl_info *mci;
> > + int i;
> > +
> > + if (!ghes_pvt)
> > + return -1;
>
> You don't need that test.
Will remove.
> > +
> > + mci = ghes_pvt->mci;
> > +
> > + if (!mci)
> > + return -1;
>
> Ditto.
Will remove
Thanks,
Fan
Powered by blists - more mailing lists