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

From: William Breathitt Gray
Date: Wed Aug 31 2022 - 09:54:12 EST


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

Attachment: signature.asc
Description: PGP signature