Re: [PATCH net-next v3 4/4] ARM64: dts: meson: odroidc2: disable advertisement EEE for GbE.
From: Jerome Brunet <hidden>
Date: 2016-11-28 12:40:47
Also in:
linux-amlogic, linux-arm-kernel, linux-devicetree, lkml
On Mon, 2016-11-28 at 13:31 +0100, Andreas Färber wrote:
Am 28.11.2016 um 10:46 schrieb Jerome Brunet:quoted
Signed-off-by: Jerome Brunet <jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> --- arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dtsb/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts index e6e3491d48a5..5624714d2b16 100644--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts@@ -46,6 +46,7 @@#include "meson-gxbb.dtsi" #include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/net/mdio.h> / { compatible = "hardkernel,odroid-c2", "amlogic,meson-gxbb";@@ -98,3 +99,18 @@pinctrl-0 = <&i2c_a_pins>; pinctrl-names = "default"; }; + +ðmac { + phy-handle = <ð_phy0>; + + mdio { + compatible = "snps,dwmac-mdio"; + #address-cells = <1>; + #size-cells = <0>; + + eth_phy0: ethernet-phy@0 { + reg = <0>; + eee-broken-modes = <MDIO_EEE_1000T>; + }; + }; +};
6I've tested this hand-applied because it applies to neither amlogic
v4.10/integ nor linux-next.git and will conflict if applied through the net-next tree.
I've rebased on net-next this morning. I just checked again and now there is a conflict indeed. Something got applied between in the last 6 ours which conflict with patch 4.
Note that there already is an ðmac node that you should be
extending
rather than duplicating:
ðmac {
status = "okay";
pinctrl-0 = <ð_rgmii_pins>;
pinctrl-names = "default";
};
If you or your colleagues could please fix the sort order of the
nodes
to be alphabetical again (ethmac after i2c_A here; between uart_A and
ir
in-tree) this wouldn't happen so easily again.OK
I therefore suggest to not apply this patch 4/4 through net-next but through the amlogic tree instead.
Agreed. The change is provided here so people can test. If the other patches get accepted, I'll submit the dts change through the amlogic tree.
Thanks, Andreas
-- 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