diff for duplicates of <1484735769.26620.69.camel@baylibre.com> diff --git a/a/content_digest b/N1/content_digest index 3530135..aa6a2ee 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -5,7 +5,7 @@ "From\0jbrunet@baylibre.com (Jerome Brunet)\0" "Subject\0[PATCH 3/5] ARM64: dts: meson-gxbb-p20x: add reset for the ethernet PHY\0" "Date\0Wed, 18 Jan 2017 11:36:09 +0100\0" - "To\0linus-amlogic@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Tue, 2017-01-17 at 22:26 +0100, Martin Blumenstingl wrote:\n" @@ -149,4 +149,4 @@ "> [1] https://github.com/khadas/linux/blob/Vim/arch/arm64/boot/dts/amlo\n" > gic/gxbb_p201.dts#L186 -9e0a2720316dc889d52c8e6f69076e5b219868b32ba5163c7dc62d8ea18ccffa +bf944b87f2febfa5c15cf654c554a5f4a3999a2901d7280d2e07a0cac4f08941
diff --git a/a/1.txt b/N2/1.txt index 2d81043..41e41d2 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,5 +1,5 @@ On Tue, 2017-01-17 at 22:26 +0100, Martin Blumenstingl wrote: -> On Tue, Jan 17, 2017 at 8:22 PM, Jerome Brunet <jbrunet@baylibre.com> +> On Tue, Jan 17, 2017 at 8:22 PM, Jerome Brunet <jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> > wrote: > > > > On Sat, 2016-12-03 at 00:47 +0100, Martin Blumenstingl wrote: @@ -14,11 +14,11 @@ On Tue, 2017-01-17 at 22:26 +0100, Martin Blumenstingl wrote: > > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemai > > > l.co > > > m> -> > > Tested-by: Neil Armstrong <narmstrong@baylibre.com> +> > > Tested-by: Neil Armstrong <narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> > > > --- -> > > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 15 +> > > arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 15 > > > +++++++++++++++ -> > > ?1 file changed, 15 insertions(+) +> > > 1 file changed, 15 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi > > > b/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi @@ -26,13 +26,13 @@ On Tue, 2017-01-17 at 22:26 +0100, Martin Blumenstingl wrote: > > > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi > > > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi > > > @@ -134,10 +134,25 @@ -> > > ??????pinctrl-names = "default"; -> > > ?}; +> > > pinctrl-names = "default"; +> > > }; > > > > > > +&mdio0 { -> > > +?????ethernet_phy0: ethernet-phy at 0 { -> > > +?????????????compatible = "ethernet-phy-ieee802.3-c22"; -> > > +?????????????reg = <0>; +> > > + ethernet_phy0: ethernet-phy@0 { +> > > + compatible = "ethernet-phy-ieee802.3-c22"; +> > > + reg = <0>; > > > > Sorry for the late reply. > > I just tried on the p200 and this patch (serie) breaks the network @@ -63,21 +63,21 @@ IP101GR from what I can see on the board. > > > > > > -> > > +?????}; +> > > + }; > > > +}; > > > + -> > > ?ðmac { -> > > ??????status = "okay"; -> > > ??????pinctrl-0 = <ð_rgmii_pins>; -> > > ??????pinctrl-names = "default"; +> > > ðmac { +> > > status = "okay"; +> > > pinctrl-0 = <ð_rgmii_pins>; +> > > pinctrl-names = "default"; > > > + -> > > +?????phy-handle = <ðernet_phy0>; +> > > + phy-handle = <ðernet_phy0>; > > > + -> > > +?????snps,reset-gpio = <&gpio GPIOZ_14 0>; -> > > +?????snps,reset-delays-us = <0 10000 1000000>; -> > > +?????snps,reset-active-low; +> > > + snps,reset-gpio = <&gpio GPIOZ_14 0>; +> > > + snps,reset-delays-us = <0 10000 1000000>; +> > > + snps,reset-active-low; > > > + -> > > +?????phy-mode = "rgmii"; +> > > + phy-mode = "rgmii"; > > > > We can't define this in p20x. actually the p201 uses an rmii. > > I have not idea about gpio reset, or the phy address for the p201. @@ -126,7 +126,7 @@ him/her, and get it working properly. After all, that's not a regression. With what we had in the meson-gxbb- p20x.dtsi so far, ethernet never worked on that board. -? + > > does that make sense? > @@ -138,3 +138,7 @@ p20x.dtsi so far, ethernet never worked on that board. > [0] https://github.com/superna9999/linux/wiki/Boards#nexbox-a95x-s905 > [1] https://github.com/khadas/linux/blob/Vim/arch/arm64/boot/dts/amlo > gic/gxbb_p201.dts#L186 +-- +To unsubscribe from this list: send the line "unsubscribe devicetree" in +the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N2/content_digest index 3530135..0d44d17 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -2,14 +2,24 @@ "ref\020161202234739.22929-4-martin.blumenstingl@googlemail.com\0" "ref\01484680941.26620.28.camel@baylibre.com\0" "ref\0CAFBinCBfJ4b1a1Bqn7kJHyj79i8UJnjJh00u3NvARzJPuUmrNA@mail.gmail.com\0" - "From\0jbrunet@baylibre.com (Jerome Brunet)\0" - "Subject\0[PATCH 3/5] ARM64: dts: meson-gxbb-p20x: add reset for the ethernet PHY\0" + "ref\0CAFBinCBfJ4b1a1Bqn7kJHyj79i8UJnjJh00u3NvARzJPuUmrNA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org\0" + "From\0Jerome Brunet <jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>\0" + "Subject\0Re: [PATCH 3/5] ARM64: dts: meson-gxbb-p20x: add reset for the ethernet PHY\0" "Date\0Wed, 18 Jan 2017 11:36:09 +0100\0" - "To\0linus-amlogic@lists.infradead.org\0" + "To\0Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>\0" + "Cc\0linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" + khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org + carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + will.deacon-5wv7dgnIgG8@public.gmane.org + catalin.marinas-5wv7dgnIgG8@public.gmane.org + mark.rutland-5wv7dgnIgG8@public.gmane.org + " robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org\0" "\00:1\0" "b\0" "On Tue, 2017-01-17 at 22:26 +0100, Martin Blumenstingl wrote:\n" - "> On Tue, Jan 17, 2017 at 8:22 PM, Jerome Brunet <jbrunet@baylibre.com>\n" + "> On Tue, Jan 17, 2017 at 8:22 PM, Jerome Brunet <jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>\n" "> wrote:\n" "> > \n" "> > On Sat, 2016-12-03 at 00:47 +0100, Martin Blumenstingl wrote:\n" @@ -24,11 +34,11 @@ "> > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemai\n" "> > > l.co\n" "> > > m>\n" - "> > > Tested-by: Neil Armstrong <narmstrong@baylibre.com>\n" + "> > > Tested-by: Neil Armstrong <narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>\n" "> > > ---\n" - "> > > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 15\n" + "> > > \302\240arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 15\n" "> > > +++++++++++++++\n" - "> > > ?1 file changed, 15 insertions(+)\n" + "> > > \302\2401 file changed, 15 insertions(+)\n" "> > > \n" "> > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi\n" "> > > b/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi\n" @@ -36,13 +46,13 @@ "> > > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi\n" "> > > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi\n" "> > > @@ -134,10 +134,25 @@\n" - "> > > ??????pinctrl-names = \"default\";\n" - "> > > ?};\n" + "> > > \302\240\302\240\302\240\302\240\302\240\302\240pinctrl-names = \"default\";\n" + "> > > \302\240};\n" "> > > \n" "> > > +&mdio0 {\n" - "> > > +?????ethernet_phy0: ethernet-phy at 0 {\n" - "> > > +?????????????compatible = \"ethernet-phy-ieee802.3-c22\";\n" - "> > > +?????????????reg = <0>;\n" + "> > > +\302\240\302\240\302\240\302\240\302\240ethernet_phy0: ethernet-phy@0 {\n" + "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240compatible = \"ethernet-phy-ieee802.3-c22\";\n" + "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240reg = <0>;\n" "> > \n" "> > Sorry for the late reply.\n" "> > I just tried on the p200 and this patch (serie) breaks the network\n" @@ -73,21 +83,21 @@ "> \n" "> > \n" "> > > \n" - "> > > +?????};\n" + "> > > +\302\240\302\240\302\240\302\240\302\240};\n" "> > > +};\n" "> > > +\n" - "> > > ?ðmac {\n" - "> > > ??????status = \"okay\";\n" - "> > > ??????pinctrl-0 = <ð_rgmii_pins>;\n" - "> > > ??????pinctrl-names = \"default\";\n" + "> > > \302\240ðmac {\n" + "> > > \302\240\302\240\302\240\302\240\302\240\302\240status = \"okay\";\n" + "> > > \302\240\302\240\302\240\302\240\302\240\302\240pinctrl-0 = <ð_rgmii_pins>;\n" + "> > > \302\240\302\240\302\240\302\240\302\240\302\240pinctrl-names = \"default\";\n" "> > > +\n" - "> > > +?????phy-handle = <ðernet_phy0>;\n" + "> > > +\302\240\302\240\302\240\302\240\302\240phy-handle = <ðernet_phy0>;\n" "> > > +\n" - "> > > +?????snps,reset-gpio = <&gpio GPIOZ_14 0>;\n" - "> > > +?????snps,reset-delays-us = <0 10000 1000000>;\n" - "> > > +?????snps,reset-active-low;\n" + "> > > +\302\240\302\240\302\240\302\240\302\240snps,reset-gpio = <&gpio GPIOZ_14 0>;\n" + "> > > +\302\240\302\240\302\240\302\240\302\240snps,reset-delays-us = <0 10000 1000000>;\n" + "> > > +\302\240\302\240\302\240\302\240\302\240snps,reset-active-low;\n" "> > > +\n" - "> > > +?????phy-mode = \"rgmii\";\n" + "> > > +\302\240\302\240\302\240\302\240\302\240phy-mode = \"rgmii\";\n" "> > \n" "> > We can't define this in p20x. actually the p201 uses an rmii.\n" "> > I have not idea about gpio reset, or the phy address for the p201.\n" @@ -136,7 +146,7 @@ "\n" "After all, that's not a regression. With what we had in the meson-gxbb-\n" "p20x.dtsi so far, ethernet never worked on that board.\n" - "?\n" + "\302\240\n" "> \n" "> does that make sense?\n" "> \n" @@ -147,6 +157,10 @@ "> \n" "> [0] https://github.com/superna9999/linux/wiki/Boards#nexbox-a95x-s905\n" "> [1] https://github.com/khadas/linux/blob/Vim/arch/arm64/boot/dts/amlo\n" - > gic/gxbb_p201.dts#L186 + "> gic/gxbb_p201.dts#L186\n" + "--\n" + "To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" + "the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -9e0a2720316dc889d52c8e6f69076e5b219868b32ba5163c7dc62d8ea18ccffa +e104eeef3e53ac8413b67e48435b12232e1fd37a51237c803444334b00184f30
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.