[PATCH 3/3] ARM: l2x0: Add OF based initialization

Rob Herring robherring2 at gmail.com
Wed Jun 1 15:01:03 EDT 2011

On 06/01/2011 01:40 PM, Olof Johansson wrote:
> On Wed, Jun 1, 2011 at 9:37 AM, Rob Herring<robherring2 at gmail.com>  wrote:
>> From: Rob Herring<rob.herring at calxeda.com>
>> This adds probing for pl310 cache controller via device tree. An example
>> binding looks like this:
>> L2: l2-cache {
>>         compatible = "arm,pl310-cache", "cache";
> "cache" is too generic to be useful. "arm,pl2x0-cache" would be a more
> meaningful fallback, I think?

I agree. This is what ePAPR says should be present for caches along with 
a more specific string.

Based on the prior discussion on pmu naming, it should be the specific 
model. There is no such thing as a pl2x0. The models of L2 controllers 
the cache-l2x0.c code supports are:


And my compatible strings reflect that.

> (Same comment as for 1/3: The bindings need to be documented. Same
> applies to 2/3, obviousy).

Will do.


More information about the linux-arm-kernel mailing list