Re: [PATCH v2 1/1] PCI: imx6: Add pcie compliance test option

["Schöfegger Stefan" <Stefan.Schoefegger@ginzinger.com>]

On Wednesday, June 14, 2017 4:11:29 PM CEST Fabio Estevam wrote:
> On Wed, Jun 14, 2017 at 2:52 AM, Sch=F6fegger Stefan
>=20
> <Stefan.Schoefegger@ginzinger.com> wrote:
> > device). The bitrate and de-emphasis must be switched. The driver (with=
out
> > this patches) does not allow to switch to gen2 because it falls back to
> > gen1. It is impossible to generate the gen2 test pattern.
>=20
> If you pass 'fsl,max-link-speed =3D <2>;' in your device tree, then it
> will allow gen2.
>=20
> From Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt:
>=20
> "- fsl,max-link-speed: Specify PCI gen for link capability. Must be '2' f=
or
>   gen2, otherwise will default to gen1. Note that the IMX6 LVDS clock
> outputs do not meet gen2 jitter requirements and thus for gen2 capability=
 a
> gen2 compliant clock generator should be used and configured."
>=20
> Wouldn't this solve the problem?

No, it only sets the maximum allowed link speed but it is forced to gen1=20
before.

During compliance test there is no real link established and there is no li=
nk=20
speed negotiation, the phy is in a special compliance test state.

Stefan