[PATCH] ARM: tegra: Fix Beaver's PCIe lane configuration

[PATCH] ARM: tegra: Fix Beaver's PCIe lane configuration

[Stephen Warren]

From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Beaver's PCIe lane configuration most closely matches x2 x2 x2 rather
than x4 x1 x1, since clocks 0 and 2 are used, and lanes 0 and 5 are used,
and the only way those align is with a x2 x2 x2 configuration.

Also, disable root port 1; there's nothing connected to it. Root port 0
is the on-board PCIe Ethernet, and port 2 is the mini-PCIe slot.

Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
This is to be applied to Thierry's WIP PCIe driver branch.

 arch/arm/boot/dts/tegra30-beaver.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
index c42dd9b..e0b8e9c 100644
--- a/arch/arm/boot/dts/tegra30-beaver.dts
+++ b/arch/arm/boot/dts/tegra30-beaver.dts
@@ -18,16 +18,16 @@
 
 		pci@1,0 {
 			status = "okay";
-			nvidia,num-lanes = <4>;
+			nvidia,num-lanes = <2>;
 		};
 
 		pci@2,0 {
-			status = "okay";
-			nvidia,num-lanes = <1>;
+			nvidia,num-lanes = <2>;
 		};
 
 		pci@3,0 {
-			nvidia,num-lanes = <1>;
+			status = "okay";
+			nvidia,num-lanes = <2>;
 		};
 	};
 
-- 
1.8.1.5

Re: [PATCH] ARM: tegra: Fix Beaver's PCIe lane configuration

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 1042 bytes --]

On Fri, Jun 21, 2013 at 02:23:52PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> Beaver's PCIe lane configuration most closely matches x2 x2 x2 rather
> than x4 x1 x1, since clocks 0 and 2 are used, and lanes 0 and 5 are used,
> and the only way those align is with a x2 x2 x2 configuration.

Looking at the schematics again I have to agree. Thanks for catching
this.

> Also, disable root port 1; there's nothing connected to it. Root port 0
> is the on-board PCIe Ethernet, and port 2 is the mini-PCIe slot.
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> This is to be applied to Thierry's WIP PCIe driver branch.

I've applied this to my tegra/next branch and will squash it into the
Beaver patch that's already there.

You mention another fix that might be required for PCIe on Tegra30
(additional write to the PADS_REFCLK_CFG1) register. Did that turn out
to fix communication of the third port?

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH] ARM: tegra: Fix Beaver's PCIe lane configuration

[Stephen Warren]

On 06/22/2013 04:45 AM, Thierry Reding wrote:
> On Fri, Jun 21, 2013 at 02:23:52PM -0600, Stephen Warren wrote:
>> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> 
>> Beaver's PCIe lane configuration most closely matches x2 x2 x2
>> rather than x4 x1 x1, since clocks 0 and 2 are used, and lanes 0
>> and 5 are used, and the only way those align is with a x2 x2 x2
>> configuration.
> 
> Looking at the schematics again I have to agree. Thanks for
> catching this.
> 
>> Also, disable root port 1; there's nothing connected to it. Root
>> port 0 is the on-board PCIe Ethernet, and port 2 is the mini-PCIe
>> slot.
>> 
>> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> --- This is to
>> be applied to Thierry's WIP PCIe driver branch.
> 
> I've applied this to my tegra/next branch and will squash it into
> the Beaver patch that's already there.
> 
> You mention another fix that might be required for PCIe on Tegra30 
> (additional write to the PADS_REFCLK_CFG1) register. Did that turn
> out to fix communication of the third port?

Yes, I have the 3rd port working. I'm just waiting on some internal IP
review in order to say something more useful in the commit description
than "here's an opaque register write that fixes something".

Re: [PATCH] ARM: tegra: Fix Beaver's PCIe lane configuration

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 1641 bytes --]

On Sun, Jun 23, 2013 at 10:21:07PM -0600, Stephen Warren wrote:
> On 06/22/2013 04:45 AM, Thierry Reding wrote:
> > On Fri, Jun 21, 2013 at 02:23:52PM -0600, Stephen Warren wrote:
> >> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >> 
> >> Beaver's PCIe lane configuration most closely matches x2 x2 x2
> >> rather than x4 x1 x1, since clocks 0 and 2 are used, and lanes 0
> >> and 5 are used, and the only way those align is with a x2 x2 x2
> >> configuration.
> > 
> > Looking at the schematics again I have to agree. Thanks for
> > catching this.
> > 
> >> Also, disable root port 1; there's nothing connected to it. Root
> >> port 0 is the on-board PCIe Ethernet, and port 2 is the mini-PCIe
> >> slot.
> >> 
> >> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> --- This is to
> >> be applied to Thierry's WIP PCIe driver branch.
> > 
> > I've applied this to my tegra/next branch and will squash it into
> > the Beaver patch that's already there.
> > 
> > You mention another fix that might be required for PCIe on Tegra30 
> > (additional write to the PADS_REFCLK_CFG1) register. Did that turn
> > out to fix communication of the third port?
> 
> Yes, I have the 3rd port working. I'm just waiting on some internal IP
> review in order to say something more useful in the commit description
> than "here's an opaque register write that fixes something".

Perhaps you should say something more useful in the comment rather than
the commit message, given that I'll probably squash the patch into one
of the others before submitting.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]