Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210520163751.27325-1-maz@kernel.org>
In-Reply-To: <20210520163751.27325-1-maz@kernel.org>
From:   Linus Walleij <linus.walleij@linaro.org>
Date:   Fri, 21 May 2021 01:33:58 +0200
Message-ID: <CACRpkdZpn3x-P9rsoMUE0uG_19aG5jHWvv7FD7srtmW92Q1oag@mail.gmail.com>
Subject: Re: [PATCH 00/39] irqdomain: Simplify interrupt handling
To:     Marc Zyngier <maz@kernel.org>
Cc:     linux-kernel <linux-kernel@vger.kernel.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Michael Ellerman <mpe@ellerman.id.au>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Paul Mackerras <paulus@samba.org>,
        Ley Foon Tan <ley.foon.tan@intel.com>,
        Chris Zankel <chris@zankel.net>,
        Max Filippov <jcmvbkbc@gmail.com>,
        Vineet Gupta <vgupta@synopsys.com>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        Robert Jarzmik <robert.jarzmik@free.fr>,
        Russell King <linux@armlinux.org.uk>,
        Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
        Yoshinori Sato <ysato@users.sourceforge.jp>,
        Rich Felker <dalias@libc.org>,
        Geert Uytterhoeven <geert@linux-m68k.org>,
        Alex Deucher <alexander.deucher@amd.com>,
        =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Rob Clark <robdclark@gmail.com>,
        Lee Jones <lee.jones@linaro.org>,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
        Rob Herring <robh@kernel.org>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Bartosz Golaszewski <bgolaszewski@baylibre.com>,
        Android Kernel Team <kernel-team@android.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, May 20, 2021 at 6:37 PM Marc Zyngier <maz@kernel.org> wrote:

> Wouldn't it be nice if the irqdomain
> would cache the irq_desc instead of forcing the core code to look it
> up on each and every interrupt? This is what this long series is all
> about.

For gpio and pinctrl bulk conversions:
Acked-by: Linus Walleij <linus.walleij@linaro.org>

I agree that Rob's idea to create a bitmap walker may be helpful
if you have the energy for it, but this as a whole is nevertheless good.

Do they have to be applied en masse?

I think there could be some clashes and new drivers that will
create weirdness in that case so an immutable branch for
maintainers to pull in will be needed if you want it all in the
next merge window.

Unless you plan to merge the bottom
patches and then let subsystem maintainers convert each
subsystem in the next merge window.

Or a base to be pulled in and then each subsystem can
apply the bulk conversion to their subsystem only.

So looking for a merging strategy!

Yours,
Linus Walleij