Re: [PATCH v3 01/14] Documentation: dt/bindings: Document pinctrl-ingenic

Paul Cercueil
2017-02-21T11:20:10Z

Le 2017-02-20 14:56, Linus Walleij a écrit :
> On Thu, Feb 9, 2017 at 6:28 PM, Paul Cercueil <paul@crapouillou.net>
> wrote:
>
>> I was thinking that instead of having one pinctrl-ingenic instance
>> covering
>> 0x600 of register space, and 6 instances of gpio-ingenic having 0x100
>> each,
>> I could just have 6 instances of pinctrl-ingenic, each one with an
>> instance
>> of gpio-ingenic declared as a sub-node, each handling just 0x100 of
>> memory
>> space.
>
> My head is spinning, but I think I get it. What is wrong with the
> solution
> I proposed with one pin control instance covering the whole 0x600 and
> with 6
> subnodes of GPIO?
>
> The GPIO nodes do not even have to have an address range associated
> with
> them you know, that can be distributed out with regmap code accessing
> the parent regmap.

OK, but then each GPIO chip 'X' still need to know its offset in the
register
area, which is (pinctrl_base + X * 0x100).
What's the best way to pass that info to the driver? (I assume it's not
with
a custom DT binding...).

Regards,
-Paul