RE: [PATCH 1/4] EDAC: add DDR4 flag
> It already has your Reviewed-by. Is it still valid? So it does ... that was a long time ago ... but not so long that anything important changed. Yes, still valid. -Tony
Re: [PATCH 1/4] EDAC: add DDR4 flag
On Thu, Dec 03, 2015 at 07:16:29PM +, Luck, Tony wrote: > part4 (the actual KNL piece) seems not to break earlier (Broadwell) > system ... but that doesn't qualify enough for Ack/Review/Tested -by. It already has your Reviewed-by. Is it still valid? -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 1/4] EDAC: add DDR4 flag
> For patch 2 and 3 I'd need an ack from Mauro/Tony. CCed. parts 2 & 3 are OK Acked-by: Tony Luck part4 (the actual KNL piece) seems not to break earlier (Broadwell) system ... but that doesn't qualify enough for Ack/Review/Tested -by. -Tony N�r��yb�X��ǧv�^�){.n�+{zX����ܨ}���Ơz�&j:+v���zZ+��+zf���h���~i���z��w���?�&�)ߢf��^jǫy�m��@A�a��� 0��h���i
Re: [PATCH 1/4] EDAC: add DDR4 flag
On Thu, Dec 03, 2015 at 10:48:51AM +0100, Hubert Chrzaniuk wrote: > From: Jim Snow > > Make EDAC aware of DDR4/RDDR4 mem types. > > Signed-off-by: Jim Snow > [hubert.chrzan...@intel.com: patch rebase to 4.4-rc3] > Signed-off-by: Hubert Chrzaniuk > --- > include/linux/edac.h | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) Applied, thanks. For patch 2 and 3 I'd need an ack from Mauro/Tony. CCed. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/4] EDAC: add DDR4 flag
On Thu, Dec 03, 2015 at 07:51:47AM +, Anaczkowski, Lukasz wrote: > Well, although it may sound strange, yeah this is exactly the case, since > EDAC for Knights Landing has been for a while in our private branch. > > Is this sort of an issue or just curiosity? Well, the fact that I'm asking, means it is not clear who did what. Please document that like this: Signed-off-by: Random J Developer [lu...@maintainer.example.org: struct foo moved from foo.c to foo.h] Signed-off-by: Lucky K Maintainer (example is from Documentation/SubmittingPatches where the whole authorship tracking is explained in detail.) Alternatively, you can put it in free text in the commit message. Thanks. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 1/4] EDAC: add DDR4 flag
-Original Message- From: Borislav Petkov [mailto:b...@alien8.de] Sent: Wednesday, December 2, 2015 8:28 PM To: Chrzaniuk, Hubert Cc: Anaczkowski, Lukasz ; mche...@osg.samsung.com; dougthomp...@xmission.com; linux-e...@vger.kernel.org; linux-kernel@vger.kernel.org; Snow, Jim M Subject: Re: [PATCH 1/4] EDAC: add DDR4 flag > On Wed, Dec 02, 2015 at 04:38:29PM +0100, Hubert Chrzaniuk wrote: >> Signed-off-by: Jim Snow >> Signed-off-by: Lukasz Anaczkowski >> Signed-off-by: Hubert Chrzaniuk > > How am I to understand this SoB chain? > > Jim wrote it, Lukasz did something to it and Hubert sends it? Well, although it may sound strange, yeah this is exactly the case, since EDAC for Knights Landing has been for a while in our private branch. Is this sort of an issue or just curiosity? Thanks, Lukasz N�r��yb�X��ǧv�^�){.n�+{zX����ܨ}���Ơz�&j:+v���zZ+��+zf���h���~i���z��w���?�&�)ߢf��^jǫy�m��@A�a��� 0��h���i
Re: [PATCH 1/4] EDAC: add DDR4 flag
On Wed, Dec 02, 2015 at 04:38:29PM +0100, Hubert Chrzaniuk wrote: > From: Jim Snow > > Make EDAC aware of DDR4/RDDR4 mem types. > > Signed-off-by: Jim Snow > Signed-off-by: Lukasz Anaczkowski > Signed-off-by: Hubert Chrzaniuk How am I to understand this SoB chain? Jim wrote it, Lukasz did something to it and Hubert sends it? -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/