From mboxrd@z Thu Jan 1 00:00:00 1970 From: jbrunet@baylibre.com (Jerome Brunet) Date: Fri, 20 Jan 2017 16:27:10 +0100 Subject: [PATCH v3 0/9] meson-gx: reset RGMII PHYs and configure TX delay In-Reply-To: <20170120152232.13943-1-martin.blumenstingl@googlemail.com> References: <20170120132650.9784-1-martin.blumenstingl@googlemail.com> <20170120152232.13943-1-martin.blumenstingl@googlemail.com> Message-ID: <1484926030.26620.80.camel@baylibre.com> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org On Fri, 2017-01-20 at 16:22 +0100, Martin Blumenstingl wrote: > This series adds the reset GPIOs for the (external) ethernet PHYs on > all > GXBB boards. > Additionally it provides a ethernet PHY node which can be used to > specify > PHY-specific properties (this may be required if more boards require > the > "eee-broken-1000t" for the RTL8211F ethernet PHY). To make all board > .dts > consistent I chose to add the PHY node also for boards which don't > have a > RTL8211F PHY. > > Patch #7 from this series also removes ethernet support for the P200 > board because it was broken anyways and nobody seems to have a board > available for testing. This was the outcome of the discussion from > [0] > > Patch #8 was taken from (older versions of) my other series (see > [1]): > "[PATCH net-next v3 0/2] stmmac: dwmac-meson8b: configurable > RGMII TX delay". > The binding changes for amlogic,tx-delay-ns were ACK'ed already. > > Changes since v2: > - fixed patch #2 to really update the P200 board (as it's the P200 > which > ? actually uses a Micrel KSZ9031 RGMII PHY). Thanks to Neil Armstrong > for > ? spotting this huge mistake! > - instead of dropping ethernet support for P201 we simply re- > configure it > ? to RMII mode (replaces patch #7) without a MDIO node (which keeps > PHY > ? auto-scanning enabled). This got Neil Armstrong's ACK off-list > already > > Changes since v1: > - do not move the MDIO bus to meson-gx as this disables PHY auto- > scanning > ? in the stmmac driver (this drops patch #1 from v1) > - add the ethernet PHY reset GPIO for nexbox a95x which was forgotten > in > ? v1 > - add the ethernet PHY reset GPIO for boards which were added since > v1 > ? (wetek hub and wetek play2) > - rebased to apply against the current v4.11/dt64 branch > - new in v2 (patch #7): disabled ethernet support for the P200 board > (see > ? the commit description for more information). this patch is > optional > - new in v2 (patch #9): removed the phy-mode property from meson-gx > (see > ? the commit description for more information). this patch is > optional > > > [0] http://lists.infradead.org/pipermail/linux-amlogic/2017-January/0 > 02053.html > [1] http://lists.infradead.org/pipermail/linux-amlogic/2016-December/ > 001834.html > > Martin Blumenstingl (9): > ? ARM64: dts: meson-gxbb-odroidc2: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-p200: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-vega-s95: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-nexbox-a95x: add the ethernet PHY's reset > GPIO > ? ARM64: dts: meson-gxbb-wetek-hub: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-wetek-play2: add the ethernet PHY's reset > GPIO > ? ARM64: dts: meson-gxbb-p201: fix ethernet support > ? ARM64: dts: amlogic: add the ethernet TX delay configuration > ? ARM64: dts: meson-gx: remove the phy-mode property from meson-gx > > ?arch/arm64/boot/dts/amlogic/meson-gx.dtsi??????????|??1 - > ?.../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts????| 17 > ++++++++++++++ > ?.../arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts |??7 ++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts????| 26 > ++++++++++++++++++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dts????| 11 +++++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi???|??6 ----- > ?.../boot/dts/amlogic/meson-gxbb-vega-s95.dtsi??????| 20 > +++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxbb-wetek-hub.dts??????| 26 > ++++++++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxbb-wetek-play2.dts????| 26 > ++++++++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxl-s905d-p230.dts??????|??2 ++ > ?.../arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts |??2 ++ > ?arch/arm64/boot/dts/amlogic/meson-gxm-q200.dts?????|??2 ++ > ?12 files changed, 139 insertions(+), 7 deletions(-) Looks good. Reviewed-by: Jerome Brunet > From mboxrd@z Thu Jan 1 00:00:00 1970 From: jbrunet@baylibre.com (Jerome Brunet) Date: Fri, 20 Jan 2017 16:27:10 +0100 Subject: [PATCH v3 0/9] meson-gx: reset RGMII PHYs and configure TX delay In-Reply-To: <20170120152232.13943-1-martin.blumenstingl@googlemail.com> References: <20170120132650.9784-1-martin.blumenstingl@googlemail.com> <20170120152232.13943-1-martin.blumenstingl@googlemail.com> Message-ID: <1484926030.26620.80.camel@baylibre.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, 2017-01-20 at 16:22 +0100, Martin Blumenstingl wrote: > This series adds the reset GPIOs for the (external) ethernet PHYs on > all > GXBB boards. > Additionally it provides a ethernet PHY node which can be used to > specify > PHY-specific properties (this may be required if more boards require > the > "eee-broken-1000t" for the RTL8211F ethernet PHY). To make all board > .dts > consistent I chose to add the PHY node also for boards which don't > have a > RTL8211F PHY. > > Patch #7 from this series also removes ethernet support for the P200 > board because it was broken anyways and nobody seems to have a board > available for testing. This was the outcome of the discussion from > [0] > > Patch #8 was taken from (older versions of) my other series (see > [1]): > "[PATCH net-next v3 0/2] stmmac: dwmac-meson8b: configurable > RGMII TX delay". > The binding changes for amlogic,tx-delay-ns were ACK'ed already. > > Changes since v2: > - fixed patch #2 to really update the P200 board (as it's the P200 > which > ? actually uses a Micrel KSZ9031 RGMII PHY). Thanks to Neil Armstrong > for > ? spotting this huge mistake! > - instead of dropping ethernet support for P201 we simply re- > configure it > ? to RMII mode (replaces patch #7) without a MDIO node (which keeps > PHY > ? auto-scanning enabled). This got Neil Armstrong's ACK off-list > already > > Changes since v1: > - do not move the MDIO bus to meson-gx as this disables PHY auto- > scanning > ? in the stmmac driver (this drops patch #1 from v1) > - add the ethernet PHY reset GPIO for nexbox a95x which was forgotten > in > ? v1 > - add the ethernet PHY reset GPIO for boards which were added since > v1 > ? (wetek hub and wetek play2) > - rebased to apply against the current v4.11/dt64 branch > - new in v2 (patch #7): disabled ethernet support for the P200 board > (see > ? the commit description for more information). this patch is > optional > - new in v2 (patch #9): removed the phy-mode property from meson-gx > (see > ? the commit description for more information). this patch is > optional > > > [0] http://lists.infradead.org/pipermail/linux-amlogic/2017-January/0 > 02053.html > [1] http://lists.infradead.org/pipermail/linux-amlogic/2016-December/ > 001834.html > > Martin Blumenstingl (9): > ? ARM64: dts: meson-gxbb-odroidc2: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-p200: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-vega-s95: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-nexbox-a95x: add the ethernet PHY's reset > GPIO > ? ARM64: dts: meson-gxbb-wetek-hub: add the ethernet PHY's reset GPIO > ? ARM64: dts: meson-gxbb-wetek-play2: add the ethernet PHY's reset > GPIO > ? ARM64: dts: meson-gxbb-p201: fix ethernet support > ? ARM64: dts: amlogic: add the ethernet TX delay configuration > ? ARM64: dts: meson-gx: remove the phy-mode property from meson-gx > > ?arch/arm64/boot/dts/amlogic/meson-gx.dtsi??????????|??1 - > ?.../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts????| 17 > ++++++++++++++ > ?.../arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts |??7 ++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts????| 26 > ++++++++++++++++++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dts????| 11 +++++++++ > ?arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi???|??6 ----- > ?.../boot/dts/amlogic/meson-gxbb-vega-s95.dtsi??????| 20 > +++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxbb-wetek-hub.dts??????| 26 > ++++++++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxbb-wetek-play2.dts????| 26 > ++++++++++++++++++++++ > ?.../boot/dts/amlogic/meson-gxl-s905d-p230.dts??????|??2 ++ > ?.../arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts |??2 ++ > ?arch/arm64/boot/dts/amlogic/meson-gxm-q200.dts?????|??2 ++ > ?12 files changed, 139 insertions(+), 7 deletions(-) Looks good. Reviewed-by: Jerome Brunet > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerome Brunet Subject: Re: [PATCH v3 0/9] meson-gx: reset RGMII PHYs and configure TX delay Date: Fri, 20 Jan 2017 16:27:10 +0100 Message-ID: <1484926030.26620.80.camel@baylibre.com> References: <20170120132650.9784-1-martin.blumenstingl@googlemail.com> <20170120152232.13943-1-martin.blumenstingl@googlemail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170120152232.13943-1-martin.blumenstingl@googlemail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Martin Blumenstingl , linux-amlogic@lists.infradead.org, khilman@baylibre.com, carlo@caione.org Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, narmstrong@baylibre.com, catalin.marinas@arm.com, will.deacon@arm.com, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org T24gRnJpLCAyMDE3LTAxLTIwIGF0IDE2OjIyICswMTAwLCBNYXJ0aW4gQmx1bWVuc3RpbmdsIHdy b3RlOgo+IFRoaXMgc2VyaWVzIGFkZHMgdGhlIHJlc2V0IEdQSU9zIGZvciB0aGUgKGV4dGVybmFs KSBldGhlcm5ldCBQSFlzIG9uCj4gYWxsCj4gR1hCQiBib2FyZHMuCj4gQWRkaXRpb25hbGx5IGl0 IHByb3ZpZGVzIGEgZXRoZXJuZXQgUEhZIG5vZGUgd2hpY2ggY2FuIGJlIHVzZWQgdG8KPiBzcGVj aWZ5Cj4gUEhZLXNwZWNpZmljIHByb3BlcnRpZXMgKHRoaXMgbWF5IGJlIHJlcXVpcmVkIGlmIG1v cmUgYm9hcmRzIHJlcXVpcmUKPiB0aGUKPiAiZWVlLWJyb2tlbi0xMDAwdCIgZm9yIHRoZSBSVEw4 MjExRiBldGhlcm5ldCBQSFkpLiBUbyBtYWtlIGFsbCBib2FyZAo+IC5kdHMKPiBjb25zaXN0ZW50 IEkgY2hvc2UgdG8gYWRkIHRoZSBQSFkgbm9kZSBhbHNvIGZvciBib2FyZHMgd2hpY2ggZG9uJ3QK PiBoYXZlIGEKPiBSVEw4MjExRiBQSFkuCj4gCj4gUGF0Y2ggIzcgZnJvbSB0aGlzIHNlcmllcyBh bHNvIHJlbW92ZXMgZXRoZXJuZXQgc3VwcG9ydCBmb3IgdGhlIFAyMDAKPiBib2FyZCBiZWNhdXNl IGl0IHdhcyBicm9rZW4gYW55d2F5cyBhbmQgbm9ib2R5IHNlZW1zIHRvIGhhdmUgYSBib2FyZAo+ IGF2YWlsYWJsZSBmb3IgdGVzdGluZy4gVGhpcyB3YXMgdGhlIG91dGNvbWUgb2YgdGhlIGRpc2N1 c3Npb24gZnJvbQo+IFswXQo+IAo+IFBhdGNoICM4IHdhcyB0YWtlbiBmcm9tIChvbGRlciB2ZXJz aW9ucyBvZikgbXkgb3RoZXIgc2VyaWVzIChzZWUKPiBbMV0pOgo+ICJbUEFUQ0ggbmV0LW5leHQg djMgMC8yXSBzdG1tYWM6IGR3bWFjLW1lc29uOGI6IGNvbmZpZ3VyYWJsZQo+IFJHTUlJIFRYIGRl bGF5Ii4KPiBUaGUgYmluZGluZyBjaGFuZ2VzIGZvciBhbWxvZ2ljLHR4LWRlbGF5LW5zIHdlcmUg QUNLJ2VkIGFscmVhZHkuCj4gCj4gQ2hhbmdlcyBzaW5jZSB2MjoKPiAtIGZpeGVkIHBhdGNoICMy IHRvIHJlYWxseSB1cGRhdGUgdGhlIFAyMDAgYm9hcmQgKGFzIGl0J3MgdGhlIFAyMDAKPiB3aGlj aAo+IMKgIGFjdHVhbGx5IHVzZXMgYSBNaWNyZWwgS1NaOTAzMSBSR01JSSBQSFkpLiBUaGFua3Mg dG8gTmVpbCBBcm1zdHJvbmcKPiBmb3IKPiDCoCBzcG90dGluZyB0aGlzIGh1Z2UgbWlzdGFrZSEK PiAtIGluc3RlYWQgb2YgZHJvcHBpbmcgZXRoZXJuZXQgc3VwcG9ydCBmb3IgUDIwMSB3ZSBzaW1w bHkgcmUtCj4gY29uZmlndXJlIGl0Cj4gwqAgdG8gUk1JSSBtb2RlIChyZXBsYWNlcyBwYXRjaCAj Nykgd2l0aG91dCBhIE1ESU8gbm9kZSAod2hpY2gga2VlcHMKPiBQSFkKPiDCoCBhdXRvLXNjYW5u aW5nIGVuYWJsZWQpLiBUaGlzIGdvdCBOZWlsIEFybXN0cm9uZydzIEFDSyBvZmYtbGlzdAo+IGFs cmVhZHkKPiAKPiBDaGFuZ2VzIHNpbmNlIHYxOgo+IC0gZG8gbm90IG1vdmUgdGhlIE1ESU8gYnVz IHRvIG1lc29uLWd4IGFzIHRoaXMgZGlzYWJsZXMgUEhZIGF1dG8tCj4gc2Nhbm5pbmcKPiDCoCBp biB0aGUgc3RtbWFjIGRyaXZlciAodGhpcyBkcm9wcyBwYXRjaCAjMSBmcm9tIHYxKQo+IC0gYWRk IHRoZSBldGhlcm5ldCBQSFkgcmVzZXQgR1BJTyBmb3IgbmV4Ym94IGE5NXggd2hpY2ggd2FzIGZv cmdvdHRlbgo+IGluCj4gwqAgdjEKPiAtIGFkZCB0aGUgZXRoZXJuZXQgUEhZIHJlc2V0IEdQSU8g Zm9yIGJvYXJkcyB3aGljaCB3ZXJlIGFkZGVkIHNpbmNlCj4gdjEKPiDCoCAod2V0ZWsgaHViIGFu ZCB3ZXRlayBwbGF5MikKPiAtIHJlYmFzZWQgdG8gYXBwbHkgYWdhaW5zdCB0aGUgY3VycmVudCB2 NC4xMS9kdDY0IGJyYW5jaAo+IC0gbmV3IGluIHYyIChwYXRjaCAjNyk6IGRpc2FibGVkIGV0aGVy bmV0IHN1cHBvcnQgZm9yIHRoZSBQMjAwIGJvYXJkCj4gKHNlZQo+IMKgIHRoZSBjb21taXQgZGVz Y3JpcHRpb24gZm9yIG1vcmUgaW5mb3JtYXRpb24pLiB0aGlzIHBhdGNoIGlzCj4gb3B0aW9uYWwK PiAtIG5ldyBpbiB2MiAocGF0Y2ggIzkpOiByZW1vdmVkIHRoZSBwaHktbW9kZSBwcm9wZXJ0eSBm cm9tIG1lc29uLWd4Cj4gKHNlZQo+IMKgIHRoZSBjb21taXQgZGVzY3JpcHRpb24gZm9yIG1vcmUg aW5mb3JtYXRpb24pLiB0aGlzIHBhdGNoIGlzCj4gb3B0aW9uYWwKPiAKPiAKPiBbMF0gaHR0cDov L2xpc3RzLmluZnJhZGVhZC5vcmcvcGlwZXJtYWlsL2xpbnV4LWFtbG9naWMvMjAxNy1KYW51YXJ5 LzAKPiAwMjA1My5odG1sCj4gWzFdIGh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL3BpcGVybWFp bC9saW51eC1hbWxvZ2ljLzIwMTYtRGVjZW1iZXIvCj4gMDAxODM0Lmh0bWwKPiAKPiBNYXJ0aW4g Qmx1bWVuc3RpbmdsICg5KToKPiDCoCBBUk02NDogZHRzOiBtZXNvbi1neGJiLW9kcm9pZGMyOiBh ZGQgdGhlIGV0aGVybmV0IFBIWSdzIHJlc2V0IEdQSU8KPiDCoCBBUk02NDogZHRzOiBtZXNvbi1n eGJiLXAyMDA6IGFkZCB0aGUgZXRoZXJuZXQgUEhZJ3MgcmVzZXQgR1BJTwo+IMKgIEFSTTY0OiBk dHM6IG1lc29uLWd4YmItdmVnYS1zOTU6IGFkZCB0aGUgZXRoZXJuZXQgUEhZJ3MgcmVzZXQgR1BJ Two+IMKgIEFSTTY0OiBkdHM6IG1lc29uLWd4YmItbmV4Ym94LWE5NXg6IGFkZCB0aGUgZXRoZXJu ZXQgUEhZJ3MgcmVzZXQKPiBHUElPCj4gwqAgQVJNNjQ6IGR0czogbWVzb24tZ3hiYi13ZXRlay1o dWI6IGFkZCB0aGUgZXRoZXJuZXQgUEhZJ3MgcmVzZXQgR1BJTwo+IMKgIEFSTTY0OiBkdHM6IG1l c29uLWd4YmItd2V0ZWstcGxheTI6IGFkZCB0aGUgZXRoZXJuZXQgUEhZJ3MgcmVzZXQKPiBHUElP Cj4gwqAgQVJNNjQ6IGR0czogbWVzb24tZ3hiYi1wMjAxOiBmaXggZXRoZXJuZXQgc3VwcG9ydAo+ IMKgIEFSTTY0OiBkdHM6IGFtbG9naWM6IGFkZCB0aGUgZXRoZXJuZXQgVFggZGVsYXkgY29uZmln dXJhdGlvbgo+IMKgIEFSTTY0OiBkdHM6IG1lc29uLWd4OiByZW1vdmUgdGhlIHBoeS1tb2RlIHBy b3BlcnR5IGZyb20gbWVzb24tZ3gKPiAKPiDCoGFyY2gvYXJtNjQvYm9vdC9kdHMvYW1sb2dpYy9t ZXNvbi1neC5kdHNpwqDCoMKgwqDCoMKgwqDCoMKgwqB8wqDCoDEgLQo+IMKgLi4uL2Jvb3QvZHRz L2FtbG9naWMvbWVzb24tZ3hiYi1uZXhib3gtYTk1eC5kdHPCoMKgwqDCoHwgMTcKPiArKysrKysr KysrKysrKwo+IMKgLi4uL2FybTY0L2Jvb3QvZHRzL2FtbG9naWMvbWVzb24tZ3hiYi1vZHJvaWRj Mi5kdHMgfMKgwqA3ICsrKysrKwo+IMKgYXJjaC9hcm02NC9ib290L2R0cy9hbWxvZ2ljL21lc29u LWd4YmItcDIwMC5kdHPCoMKgwqDCoHwgMjYKPiArKysrKysrKysrKysrKysrKysrKysrCj4gwqBh cmNoL2FybTY0L2Jvb3QvZHRzL2FtbG9naWMvbWVzb24tZ3hiYi1wMjAxLmR0c8KgwqDCoMKgfCAx MSArKysrKysrKysKPiDCoGFyY2gvYXJtNjQvYm9vdC9kdHMvYW1sb2dpYy9tZXNvbi1neGJiLXAy MHguZHRzacKgwqDCoHzCoMKgNiAtLS0tLQo+IMKgLi4uL2Jvb3QvZHRzL2FtbG9naWMvbWVzb24t Z3hiYi12ZWdhLXM5NS5kdHNpwqDCoMKgwqDCoMKgfCAyMAo+ICsrKysrKysrKysrKysrKysrCj4g wqAuLi4vYm9vdC9kdHMvYW1sb2dpYy9tZXNvbi1neGJiLXdldGVrLWh1Yi5kdHPCoMKgwqDCoMKg wqB8IDI2Cj4gKysrKysrKysrKysrKysrKysrKysrKwo+IMKgLi4uL2Jvb3QvZHRzL2FtbG9naWMv bWVzb24tZ3hiYi13ZXRlay1wbGF5Mi5kdHPCoMKgwqDCoHwgMjYKPiArKysrKysrKysrKysrKysr KysrKysrCj4gwqAuLi4vYm9vdC9kdHMvYW1sb2dpYy9tZXNvbi1neGwtczkwNWQtcDIzMC5kdHPC oMKgwqDCoMKgwqB8wqDCoDIgKysKPiDCoC4uLi9hcm02NC9ib290L2R0cy9hbWxvZ2ljL21lc29u LWd4bS1uZXhib3gtYTEuZHRzIHzCoMKgMiArKwo+IMKgYXJjaC9hcm02NC9ib290L2R0cy9hbWxv Z2ljL21lc29uLWd4bS1xMjAwLmR0c8KgwqDCoMKgwqB8wqDCoDIgKysKPiDCoDEyIGZpbGVzIGNo YW5nZWQsIDEzOSBpbnNlcnRpb25zKCspLCA3IGRlbGV0aW9ucygtKQoKTG9va3MgZ29vZC4KClJl dmlld2VkLWJ5OiBKZXJvbWUgQnJ1bmV0IDxqYnJ1bmV0QGJheWxpYnJlLmNvbT4KCj4gCgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2Vy bmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0 cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVs Cg==