On Thu, Apr 11, 2019 at 10:33:55PM +0530, Manikanta Maddireddy wrote:
> Link retry log is info log not error, so change it to INFO log level.
The wording here is confusing. The log is clearly error level right now.
Perhaps you meant to say something like this:
Letting users know that a link is down and in the process of being
brought up again is informational, not an error condition.
Thierry
>
> Signed-off-by: Manikanta Maddireddy <mmaddireddy@xxxxxxxxxx>
> ---
> drivers/pci/controller/pci-tegra.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/controller/pci-tegra.c b/drivers/pci/controller/pci-tegra.c
> index 75873e6627f9..55f8ff4f3e07 100644
> --- a/drivers/pci/controller/pci-tegra.c
> +++ b/drivers/pci/controller/pci-tegra.c
> @@ -2461,7 +2461,7 @@ static bool tegra_pcie_port_check_link(struct tegra_pcie_port *port)
> } while (--timeout);
>
> if (!timeout) {
> - dev_err(dev, "link %u down, retrying\n", port->index);
> + dev_info(dev, "link %u down, retrying\n", port->index);
> goto retry;
> }
>
> --
> 2.17.1
>
Attachment:
signature.asc
Description: PGP signature
