diff mbox

[1/1] undo workaround for i.mx6 silicon errata ERR006687 which resulted in an unstable ethernet connection (lost interrupts?)

Message ID 1407515584.19455.9.camel@sharkbay.at (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Scheiblauer Aug. 8, 2014, 4:33 p.m. UTC
> > 
> > On Fre, 2014-08-08 at 12:46 -0300, Fabio Estevam wrote:
> > Hi Thomas,
> > 
> > On Fri, Aug 8, 2014 at 11:58 AM, Thomas Scheiblauer <tom@sharkbay.at> wrote:
> > 
> > >> So, what I'm asking is to try without this workaround in case there is
> > >> some interaction there, thus testing the same interrupt configuration
> > >> that I've had here for the last 10 months without the kind of issues
> > >> you're reporting.  Sometimes, workarounds for one problem end up causing
> > >> other problem elsewhere.
> > >>
> > >
> > > Indeed, commenting out these 2 lines in
> > > arch/arm/boot/dts/imx6qdl-wandboard.dtsi fixes my problem without having
> > > to increase TX_RING_SIZE.
> > > Thank you very much!
> > 
> > Could you please send a patch for imx6qdl-wandboard.dtsi? Please
> > provide the details of the problem in the commit log.
> > 
> > Thanks

Ok, here is my attempt on the patch. I've also removed the respective lines from all the other imx6qdl-*.dtsi files.

Signed-off-by: Thomas Scheiblauer <tom@sharkbay.at>
---
 arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 2 --
 arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 2 --
 arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 2 --
 arch/arm/boot/dts/imx6qdl-wandboard.dtsi  | 2 --
 4 files changed, 8 deletions(-)

Comments

Fabio Estevam Aug. 8, 2014, 5:01 p.m. UTC | #1
Hi Thomas,

On Fri, Aug 8, 2014 at 1:33 PM, Thomas Scheiblauer <tom@sharkbay.at> wrote:

> Ok, here is my attempt on the patch. I've also removed the respective lines from all the other imx6qdl-*.dtsi files.

Please create a clean patch and submit it via git send-mail instead of
sending it inside this thread.

Also, please check the style of the Subject. It should be "ARM: dts:
imx6qdl-wandboard: ....."

You need to copy the imx maintainer Shawn Guo and it would be nice to
also copy commit Sascha Silbe, who introduced this patch for
wandboard:

9fc77821b17155c6e0ab50b1e1dd80c2b0e63e98
Author: Sascha Silbe <x-linux@infra-silbe.de>
Date:   Thu Feb 6 23:24:13 2014 +0100

    ARM: dts: imx6qdl-wandboard: use GPIO_6 for FEC interrupt

, and also Troy Kisky to get their inputs.

>
> Signed-off-by: Thomas Scheiblauer <tom@sharkbay.at>
> ---
>  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 2 --
>  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 2 --
>  arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 2 --
>  arch/arm/boot/dts/imx6qdl-wandboard.dtsi  | 2 --
>  4 files changed, 8 deletions(-)

I would suggest by starting only with
arch/arm/boot/dts/imx6qdl-wandboard.dtsi as this is the one you have
really tested.

> diff --git a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
> b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
> index 6461f66..3b61c9a 100644
> --- a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
> @@ -257,8 +257,6 @@
>         pinctrl-0 = <&pinctrl_enet>;
>         phy-mode = "rgmii";
>         phy-reset-gpios = <&gpio3 29 0>;
> -       interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>,
> -                             <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>;

You should also remove the MX6QDL_PAD_GPIO_6__ENET_IRQ.

Actually it would easier if you just send a patch reverting 9fc77821b17.

You need to put a detailed commit log explaining what are the tests
you are running that show an unstable ethernet connection, add the
logs, command line tests, etc.

Thanks
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
index 42ff525..cf4e6db 100644
--- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
@@ -169,8 +169,6 @@ 
 	txd1-skew-ps = <0>;
 	txd2-skew-ps = <0>;
 	txd3-skew-ps = <0>;
-	interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>,
-			      <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>;
 	status = "okay";
 };
 
diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
index 009abd6..f47e82f 100644
--- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
@@ -65,8 +65,6 @@ 
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_enet>;
 	phy-mode = "rgmii";
-	interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>,
-			      <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>;
 	status = "okay";
 };
 
diff --git a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
index 0a36129..ebfcab4 100644
--- a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
@@ -168,8 +168,6 @@ 
 	txd1-skew-ps = <0>;
 	txd2-skew-ps = <0>;
 	txd3-skew-ps = <0>;
-	interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>,
-			      <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>;
 	status = "okay";
 };
 
diff --git a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
index 6461f66..3b61c9a 100644
--- a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi
@@ -257,8 +257,6 @@ 
 	pinctrl-0 = <&pinctrl_enet>;
 	phy-mode = "rgmii";
 	phy-reset-gpios = <&gpio3 29 0>;
-	interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>,
-			      <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>;
 	status = "okay";
 };