[v1 PATCH 0/4] Add INT mode support for EDAC drivers on Maple

2009-05-18 Thread Harry Ciao
Hi Ben, This is the v2 patches that have integrated your suggestions to remove the refcount for a hwriq2virq mapping as long as we don't dispose it, the changes are mostly within the 1/4 patch where the unnecessary refcount and irqmap structure are removed, and callings to edac_put_mpic_irq() are

[v1 PATCH 1/4] EDAC: MPIC Hypertransport IRQ support

2009-05-18 Thread Harry Ciao
just don't call irq_dispose_mapping() against it. This won't occupy unnecessary resource since irq_map[] is of fixed size(==512). The edac_mpic_irq.c is inert for EDAC drivers where related hardware is not connecting to MPIC, so it should be controlled by CONFIG_MPIC. Signed-off

[v1 PATCH 3/4] EDAC: INT mode support for AMD8111 driver

2009-05-18 Thread Harry Ciao
errors correctly. Signed-off-by: Harry Ciao --- drivers/edac/amd8111_edac.c | 348 +-- drivers/edac/amd8111_edac.h | 43 +- 2 files changed, 343 insertions(+), 48 deletions(-) diff --git a/drivers/edac/amd8111_edac.c b/drivers/edac/amd8111_edac.c index

[v1 PATCH 2/4] EDAC: MCE & INT mode support for CPC925 driver

2009-05-18 Thread Harry Ciao
ctly. Signed-off-by: Harry Ciao --- arch/powerpc/kernel/traps.c | 16 +++ drivers/edac/cpc925_edac.c | 275 --- drivers/edac/edac_stub.c|6 + include/linux/edac.h|6 + 4 files changed, 284 insertions(+), 19 deletions(-) diff --git a

[v1 PATCH 4/4] EDAC: INT mode support for AMD8131 driver

2009-05-18 Thread Harry Ciao
errors correctly. Signed-off-by: Harry Ciao --- drivers/edac/amd8131_edac.c | 169 ++- drivers/edac/amd8131_edac.h | 20 +- 2 files changed, 170 insertions(+), 19 deletions(-) diff --git a/drivers/edac/amd8131_edac.c b/drivers/edac/amd8131_edac.c index

[v0 PATCH 4/4] EDAC: INT mode support for AMD8131 driver

2009-05-15 Thread Harry Ciao
errors correctly. Signed-off-by: Harry Ciao diff --git a/drivers/edac/amd8131_edac.c b/drivers/edac/amd8131_edac.c index b432d60..913be34 100644 --- a/drivers/edac/amd8131_edac.c +++ b/drivers/edac/amd8131_edac.c @@ -28,6 +28,7 @@ #include #include #include +#include #include "edac_c

[v0 PATCH 1/4] EDAC: MPIC Hypertransport IRQ support

2009-05-15 Thread Harry Ciao
o it should be controlled by CONFIG_MPIC. Signed-off-by: Harry Ciao diff --git a/drivers/edac/Makefile b/drivers/edac/Makefile index 07a31cf..62778ee 100644 --- a/drivers/edac/Makefile +++ b/drivers/edac/Makefile @@ -17,6 +17,10 @@ ifdef CONFIG_PCI edac_core-objs += edac_pci.o edac_pci_sysfs.o e

[v0 PATCH 2/4] EDAC: MCE & INT mode support for CPC925 driver

2009-05-15 Thread Harry Ciao
ctly. Signed-off-by: Harry Ciao diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index 678fbff..1ae3465 100644 --- a/arch/powerpc/kernel/traps.c +++ b/arch/powerpc/kernel/traps.c @@ -57,6 +57,10 @@ #include #endif +#ifdef CONFIG_EDAC +#include +#endif + #if defined(CONFIG_

[v0 PATCH 3/4] EDAC: INT mode support for AMD8111 driver

2009-05-15 Thread Harry Ciao
errors correctly. Signed-off-by: Harry Ciao diff --git a/drivers/edac/amd8111_edac.c b/drivers/edac/amd8111_edac.c index 35b78d0..022a5bc 100644 --- a/drivers/edac/amd8111_edac.c +++ b/drivers/edac/amd8111_edac.c @@ -38,6 +38,11 @@ #define PCI_DEVICE_ID_AMD_8111_PCI 0x7460 +static int

[v0 PATCH 0/4] Add INT mode support for EDAC drivers on Maple

2009-05-15 Thread Harry Ciao
Comments: - What to be added - 1, Support EDAC INT mode on Maple platform, where CPC925 Hypertransport hostbridge controller will latch MPIC INT0 pin on receiving upstream NMI request messages with vector == 0 that posted from Hypertransport southbridges su

