Re: [PATCH] irqchip: add keystone irq controller ip driver

From: Santosh Shilimkar
Date: Fri Jul 18 2014 - 09:51:19 EST


Hi Jason,

On Friday 18 July 2014 08:59 AM, Jason Cooper wrote:
> Grygorii,
>
> On Mon, Jul 14, 2014 at 06:27:57PM +0300, Grygorii Strashko wrote:
>> On Keystone SOCs, DSP cores can send interrupts to ARM
>> host using the IRQ controller IP. It provides 28 IRQ
>> signals to ARM. The IRQ handler running on HOST OS can
>> identify DSP signal source by analyzing SRCCx bits in
>> IPCARx registers. This is one of the component used by
>> the IPC mechanism used on Keystone SOCs.
>>
>> Signed-off-by: Grygorii Strashko <grygorii.strashko@xxxxxx>
>> ---
>> .../interrupt-controller/ti,keystone-irq.txt | 36 +++
>> drivers/irqchip/Kconfig | 7 +
>> drivers/irqchip/Makefile | 1 +
>> drivers/irqchip/irq-keystone.c | 235 ++++++++++++++++++++
>> 4 files changed, 279 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/ti,keystone-irq.txt
>> create mode 100644 drivers/irqchip/irq-keystone.c
>>


[..]

>> +
>> +static const struct of_device_id keystone_irq_dt_ids[] = {
>> + { .compatible = "ti,keystone-irq", },
>> + {},
>> +};
>> +MODULE_DEVICE_TABLE(of, keystone_irq_dt_ids);
>> +
>> +static struct platform_driver keystone_irq_device_driver = {
>> + .probe = keystone_irq_probe,
>> + .remove = keystone_irq_remove,
>> + .driver = {
>> + .name = "keystone_irq",
>> + .owner = THIS_MODULE,
>> + .of_match_table = of_match_ptr(keystone_irq_dt_ids),
>> + }
>> +};
>> +
>> +module_platform_driver(keystone_irq_device_driver);
>
> My understanding of DSP use-cases is a little sparse, are there
> legitimate scenarios where you might remove this driver during runtime?
> Perhaps IRQCHIP_DECLARE() might be better?
>
There is no scenario where driver needs to hotpluged out. Usecase is
simple. Its really any other IRQCHIP. The difference is really the
source of interrupts. Instead of peripherals interrupting the host
OS9linux), here the DSP cores can send interrupts to Host OS.

Hope this clarifies.

Regards,
Santosh

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/