On Sun, 2017-07-02 at 22:02 +0000, Chris Packham wrote:

quoted

quoted

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 |