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
| ||
|
Date: Tue, 18 Oct 2022 07:21:54 +0100 From: Jon Hunter <jonathanh@...dia.com> To: Vidya Sagar <vidyas@...dia.com>, jingoohan1@...il.com, gustavo.pimentel@...opsys.com, lpieralisi@...nel.org, robh@...nel.org, kw@...ux.com, bhelgaas@...gle.com, treding@...dia.com Cc: linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org, kthota@...dia.com, mmaddireddy@...dia.com, sagar.tv@...il.com, "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org> Subject: Re: [PATCH V1] PCI: dwc: Use dev_info for PCIe link down event logging Hi Bjorn, On 13/09/2022 11:12, Vidya Sagar wrote: > Some of the platforms (like Tegra194 and Tegra234) have open slots and > not having an endpoint connected to the slot is not an error. > So, changing the macro from dev_err to dev_info to log the event. > > Signed-off-by: Vidya Sagar <vidyas@...dia.com> > --- > drivers/pci/controller/dwc/pcie-designware.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/dwc/pcie-designware.c b/drivers/pci/controller/dwc/pcie-designware.c > index 650a7f22f9d0..25154555aa7a 100644 > --- a/drivers/pci/controller/dwc/pcie-designware.c > +++ b/drivers/pci/controller/dwc/pcie-designware.c > @@ -456,7 +456,7 @@ int dw_pcie_wait_for_link(struct dw_pcie *pci) > } > > if (retries >= LINK_WAIT_MAX_RETRIES) { > - dev_err(pci->dev, "Phy link never came up\n"); > + dev_info(pci->dev, "Phy link never came up\n"); > return -ETIMEDOUT; > } > Are you OK to take this change? Acked-by: Jon Hunter <jonathanh@...dia.com> Tested-by: Jon Hunter <jonathanh@...dia.com> Thanks Jon -- nvpublic
Powered by blists - more mailing lists