On Mon, Jul 29, 2019 at 6:55 PM Paul Burton <paul.burton@xxxxxxxx> wrote: > > Lee, Arnd, > > On Wed, Jul 24, 2019 at 01:16:06PM -0400, Paul Cercueil wrote: > > device_node_to_regmap() is exactly like syscon_node_to_regmap(), but it > > does not check that the node is compatible with "syscon", and won't > > attach the first clock it finds to the regmap. > > > > The rationale behind this, is that one device node with a standard > > compatible string "foo,bar" can be covered by multiple drivers sharing a > > regmap, or by a single driver doing all the job without a regmap, but > > these are implementation details which shouldn't reflect on the > > devicetree. > > Does this looks like a good path forwards to you? Its use in this case > is described by Documentation/devicetree/bindings/timer/ingenic,tcu.txt > in patch 3 of the series. > > If you're OK with it an ack would be appreciated so I can take the > series through mips-next, otherwise I guess we'd need to go back to the > v14 approach. Yes, I guess this is ok, sorry for missing the submission earlier. Acked-by: Arnd Bergmann <arnd@xxxxxxxx>