RE: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-05 Thread Bounine, Alexandre
@lists.ozlabs.org; linux-ker...@vger.kernel.org; thomas.m...@sysgo.com; Kumar Gala Subject: Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port In message 0ce8b6be3c4ad74ab97d9d29bd24e55201143...@corpexch1.na.ads.idt.com you wrote: Yang Li pointed to these patches in his post from July 23

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-05 Thread Kumar Gala
On Aug 5, 2010, at 12:25 PM, Bounine, Alexandre wrote: Below is a copy of Leo's message with pointers to the patches. Alex. Subject: [PATCH] RapidIO,powerpc/85xx: remove MCSR_MASK in fsl_rio Fixes compile problem caused by MCSR_MASK removal from book-E definitions. Hi Alex, Only

RE: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-05 Thread Bounine, Alexandre
I'm looking at this now and wondering what we added the mcheck handler for in the first place and what its trying to accomplish. - k This protects system from hanging if RIO link fails or enters error state. In some situations following maintenance read may initiate link recovery from error

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-04 Thread Michael Neuling
-ker...@vger.kernel.org; thomas.m...@sysgo.com; Kumar Gala Subject: Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port =20 On Tue, Aug 3, 2010 at 7:17 AM, Bounine, Alexandre alexandre.boun...@idt.com wrote: This happened after change to book-e definitions

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Michael Neuling
MCSR_MASK is not defined anywhere, so when I compile this code, I get this: Never mind. I see that it's been fixed already, and that the patch that removed MCSR_MASK was posted around the same time that this patch was posted. I don't know what happened here but 2.6.35 is broken because of

RE: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Bounine, Alexandre
...@vger.kernel.org; thomas.m...@sysgo.com Subject: Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port MCSR_MASK is not defined anywhere, so when I compile this code, I get this: Never mind. I see that it's been fixed already, and that the patch that removed MCSR_MASK was posted

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Timur Tabi
On Tue, Aug 3, 2010 at 7:17 AM, Bounine, Alexandre alexandre.boun...@idt.com wrote: This happened after change to book-e definitions. There are patches that address this issue. And those patches should have been applied before 2.6.35 was released. Someone dropped the ball. 2.6.35 is broken

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-06-30 Thread Timur Tabi
On Mon, Mar 8, 2010 at 2:10 PM, Alexandre Bounine aboun...@tundra.com wrote: From: Alexandre Bounine alexandre.boun...@idt.com Add Machine Check exception handling into RapidIO port driver for Freescale SoCs (MPC85xx). Signed-off-by: Alexandre Bounine alexandre.boun...@idt.com Tested-by:

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-06-30 Thread Timur Tabi
On Wed, Jun 30, 2010 at 3:55 PM, Timur Tabi timur.t...@gmail.com wrote: MCSR_MASK is not defined anywhere, so when I compile this code, I get this: Never mind. I see that it's been fixed already, and that the patch that removed MCSR_MASK was posted around the same time that this patch was

[PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-03-08 Thread Alexandre Bounine
From: Alexandre Bounine alexandre.boun...@idt.com Add Machine Check exception handling into RapidIO port driver for Freescale SoCs (MPC85xx). Signed-off-by: Alexandre Bounine alexandre.boun...@idt.com Tested-by: Thomas Moll thomas.m...@sysgo.com --- fsl_rio.c | 74