[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1922e611-7a28-4198-8496-7e2a3e9c48ec@163.com>
Date: Fri, 21 Mar 2025 21:59:20 +0800
From: Hans Zhang <18255117159@....com>
To: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Cc: lpieralisi@...nel.org, kw@...ux.com, robh@...nel.org,
bhelgaas@...gle.com, jingoohan1@...il.com, thomas.richard@...tlin.com,
linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [v4 4/4] PCI: cadence: Use cdns_pcie_find_*capability to find
capability offset instead of hardcore
On 2025/3/21 21:18, Manivannan Sadhasivam wrote:
> On Fri, Mar 21, 2025 at 06:17:10PM +0800, Hans Zhang wrote:
>> The offset address of capability or extended capability designed by
>> different SOC design companies may not be the same. Therefore, a flexible
>> public API is required to find the offset address of a capability or
>> extended capability in the configuration space.
>>
>
> The PCIe capability/extended capability offsets are not guaranteed to be
> the same across all SoCs integrating the Cadence PCIe IP. Hence, use the
> cdns_pcie_find_{ext}_capability() APIs for finding them.
>
> This avoids hardcoding the offsets in the driver.
>
Hi Mani,
Thank you very much for your advice. Will change.
Best regards,
Hans
> - Mani
>
Powered by blists - more mailing lists