RE: [PATCH 4/7] EDAC/{skx_common,imh}: Add EDAC driver for Intel Diamond Rapids servers

[Luck, Tony]

> So is adding the new functionality going to make the old one ugly enough to
> warrant the clean slate?

Yes.

> Is the integrated memory/IO hub the new thing that is
> going to be done going forward?

The next CPU generation will definitely do the same thing. After that
my crystal ball is too hazy :-(

> It doesn't get clear to me from the history (parts of which I remember too,
> btw. :))
>
> Don't get me wrong - you're going to end up maintaining all this so I'm just
> playing the devil's advocate here.

Understood.

-Tony