[PATCH] powerpc/85xx: fix memory controller compatible for edac

Xie Shaohui-B21989 B21989 at freescale.com
Mon Aug 1 12:44:53 EST 2011


>From: Grant Likely [mailto:glikely at secretlab.ca] On Behalf Of Grant Likely
>Sent: Sunday, July 31, 2011 12:03 PM
>To: Xie Shaohui-B21989
>Cc: linuxppc-dev at lists.ozlabs.org; Gala Kumar-B11780; mm-
>commits at vger.kernel.org; avorontsov at mvista.com; davem at davemloft.net;
>akpm at linux-foundation.org
>Subject: Re: [PATCH] powerpc/85xx: fix memory controller compatible for
>edac
>
>On Tue, Jul 26, 2011 at 01:46:33PM +0800, Shaohui Xie wrote:
>> compatible in dts has been changed, so driver need to update accordingly.
>>
>> Signed-off-by: Shaohui Xie <Shaohui.Xie at freescale.com>
>> ---
>> apply for
>> http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git
>> 'next' branch.
>>
>>  drivers/edac/mpc85xx_edac.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c
>> index 13f6cc5..94c064a 100644
>> --- a/drivers/edac/mpc85xx_edac.c
>> +++ b/drivers/edac/mpc85xx_edac.c
>> @@ -1253,7 +1253,7 @@ static struct of_device_id
>mpc85xx_mc_err_of_match[] = {
>>  	{ .compatible = "fsl,p1020-memory-controller", },
>>  	{ .compatible = "fsl,p1021-memory-controller", },
>>  	{ .compatible = "fsl,p2020-memory-controller", },
>> -	{ .compatible = "fsl,p4080-memory-controller", },
>> +	{ .compatible = "fsl,qoriq-memory-controller", },
>
>Are there any implementations in the field that depend on the p4080 value?
>

[Xie Shaohui] The 'p4080' is introduced by commit cd1542c8197

edac: mpc85xx: add support for new MPCxxx/Pxxxx EDAC controllers

Simply add proper IDs into the device table.


My patch intended for fix the compliance of dts and driver.



Best Regards, 
Shaohui Xie



More information about the Linuxppc-dev mailing list