* [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
@ 2016-01-04 7:45 Zhiqiang Hou
2016-02-01 3:57 ` Zhiqiang Hou
0 siblings, 1 reply; 5+ messages in thread
From: Zhiqiang Hou @ 2016-01-04 7:45 UTC (permalink / raw)
To: linuxppc-dev, benh, paulus, mpe
Cc: scottwood, akpm, robh, Mingkai.Hu, Hou Zhiqiang
From: Hou Zhiqiang <Zhiqiang.Hou@freescale.com>
Only the value PCIE_LTSSM_L0 stands for the link is up, instead
of >= PCIE_LTSSM_L0.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@freescale.com>
---
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
2016-01-04 7:45 [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM Zhiqiang Hou
@ 2016-02-01 3:57 ` Zhiqiang Hou
2016-02-01 4:07 ` Scott Wood
0 siblings, 1 reply; 5+ messages in thread
From: Zhiqiang Hou @ 2016-02-01 3:57 UTC (permalink / raw)
To: Zhiqiang Hou, linuxppc-dev, benh, paulus, mpe
Cc: Scott Wood, akpm, robh, Mingkai.Hu
SGksDQoNCkFueSBjb21tZW50cz8NCg0KPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBG
cm9tOiBaaGlxaWFuZyBIb3UgW21haWx0bzpaaGlxaWFuZy5Ib3VAZnJlZXNjYWxlLmNvbV0NCj4g
U2VudDogMjAxNsTqMdTCNMjVIDE1OjQ1DQo+IFRvOiBsaW51eHBwYy1kZXZAbGlzdHMub3psYWJz
Lm9yZzsgYmVuaEBrZXJuZWwuY3Jhc2hpbmcub3JnOyBwYXVsdXNAc2FtYmEub3JnOw0KPiBtcGVA
ZWxsZXJtYW4uaWQuYXUNCj4gQ2M6IFNjb3R0IFdvb2QgPHNjb3R0d29vZEBmcmVlc2NhbGUuY29t
PjsgYWtwbUBsaW51eC1mb3VuZGF0aW9uLm9yZzsNCj4gcm9iaEBrZXJuZWwub3JnOyBNaW5na2Fp
Lkh1QGZyZWVzY2FsZS5jb207IEhvdSBaaGlxaWFuZw0KPiA8WmhpcWlhbmcuSG91QGZyZWVzY2Fs
ZS5jb20+DQo+IFN1YmplY3Q6IFtQQVRDSF0gcG93ZXJwYy9mc2wtcGNpOiBDb3JyZWN0IHRoZSBj
b21wYXJpc29uIG9mIHRoZSBMVFNTTQ0KPiANCj4gRnJvbTogSG91IFpoaXFpYW5nIDxaaGlxaWFu
Zy5Ib3VAZnJlZXNjYWxlLmNvbT4NCj4gDQo+IE9ubHkgdGhlIHZhbHVlIFBDSUVfTFRTU01fTDAg
c3RhbmRzIGZvciB0aGUgbGluayBpcyB1cCwgaW5zdGVhZCBvZiA+PQ0KPiBQQ0lFX0xUU1NNX0ww
Lg0KPiANCj4gU2lnbmVkLW9mZi1ieTogSG91IFpoaXFpYW5nIDxaaGlxaWFuZy5Ib3VAZnJlZXNj
YWxlLmNvbT4NCj4gLS0tDQo+ICBhcmNoL3Bvd2VycGMvc3lzZGV2L2ZzbF9wY2kuYyB8IDIgKy0N
Cj4gIDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiANCj4g
ZGlmZiAtLWdpdCBhL2FyY2gvcG93ZXJwYy9zeXNkZXYvZnNsX3BjaS5jIGIvYXJjaC9wb3dlcnBj
L3N5c2Rldi9mc2xfcGNpLmMgaW5kZXgNCj4gNjEwZjQ3Mi4uMTU5NWY1NSAxMDA2NDQNCj4gLS0t
IGEvYXJjaC9wb3dlcnBjL3N5c2Rldi9mc2xfcGNpLmMNCj4gKysrIGIvYXJjaC9wb3dlcnBjL3N5
c2Rldi9mc2xfcGNpLmMNCj4gQEAgLTczLDcgKzczLDcgQEAgc3RhdGljIGludCBmc2xfcGNpZV9j
aGVja19saW5rKHN0cnVjdCBwY2lfY29udHJvbGxlciAqaG9zZSkNCj4gIAkJCQkJICAgICAgIFBD
SUVfTFRTU00sIDQsICZ2YWwpOw0KPiAgCQllbHNlDQo+ICAJCQllYXJseV9yZWFkX2NvbmZpZ19k
d29yZChob3NlLCAwLCAwLCBQQ0lFX0xUU1NNLCAmdmFsKTsNCj4gLQkJaWYgKHZhbCA8IFBDSUVf
TFRTU01fTDApDQo+ICsJCWlmICh2YWwgIT0gUENJRV9MVFNTTV9MMCkNCj4gIAkJCXJldHVybiAx
Ow0KPiAgCX0gZWxzZSB7DQo+ICAJCXN0cnVjdCBjY3NyX3BjaSBfX2lvbWVtICpwY2kgPSBob3Nl
LT5wcml2YXRlX2RhdGE7DQo+IC0tDQo+IDIuMS4wLjI3Lmc5NmRiMzI0DQoNClRoYW5rcywNClpo
aXFpYW5nDQo=
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
2016-02-01 3:57 ` Zhiqiang Hou
@ 2016-02-01 4:07 ` Scott Wood
2016-02-03 21:32 ` Edward L Swarthout
0 siblings, 1 reply; 5+ messages in thread
From: Scott Wood @ 2016-02-01 4:07 UTC (permalink / raw)
To: Zhiqiang Hou, linuxppc-dev, mpe; +Cc: Mingkai.Hu
[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:Zhiqiang.Hou@freescale.com]
> > Sent: 2016年1月4日 15:45
> > To: linuxppc-dev@lists.ozlabs.org; benh@kernel.crashing.org;
> > paulus@samba.org;
> > mpe@ellerman.id.au
> > Cc: Scott Wood <scottwood@freescale.com>; akpm@linux-foundation.org;
> > robh@kernel.org; Mingkai.Hu@freescale.com; Hou Zhiqiang
> > <Zhiqiang.Hou@freescale.com>
> > Subject: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
> >
> > From: Hou Zhiqiang <Zhiqiang.Hou@freescale.com>
> >
> > Only the value PCIE_LTSSM_L0 stands for the link is up, instead of >=
> > PCIE_LTSSM_L0.
> >
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@freescale.com>
> > ---
> > 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
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
2016-02-01 4:07 ` Scott Wood
@ 2016-02-03 21:32 ` Edward L Swarthout
2016-02-03 23:03 ` Scott Wood
0 siblings, 1 reply; 5+ messages in thread
From: Edward L Swarthout @ 2016-02-03 21:32 UTC (permalink / raw)
To: Scott Wood, Zhiqiang Hou, linuxppc-dev, mpe; +Cc: Mingkai.Hu
From: Scott Wood:
>On Mon, 2016-02-01 at 03:57 +0000, Zhiqiang Hou wrote:
>> 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 stat=
ement
> of what the actual symptoms are.
>
> > -----Original Message-----
> > From: Zhiqiang Hou [mailto:Zhiqiang.Hou@freescale.com]
> > To: linuxppc-dev@lists.ozlabs.org; benh@kernel.crashing.org;
> > Subject: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
> >
> > Only the value PCIE_LTSSM_L0 stands for the link is up, instead of >=3D
> > PCIE_LTSSM_L0.
> >
> > diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pc=
i.c
> > @@ -73,7 +73,7 @@ static int fsl_pcie_check_link(struct pci_controller
> > early_read_config_dword(hose, 0, 0, PCIE_LTSSM,
> > - if (val < PCIE_LTSSM_L0)
> > + if (val !=3D PCIE_LTSSM_L0)
> > return 1;
This patch is incorrect as there are actually two states L0 and L0s (standb=
y)=20
that indicate an active link (a LTSSM value of 0x11 or 0x12).
The next user manual revision should contain this detail.
Ed
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
2016-02-03 21:32 ` Edward L Swarthout
@ 2016-02-03 23:03 ` Scott Wood
0 siblings, 0 replies; 5+ messages in thread
From: Scott Wood @ 2016-02-03 23:03 UTC (permalink / raw)
To: Edward L Swarthout, Zhiqiang Hou, linuxppc-dev, mpe; +Cc: Mingkai.Hu
On Wed, 2016-02-03 at 21:32 +0000, Edward L Swarthout wrote:
> From: Scott Wood:
> > On Mon, 2016-02-01 at 03:57 +0000, Zhiqiang Hou wrote:
> > > 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.
> >
> > > -----Original Message-----
> > > From: Zhiqiang Hou [mailto:Zhiqiang.Hou@freescale.com]
> > > To: linuxppc-dev@lists.ozlabs.org; benh@kernel.crashing.org;
> > > Subject: [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM
> > >
> > > Only the value PCIE_LTSSM_L0 stands for the link is up, instead of >=
> > > PCIE_LTSSM_L0.
> > >
> > > diff --git 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
> > > early_read_config_dword(hose, 0, 0, PCIE_LTSSM,
> > > - if (val < PCIE_LTSSM_L0)
> > > + if (val != PCIE_LTSSM_L0)
> > > return 1;
>
> This patch is incorrect as there are actually two states L0 and L0s
> (standby)
> that indicate an active link (a LTSSM value of 0x11 or 0x12).
>
> The next user manual revision should contain this detail.
The next manual of what chip?
Are there any non-active states for which the current test does not work?
-Scott
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-02-03 23:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-04 7:45 [PATCH] powerpc/fsl-pci: Correct the comparison of the LTSSM Zhiqiang Hou
2016-02-01 3:57 ` Zhiqiang Hou
2016-02-01 4:07 ` Scott Wood
2016-02-03 21:32 ` Edward L Swarthout
2016-02-03 23:03 ` Scott Wood
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.