Среда, 5 марта 2014, 9:11 +08:00 от Linus Walleij <[email protected]>: > On Tue, Feb 25, 2014 at 11:09 PM, Alexander Shiyan <[email protected]> wrote: > > On Tue, 18 Feb 2014 15:25:20 -0800 > > Olof Johansson <[email protected]> wrote: > > >> If we have more drivers like these down the road we can make a common > >> shared > >> binding, but until then I don't think there's much point in it. > > > > Is this proposed to remove the generic implementation? > > Linus, what is your opinion on this? > > That is better, because it relieves me from the pain about this > driver. (Sorry for slow answers.) > > So instead of this: > > +- compatible: Should contain one of the following: > + - "clps711x,syscon-gpio-mctrl": For ARM CLPS711X SYSFLG1 MCTRL GPIOs. > > Something more like > > "cirrus,clps711x-syscon-gpio" > > I guess?
I think that Olof meant to completely remove the concept of "syscon" from the DT description. This can be done by changing the compatibility string with "cirrus,clps711x-mctrl-gpio" for my case and hide "syscon" in the internal structure of the driver to use the syscon_regmap_lookup_by_compatible() function. If so, then I'll make the next version. ---
