Re: [PATCH 02/19] EDAC: Remove EDAC_DIMM_OFF() macro
From: Robert Richter
Date: Fri Oct 11 2019 - 07:36:32 EST
Thanks for review.
On 11.10.19 07:09:03, Mauro Carvalho Chehab wrote:
> Em Thu, 10 Oct 2019 20:25:07 +0000
> Robert Richter <rrichter@xxxxxxxxxxx> escreveu:
>
> > The EDAC_DIMM_OFF() macro takes 5 arguments to get the DIMM's index.
> > This can be much simplified now as the offset is already stored in
> > struct dimm_info. Use it directly and completely remove the
> > EDAC_DIMM_OFF() macro.
>
> Ah, now I understand why you added the dimm idx field. One more reason
> why to split it on a separate patch (or perhaps merge it here).
Right, it needs to be part of the this patch.
-Robert