lkml.org 
[lkml]   [2017]   [Jul]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] irqchip: create a Kconfig menu for irqchip drivers
2017-07-26 23:24 GMT+09:00 Thomas Gleixner <tglx@linutronix.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@savoirfairelinux.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@savoirfairelinux.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Pawel Moll <pawel.moll@arm.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>
Cc: Kumar Gala <galak@codeaurora.org>
Cc: kernel@savoirfairelinux.com
Link: http://lkml.kernel.org/r/1450728683-31416-2-git-send-email-damien.riegel@savoirfairelinux.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>






--
Best Regards
Masahiro Yamada

\
 
 \ /
  Last update: 2017-07-26 16:49    [W:0.168 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site