Re: [PATCH v4] gpio: Driver for SYSCON-based GPIOs

[Alexander Shiyan <shc_work@xxxxxxx>]

Среда,  5 марта 2014, 9:11 +08:00 от Linus Walleij <linus.walleij@xxxxxxxxxx>:
> On Tue, Feb 25, 2014 at 11:09 PM, Alexander Shiyan <shc_work@xxxxxxx> wrote:
> > On Tue, 18 Feb 2014 15:25:20 -0800
> > Olof Johansson <olof@xxxxxxxxx> 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.

---
��.n��������+%������w��{.n�����{��
b���ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f