On Mon, Aug 29, 2016 at 12:21 PM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Mon, Aug 29, 2016 at 4:06 PM, Rob Herring <robh+dt@xxxxxxxxxx> wrote: >> [Me] >>> That is very close to how >>> Documentation/devicetree/bindings/bus/imx-weim.txt >>> drivers/bus/imx-weim.c >>> does things, just with the additional property arrays. >> >> Maybe so, but I'm trying to bring some consistency here, so I'd >> suggest looking at newer ones like atmel IIRC. > > The Uniphier bindings also look like this and it's the most recent > bus driver ... > Documentation/devicetree/bindings/bus/uniphier-system-bus.txt > > But OK it's probably best to follow an example so I get it the way > you prefer. Which binding and driver is that? > > There's no Atmel stuff in Documentation/devicetree/bindings/bus/ > nor drivers/bus from what I can tell, so just point out what you're > referring to here and I'll do my best to follow that example. Argg, that's because it is in a different spot: Documentation/devicetree/bindings/memory-controllers/atmel,ebi.txt Time to clean things up... bindings/bus is supposed to be internal, on-chip buses. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html