[PATCH 5/6] arm: mvebu: add Aurora L2 Cache Controller to the DT

[Gregory CLEMENT]

On 08/27/2012 11:25 AM, Ian Molton wrote:
> On 24/08/12 17:15, Gregory CLEMENT wrote:
>> On 08/24/2012 04:56 PM, Ian Molton wrote:> On 24/08/12 11:09, Gregory CLEMENT wrote:
>>>> + compatible = "marvell,aurora-cache-with-outer";
>>> This reads wrong.
>>>
>>> perhaps aurora-with-outer-cache?
>> We can use
>> compatible = "marvell,aurora-outer-cache"
>> and
>> compatible = "marvell,aurora-system-cache"
>>
>> which is a better description of the L2 cache controller indeed.
> 
> Yes, much clearer.

OK I will change this in next version.

> 
>> I guess you meant:
>>
>> compatible = "marvell,armada370", "marvell,armada-370-xp";
>>     L2: l2-cache {										
>>         compatible = "marvell,aurora-cache";
>>         reg = <0xd0008000 0x1000>;
>>         cache-id-part = <0x100>;
>>         wt-override;
>>         with-outer="yes";
>>     };
> 
> Yes :)
> 
> -Ian
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com