[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4419b25f-dd71-9c14-5de2-4425f9367e29@opensource.wdc.com>
Date: Tue, 11 Jan 2022 08:58:15 +0900
From: Damien Le Moal <damien.lemoal@...nsource.wdc.com>
To: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Cc: Rob Herring <robh+dt@...nel.org>,
Andy Shevchenko <andy.shevchenko@...il.com>,
Prabhakar <prabhakar.csengg@...il.com>,
linux-ide@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5] ata: pata_of_platform: Use platform_get_irq_optional()
to get the interrupt
On 1/6/22 03:17, Lad Prabhakar wrote:
> platform_get_resource(pdev, IORESOURCE_IRQ, ..) relies on static
> allocation of IRQ resources in DT core code, this causes an issue
> when using hierarchical interrupt domains using "interrupts" property
> in the node as this bypasses the hierarchical setup and messes up the
> irq chaining.
>
> In preparation for removal of static setup of IRQ resource from DT core
> code use platform_get_irq_optional().
>
> Note the code does not set the IRQ flags as this is handled automatically
> for DT.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> ---
> Hi All,
>
> This patch is part of series [1]. I'll re-visit merging of pata_of_platform
> into pata_platform at later point. As my primary focus is removal of static
> setup of IRQ resource from DT core code.
>
> [1] https://patchwork.ozlabs.org/project/linux-ide/list/?series=278349
>
> v4->v5
> * Set end member of IRQ resource
> * Clear irq_res un-conditionally.
>
> Cheers,
> Prabhakar
Applied to for-5.17. Thanks !
--
Damien Le Moal
Western Digital Research
Powered by blists - more mailing lists