[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMRc=Mdh9fx4-1kHOWWJa8uAqvg-8eTSLdM0A3RgVBggmMYrvg@mail.gmail.com>
Date: Tue, 20 Jun 2017 12:31:49 +0200
From: Bartosz Golaszewski <brgl@...ev.pl>
To: Thomas Gleixner <tglx@...utronix.de>,
Marc Zyngier <marc.zyngier@....com>,
Jonathan Corbet <corbet@....net>
Cc: linux-kernel@...r.kernel.org, linux-doc@...r.kernel.org,
Bartosz Golaszewski <brgl@...ev.pl>
Subject: Re: [PATCH 0/5] irq: generic-chip: resource management improvements
2017-05-31 18:06 GMT+02:00 Bartosz Golaszewski <brgl@...ev.pl>:
> This series is a follow-up to [1].
>
> Some users of irq_alloc_generic_chip() are modules which can be
> removed (e.g. gpio-ml-ioh) but have no means of freeing the allocated
> generic chip.
>
> Last time it was suggested to provide irq_destroy_generic_chip() which
> would undo both irq_remove_generic_chip() and irq_alloc_generic_chip().
>
> This functionality is provided by patch 2/5 with 1/5 adding the option
> to only free the allocated memory.
>
> Patch 3/5 exports a function that will be used in the devres variant
> of irq_alloc_generic_chip().
>
> Patches 4/5 and 5/5 add resource managed versions of
> irq_alloc_generic_chip() & irq_setup_generic_chip(). They will be used
> in drivers where applicable. Device resources are released in reverse
> order so it's ok to call devm_irq_alloc_generic_chip() and then
> devm_irq_setup_generic_chip().
>
> [1] https://lkml.org/lkml/2017/3/8/550
>
> Bartosz Golaszewski (5):
> irq: generic-chip: provide irq_free_generic_chip()
> irq: generic-chip: provide irq_destroy_generic_chip()
> irq: generic-chip: export irq_init_generic_chip() locally
> irq: generic-chip: provide devm_irq_alloc_generic_chip()
> irq: generic-chip: provide devm_irq_setup_generic_chip()
>
> Documentation/driver-model/devres.txt | 2 +
> include/linux/irq.h | 22 +++++++++
> kernel/irq/devres.c | 86 +++++++++++++++++++++++++++++++++++
> kernel/irq/generic-chip.c | 7 ++-
> kernel/irq/internals.h | 11 +++++
> 5 files changed, 124 insertions(+), 4 deletions(-)
>
> --
> 2.9.3
>
Ping for v4.13.
Is there any reason not to merge it?
Thanks,
Bartosz
Powered by blists - more mailing lists