Messages in this thread | | | From | Arnd Bergmann <> | Subject | Re: [PATCH v3 5/7] ARM: brcmstb: add misc. DT bindings for brcm,brcmstb-* | Date | Wed, 15 Jan 2014 18:15:10 +0100 |
| |
On Wednesday 15 January 2014 17:11:55 Mark Rutland wrote: > > + > > +Further, a node with the following compatible string shall be defined: > > + > > + - compatible: "brcm,brcmstb-gen-ctrl-v1" > > It's probably better to say a brcmstb-gen-ctrl node (described below) > should be present, or you'll have two places to update the compatible > strings for each new variant...
See my other reply on this node. The previous patch version had a different approach to these registers, and I suggested doing yet another one and using syscon.
Arnd
| |