Received-SPF: pass (google.com: domain of linux-kernel+bounces-83735-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Tue, 27 Feb 2024 12:34:15 -0500
From: Frank Li <Frank.li@nxp.com>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>,
	Gustavo Pimentel <gustavo.pimentel@synopsys.com>,
	Lorenzo Pieralisi <lpieralisi@kernel.org>,
	Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= <kw@linux.com>,
	Rob Herring <robh@kernel.org>, Bjorn Helgaas <bhelgaas@google.com>,
	Marek Vasut <marek.vasut+renesas@gmail.com>,
	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Vidya Sagar <vidyas@nvidia.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Richard Zhu <hongxing.zhu@nxp.com>,
	Lucas Stach <l.stach@pengutronix.de>,
	Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Minghuan Lian <minghuan.Lian@nxp.com>,
	Mingkai Hu <mingkai.hu@nxp.com>, Roy Zang <roy.zang@nxp.com>,
	Kunihiko Hayashi <hayashi.kunihiko@socionext.com>,
	Masami Hiramatsu <mhiramat@kernel.org>,
	Kishon Vijay Abraham I <kishon@kernel.org>,
	linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-renesas-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	linux-tegra@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org,
	Niklas Cassel <cassel@kernel.org>
Subject: Re: [PATCH v8 09/10] PCI: qcom-ep: Use the generic
 dw_pcie_ep_linkdown() API to handle LINK_DOWN event
Message-ID: <Zd4dFyM78Nc1f7fk@lizhi-Precision-Tower-5810>
References: <20240224-pci-dbi-rework-v8-0-64c7fd0cfe64@linaro.org>
 <20240224-pci-dbi-rework-v8-9-64c7fd0cfe64@linaro.org>
 <ZdzIada1H95ike0t@lizhi-Precision-Tower-5810>
 <20240227123230.GP2587@thinkpad>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20240227123230.GP2587@thinkpad>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?T1I3OG93TFd3OEhJblNRbGtxYlFJUHJzVTZpTkg2c29CU2R5QUFUTWlrZDJV?=
 =?utf-8?B?OEh4MkdXekJlSlYrOVJvNlBaUjVBWkgvQVJzcVlIR1hHMmgzTmhwRnRtSy94?=
 =?utf-8?B?WnY5UUl3NG1iQTB0UlYzd3BSTjYwdmEzMkRaaHpSYWgvLzE5NW1GamRiS0E3?=
 =?utf-8?B?YVhiNE9LaXNZL3g5anpSZlBpcEZwRlFudU5DM1poMFFMbGRhdDJwYzFIUmhu?=
 =?utf-8?B?eUd6dXlPaGg0LzNHdzMwV0RjZnRRNUswVHBFdjdweGhvcDdWUkZkTmptVHpO?=
 =?utf-8?B?dFR4M1IwQ00rbVRPWU1rYzlwVTI0cFBJRHV4b0w0VWVVcEZGc2pFOTlpNi9j?=
 =?utf-8?B?RkZpcUhpRU1QNkI4ZFRSajlPcm5uN0VmU25sQUxuMDBhTmEzVWdsQUFJN1ZT?=
 =?utf-8?B?aTZWRm1FQjZhdENsN3RXQTc5VW82RVk1dFIyNW1vcXFvUG1GcFBTZDVCQnBB?=
 =?utf-8?B?dVEvNU1hdTFOS2pJdFpNMHIvUGNIRVdVa042TXNaYSs2cStIWHpiTkh2aS93?=
 =?utf-8?B?cjhHc0l6NGUwQW5hRHBWcFdHZHRKZUhYNy9pY0FhR0lSdzNBRE95RU1NY2VO?=
 =?utf-8?B?QkkyNmhyb1E2TnpESXZ4Q0N5SjFjQ3Bka05DeWMyY0lLbFlwWVZoRG1KNUM5?=
 =?utf-8?B?eWpzcUJGODFQTFhGTU1tdnNTR0E5WGtIa1NiaGZvLzYwekhvdnRRTE9VVWJ1?=
 =?utf-8?B?RTUzUjM3OERIalIyRVcvWFRaMHBDWHJYQmIvR1JjTVpiY0dDNzNNUzlvMWRv?=
 =?utf-8?B?M2srVFN6STQyOGx3eGdzUE1nVE1WRVl2UHRPVDFXc1F5bkJ2QmRTUHlmY0Jz?=
 =?utf-8?B?U3VvVUl5SXR0aHJ2QzlMWEtPZ01MU0JIdmhVS3NSYjJLZXhTOFU2T2RFN0NC?=
 =?utf-8?B?bUMzRGJHWE5YNDg4RmlXUGZzTFc4dWpaMFIzNjdHQ04wRlhlZmswN1BxQTdh?=
 =?utf-8?B?dkREN3BsZk9Xc0FlVjRyRjZHUFcvYVppblVua05pZzZnMkUrejd1M3dYMnNB?=
 =?utf-8?B?M2pJNkw0SFk5anBPQ2RaUkJ1Wk5DV3NHQWVlK243Sy9ralRvQWxLdUxqWFAz?=
 =?utf-8?B?TlN1c3VXNFFjSnBLOVNmeFFicGY1NlR4bXdvQkJsVTBsU1Yvc1luU0FtQm5W?=
 =?utf-8?B?YUxHeFNXSTM5SmRYaHVncmdGSUVPQU9GN0h5Wngxd09zaXFadFdObzloWmxF?=
 =?utf-8?B?YnJIMWsxOVl0RldKaXgwWVdrNjE0VWluQlNTSFZPVVJzaFJXYnpXTzdXNjMr?=
 =?utf-8?B?VG9sOVg3RHBSdWZ4dXlmMFdZT0NiTmhJdFUwc3N0bzR6Ujk0Q2hWMW42ZXhs?=
 =?utf-8?B?YkYrOFk0dXFxcGZoRHJpSFp0VGVUT210ckFZSWN3VnMzbE9hNlgxbjdDU05S?=
 =?utf-8?B?a0lRdW5zK1BUYXkzdGZKQnhURVVsYnlha3JpeEV2b0pRcTRNSkNuNzNDbjhD?=
 =?utf-8?B?L1QwRENMTFlmdnJhV2pGUng0cjVZUVVrQ1NRT0ZIU1dHdHdVRllYUjhsbFFC?=
 =?utf-8?B?UDk3Wjg5NmxCQ0NRQnBVSms5cE5jeXZaSEFZa29tak41VFZhbkFSQmhGdStH?=
 =?utf-8?B?YU5LZzZKK1lvTGdXd09OSmhIVUJ0S2p3Q2FRMjYxNU56d003OGJzVnV0eWww?=
 =?utf-8?B?MzRxTUNpWmQ3a0w0NUloSEJQeDhxVVZCcERGazZWMnVPd0tXRXRzdGhDMjg0?=
 =?utf-8?B?cVR5bG45Z01FOHhkOVd5dTdjc01XdlVlUWJaQlRTSXkydXNwdmpOcTdtZXRM?=
 =?utf-8?B?VDNiTW0vaHF2Z1pRUjRsU1BmMXo0KzlQSXJkcGtpb3kwekliVnRiUkZZYmhL?=
 =?utf-8?B?eERTNkNHU3Fha2J6cDE5N1RDNGUyUWlDL1BmNmhxcndJTkVNcUhXanEwSUJE?=
 =?utf-8?B?WWR4ejJMWU82SWpDL2o5cS9lZkMySzFCZ09jdENEYk9OUXEyTCsrRFByQitL?=
 =?utf-8?B?d0hNNTcwMS91OUtLMENIUjBleGNJbml4SEMrWEd4YndsNHUzVHpqNFJlbm9i?=
 =?utf-8?B?OEZBZDl2bzBCK0tGV1d3V3AzL1B2Z0hnZU9Sclk4aUNuS2ZETUpUT3JNd283?=
 =?utf-8?B?azhwc3FybnFOTGhvcWJOb1h0UnVVWkx5RWsvdjhjazBRUk1CbUNkcWhLa2hN?=
 =?utf-8?Q?vsKSI8RlRCZByOmoTR2PhiIFn?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a7cfa71f-a0c8-4cb4-b7a9-08dc37ba58b9
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Feb 2024 17:34:27.3019
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: cxsR+1HjWqRYAIXgb/UbZv+Cq7880d3XNYOBLbJq8KhyCR6uLKTxK5wj+JlCRNt87Ynop6D5+B/W1n/8IpI2Ww==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA1PR04MB10171

On Tue, Feb 27, 2024 at 06:02:30PM +0530, Manivannan Sadhasivam wrote:
> On Mon, Feb 26, 2024 at 12:20:41PM -0500, Frank Li wrote:
> > On Sat, Feb 24, 2024 at 12:24:15PM +0530, Manivannan Sadhasivam wrote:
> > > Now that the API is available, let's make use of it. It also handles the
> > > reinitialization of DWC non-sticky registers in addition to sending the
> > > notification to EPF drivers.
> > > 
> > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > ---
> > >  drivers/pci/controller/dwc/pcie-qcom-ep.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/pci/controller/dwc/pcie-qcom-ep.c b/drivers/pci/controller/dwc/pcie-qcom-ep.c
> > > index 2fb8c15e7a91..4e45bc4bca45 100644
> > > --- a/drivers/pci/controller/dwc/pcie-qcom-ep.c
> > > +++ b/drivers/pci/controller/dwc/pcie-qcom-ep.c
> > > @@ -640,7 +640,7 @@ static irqreturn_t qcom_pcie_ep_global_irq_thread(int irq, void *data)
> > >  	if (FIELD_GET(PARF_INT_ALL_LINK_DOWN, status)) {
> > >  		dev_dbg(dev, "Received Linkdown event\n");
> > >  		pcie_ep->link_status = QCOM_PCIE_EP_LINK_DOWN;
> > > -		pci_epc_linkdown(pci->ep.epc);
> > > +		dw_pcie_ep_linkdown(&pci->ep);
> > 
> > Suppose pci_epc_linkdown() will call dw_pcie_ep_linkdown() ?
> > why need direct call dw_pcie_ep_linkdown() here?
> > 
> 
> I've already justified this in the commit message. Here is the excerpt:
> 
> "It also handles the reinitialization of DWC non-sticky registers in addition
> to sending the notification to EPF drivers."

API function name is too similar. It is hard to know difference from API
naming. It'd better to know what function do from function name.

Frank
> 
> - Mani
> 
> -- 
> மணிவண்ணன் சதாசிவம்