Re: [PATCH 0/6] isa: Ensure number of irq matches number of base

From: Bartosz Golaszewski
Date: Wed Aug 31 2022 - 10:00:45 EST


On Wed, Aug 31, 2022 at 3:54 PM William Breathitt Gray
<william.gray@xxxxxxxxxx> wrote:
>
> On Wed, Aug 31, 2022 at 02:35:51PM +0200, Bartosz Golaszewski wrote:
> > On Fri, Aug 19, 2022 at 1:11 AM William Breathitt Gray
> > <william.gray@xxxxxxxxxx> wrote:
> > >
> > > Several ISA drivers support IRQ and call devm_request_irq() in their
> > > device probe callbacks. These drivers typically provide an "irq" array
> > > module parameter, which matches with the respective "base" array module
> > > parameter, to specify what IRQ lines are used for each device. To reduce
> > > code repetition, a module_isa_driver_with_irq helper macro is introduced
> > > providing a check ensuring that the number of "irq" passed to the module
> > > matches with the respective number of "base". The relevant ISA drivers
> > > are updated accordingly to utilize the new module_isa_driver_with_irq
> > > macro.
> > >
> > > William Breathitt Gray (6):
> > > isa: Introduce the module_isa_driver_with_irq helper macro
> > > counter: 104-quad-8: Ensure number of irq matches number of base
> > > gpio: 104-dio-48e: Ensure number of irq matches number of base
> > > gpio: 104-idi-48: Ensure number of irq matches number of base
> > > gpio: 104-idio-16: Ensure number of irq matches number of base
> > > gpio: ws16c48: Ensure number of irq matches number of base
> > >
> > > drivers/counter/104-quad-8.c | 5 ++--
> > > drivers/gpio/gpio-104-dio-48e.c | 5 ++--
> > > drivers/gpio/gpio-104-idi-48.c | 5 ++--
> > > drivers/gpio/gpio-104-idio-16.c | 5 ++--
> > > drivers/gpio/gpio-ws16c48.c | 5 ++--
> > > include/linux/isa.h | 52 ++++++++++++++++++++++++++-------
> > > 6 files changed, 57 insertions(+), 20 deletions(-)
> > >
> > >
> > > base-commit: 568035b01cfb107af8d2e4bd2fb9aea22cf5b868
> > > --
> > > 2.37.2
> > >
> >
> > Looks good to me, do you send your PRs directly to Linus? Do you want
> > me to take this through the GPIO tree?
> >
> > Bart
>
> Hi Bart,
>
> Would you take this through the GPIO tree? I'm planning on submitting
> some more patches later for these GPIO drivers so it will be convenient
> to keep all these changes within the same tree.
>
> Thanks,
>
> William Breathitt Gray

Sure, just leave your Acks on the first two patches.

Bart