[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <swpcfwykjzktuijaln2vzghjg35zppfyeiuarvrcpftf66hd6d@cfsznto43sqx>
Date: Wed, 21 Jan 2026 14:43:54 +0530
From: Manivannan Sadhasivam <mani@...nel.org>
To: Niklas Cassel <cassel@...nel.org>
Cc: manivannan.sadhasivam@....qualcomm.com,
Jingoo Han <jingoohan1@...il.com>, Lorenzo Pieralisi <lpieralisi@...nel.org>,
Krzysztof Wilczyński <kwilczynski@...nel.org>, Rob Herring <robh@...nel.org>,
Bjorn Helgaas <bhelgaas@...gle.com>, linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org,
vincent.guittot@...aro.org, zhangsenchuan@...incomputing.com,
Shawn Lin <shawn.lin@...k-chips.com>, Richard Zhu <hongxing.zhu@....com>
Subject: Re: [PATCH v4 4/5] PCI: dwc: Rework the error print of
dw_pcie_wait_for_link()
On Wed, Jan 21, 2026 at 09:23:58AM +0100, Niklas Cassel wrote:
> On Tue, Jan 20, 2026 at 11:17:43PM +0530, Manivannan Sadhasivam via B4 Relay wrote:
> > From: Manivannan Sadhasivam <manivannan.sadhasivam@....qualcomm.com>
> >
> > If the link fails to come up even after detecting the device on the bus
> > i.e., if the LTSSM is not in Detect.Quiet and Detect.Active states, then
> > dw_pcie_wait_for_link() should log it as an error.
>
> Nit: maybe you should mention Poll.Active and Poll.Compliance here too?
>
Sure, I missed it. Will ammend it while applying.
- Mani
--
மணிவண்ணன் சதாசிவம்
Powered by blists - more mailing lists