arm: a37xx: pci: Don't spam about PIO Response Status
authorMarek Behún <marek.behun@nic.cz>
Tue, 7 Sep 2021 15:27:08 +0000 (17:27 +0200)
committerStefan Roese <sr@denx.de>
Fri, 10 Sep 2021 07:01:51 +0000 (09:01 +0200)
Use dev_dbg() instead of dev_err() in pcie_advk_check_pio_status().

For example CRS is not an error status, it just says that the request
should be retried.

Without this, U-Boot spams the terminal with
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x100000
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x108000
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x110000
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x120000
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x128000
  pcie_advk pcie@d0070000: Non-posted PIO Response Status: UR, 0xc80 @ 0x130000
  ...
when a device is not connected to a PCIe switch (Unsupported Request
from the switch).

Signed-off-by: Marek Behún <marek.behun@nic.cz>
Reviewed-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Stefan Roese <sr@denx.de>
drivers/pci/pci-aardvark.c

index 74797e9..cf6e30f 100644 (file)
@@ -327,7 +327,7 @@ static int pcie_advk_check_pio_status(struct pcie_advk *pcie,
        else
                str_posted = "Posted";
 
-       dev_err(pcie->dev, "%s PIO Response Status: %s, %#x @ %#x\n",
+       dev_dbg(pcie->dev, "%s PIO Response Status: %s, %#x @ %#x\n",
                str_posted, strcomp_status, reg,
                advk_readl(pcie, PIO_ADDR_LS));