Re: [PATCH 10/11] arm64: broadcom: Removed forced select of interrupt controllers

From: Arnd Bergmann
Date: Sat Sep 25 2021 - 13:46:27 EST


On Sat, Sep 25, 2021 at 2:10 PM Marc Zyngier <maz@xxxxxxxxxx> wrote:
> On Fri, 24 Sep 2021 18:05:45 +0100, Florian Fainelli <f.fainelli@xxxxxxxxx> wrote:
>
> How does the user know about that? People will build a kernel
> selecting their platform, and find out it doesn't work. This seems
> terribly counter-productive to me.

It's default-enabled on the platforms that need it, see one of the
earlier patches.

Having the option to make it a loadable module is a definite benefit as
far as I'm concerned, and I generally like the idea of having individually
selectable symbols for consistency as that is what we have in other
subsystems as well.

Ideally I'd do away with all the 'select' statements for the platforms and
only have them control dependencies as we do for most other subsystems.
irqchip is one of the few exceptions here, though I understand the reason
for having the most important drivers tied to the platform more closely.

Arnd