ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[codekipper-Re5JQEeQqe8AvxtiuMwx3w]

From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

The enabling of the emac was reverted when stability issues were
discovered.

This reverts partially commit 6b0e06df5cad (Revert "ARM: dts:
sun8i: h3: Enable dwmac-sun8i on the Beelink X2")

Signed-off-by: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 arch/arm/boot/dts/sun8i-h3-beelink-x2.dts | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm/boot/dts/sun8i-h3-beelink-x2.dts b/arch/arm/boot/dts/sun8i-h3-beelink-x2.dts
index cf1f970b0c6f..5d23667dc2d2 100644
--- a/arch/arm/boot/dts/sun8i-h3-beelink-x2.dts
+++ b/arch/arm/boot/dts/sun8i-h3-beelink-x2.dts
@@ -123,6 +123,13 @@
 	status = "okay";
 };
 
+&emac {
+	phy-handle = <&int_mii_phy>;
+	phy-mode = "mii";
+	allwinner,leds-active-low;
+	status = "okay";
+};
+
 &hdmi {
 	status = "okay";
 };
-- 
2.18.0

Re: ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Maxime Ripard]

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

Hi,

On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote:
> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> 
> The enabling of the emac was reverted when stability issues were
> discovered.

What stability issues exactly? Have they been fixed? If so, by which
commit?

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

Re: ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Code Kipper]

On 3 July 2018 at 16:50, Maxime Ripard <maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org> wrote:
> Hi,
>
> On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote:
>> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>
>> The enabling of the emac was reverted when stability issues were
>> discovered.
>
> What stability issues exactly? Have they been fixed? If so, by which
> commit?

They were fixed by 4904337fe34f (ARM: dts: sunxi: Restore EMAC changes
(boards)) but this board was missed.
CK
>
> Maxime
>
> --
> Maxime Ripard, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com

Re: ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Maxime Ripard]

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

On Tue, Jul 03, 2018 at 07:19:16PM +0200, Code Kipper wrote:
> On 3 July 2018 at 16:50, Maxime Ripard <maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org> wrote:
> > Hi,
> >
> > On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote:
> >> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> >>
> >> The enabling of the emac was reverted when stability issues were
> >> discovered.
> >
> > What stability issues exactly? Have they been fixed? If so, by which
> > commit?
> 
> They were fixed by 4904337fe34f (ARM: dts: sunxi: Restore EMAC changes
> (boards)) but this board was missed.

Please add that to the commit log, and what the issue was.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

Re: ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Code Kipper]

On 4 July 2018 at 09:44, Maxime Ripard <maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org> wrote:
> On Tue, Jul 03, 2018 at 07:19:16PM +0200, Code Kipper wrote:
>> On 3 July 2018 at 16:50, Maxime Ripard <maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org> wrote:
>> > Hi,
>> >
>> > On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote:
>> >> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> >>
>> >> The enabling of the emac was reverted when stability issues were
>> >> discovered.
>> >
>> > What stability issues exactly? Have they been fixed? If so, by which
>> > commit?
>>
>> They were fixed by 4904337fe34f (ARM: dts: sunxi: Restore EMAC changes
>> (boards)) but this board was missed.
>
> Please add that to the commit log, and what the issue was.
(A)CK
>
> Maxime
>
> --
> Maxime Ripard, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com