Subject: Re: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM

From: Scott Wood <[email-masked]>
Date: Sun, 31 Jan 2016 22:07:29 -0600

[cc list trimmed]

On Mon, 2016-02-01 at 03:57 +0000, Zhiqiang Hou wrote:
> Hi,
> Any comments?

Only that it was submitted too late for the last merge window, and I have not
yet begun processing patches for the next one.  If you meant this to be a fix
important enough to go in outside the merge window, it's missing any statement
of what the actual symptoms are.

-Scott

> 
> > -----Original Message-----
> > From: Zhiqiang Hou [mailto:[email-masked]]
> > Sent: 2016年1月4日 15:45
> > To: [email-masked]; [email-masked]; 
> > [email-masked];
> > [email-masked]
> > Cc: Scott Wood <[email-masked]>; [email-masked];
> > [email-masked]; [email-masked]; Hou Zhiqiang
> > <[email-masked]>
> > Subject: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
> > 
> > From: Hou Zhiqiang <[email-masked]>
> > 
> > Only the value PCIE_LTSSM_L0 stands for the link is up, instead of >=
> > PCIE_LTSSM_L0.
> > 
> > Signed-off-by: Hou Zhiqiang <[email-masked]>
> > ---
> >  arch/powerpc/sysdev/fsl_pci.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pci.c
> > index
> > 610f472..1595f55 100644
> > --- a/arch/powerpc/sysdev/fsl_pci.c
> > +++ b/arch/powerpc/sysdev/fsl_pci.c
> > @@ -73,7 +73,7 @@ static int fsl_pcie_check_link(struct pci_controller
> > *hose)
> >  					       PCIE_LTSSM, 4, &val);
> >  		else
> >  			early_read_config_dword(hose, 0, 0, PCIE_LTSSM,
> > &val);
> > -		if (val < PCIE_LTSSM_L0)
> > +		if (val != PCIE_LTSSM_L0)
> >  			return 1;
> >  	} else {
> >  		struct ccsr_pci __iomem *pci = hose->private_data;
> > --
> > 2.1.0.27.g96db324
> 
> Thanks,
> Zhiqiang
> _______________________________________________
> Linuxppc-dev mailing list
> [email-masked]
> https://lists.ozlabs.org/listinfo/linuxppc-dev