Re: [PATCH] EDAC, sb_edac: Fixed logic error in sb_edac driver

From: Borislav Petkov
Date: Tue Mar 08 2016 - 05:28:35 EST


On Tue, Mar 08, 2016 at 09:18:33AM +0000, Chrzaniuk, Hubert wrote:
> Yes, thatâs right:
> 1. d0cdf9003140 is the patch I am referring to
> 2. EDAC for KNL platform on 4.5 will not cause any problems but will not be functional for DIMMs (only for MCDRAM)

Ok, thanks for confirming.

> Thanks, that is a valid point, it will be refactored in a separate
> patch. However it would good to fix what we have now, so I still hope
> to have the patch merged anyway...

Of course - it is too late for cleanups anyway. I was suggesting that
you guys put it on your radar and start cleaning it up at some point,
when you get a chance. It is a low prio thing and can wait.

Thanks.

--
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.