Hi Rob, Thanks a lot for your review!
> -----Original Message----- > From: Rob Herring <[email protected]> > Sent: 2020年9月15日 9:20 > To: Z.q. Hou <[email protected]> > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; Leo Li <[email protected]>; > [email protected]; [email protected]; M.h. Lian > <[email protected]>; Mingkai Hu <[email protected]>; Roy Zang > <[email protected]> > Subject: Re: [PATCH 2/7] PCI: layerscape: Change to use the DWC common > link-up check function > > On Mon, Sep 07, 2020 at 01:37:56PM +0800, Zhiqiang Hou wrote: > > From: Hou Zhiqiang <[email protected]> > > > > The current Layerscape PCIe driver directly uses the physical layer > > LTSSM code to check the link-up state, which treats the > L0 states as > > link-up. This is not correct, since there is not explicit map between > > link-up state and LTSSM. So this patch changes to use the DWC common > > link-up check function. > > > > Signed-off-by: Hou Zhiqiang <[email protected]> > > --- > > drivers/pci/controller/dwc/pci-layerscape.c | 141 > > ++------------------ > > 1 file changed, 10 insertions(+), 131 deletions(-) > > IIRC, the common function uses a debug register. I've been wondering do > the common PCIe config space registers not work on DWC? If you have an > answer, that would be great for some potential additional cleanups. > You're right it uses a port debug register, but I'm not sure if the Link status Register of common PCIe config space works or not on DWC. Gustavo, can you help answer this query? Regards, Zhiqiang > Either way, > > Reviewed-by: Rob Herring <[email protected]>
