IronPort-SDR: JbIvHmVC/do0ygU3Yq8IX8X46ABz2nr/e6vRVTT5Wqxrcyb1RJFv50+bYrwyPTP8okg69u8BmI
 LfR+ONFp4oW5Tx9xnSrdVcY54Kb6T44544M/v6qEwBLfgvCknnVcTllIEOpAazH3l88yn7fjrr
 KEWSIzsa/Zn1EJ4tBiyMipVjN9jUJ0OKtZ5lzxRvKPUK4U5l4TxhQN5Fozoq3leQR+tLYDbcJm
 w4Rj7zwqY4boHX0PenEnqA1iRBqImiZDqjNQz47prM1vGxBqD4FfrzGoHHCsJCNMVYY6MWICKE
 WxIBy8pQ2rDwfGD+R8xcPLnF
IronPort-SDR: bqgn9pOLOtMCnXPtaIAthvfI6MILj0bu5dI8PJkfiQTOa/fjwcGq8F661LFSCpyTsdkLVSocY3
 ec7ZAcG6nwYmNC6uDYel5HygDDEgm6Rn9jrhowpYJPve5q5GNO+XhpLxSpKewfHLT8SoUQ9gzF
 EH/6voITHafDWkpJ7sz5gXEfGGWufmuaeBcrGbkFkY08w0YSx7pL/NGaAfkHlf7LN8U80VMrx2
 wzIdGFUVQBHbkMP884Se0hmP3jLCiEkQJ2ek6jOM4qHkkb5vJJI9zczMXVoHZSSn95sXOrT1SR
 Oe4=
WDCIronportException: Internal
Message-ID: <4419b25f-dd71-9c14-5de2-4425f9367e29@opensource.wdc.com>
Date:   Tue, 11 Jan 2022 08:58:15 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.4.0
Subject: Re: [PATCH v5] ata: pata_of_platform: Use platform_get_irq_optional()
 to get the interrupt
Content-Language: en-US
To:     Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Andy Shevchenko <andy.shevchenko@gmail.com>,
        Prabhakar <prabhakar.csengg@gmail.com>,
        linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220105181721.13087-1-prabhakar.mahadev-lad.rj@bp.renesas.com>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <20220105181721.13087-1-prabhakar.mahadev-lad.rj@bp.renesas.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@bp.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