Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1565835

Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback for GPIO chips

Show all headers | View raw

On Tue, Jan 24, 2017 at 12:11 PM, Mika Westerberg
<mika.westerberg@linux.intel.com> wrote:
> On Mon, Jan 23, 2017 at 06:11:07PM +0100, Johan Hovold wrote:
>> On Mon, Jan 23, 2017 at 03:34:34PM +0300, Mika Westerberg wrote:
>> > Currently we already have two pin configuration related callbacks
>> > available for GPIO chips .set_single_ended() and .set_debounce(). In
>> > future we expect to have even more, which does not scale well if we need
>> > to add yet another callback to the GPIO chip structure for each possible
>> > configuration parameter.
>> >
>> > Better solution is to reuse what we already have available in the
>> > generic pinconf.
>> >
>> > To support this, we introduce a new .set_config() callback for GPIO
>> > chips. The callback takes a single packed pin configuration value as
>> > parameter. This can then be extended easily beyond what is currently
>> > supported by just adding new types to the generic pinconf enum.
>> >
>> > If the GPIO driver is backed up by a pinctrl driver the GPIO driver can
>> > just assign gpiochip_generic_config() (introduced in this patch) to
>> > .set_config and that will take care configuration requests are directed
>> > to the pinctrl driver.
>> >
>> > We then convert the existing drivers over .set_config() and finally
>> > remove the .set_single_ended() and .set_debounce() callbacks.
>> >
>> > Suggested-by: Linus Walleij <linus.walleij@linaro.org>
>> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
>> For greybus and USB serial:

>>
>> Acked-by: Johan Hovold <johan@kernel.org>
>
> Thanks!
>
>> Note however that this patch fails to apply to linux-next (conflicts in
>> pinctrl as well as staging).
>
> Indeed, it does. I did the series on top of v4.10-rc5 but looks like
> there are some changes in linux-next that I missed.
>
> I'll rebase the series on top of linux-next and resend.

If the conflicts are just with the GPIO tree then the  "devel" branch
in the GPIO tree is what you should base it on.

If there are conflicts with other trees including pinctrl it should
probably be based on v4.10-rcN and end up in my face, in this case
maybe I should just make an immutable branch in the GPIO tree and
pull to both itself and pincontrol and resolve the conflicts if they are
clashing.

Yours,
Linus Walleij

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/3] pinctrl / gpio: Allow GPIO chips to use generic pinconfig Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-23 13:40 +0100
  [PATCH v2 2/3] pinctrl: Allow configuration of pins from gpiolib based drivers Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-23 13:40 +0100
  Re: [PATCH v2 0/3] pinctrl / gpio: Allow GPIO chips to use generic  pinconfig Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-01-23 15:30 +0100
  Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Johan Hovold <johan@kernel.org> - 2017-01-23 18:20 +0100
    Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-24 12:20 +0100
      Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Linus Walleij <linus.walleij@linaro.org> - 2017-01-24 14:00 +0100
        Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-24 14:10 +0100
          Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Johan Hovold <johan@kernel.org> - 2017-01-24 14:50 +0100
            Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-24 15:30 +0100
              Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-25 11:20 +0100
                Re: [PATCH v2 3/3] pinctrl / gpio: Introduce .set_config() callback  for GPIO chips Linus Walleij <linus.walleij@linaro.org> - 2017-01-26 15:40 +0100
  Re: [PATCH v2 0/3] pinctrl / gpio: Allow GPIO chips to use generic pinconfig Linus Walleij <linus.walleij@linaro.org> - 2017-01-26 15:30 +0100

csiph-web