[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20121217211124.GB3012@jshin-Toonie>
Date: Mon, 17 Dec 2012 15:11:24 -0600
From: Jacob Shin <jacob.shin@....com>
To: Borislav Petkov <bp@...en8.de>, Joe Perches <joe@...ches.com>,
Doug Thompson <dougthompson@...ssion.com>,
<linux-edac@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] MCE, AMD: Make MC2 decoding part of amd_decoder_ops
as well
On Mon, Dec 17, 2012 at 10:08:02PM +0100, Borislav Petkov wrote:
> On Mon, Dec 17, 2012 at 12:59:00PM -0800, Joe Perches wrote:
> > The new pr_emerg is effectively a termination of the
> > previous one. There's no need for 2 consecutive uses
> > of pr_emerg(HW_ERR ...) here.
>
> Actually, in decode_mc5_mce(), the second pr_emerg needs to stay like
> what it is now.
>
> So Jacob, please leave those changes out, I'll sort them out later.
Okay, will do, I'll change fix it in my changes then. But I'll wait
for your feedback tomorrow, before sending out V2.
Thanks!
-Jacob
>
> Thanks.
>
> --
> Regards/Gruss,
> Boris.
>
> Sent from a fat crate under my desk. Formatting is fine.
> --
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists