mbox series

[v3,0/5] gpiolib: switch to fwnode in the core

Message ID 20210304201253.14652-1-andriy.shevchenko@linux.intel.com
Headers show
Series gpiolib: switch to fwnode in the core | expand

Message

Andy Shevchenko March 4, 2021, 8:12 p.m. UTC
GPIO library uses of_node and fwnode in the core in non-unified way.
The series cleans this up and improves IRQ domain creation for non-OF cases
where currently the names of the domain are 'unknown'.

This has been tested on Intel Galileo Gen 2.

In v3:
- fix subtle bug in gpiod_count
- make irq_domain_add_simple() static inline (Marc)

In v2:
- added a new patch due to functionality in irq_comain_add_simple() (Linus)
- tagged patches 2-4 (Linus)
- Cc'ed to Rafael

Andy Shevchenko (5):
  irqdomain: Introduce irq_domain_create_simple() API
  gpiolib: Unify the checks on fwnode type
  gpiolib: Move of_node operations to gpiolib-of and correct fwnode use
  gpiolib: Introduce acpi_gpio_dev_init() and call it from core
  gpiolib: Reuse device's fwnode to create IRQ domain

 Documentation/core-api/irq/irq-domain.rst | 22 ++++----
 drivers/gpio/gpiolib-acpi.c               |  7 +++
 drivers/gpio/gpiolib-acpi.h               |  4 ++
 drivers/gpio/gpiolib-of.c                 |  6 ++-
 drivers/gpio/gpiolib.c                    | 66 +++++++++--------------
 include/linux/irqdomain.h                 | 19 +++++--
 kernel/irq/irqdomain.c                    | 20 +++----
 7 files changed, 77 insertions(+), 67 deletions(-)

Comments

Rafael J. Wysocki March 8, 2021, 6:22 p.m. UTC | #1
On Thu, Mar 4, 2021 at 9:13 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>

> GPIO library uses of_node and fwnode in the core in non-unified way.

> The series cleans this up and improves IRQ domain creation for non-OF cases

> where currently the names of the domain are 'unknown'.

>

> This has been tested on Intel Galileo Gen 2.

>

> In v3:

> - fix subtle bug in gpiod_count

> - make irq_domain_add_simple() static inline (Marc)

>

> In v2:

> - added a new patch due to functionality in irq_comain_add_simple() (Linus)

> - tagged patches 2-4 (Linus)

> - Cc'ed to Rafael

>

> Andy Shevchenko (5):

>   irqdomain: Introduce irq_domain_create_simple() API

>   gpiolib: Unify the checks on fwnode type

>   gpiolib: Move of_node operations to gpiolib-of and correct fwnode use

>   gpiolib: Introduce acpi_gpio_dev_init() and call it from core

>   gpiolib: Reuse device's fwnode to create IRQ domain


[1-4/5] applied as 5.13 material and I have a minor comment regarding
the last patch (will send separately).

Thanks!
Bartosz Golaszewski March 8, 2021, 7:22 p.m. UTC | #2
On Mon, Mar 8, 2021 at 7:22 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
>

> On Thu, Mar 4, 2021 at 9:13 PM Andy Shevchenko

> <andriy.shevchenko@linux.intel.com> wrote:

> >

> > GPIO library uses of_node and fwnode in the core in non-unified way.

> > The series cleans this up and improves IRQ domain creation for non-OF cases

> > where currently the names of the domain are 'unknown'.

> >

> > This has been tested on Intel Galileo Gen 2.

> >

> > In v3:

> > - fix subtle bug in gpiod_count

> > - make irq_domain_add_simple() static inline (Marc)

> >

> > In v2:

> > - added a new patch due to functionality in irq_comain_add_simple() (Linus)

> > - tagged patches 2-4 (Linus)

> > - Cc'ed to Rafael

> >

> > Andy Shevchenko (5):

> >   irqdomain: Introduce irq_domain_create_simple() API

> >   gpiolib: Unify the checks on fwnode type

> >   gpiolib: Move of_node operations to gpiolib-of and correct fwnode use

> >   gpiolib: Introduce acpi_gpio_dev_init() and call it from core

> >   gpiolib: Reuse device's fwnode to create IRQ domain

>

> [1-4/5] applied as 5.13 material and I have a minor comment regarding

> the last patch (will send separately).

>

> Thanks!


Hi Rafael!

AFAICT this should go through the GPIO tree as usual. Any reason for
you to pick these patches this time?

Bartosz
Rafael J. Wysocki March 8, 2021, 7:26 p.m. UTC | #3
On Mon, Mar 8, 2021 at 8:23 PM Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
>

> On Mon, Mar 8, 2021 at 7:22 PM Rafael J. Wysocki <rafael@kernel.org> wrote:

> >

> > On Thu, Mar 4, 2021 at 9:13 PM Andy Shevchenko

> > <andriy.shevchenko@linux.intel.com> wrote:

> > >

> > > GPIO library uses of_node and fwnode in the core in non-unified way.

> > > The series cleans this up and improves IRQ domain creation for non-OF cases

> > > where currently the names of the domain are 'unknown'.

> > >

> > > This has been tested on Intel Galileo Gen 2.

> > >

> > > In v3:

> > > - fix subtle bug in gpiod_count

> > > - make irq_domain_add_simple() static inline (Marc)

> > >

> > > In v2:

> > > - added a new patch due to functionality in irq_comain_add_simple() (Linus)

> > > - tagged patches 2-4 (Linus)

> > > - Cc'ed to Rafael

> > >

> > > Andy Shevchenko (5):

> > >   irqdomain: Introduce irq_domain_create_simple() API

> > >   gpiolib: Unify the checks on fwnode type

> > >   gpiolib: Move of_node operations to gpiolib-of and correct fwnode use

> > >   gpiolib: Introduce acpi_gpio_dev_init() and call it from core

> > >   gpiolib: Reuse device's fwnode to create IRQ domain

> >

> > [1-4/5] applied as 5.13 material and I have a minor comment regarding

> > the last patch (will send separately).

> >

> > Thanks!

>

> Hi Rafael!

>

> AFAICT this should go through the GPIO tree as usual. Any reason for

> you to pick these patches this time?


My impression was that Andy wanted me to take them.

However, if you'd rather take care of them yourself, there you go!

I'll drop them now and assume that they will be routed through the GPIO tree.

Thanks!
Bartosz Golaszewski March 8, 2021, 7:29 p.m. UTC | #4
On Mon, Mar 8, 2021 at 8:26 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
>
> On Mon, Mar 8, 2021 at 8:23 PM Bartosz Golaszewski
> <bgolaszewski@baylibre.com> wrote:
> >
> > On Mon, Mar 8, 2021 at 7:22 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
> > >
> > > On Thu, Mar 4, 2021 at 9:13 PM Andy Shevchenko
> > > <andriy.shevchenko@linux.intel.com> wrote:
> > > >
> > > > GPIO library uses of_node and fwnode in the core in non-unified way.
> > > > The series cleans this up and improves IRQ domain creation for non-OF cases
> > > > where currently the names of the domain are 'unknown'.
> > > >
> > > > This has been tested on Intel Galileo Gen 2.
> > > >
> > > > In v3:
> > > > - fix subtle bug in gpiod_count
> > > > - make irq_domain_add_simple() static inline (Marc)
> > > >
> > > > In v2:
> > > > - added a new patch due to functionality in irq_comain_add_simple() (Linus)
> > > > - tagged patches 2-4 (Linus)
> > > > - Cc'ed to Rafael
> > > >
> > > > Andy Shevchenko (5):
> > > >   irqdomain: Introduce irq_domain_create_simple() API
> > > >   gpiolib: Unify the checks on fwnode type
> > > >   gpiolib: Move of_node operations to gpiolib-of and correct fwnode use
> > > >   gpiolib: Introduce acpi_gpio_dev_init() and call it from core
> > > >   gpiolib: Reuse device's fwnode to create IRQ domain
> > >
> > > [1-4/5] applied as 5.13 material and I have a minor comment regarding
> > > the last patch (will send separately).
> > >
> > > Thanks!
> >
> > Hi Rafael!
> >
> > AFAICT this should go through the GPIO tree as usual. Any reason for
> > you to pick these patches this time?
>
> My impression was that Andy wanted me to take them.
>
> However, if you'd rather take care of them yourself, there you go!
>
> I'll drop them now and assume that they will be routed through the GPIO tree.
>
> Thanks!

They touch a lot of core GPIO code and are likely to conflict if any
other changes show up this release cycle. I'd rather take them through
the usual channel. Thanks!

Bartosz
Andy Shevchenko March 8, 2021, 7:52 p.m. UTC | #5
On Mon, Mar 08, 2021 at 09:36:52PM +0200, Andy Shevchenko wrote:
> On Mon, Mar 08, 2021 at 08:29:27PM +0100, Bartosz Golaszewski wrote:

> > On Mon, Mar 8, 2021 at 8:26 PM Rafael J. Wysocki <rafael@kernel.org> wrote:

> > > On Mon, Mar 8, 2021 at 8:23 PM Bartosz Golaszewski

> > > <bgolaszewski@baylibre.com> wrote:

> 

> ...

> 

> > > My impression was that Andy wanted me to take them.

> > >

> > > However, if you'd rather take care of them yourself, there you go!

> > >

> > > I'll drop them now and assume that they will be routed through the GPIO tree.

> > >

> > > Thanks!

> > 

> > They touch a lot of core GPIO code and are likely to conflict if any

> > other changes show up this release cycle. I'd rather take them through

> > the usual channel. Thanks!

> 

> Since now we have v4 based on Rafael's bleeding-edge, what do you want me to

> do? Resend a v5 with all patches included?


I have decided to resend as usually it's better for maintainers.

But it appears I was too quick to miss Rafael's review tag / comments.

So, I will send v6 with those included.

-- 
With Best Regards,
Andy Shevchenko
Bartosz Golaszewski March 9, 2021, 8:19 a.m. UTC | #6
On Mon, Mar 8, 2021 at 8:52 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> On Mon, Mar 08, 2021 at 09:36:52PM +0200, Andy Shevchenko wrote:
> > On Mon, Mar 08, 2021 at 08:29:27PM +0100, Bartosz Golaszewski wrote:
> > > On Mon, Mar 8, 2021 at 8:26 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
> > > > On Mon, Mar 8, 2021 at 8:23 PM Bartosz Golaszewski
> > > > <bgolaszewski@baylibre.com> wrote:
> >
> > ...
> >
> > > > My impression was that Andy wanted me to take them.
> > > >
> > > > However, if you'd rather take care of them yourself, there you go!
> > > >
> > > > I'll drop them now and assume that they will be routed through the GPIO tree.
> > > >
> > > > Thanks!
> > >
> > > They touch a lot of core GPIO code and are likely to conflict if any
> > > other changes show up this release cycle. I'd rather take them through
> > > the usual channel. Thanks!
> >
> > Since now we have v4 based on Rafael's bleeding-edge, what do you want me to
> > do? Resend a v5 with all patches included?
>
> I have decided to resend as usually it's better for maintainers.
>
> But it appears I was too quick to miss Rafael's review tag / comments.
>
> So, I will send v6 with those included.
>
> --
> With Best Regards,
> Andy Shevchenko
>
>

Does this series depend on patches already in Rafael's tree? If so,
maybe Rafael can provide me with an immutable tag to merge in?

Bartosz
Andy Shevchenko March 9, 2021, 9:41 a.m. UTC | #7
On Tue, Mar 09, 2021 at 09:19:19AM +0100, Bartosz Golaszewski wrote:
> On Mon, Mar 8, 2021 at 8:52 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Mon, Mar 08, 2021 at 09:36:52PM +0200, Andy Shevchenko wrote:

...

> > So, I will send v6 with those included.
> 
> Does this series depend on patches already in Rafael's tree? If so,
> maybe Rafael can provide me with an immutable tag to merge in?

Not anymore since v5.12-rc2 has a necessary fix.

In any case I have sent a v6. It should be clean to apply on top of your for-next.