On Thursday 30 October 2014 10:02:25 Philipp Zabel wrote: > Since the reset controller driver accesses registers solely through the > syscon regmap, I'd prefer to keep with the device tree control graph > concept and make the reset-controller nodes children of the syscon > nodes. I've brought this up before: https://lkml.org/lkml/2014/5/27/422, > and I think this is another case where child node support for syscon > makes sense: > > infracfg: syscon@10001000 { > compatible = "mediatek,mt8135-infracfg", "syscon"; > reg = <0 0x10001000 0 0x1000>; > > infrarst: reset-controller@30 { > #reset-cells = <1>; > compatible = "mediatek,mt8135-infracfg-reset", "mediatek,reset"; > reg = <0x30 0x8>; > }; > }; > Yes, this looks much better to me too. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html