[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[codekipper at gmail.com]

From: Marcus Cooper <codekipper@gmail.com>

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@gmail.com>
---
 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

[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Maxime Ripard]

Hi,

On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper at gmail.com wrote:
> From: Marcus Cooper <codekipper@gmail.com>
> 
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180703/4a705a6f/attachment.sig>

[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Code Kipper]

On 3 July 2018 at 16:50, Maxime Ripard <maxime.ripard@bootlin.com> wrote:
> Hi,
>
> On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper at gmail.com wrote:
>> From: Marcus Cooper <codekipper@gmail.com>
>>
>> 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

[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Maxime Ripard]

On Tue, Jul 03, 2018 at 07:19:16PM +0200, Code Kipper wrote:
> On 3 July 2018 at 16:50, Maxime Ripard <maxime.ripard@bootlin.com> wrote:
> > Hi,
> >
> > On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper at gmail.com wrote:
> >> From: Marcus Cooper <codekipper@gmail.com>
> >>
> >> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180704/98da5e68/attachment.sig>

[linux-sunxi] ARM: dts: sun8i: h3: Enable dwmac-sun8i on the Beelink X2

[Code Kipper]

On 4 July 2018 at 09:44, Maxime Ripard <maxime.ripard@bootlin.com> 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@bootlin.com> wrote:
>> > Hi,
>> >
>> > On Sat, Jun 30, 2018 at 08:27:20AM +0200, codekipper at gmail.com wrote:
>> >> From: Marcus Cooper <codekipper@gmail.com>
>> >>
>> >> 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