[PATCH 4/8] ARM: l2x0: add arm,ecc-enable property for aurora

Jan Lübbe jlu at pengutronix.de
Mon Jul 17 03:11:54 PDT 2017


On Sun, 2017-07-02 at 22:02 +0000, Chris Packham wrote:
> > > index d9650c1788f4..6316e673307a 100644
> > > --- a/Documentation/devicetree/bindings/arm/l2c2x0.txt
> > > +++ b/Documentation/devicetree/bindings/arm/l2c2x0.txt
> > > @@ -76,6 +76,8 @@ Optional properties:
> > >      specified to indicate that such transforms are precluded.
> > >    - arm,parity-enable : enable parity checking on the L2 cache
> > > (L220 or PL310).
> > >    - arm,parity-disable : disable parity checking on the L2 cache
> > > (L220 or PL310).
> > > +- arm,ecc-enable : enable ECC protection on the L2 cache
> > > +- arm,ecc-disable : disable ECC protection on the L2 cache
> > 
> > Why should this be prefixed by a vendor of "arm" ?
> > 
> 
> I was following the parity-enable example. But I'm guessing this
> should either be "aurora" or "marvell" any preference?

I'd prefer 'marvell' as they are the vendor and the prefix is already
in vendor-prefixes.txt. If nobody objects, i'll change it to
marvell,ecc-* in v2.
-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the linux-arm-kernel mailing list