[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK7LNARv=pE1h_BBiD51W4VOo4m5TnyHVOQyfojuQTOWZ2oyKw@mail.gmail.com>
Date: Wed, 26 Jul 2017 23:48:51 +0900
From: Masahiro Yamada <yamada.masahiro@...ionext.com>
To: Thomas Gleixner <tglx@...utronix.de>
Cc: Marc Zyngier <marc.zyngier@....com>,
Jason Cooper <jason@...edaemon.net>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] irqchip: create a Kconfig menu for irqchip drivers
2017-07-26 23:24 GMT+09:00 Thomas Gleixner <tglx@...utronix.de>:
> On Wed, 26 Jul 2017, Marc Zyngier wrote:
>> On 26/07/17 14:14, Masahiro Yamada wrote:
>> > The irqchip maintainers applied drivers
>> > with user-configurable Kconfig entries.
>>
>> They are *not* user-selectable, since there is *NO* menu entry. *You*
>> are making them user-selectable, and I'm objecting to that.
>
> Masahiro is right. They are user selectable.
>
> Enable CONFIG_SOC_IMX51 and you'll get a prompt for
>
> TS-4800 IRQ controller
>
> in the middle of the 'Device Drivers' menu. Same for the other options,
> which have prompts. The irqchip Kconfig is included from drivers/Kconfig so
> there is menu context, just not a proper one.
>
> I agree that this is crap and these prompts should be removed and the irq
> thingy selected by the stuff which needs it.
I could not find any corresponding platform that selects CONFIG_TS4800_IRQ.
I am not familiar with this driver at all, but
is this a corner case according to the commit message below?
If the main SoC selects the FPGA IP, it is weird
because this is really board-specific connection.
commit d01f8633d52e4dac5ee598b87d49fd23346ccfd6
Author: Damien Riegel <damien.riegel@...oirfairelinux.com>
Date: Mon Dec 21 15:11:23 2015 -0500
irqchip/ts4800: Add TS-4800 interrupt controller
This commit adds support for the TS-4800 interrupt controller. This
controller is instantiated in a companion FPGA, and multiplex interrupts
for other FPGA IPs.
As this component is external to the SoC, the SoC might need to reserve
pins, so this controller is implemented as a platform driver and doesn't
use the IRQCHIP_DECLARE construct.
Signed-off-by: Damien Riegel <damien.riegel@...oirfairelinux.com>
Cc: Jason Cooper <jason@...edaemon.net>
Cc: Marc Zyngier <marc.zyngier@....com>
Cc: Rob Herring <robh+dt@...nel.org>
Cc: Pawel Moll <pawel.moll@....com>
Cc: Mark Rutland <mark.rutland@....com>
Cc: Ian Campbell <ijc+devicetree@...lion.org.uk>
Cc: Kumar Gala <galak@...eaurora.org>
Cc: kernel@...oirfairelinux.com
Link: http://lkml.kernel.org/r/1450728683-31416-2-git-send-email-damien.riegel@savoirfairelinux.com
Signed-off-by: Thomas Gleixner <tglx@...utronix.de>
--
Best Regards
Masahiro Yamada
Powered by blists - more mailing lists