Messages in this thread | | | Date | Tue, 26 Feb 2019 14:24:16 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCH 4/5] EDAC/amd64: Support more than two Controllers for Chip Select handling |
| |
On Tue, Feb 19, 2019 at 08:26:09PM +0000, Ghannam, Yazen wrote: > From: Yazen Ghannam <yazen.ghannam@amd.com> > > The struct chip_select array that's used for saving Chip Select bases > and masks is fixed at length of two. There should be one struct > chip_select for each controller, so this array should be increased to > support systems that may have more than two controllers. > > Increase the size of the struct chip_select array to eight, which is the > largest number of controllers per die currently supported on AMD > systems. > > Also, carve out the Fam17h+ reading of the bases/masks into a separate > function. This effectively reverts the original bases/masks reading code > to pre-Fam17h support.
I like it simpler again. :-)
-- Regards/Gruss, Boris.
Good mailing practices for 400: avoid top-posting and trim the reply.
| |