* Re: [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
@ 2009-07-09 23:24 Doug Thompson
2009-07-10 15:04 ` Ira W. Snyder
0 siblings, 1 reply; 5+ messages in thread
From: Doug Thompson @ 2009-07-09 23:24 UTC (permalink / raw)
To: bluesmoke-devel, Dave Jiang, Kumar Gala, linuxppc-dev,
Ira W. Snyder
Ok, is this the one you want me to push upstream?=0A=0Adoug t=0A=0A=0A--- O=
n Thu, 7/9/09, Ira W. Snyder <iws@ovro.caltech.edu> wrote:=0A=0AFrom: Ira W=
. Snyder <iws@ovro.caltech.edu>=0ASubject: [PATCH v2] edac: mpc85xx: add su=
pport for mpc83xx memory controller=0ATo: bluesmoke-devel@lists.sourceforge=
.net, "Dave Jiang" <djiang@mvista.com>, "Kumar Gala" <galak@kernel.crashing=
.org>, linuxppc-dev@ozlabs.org=0ADate: Thursday, July 9, 2009, 1:40 PM=0A=
=0AAdd support for the Freescale MPC83xx memory controller to the existing=
=0Adriver for the Freescale MPC85xx memory controller. The only difference=
=0Abetween the two processors are in the CS_BNDS register parsing code, whi=
ch=0Ahas been changed so it will work on both processors.=0A=0AThe L2 cache=
controller does not exist on the MPC83xx, but the OF subsystem=0Awill not =
use the driver if the device is not present in the OF device tree.=0A=0ASig=
ned-off-by: Ira W. Snyder <iws@ovro.caltech.edu>=0A---=0A=0AKumar, I had to=
change the nr_pages calculation to make the math work=0Aout. I checked it =
on my board and did the math by hand for a 64GB 85xx=0Ausing 64K pages. In =
both cases, nr_pages * PAGE_SIZE comes out to the=0Acorrect value. Thanks f=
or the help.=0A=0Av1 -> v2:=0A=A0 * Use PAGE_SHIFT to parse cs_bnds regardl=
ess of board type=0A=A0 * Remove special-casing for the 83xx processor=0A=
=0A drivers/edac/Kconfig=A0 =A0 =A0 =A0 |=A0 =A0 6 +++---=0A drivers/edac/m=
pc85xx_edac.c |=A0=A0=A028 +++++++++++++++++++---------=0A 2 files changed,=
22 insertions(+), 12 deletions(-)=0A=0Adiff --git a/drivers/edac/Kconfig b=
/drivers/edac/Kconfig=0Aindex 4339b1a..78303f9 100644=0A--- a/drivers/edac/=
Kconfig=0A+++ b/drivers/edac/Kconfig=0A@@ -176,11 +176,11 @@ config EDAC_I5=
100=0A =A0=A0=A0 =A0 San Clemente MCH.=0A =0A config EDAC_MPC85XX=0A-=A0=A0=
=A0 tristate "Freescale MPC85xx"=0A-=A0=A0=A0 depends on EDAC_MM_EDAC && FS=
L_SOC && MPC85xx=0A+=A0=A0=A0 tristate "Freescale MPC83xx / MPC85xx"=0A+=A0=
=A0=A0 depends on EDAC_MM_EDAC && FSL_SOC && (PPC_83xx || MPC85xx)=0A =A0=
=A0=A0 help=0A =A0=A0=A0 =A0 Support for error detection and correction on =
the Freescale=0A-=A0=A0=A0 =A0 MPC8560, MPC8540, MPC8548=0A+=A0=A0=A0 =A0 M=
PC8349, MPC8560, MPC8540, MPC8548=0A =0A config EDAC_MV64X60=0A =A0=A0=A0 t=
ristate "Marvell MV64x60"=0Adiff --git a/drivers/edac/mpc85xx_edac.c b/driv=
ers/edac/mpc85xx_edac.c=0Aindex b4f5c63..ba2a264 100644=0A--- a/drivers/eda=
c/mpc85xx_edac.c=0A+++ b/drivers/edac/mpc85xx_edac.c=0A@@ -43,7 +43,9 @@ st=
atic u32 orig_pci_err_en;=0A #endif=0A =0A static u32 orig_l2_err_disable;=
=0A+#ifdef CONFIG_MPC85xx=0A static u32 orig_hid1[2];=0A+#endif=0A =0A /***=
********************* MC SYSFS parts ***********************************/=
=0A =0A@@ -790,19 +792,20 @@ static void __devinit mpc85xx_init_csrows(stru=
ct mem_ctl_info *mci)=0A =A0=A0=A0 =A0=A0=A0 csrow =3D &mci->csrows[index];=
=0A =A0=A0=A0 =A0=A0=A0 cs_bnds =3D in_be32(pdata->mc_vbase + MPC85XX_MC_CS=
_BNDS_0 +=0A =A0=A0=A0 =A0=A0=A0 =A0=A0=A0 =A0=A0=A0 =A0 (index * MPC85XX_M=
C_CS_BNDS_OFS));=0A-=A0=A0=A0 =A0=A0=A0 start =3D (cs_bnds & 0xfff0000) << =
4;=0A-=A0=A0=A0 =A0=A0=A0 end =3D ((cs_bnds & 0xfff) << 20);=0A-=A0=A0=A0 =
=A0=A0=A0 if (start)=0A-=A0=A0=A0 =A0=A0=A0 =A0=A0=A0 start |=3D 0xfffff;=
=0A-=A0=A0=A0 =A0=A0=A0 if (end)=0A-=A0=A0=A0 =A0=A0=A0 =A0=A0=A0 end |=3D =
0xfffff;=0A+=0A+=A0=A0=A0 =A0=A0=A0 start =3D (cs_bnds & 0xffff0000) >> 16;=
=0A+=A0=A0=A0 =A0=A0=A0 end=A0=A0=A0=3D (cs_bnds & 0x0000ffff);=0A =0A =A0=
=A0=A0 =A0=A0=A0 if (start =3D=3D end)=0A =A0=A0=A0 =A0=A0=A0 =A0=A0=A0 con=
tinue;=A0=A0=A0 /* not populated */=0A =0A+=A0=A0=A0 =A0=A0=A0 start <<=3D =
(24 - PAGE_SHIFT);=0A+=A0=A0=A0 =A0=A0=A0 end=A0=A0=A0<<=3D (24 - PAGE_SHIF=
T);=0A+=A0=A0=A0 =A0=A0=A0 end=A0 =A0 |=3D (1 << (24 - PAGE_SHIFT)) - 1;=0A=
+=0A =A0=A0=A0 =A0=A0=A0 csrow->first_page =3D start >> PAGE_SHIFT;=0A =A0=
=A0=A0 =A0=A0=A0 csrow->last_page =3D end >> PAGE_SHIFT;=0A-=A0=A0=A0 =A0=
=A0=A0 csrow->nr_pages =3D csrow->last_page + 1 - csrow->first_page;=0A+=A0=
=A0=A0 =A0=A0=A0 csrow->nr_pages =3D end + 1 - start;=0A =A0=A0=A0 =A0=A0=
=A0 csrow->grain =3D 8;=0A =A0=A0=A0 =A0=A0=A0 csrow->mtype =3D mtype;=0A =
=A0=A0=A0 =A0=A0=A0 csrow->dtype =3D DEV_UNKNOWN;=0A@@ -986,6 +989,7 @@ sta=
tic struct of_device_id mpc85xx_mc_err_of_match[] =3D {=0A =A0=A0=A0 { .com=
patible =3D "fsl,mpc8560-memory-controller", },=0A =A0=A0=A0 { .compatible =
=3D "fsl,mpc8568-memory-controller", },=0A =A0=A0=A0 { .compatible =3D "fsl=
,mpc8572-memory-controller", },=0A+=A0=A0=A0 { .compatible =3D "fsl,mpc8349=
-memory-controller", },=0A =A0=A0=A0 {},=0A };=0A =0A@@ -1001,13 +1005,13 @=
@ static struct of_platform_driver mpc85xx_mc_err_driver =3D {=0A =A0=A0=A0=
=A0=A0=A0 =A0=A0=A0},=0A };=0A =0A-=0A+#ifdef CONFIG_MPC85xx=0A static voi=
d __init mpc85xx_mc_clear_rfxe(void *data)=0A {=0A =A0=A0=A0 orig_hid1[smp_=
processor_id()] =3D mfspr(SPRN_HID1);=0A =A0=A0=A0 mtspr(SPRN_HID1, (orig_h=
id1[smp_processor_id()] & ~0x20000));=0A }=0A-=0A+#endif=0A =0A static int =
__init mpc85xx_mc_init(void)=0A {=0A@@ -1040,26 +1044,32 @@ static int __in=
it mpc85xx_mc_init(void)=0A =A0=A0=A0 =A0=A0=A0 printk(KERN_WARNING EDAC_MO=
D_STR "PCI fails to register\n");=0A #endif=0A =0A+#ifdef CONFIG_MPC85xx=0A=
=A0=A0=A0 /*=0A =A0=A0=A0=A0=A0* need to clear HID1[RFXE] to disable machi=
ne check int=0A =A0=A0=A0=A0=A0* so we can catch it=0A =A0=A0=A0=A0=A0*/=0A=
=A0=A0=A0 if (edac_op_state =3D=3D EDAC_OPSTATE_INT)=0A =A0=A0=A0 =A0=A0=
=A0 on_each_cpu(mpc85xx_mc_clear_rfxe, NULL, 0);=0A+#endif=0A =0A =A0=A0=A0=
return 0;=0A }=0A =0A module_init(mpc85xx_mc_init);=0A =0A+#ifdef CONFIG_M=
PC85xx=0A static void __exit mpc85xx_mc_restore_hid1(void *data)=0A {=0A =
=A0=A0=A0 mtspr(SPRN_HID1, orig_hid1[smp_processor_id()]);=0A }=0A+#endif=
=0A =0A static void __exit mpc85xx_mc_exit(void)=0A {=0A+#ifdef CONFIG_MPC8=
5xx=0A =A0=A0=A0 on_each_cpu(mpc85xx_mc_restore_hid1, NULL, 0);=0A+#endif=
=0A #ifdef CONFIG_PCI=0A =A0=A0=A0 of_unregister_platform_driver(&mpc85xx_p=
ci_err_driver);=0A #endif=0A-- =0A1.5.4.3=0A=0A=0A-------------------------=
-----------------------------------------------------=0AEnter the BlackBerr=
y Developer Challenge=A0 =0AThis is your chance to win up to $100,000 in pr=
izes! For a limited time, =0Avendors submitting new applications to BlackBe=
rry App World(TM) will have=0Athe opportunity to enter the BlackBerry Devel=
oper Challenge. See full prize=A0 =0Adetails at: http://p.sf.net/sfu/Challe=
nge=0A_______________________________________________=0Abluesmoke-devel mai=
ling list=0Abluesmoke-devel@lists.sourceforge.net=0Ahttps://lists.sourcefor=
ge.net/lists/listinfo/bluesmoke-devel=0A
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
2009-07-09 23:24 [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller Doug Thompson
@ 2009-07-10 15:04 ` Ira W. Snyder
2009-07-10 15:36 ` Kumar Gala
0 siblings, 1 reply; 5+ messages in thread
From: Ira W. Snyder @ 2009-07-10 15:04 UTC (permalink / raw)
To: Doug Thompson; +Cc: linuxppc-dev, bluesmoke-devel, Dave Jiang
On Thu, Jul 09, 2009 at 04:24:38PM -0700, Doug Thompson wrote:
>
> Ok, is this the one you want me to push upstream?
>
Yep, this is the finished version.
Thanks,
Ira
> doug t
>
>
> --- On Thu, 7/9/09, Ira W. Snyder <iws@ovro.caltech.edu> wrote:
>
> From: Ira W. Snyder <iws@ovro.caltech.edu>
> Subject: [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
> To: bluesmoke-devel@lists.sourceforge.net, "Dave Jiang" <djiang@mvista.com>, "Kumar Gala" <galak@kernel.crashing.org>, linuxppc-dev@ozlabs.org
> Date: Thursday, July 9, 2009, 1:40 PM
>
> Add support for the Freescale MPC83xx memory controller to the existing
> driver for the Freescale MPC85xx memory controller. The only difference
> between the two processors are in the CS_BNDS register parsing code, which
> has been changed so it will work on both processors.
>
> The L2 cache controller does not exist on the MPC83xx, but the OF subsystem
> will not use the driver if the device is not present in the OF device tree.
>
> Signed-off-by: Ira W. Snyder <iws@ovro.caltech.edu>
> ---
>
> Kumar, I had to change the nr_pages calculation to make the math work
> out. I checked it on my board and did the math by hand for a 64GB 85xx
> using 64K pages. In both cases, nr_pages * PAGE_SIZE comes out to the
> correct value. Thanks for the help.
>
> v1 -> v2:
> * Use PAGE_SHIFT to parse cs_bnds regardless of board type
> * Remove special-casing for the 83xx processor
>
> drivers/edac/Kconfig | 6 +++---
> drivers/edac/mpc85xx_edac.c | 28 +++++++++++++++++++---------
> 2 files changed, 22 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
> index 4339b1a..78303f9 100644
> --- a/drivers/edac/Kconfig
> +++ b/drivers/edac/Kconfig
> @@ -176,11 +176,11 @@ config EDAC_I5100
> San Clemente MCH.
>
> config EDAC_MPC85XX
> - tristate "Freescale MPC85xx"
> - depends on EDAC_MM_EDAC && FSL_SOC && MPC85xx
> + tristate "Freescale MPC83xx / MPC85xx"
> + depends on EDAC_MM_EDAC && FSL_SOC && (PPC_83xx || MPC85xx)
> help
> Support for error detection and correction on the Freescale
> - MPC8560, MPC8540, MPC8548
> + MPC8349, MPC8560, MPC8540, MPC8548
>
> config EDAC_MV64X60
> tristate "Marvell MV64x60"
> diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c
> index b4f5c63..ba2a264 100644
> --- a/drivers/edac/mpc85xx_edac.c
> +++ b/drivers/edac/mpc85xx_edac.c
> @@ -43,7 +43,9 @@ static u32 orig_pci_err_en;
> #endif
>
> static u32 orig_l2_err_disable;
> +#ifdef CONFIG_MPC85xx
> static u32 orig_hid1[2];
> +#endif
>
> /************************ MC SYSFS parts ***********************************/
>
> @@ -790,19 +792,20 @@ static void __devinit mpc85xx_init_csrows(struct mem_ctl_info *mci)
> csrow = &mci->csrows[index];
> cs_bnds = in_be32(pdata->mc_vbase + MPC85XX_MC_CS_BNDS_0 +
> (index * MPC85XX_MC_CS_BNDS_OFS));
> - start = (cs_bnds & 0xfff0000) << 4;
> - end = ((cs_bnds & 0xfff) << 20);
> - if (start)
> - start |= 0xfffff;
> - if (end)
> - end |= 0xfffff;
> +
> + start = (cs_bnds & 0xffff0000) >> 16;
> + end = (cs_bnds & 0x0000ffff);
>
> if (start == end)
> continue; /* not populated */
>
> + start <<= (24 - PAGE_SHIFT);
> + end <<= (24 - PAGE_SHIFT);
> + end |= (1 << (24 - PAGE_SHIFT)) - 1;
> +
> csrow->first_page = start >> PAGE_SHIFT;
> csrow->last_page = end >> PAGE_SHIFT;
> - csrow->nr_pages = csrow->last_page + 1 - csrow->first_page;
> + csrow->nr_pages = end + 1 - start;
> csrow->grain = 8;
> csrow->mtype = mtype;
> csrow->dtype = DEV_UNKNOWN;
> @@ -986,6 +989,7 @@ static struct of_device_id mpc85xx_mc_err_of_match[] = {
> { .compatible = "fsl,mpc8560-memory-controller", },
> { .compatible = "fsl,mpc8568-memory-controller", },
> { .compatible = "fsl,mpc8572-memory-controller", },
> + { .compatible = "fsl,mpc8349-memory-controller", },
> {},
> };
>
> @@ -1001,13 +1005,13 @@ static struct of_platform_driver mpc85xx_mc_err_driver = {
> },
> };
>
> -
> +#ifdef CONFIG_MPC85xx
> static void __init mpc85xx_mc_clear_rfxe(void *data)
> {
> orig_hid1[smp_processor_id()] = mfspr(SPRN_HID1);
> mtspr(SPRN_HID1, (orig_hid1[smp_processor_id()] & ~0x20000));
> }
> -
> +#endif
>
> static int __init mpc85xx_mc_init(void)
> {
> @@ -1040,26 +1044,32 @@ static int __init mpc85xx_mc_init(void)
> printk(KERN_WARNING EDAC_MOD_STR "PCI fails to register\n");
> #endif
>
> +#ifdef CONFIG_MPC85xx
> /*
> * need to clear HID1[RFXE] to disable machine check int
> * so we can catch it
> */
> if (edac_op_state == EDAC_OPSTATE_INT)
> on_each_cpu(mpc85xx_mc_clear_rfxe, NULL, 0);
> +#endif
>
> return 0;
> }
>
> module_init(mpc85xx_mc_init);
>
> +#ifdef CONFIG_MPC85xx
> static void __exit mpc85xx_mc_restore_hid1(void *data)
> {
> mtspr(SPRN_HID1, orig_hid1[smp_processor_id()]);
> }
> +#endif
>
> static void __exit mpc85xx_mc_exit(void)
> {
> +#ifdef CONFIG_MPC85xx
> on_each_cpu(mpc85xx_mc_restore_hid1, NULL, 0);
> +#endif
> #ifdef CONFIG_PCI
> of_unregister_platform_driver(&mpc85xx_pci_err_driver);
> #endif
> --
> 1.5.4.3
>
>
> ------------------------------------------------------------------------------
> Enter the BlackBerry Developer Challenge
> This is your chance to win up to $100,000 in prizes! For a limited time,
> vendors submitting new applications to BlackBerry App World(TM) will have
> the opportunity to enter the BlackBerry Developer Challenge. See full prize
> details at: http://p.sf.net/sfu/Challenge
> _______________________________________________
> bluesmoke-devel mailing list
> bluesmoke-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/bluesmoke-devel
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
2009-07-10 15:04 ` Ira W. Snyder
@ 2009-07-10 15:36 ` Kumar Gala
2009-07-10 17:17 ` Ira W. Snyder
0 siblings, 1 reply; 5+ messages in thread
From: Kumar Gala @ 2009-07-10 15:36 UTC (permalink / raw)
To: Ira W. Snyder; +Cc: linuxppc-dev, Doug Thompson, bluesmoke-devel, Dave Jiang
On Jul 10, 2009, at 10:04 AM, Ira W. Snyder wrote:
> On Thu, Jul 09, 2009 at 04:24:38PM -0700, Doug Thompson wrote:
>>
>> Ok, is this the one you want me to push upstream?
>>
>
> Yep, this is the finished version.
>
> Thanks,
> Ira
>
>> doug t
>
is that for .31 or .32? If .31 I'm fine. If for .32 I still have
some things I like to deal with. (like the other 85xx specific bits)
- k
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
2009-07-10 15:36 ` Kumar Gala
@ 2009-07-10 17:17 ` Ira W. Snyder
0 siblings, 0 replies; 5+ messages in thread
From: Ira W. Snyder @ 2009-07-10 17:17 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev, Doug Thompson, bluesmoke-devel, Dave Jiang
On Fri, Jul 10, 2009 at 10:36:20AM -0500, Kumar Gala wrote:
>
> On Jul 10, 2009, at 10:04 AM, Ira W. Snyder wrote:
>
>> On Thu, Jul 09, 2009 at 04:24:38PM -0700, Doug Thompson wrote:
>>>
>>> Ok, is this the one you want me to push upstream?
>>>
>>
>> Yep, this is the finished version.
>>
>> Thanks,
>> Ira
>>
>>> doug t
>>
>
> is that for .31 or .32? If .31 I'm fine. If for .32 I still have some
> things I like to deal with. (like the other 85xx specific bits)
>
.32 is fine for me. This is a new feature, not really a bugfix. I can
maintain it in my (growing) set of patches against mainline until it
gets merged.
Ira
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller
@ 2009-07-09 19:40 Ira W. Snyder
0 siblings, 0 replies; 5+ messages in thread
From: Ira W. Snyder @ 2009-07-09 19:40 UTC (permalink / raw)
To: bluesmoke-devel, Dave Jiang, Kumar Gala, linuxppc-dev
Add support for the Freescale MPC83xx memory controller to the existing
driver for the Freescale MPC85xx memory controller. The only difference
between the two processors are in the CS_BNDS register parsing code, which
has been changed so it will work on both processors.
The L2 cache controller does not exist on the MPC83xx, but the OF subsystem
will not use the driver if the device is not present in the OF device tree.
Signed-off-by: Ira W. Snyder <iws@ovro.caltech.edu>
---
Kumar, I had to change the nr_pages calculation to make the math work
out. I checked it on my board and did the math by hand for a 64GB 85xx
using 64K pages. In both cases, nr_pages * PAGE_SIZE comes out to the
correct value. Thanks for the help.
v1 -> v2:
* Use PAGE_SHIFT to parse cs_bnds regardless of board type
* Remove special-casing for the 83xx processor
drivers/edac/Kconfig | 6 +++---
drivers/edac/mpc85xx_edac.c | 28 +++++++++++++++++++---------
2 files changed, 22 insertions(+), 12 deletions(-)
diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
index 4339b1a..78303f9 100644
--- a/drivers/edac/Kconfig
+++ b/drivers/edac/Kconfig
@@ -176,11 +176,11 @@ config EDAC_I5100
San Clemente MCH.
config EDAC_MPC85XX
- tristate "Freescale MPC85xx"
- depends on EDAC_MM_EDAC && FSL_SOC && MPC85xx
+ tristate "Freescale MPC83xx / MPC85xx"
+ depends on EDAC_MM_EDAC && FSL_SOC && (PPC_83xx || MPC85xx)
help
Support for error detection and correction on the Freescale
- MPC8560, MPC8540, MPC8548
+ MPC8349, MPC8560, MPC8540, MPC8548
config EDAC_MV64X60
tristate "Marvell MV64x60"
diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c
index b4f5c63..ba2a264 100644
--- a/drivers/edac/mpc85xx_edac.c
+++ b/drivers/edac/mpc85xx_edac.c
@@ -43,7 +43,9 @@ static u32 orig_pci_err_en;
#endif
static u32 orig_l2_err_disable;
+#ifdef CONFIG_MPC85xx
static u32 orig_hid1[2];
+#endif
/************************ MC SYSFS parts ***********************************/
@@ -790,19 +792,20 @@ static void __devinit mpc85xx_init_csrows(struct mem_ctl_info *mci)
csrow = &mci->csrows[index];
cs_bnds = in_be32(pdata->mc_vbase + MPC85XX_MC_CS_BNDS_0 +
(index * MPC85XX_MC_CS_BNDS_OFS));
- start = (cs_bnds & 0xfff0000) << 4;
- end = ((cs_bnds & 0xfff) << 20);
- if (start)
- start |= 0xfffff;
- if (end)
- end |= 0xfffff;
+
+ start = (cs_bnds & 0xffff0000) >> 16;
+ end = (cs_bnds & 0x0000ffff);
if (start == end)
continue; /* not populated */
+ start <<= (24 - PAGE_SHIFT);
+ end <<= (24 - PAGE_SHIFT);
+ end |= (1 << (24 - PAGE_SHIFT)) - 1;
+
csrow->first_page = start >> PAGE_SHIFT;
csrow->last_page = end >> PAGE_SHIFT;
- csrow->nr_pages = csrow->last_page + 1 - csrow->first_page;
+ csrow->nr_pages = end + 1 - start;
csrow->grain = 8;
csrow->mtype = mtype;
csrow->dtype = DEV_UNKNOWN;
@@ -986,6 +989,7 @@ static struct of_device_id mpc85xx_mc_err_of_match[] = {
{ .compatible = "fsl,mpc8560-memory-controller", },
{ .compatible = "fsl,mpc8568-memory-controller", },
{ .compatible = "fsl,mpc8572-memory-controller", },
+ { .compatible = "fsl,mpc8349-memory-controller", },
{},
};
@@ -1001,13 +1005,13 @@ static struct of_platform_driver mpc85xx_mc_err_driver = {
},
};
-
+#ifdef CONFIG_MPC85xx
static void __init mpc85xx_mc_clear_rfxe(void *data)
{
orig_hid1[smp_processor_id()] = mfspr(SPRN_HID1);
mtspr(SPRN_HID1, (orig_hid1[smp_processor_id()] & ~0x20000));
}
-
+#endif
static int __init mpc85xx_mc_init(void)
{
@@ -1040,26 +1044,32 @@ static int __init mpc85xx_mc_init(void)
printk(KERN_WARNING EDAC_MOD_STR "PCI fails to register\n");
#endif
+#ifdef CONFIG_MPC85xx
/*
* need to clear HID1[RFXE] to disable machine check int
* so we can catch it
*/
if (edac_op_state == EDAC_OPSTATE_INT)
on_each_cpu(mpc85xx_mc_clear_rfxe, NULL, 0);
+#endif
return 0;
}
module_init(mpc85xx_mc_init);
+#ifdef CONFIG_MPC85xx
static void __exit mpc85xx_mc_restore_hid1(void *data)
{
mtspr(SPRN_HID1, orig_hid1[smp_processor_id()]);
}
+#endif
static void __exit mpc85xx_mc_exit(void)
{
+#ifdef CONFIG_MPC85xx
on_each_cpu(mpc85xx_mc_restore_hid1, NULL, 0);
+#endif
#ifdef CONFIG_PCI
of_unregister_platform_driver(&mpc85xx_pci_err_driver);
#endif
--
1.5.4.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-07-10 17:17 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-09 23:24 [PATCH v2] edac: mpc85xx: add support for mpc83xx memory controller Doug Thompson
2009-07-10 15:04 ` Ira W. Snyder
2009-07-10 15:36 ` Kumar Gala
2009-07-10 17:17 ` Ira W. Snyder
-- strict thread matches above, loose matches on Subject: below --
2009-07-09 19:40 Ira W. Snyder
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).