Re: [PATCH] edac, highbank: remove dependency on ARCH_HIGHBANK

[Rob Herring <robherring2@gmail.com>]

On 11/11/2013 05:09 AM, Robert Richter wrote:
> On 05.11.13 21:25:52, Rob Herring wrote:
>> From: Rob Herring <rob.herring@calxeda.com>
>>
>> In order to enable on arm64 and improve the build coverage, remove the
>> dependency on ARCH_HIGHBANK.
>>
>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>> Cc: Doug Thompson <dougthompson@xmission.com>
>> Cc: Robert Richter <rric@kernel.org>
>> Cc: linux-edac@vger.kernel.org
> 
> Rob, I will apply your patch with the changes we discussed if this is
> ok for you.

Thanks. It should be (ARM || ARM64 || COMPILE_TEST) rather than
ARM_MULTI_V7. The L2 ECC driver can be ARM only.

Rob

> 
> -Robert
> 
>> ---
>>  drivers/edac/Kconfig | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
>> index 878f090..fc8aaaf 100644
>> --- a/drivers/edac/Kconfig
>> +++ b/drivers/edac/Kconfig
>> @@ -328,14 +328,14 @@ config EDAC_TILE
>>  
>>  config EDAC_HIGHBANK_MC
>>  	tristate "Highbank Memory Controller"
>> -	depends on EDAC_MM_EDAC && ARCH_HIGHBANK
>> +	depends on EDAC_MM_EDAC
>>  	help
>>  	  Support for error detection and correction on the
>>  	  Calxeda Highbank memory controller.
>>  
>>  config EDAC_HIGHBANK_L2
>>  	tristate "Highbank L2 Cache"
>> -	depends on EDAC_MM_EDAC && ARCH_HIGHBANK
>> +	depends on EDAC_MM_EDAC
>>  	help
>>  	  Support for error detection and correction on the
>>  	  Calxeda Highbank memory controller.
>> -- 
>> 1.8.1.2
>>