[v3 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-15 Thread Harry Ciao
Hi Andrew and Michael, This is the modified 3/3 patch that corrects CPC925 memory controller DTB node on Maple and setup related platform device for CPC925 EDAC driver. Michael's suggestion of being more discernible for the Maple platform only has been adopted. v2 1/3 & 2/3 patches have been

[v3 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-15 Thread Harry Ciao
Fixup the number of cells for the values of CPC925 Memory Controller, and setup related platform device during system booting up, against which CPC925 Memory Controller EDAC driver would be matched. Signed-off-by: Harry Ciao --- arch/powerpc/kernel/prom_init.c | 40

Re: + edac-cpc925-mc-platform-device-setup.patch added to -mm tree

2009-04-15 Thread Harry Ciao
Michael Ellerman wrote: On Thu, 2009-04-16 at 09:57 +0800, Harry Ciao wrote: Kumar Gala wrote: On Apr 15, 2009, at 5:27 PM, a...@linux-foundation.org wrote: arch/powerpc/kernel/prom_init.c | 33 + arch/powerpc/platforms/maple/setup.c | 59

Re: + edac-cpc925-mc-platform-device-setup.patch added to -mm tree

2009-04-15 Thread Harry Ciao
code *** See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ -- Subject: edac: cpc925 MC platform device setup From: Harry Ciao

[v2 PATCH 0/3] Add CPC925 Memory Controller EDAC drivers

2009-04-12 Thread Harry Ciao
Hi Doug and Michael, This is the latest v2 patches, the 1/3 of CPC925 MC EDAC driver remains the same as before, the 2/3 has been pushed to Andrew already, and the 3/3 has integrated Michael's suggestions to add a fixup routine for the memory controller on Maple that has incorrect number of cel

[v2 PATCH 1/3] EDAC: Add CPC925 Memory Controller driver

2009-04-12 Thread Harry Ciao
Introduce IBM CPC925 EDAC driver, which makes use of ECC, CPU and HyperTransport Link error detections and corrections on the IBM CPC925 Bridge and Memory Controller. Signed-off-by: Harry Ciao --- drivers/edac/Kconfig |9 + drivers/edac/Makefile |1 + drivers/edac

[v2 PATCH 2/3] EDAC: Add edac_device_alloc_index()

2009-04-12 Thread Harry Ciao
Add edac_device_alloc_index(), because for MAPLE platform there may exist several EDAC driver modules that could make use of edac_device_ctl_info structure at the same time. The index allocation for these structures should be taken care of by EDAC core. Signed-off-by: Harry Ciao --- drivers

[v2 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-12 Thread Harry Ciao
Fixup the number of cells for the values of CPC925 Memory Controller, and setup related platform device during system booting up, against which CPC925 Memory Controller EDAC driver would be matched. Signed-off-by: Harry Ciao --- arch/powerpc/kernel/prom_init.c | 33

Re: [v1 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-03 Thread Harry Ciao
Michael Ellerman 写道: > On Fri, 2009-04-03 at 14:10 +0800, Harry Ciao wrote: > >> Setup a platform device for the CPC925 Memory Controller during system >> booting up, against which CPC925 MC EDAC driver would be matched. >> >> Signed-off-by: Harry Ciao >>

Re: [v0 PATCH 5/5] EDAC: CPC925 MC platform device setup

2009-04-03 Thread Harry Ciao
Arnd Bergmann 写道: On Thursday 02 April 2009, Harry Ciao wrote: +#ifdef CONFIG_EDAC +#define CPC925_MC_START0xf800 +#define CPC925_MC_END 0xf8ff /* sizeof 16MB */ +/* Register a platform device for CPC925 memory controller */ +static int __init

[v1 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-02 Thread Harry Ciao
Setup a platform device for the CPC925 Memory Controller during system booting up, against which CPC925 MC EDAC driver would be matched. Signed-off-by: Harry Ciao --- arch/powerpc/platforms/maple/setup.c | 47 ++ 1 files changed, 47 insertions(+), 0 deletions

[v1 PATCH 3/3] EDAC: CPC925 MC platform device setup

2009-04-02 Thread Harry Ciao
Hi, Sorry, my git-send-mail died and failed to send out 3/3 patch. Harry ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[v1 PATCH 2/3] EDAC: Add edac_device_alloc_index()

2009-04-02 Thread Harry Ciao
Add edac_device_alloc_index(), because for MAPLE platform there may exist several EDAC driver modules that could make use of edac_device_ctl_info structure at the same time. The index allocation for these structures should be taken care of by EDAC core. Signed-off-by: Harry Ciao --- drivers

[v1 PATCH 2/3] EDAC: Add edac_device_alloc_index()

2009-04-02 Thread Harry Ciao
Hi, Sorry, my git-send-mail died and failed to send out the 2/3 patch. Harry ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[v1 PATCH 1/3] EDAC: Add CPC925 Memory Controller driver

2009-04-02 Thread Harry Ciao
Introduce IBM CPC925 EDAC driver, which makes use of ECC, CPU and HyperTransport Link error detections and corrections on the IBM CPC925 Bridge and Memory Controller. Signed-off-by: Harry Ciao --- drivers/edac/Kconfig |9 + drivers/edac/Makefile |1 + drivers/edac

[v1 PATCH 0/3] Add CPC925 Memory Controller EDAC drivers

2009-04-02 Thread Harry Ciao
Hi Doug and PPC developers, This is v1 series of patches for CPC925 Memory Controller EDAC driver that works on PPC970FX. I have taken Arnd's suggestion to fold header/source/etc into one file and makes CONFIG_EDAC_CPC925 depend on CONFIG_PPC64. Please take a look and give your comments, many

Re: [v0 PATCH 5/5] EDAC: CPC925 MC platform device setup

2009-04-02 Thread Harry Ciao
Harry Ciao 写道: Arnd Bergmann 写道: On Thursday 02 April 2009, Harry Ciao wrote: +#ifdef CONFIG_EDAC +#define CPC925_MC_START0xf800 +#define CPC925_MC_END 0xf8ff /* sizeof 16MB */ +/* Register a platform device for CPC925 memory controller */ +static int __init