[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAHQ1cqGHic4Osyzy9jtPc_7tPqEanmSVJo8n-NAV0uzcpJbxqA@mail.gmail.com>
Date: Sun, 14 Apr 2019 11:47:31 -0700
From: Andrey Smirnov <andrew.smirnov@...il.com>
To: Lucas Stach <l.stach@...gutronix.de>
Cc: linux-pci@...r.kernel.org,
Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Fabio Estevam <fabio.estevam@....com>,
Chris Healy <cphealy@...il.com>,
Leonard Crestez <leonard.crestez@....com>,
"A.s. Dong" <aisheng.dong@....com>,
Richard Zhu <hongxing.zhu@....com>,
dl-linux-imx <linux-imx@....com>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 02/11] PCI: imx6: Remove redundant debug tracing
On Fri, Apr 12, 2019 at 8:53 AM Lucas Stach <l.stach@...gutronix.de> wrote:
>
> Am Sonntag, den 31.03.2019, 21:25 -0700 schrieb Andrey Smirnov:
> > All calls to imx6_pcie_wait_for_link() share the same error path and
> > the state of PHY debug registers will already be printed there.
>
> With this change we can drop imx6_pcie_wait_for_link altogether, as
> it's now just wrapper around dw_pcie_wait_for_link that pulls out the
> dw_pcie struct from imx6_pcie. Both callers have dw_pcie already
> available, so there is no point in keeping this function.
>
Good point, will do.
Thanks,
Andrey Smirnov
Powered by blists - more mailing lists