lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
 <CH2PPF4D26F8E1C22F8726D1ABA60E1A819A262A@CH2PPF4D26F8E1C.namprd07.prod.outlook.com>
Date: Mon, 2 Jun 2025 01:24:27 +0000
From: Manikandan Karunakaran Pillai <mpillai@...ence.com>
To: "manivannan.sadhasivam@...aro.org" <manivannan.sadhasivam@...aro.org>
CC: Krzysztof Kozlowski <krzk@...nel.org>,
        "hans.zhang@...tech.com"
	<hans.zhang@...tech.com>,
        "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
        "lpieralisi@...nel.org" <lpieralisi@...nel.org>,
        "kw@...ux.com"
	<kw@...ux.com>, "robh@...nel.org" <robh@...nel.org>,
        "krzk+dt@...nel.org"
	<krzk+dt@...nel.org>,
        "conor+dt@...nel.org" <conor+dt@...nel.org>,
        Milind
 Parab <mparab@...ence.com>,
        "peter.chen@...tech.com"
	<peter.chen@...tech.com>,
        "linux-pci@...r.kernel.org"
	<linux-pci@...r.kernel.org>,
        "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v4 5/5] PCI: cadence: Add callback functions for RP and EP
 controller

Hi,

The next patch v5 still under works will have Rob's comments addressed by separating out the drivers for legacy(LGA)
and High performance arch(HPA) replacing the ops method that are in current patches.


>-----Original Message-----
>From: manivannan.sadhasivam@...aro.org
><manivannan.sadhasivam@...aro.org>
>Sent: Sunday, June 1, 2025 8:11 PM
>To: Manikandan Karunakaran Pillai <mpillai@...ence.com>
>Cc: Krzysztof Kozlowski <krzk@...nel.org>; hans.zhang@...tech.com;
>bhelgaas@...gle.com; lpieralisi@...nel.org; kw@...ux.com;
>robh@...nel.org; krzk+dt@...nel.org; conor+dt@...nel.org; Milind Parab
><mparab@...ence.com>; peter.chen@...tech.com; linux-pci@...r.kernel.org;
>devicetree@...r.kernel.org; linux-kernel@...r.kernel.org
>Subject: Re: [PATCH v4 5/5] PCI: cadence: Add callback functions for RP and EP
>controller
>
>EXTERNAL MAIL
>
>
>On Sun, Apr 27, 2025 at 03:52:13AM +0000, Manikandan Karunakaran Pillai
>wrote:
>> >
>> >> ---
>> >>  drivers/pci/controller/cadence/pci-j721e.c    |  12 +
>> >>  .../pci/controller/cadence/pcie-cadence-ep.c  |  29 +-
>> >>  .../controller/cadence/pcie-cadence-host.c    | 263 ++++++++++++++++--
>> >>  .../controller/cadence/pcie-cadence-plat.c    |  27 +-
>> >>  drivers/pci/controller/cadence/pcie-cadence.c | 197 ++++++++++++-
>> >>  drivers/pci/controller/cadence/pcie-cadence.h |  11 +-
>> >>  6 files changed, 495 insertions(+), 44 deletions(-)
>> >>
>> >> diff --git a/drivers/pci/controller/cadence/pci-j721e.c
>> >b/drivers/pci/controller/cadence/pci-j721e.c
>> >> index ef1cfdae33bb..154b36c30101 100644
>> >> --- a/drivers/pci/controller/cadence/pci-j721e.c
>> >> +++ b/drivers/pci/controller/cadence/pci-j721e.c
>> >> @@ -164,6 +164,14 @@ static const struct cdns_pcie_ops j721e_pcie_ops
>= {
>> >>  	.start_link = j721e_pcie_start_link,
>> >>  	.stop_link = j721e_pcie_stop_link,
>> >>  	.link_up = j721e_pcie_link_up,
>> >> +	.host_init_root_port = cdns_pcie_host_init_root_port,
>> >> +	.host_bar_ib_config = cdns_pcie_host_bar_ib_config,
>> >> +	.host_init_address_translation =
>> >cdns_pcie_host_init_address_translation,
>> >> +	.detect_quiet_min_delay_set =
>> >cdns_pcie_detect_quiet_min_delay_set,
>> >> +	.set_outbound_region = cdns_pcie_set_outbound_region,
>> >> +	.set_outbound_region_for_normal_msg =
>> >> +
>> >cdns_pcie_set_outbound_region_for_normal_msg,
>> >> +	.reset_outbound_region = cdns_pcie_reset_outbound_region,
>> >
>> >How did you resolve Rob's comments?
>> >
>> >These were repeated I think three times finally with:
>> >
>> >"Please listen when I say we do not want the ops method used in other
>> >drivers. "
>> >
>> >I think you just send the same ignoring previous discussion which is the
>> >shortest way to get yourself NAKed.
>> >
>> >
>>
>> I was waiting to check if there are additional comments on the approach,
>because this approach was taken
>> based on an earlier comments on the patches. Since we have not got any
>adverse comments from other
>> maintainers on this, I will separate out the entire driver for old and new
>architecture. The few common functions
>> will be moved to a common file, to be used as library functions. There will be
>repetitions of
>> code but from Rob's comments, I believe it is fine.
>>
>
>I agree with Rob. We should really get rid of the callbacks and try to make the
>common code a library.
>
>- Mani
>
>--
>மணிவண்ணன் சதாசிவம்

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