Linux-Amlogic Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
@ 2020-07-30 10:02 Anand Moon
  2020-07-30 10:02 ` [PATCH v1 1/3] arm64: dts: amlogic: meson-gx: add missing ethernet reset ID Anand Moon
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Anand Moon @ 2020-07-30 10:02 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Neil Armstrong, Rob Herring, Kevin Hilman,
	Jerome Brunet

These patch fix the Ethernet issue on Odroid C2 & Odroid N2 
Some time Ethernet interface come up but no DHCP request
is forwaded to the Router, this happens quite offern on
my side. Adding Reset controller to ethernet node FIx the
inilization issue.

-Anand

Anand Moon (3):
  arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
  arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
  arm64: dts: amlogic: meson-g12: add missing ethernet reset ID

 arch/arm64/boot/dts/amlogic/meson-axg.dtsi        | 2 ++
 arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 2 ++
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi         | 3 +++
 3 files changed, 7 insertions(+)

-- 
2.28.0


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH v1 1/3] arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
  2020-07-30 10:02 [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Anand Moon
@ 2020-07-30 10:02 ` Anand Moon
  2020-07-30 10:02 ` [PATCH v1 2/3] arm64: dts: amlogic: meson-axg: " Anand Moon
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 9+ messages in thread
From: Anand Moon @ 2020-07-30 10:02 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Neil Armstrong, Rob Herring, Kevin Hilman,
	Jerome Brunet

Add missing etherent reset ID for reset controller to reset
ethernet mac controller.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
index ba63c36b22e0..8720c83990ce 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
@@ -12,6 +12,7 @@
 #include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/reset/amlogic,meson-gxbb-reset.h>
 #include <dt-bindings/thermal/thermal.h>
 
 / {
@@ -574,6 +575,8 @@ ethmac: ethernet@c9410000 {
 			interrupt-names = "macirq";
 			rx-fifo-depth = <4096>;
 			tx-fifo-depth = <2048>;
+			resets = <&reset RESET_ETHERNET>;
+			reset-names = "stmmaceth";
 			status = "disabled";
 		};
 
-- 
2.28.0


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH v1 2/3] arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
  2020-07-30 10:02 [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Anand Moon
  2020-07-30 10:02 ` [PATCH v1 1/3] arm64: dts: amlogic: meson-gx: add missing ethernet reset ID Anand Moon
@ 2020-07-30 10:02 ` Anand Moon
  2020-07-30 10:02 ` [PATCH v1 3/3] arm64: dts: amlogic: meson-g12: " Anand Moon
  2020-07-30 11:57 ` [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Jerome Brunet
  3 siblings, 0 replies; 9+ messages in thread
From: Anand Moon @ 2020-07-30 10:02 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Neil Armstrong, Rob Herring, Kevin Hilman,
	Jerome Brunet

Add missing etherent reset ID for reset controller to reset
ethernet mac controller.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi
index 8e6281c685fa..61adc5592211 100644
--- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi
@@ -185,6 +185,8 @@ ethmac: ethernet@ff3f0000 {
 			clock-names = "stmmaceth", "clkin0", "clkin1";
 			rx-fifo-depth = <4096>;
 			tx-fifo-depth = <2048>;
+			resets = <&reset RESET_ETHERNET>;
+			reset-names = "stmmaceth";
 			status = "disabled";
 		};
 
-- 
2.28.0


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH v1 3/3] arm64: dts: amlogic: meson-g12: add missing ethernet reset ID
  2020-07-30 10:02 [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Anand Moon
  2020-07-30 10:02 ` [PATCH v1 1/3] arm64: dts: amlogic: meson-gx: add missing ethernet reset ID Anand Moon
  2020-07-30 10:02 ` [PATCH v1 2/3] arm64: dts: amlogic: meson-axg: " Anand Moon
@ 2020-07-30 10:02 ` Anand Moon
  2020-07-30 11:57 ` [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Jerome Brunet
  3 siblings, 0 replies; 9+ messages in thread
From: Anand Moon @ 2020-07-30 10:02 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Neil Armstrong, Rob Herring, Kevin Hilman,
	Jerome Brunet

Add missing etherent reset ID for reset controller to reset
ethernet mac controller.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
index 593a006f4b7b..7f291e6e9847 100644
--- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
@@ -189,6 +189,8 @@ ethmac: ethernet@ff3f0000 {
 			clock-names = "stmmaceth", "clkin0", "clkin1";
 			rx-fifo-depth = <4096>;
 			tx-fifo-depth = <2048>;
+			resets = <&reset RESET_ETHERNET>;
+			reset-names = "stmmaceth";
 			status = "disabled";
 
 			mdio0: mdio {
-- 
2.28.0


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
  2020-07-30 10:02 [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Anand Moon
                   ` (2 preceding siblings ...)
  2020-07-30 10:02 ` [PATCH v1 3/3] arm64: dts: amlogic: meson-g12: " Anand Moon
@ 2020-07-30 11:57 ` Jerome Brunet
  2020-07-30 12:14   ` Jerome Brunet
  3 siblings, 1 reply; 9+ messages in thread
From: Jerome Brunet @ 2020-07-30 11:57 UTC (permalink / raw)
  To: Anand Moon, devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Kevin Hilman, Rob Herring, Neil Armstrong


On Thu 30 Jul 2020 at 12:02, Anand Moon <linux.amoon@gmail.com> wrote:

> These patch fix the Ethernet issue on Odroid C2 & Odroid N2 
> Some time Ethernet interface come up but no DHCP request
> is forwaded to the Router, this happens quite offern on
> my side. Adding Reset controller to ethernet node FIx the
> inilization issue.
>

I'm seeing this on other device as well. Strange that it pops up only
now. Change tested on S905x, seems to help.

> -Anand
>
> Anand Moon (3):
>   arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
>   arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
>   arm64: dts: amlogic: meson-g12: add missing ethernet reset ID

None of the patches apply, please rebase on kevin's dt64 branch.

With that, you can add

Tested-by: Jerome Brunet <jbrunet@baylibre.com>

>
>  arch/arm64/boot/dts/amlogic/meson-axg.dtsi        | 2 ++
>  arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 2 ++
>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi         | 3 +++
>  3 files changed, 7 insertions(+)


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
  2020-07-30 11:57 ` [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Jerome Brunet
@ 2020-07-30 12:14   ` Jerome Brunet
  2020-07-30 13:46     ` Anand Moon
  0 siblings, 1 reply; 9+ messages in thread
From: Jerome Brunet @ 2020-07-30 12:14 UTC (permalink / raw)
  To: Anand Moon, devicetree, linux-arm-kernel, linux-amlogic
  Cc: Martin Blumenstingl, Kevin Hilman, Rob Herring, Neil Armstrong


On Thu 30 Jul 2020 at 13:57, Jerome Brunet <jbrunet@baylibre.com> wrote:

> On Thu 30 Jul 2020 at 12:02, Anand Moon <linux.amoon@gmail.com> wrote:
>
>> These patch fix the Ethernet issue on Odroid C2 & Odroid N2 
>> Some time Ethernet interface come up but no DHCP request
>> is forwaded to the Router, this happens quite offern on
>> my side. Adding Reset controller to ethernet node FIx the
>> inilization issue.
>>
>
> I'm seeing this on other device as well. Strange that it pops up only
> now. Change tested on S905x, seems to help.

Strike that one - still seeing random DHCP issue on boot with this
applied

>
>> -Anand
>>
>> Anand Moon (3):
>>   arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
>>   arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
>>   arm64: dts: amlogic: meson-g12: add missing ethernet reset ID
>
> None of the patches apply, please rebase on kevin's dt64 branch.
>
> With that, you can add
>
> Tested-by: Jerome Brunet <jbrunet@baylibre.com>

So still worth rebasing and applying on I suppose but I don't think it
solves the problem as described in this cover letter.

>
>>
>>  arch/arm64/boot/dts/amlogic/meson-axg.dtsi        | 2 ++
>>  arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 2 ++
>>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi         | 3 +++
>>  3 files changed, 7 insertions(+)


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
  2020-07-30 12:14   ` Jerome Brunet
@ 2020-07-30 13:46     ` Anand Moon
  2020-07-30 13:55       ` Jerome Brunet
  0 siblings, 1 reply; 9+ messages in thread
From: Anand Moon @ 2020-07-30 13:46 UTC (permalink / raw)
  To: Jerome Brunet
  Cc: devicetree, Neil Armstrong, Martin Blumenstingl, Kevin Hilman,
	Rob Herring, linux-amlogic, linux-arm-kernel

Hi Jerome

On Thu, 30 Jul 2020 at 17:44, Jerome Brunet <jbrunet@baylibre.com> wrote:
>
>
> On Thu 30 Jul 2020 at 13:57, Jerome Brunet <jbrunet@baylibre.com> wrote:
>
> > On Thu 30 Jul 2020 at 12:02, Anand Moon <linux.amoon@gmail.com> wrote:
> >
> >> These patch fix the Ethernet issue on Odroid C2 & Odroid N2
> >> Some time Ethernet interface come up but no DHCP request
> >> is forwaded to the Router, this happens quite offern on
> >> my side. Adding Reset controller to ethernet node FIx the
> >> inilization issue.
> >>
> >
> > I'm seeing this on other device as well. Strange that it pops up only
> > now. Change tested on S905x, seems to help.
>
> Strike that one - still seeing random DHCP issue on boot with this
> applied
>
> >
> >> -Anand
> >>
> >> Anand Moon (3):
> >>   arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
> >>   arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
> >>   arm64: dts: amlogic: meson-g12: add missing ethernet reset ID
> >
> > None of the patches apply, please rebase on kevin's dt64 branch.
> >
> > With that, you can add
> >
> > Tested-by: Jerome Brunet <jbrunet@baylibre.com>
>
So I have to drop the Tested-by  ?

> So still worth rebasing and applying on I suppose but I don't think it
> solves the problem as described in this cover letter.
>

I have a mac address mapped to the ip address on my router,
So I am just testing with the latest u-boot-2000.07 on my boards.
Usually the ethernet comes up with no packet getting forwarded,
Even ping to the device is not working, which is what I observed at
my end many times.

I will try to rebase on top of Kevin's v5.9/dt64 if needed.
Please let me know.

-Anand

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
  2020-07-30 13:46     ` Anand Moon
@ 2020-07-30 13:55       ` Jerome Brunet
  2020-07-30 18:04         ` Anand Moon
  0 siblings, 1 reply; 9+ messages in thread
From: Jerome Brunet @ 2020-07-30 13:55 UTC (permalink / raw)
  To: Anand Moon
  Cc: devicetree, Neil Armstrong, Martin Blumenstingl, Kevin Hilman,
	Rob Herring, linux-amlogic, linux-arm-kernel


On Thu 30 Jul 2020 at 15:46, Anand Moon <linux.amoon@gmail.com> wrote:

> Hi Jerome
>
> On Thu, 30 Jul 2020 at 17:44, Jerome Brunet <jbrunet@baylibre.com> wrote:
>>
>>
>> On Thu 30 Jul 2020 at 13:57, Jerome Brunet <jbrunet@baylibre.com> wrote:
>>
>> > On Thu 30 Jul 2020 at 12:02, Anand Moon <linux.amoon@gmail.com> wrote:
>> >
>> >> These patch fix the Ethernet issue on Odroid C2 & Odroid N2
>> >> Some time Ethernet interface come up but no DHCP request
>> >> is forwaded to the Router, this happens quite offern on
>> >> my side. Adding Reset controller to ethernet node FIx the
>> >> inilization issue.
>> >>
>> >
>> > I'm seeing this on other device as well. Strange that it pops up only
>> > now. Change tested on S905x, seems to help.
>>
>> Strike that one - still seeing random DHCP issue on boot with this
>> applied
>>
>> >
>> >> -Anand
>> >>
>> >> Anand Moon (3):
>> >>   arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
>> >>   arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
>> >>   arm64: dts: amlogic: meson-g12: add missing ethernet reset ID
>> >
>> > None of the patches apply, please rebase on kevin's dt64 branch.
>> >
>> > With that, you can add
>> >
>> > Tested-by: Jerome Brunet <jbrunet@baylibre.com>
>>
> So I have to drop the Tested-by  ?

Yes

>
>> So still worth rebasing and applying on I suppose but I don't think it
>> solves the problem as described in this cover letter.
>>
>
> I have a mac address mapped to the ip address on my router,
> So I am just testing with the latest u-boot-2000.07 on my boards.
> Usually the ethernet comes up with no packet getting forwarded,
> Even ping to the device is not working, which is what I observed at
> my end many times.

This is not the use case descibed in the cover letter. Please clarify if
you resubmit.

>
> I will try to rebase on top of Kevin's v5.9/dt64 if needed.

If you intend on submitting the series again, rebasing is necessary.

> Please let me know.
>
> -Anand


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC
  2020-07-30 13:55       ` Jerome Brunet
@ 2020-07-30 18:04         ` Anand Moon
  0 siblings, 0 replies; 9+ messages in thread
From: Anand Moon @ 2020-07-30 18:04 UTC (permalink / raw)
  To: Jerome Brunet
  Cc: devicetree, Neil Armstrong, Martin Blumenstingl, Kevin Hilman,
	Rob Herring, linux-amlogic, linux-arm-kernel

Hi Jerome,

On Thu, 30 Jul 2020 at 19:25, Jerome Brunet <jbrunet@baylibre.com> wrote:
>
>
> On Thu 30 Jul 2020 at 15:46, Anand Moon <linux.amoon@gmail.com> wrote:
>
> > Hi Jerome
> >
> > On Thu, 30 Jul 2020 at 17:44, Jerome Brunet <jbrunet@baylibre.com> wrote:
> >>
> >>
> >> On Thu 30 Jul 2020 at 13:57, Jerome Brunet <jbrunet@baylibre.com> wrote:
> >>
> >> > On Thu 30 Jul 2020 at 12:02, Anand Moon <linux.amoon@gmail.com> wrote:
> >> >
> >> >> These patch fix the Ethernet issue on Odroid C2 & Odroid N2
> >> >> Some time Ethernet interface come up but no DHCP request
> >> >> is forwaded to the Router, this happens quite offern on
> >> >> my side. Adding Reset controller to ethernet node FIx the
> >> >> inilization issue.
> >> >>
> >> >
> >> > I'm seeing this on other device as well. Strange that it pops up only
> >> > now. Change tested on S905x, seems to help.
> >>
> >> Strike that one - still seeing random DHCP issue on boot with this
> >> applied
> >>
> >> >
> >> >> -Anand
> >> >>
> >> >> Anand Moon (3):
> >> >>   arm64: dts: amlogic: meson-gx: add missing ethernet reset ID
> >> >>   arm64: dts: amlogic: meson-axg: add missing ethernet reset ID
> >> >>   arm64: dts: amlogic: meson-g12: add missing ethernet reset ID
> >> >
> >> > None of the patches apply, please rebase on kevin's dt64 branch.
> >> >
> >> > With that, you can add
> >> >
> >> > Tested-by: Jerome Brunet <jbrunet@baylibre.com>
> >>
> > So I have to drop the Tested-by  ?
>
> Yes
>
> >
> >> So still worth rebasing and applying on I suppose but I don't think it
> >> solves the problem as described in this cover letter.
> >>
> >
> > I have a mac address mapped to the ip address on my router,
> > So I am just testing with the latest u-boot-2000.07 on my boards.
> > Usually the ethernet comes up with no packet getting forwarded,
> > Even ping to the device is not working, which is what I observed at
> > my end many times.
>
> This is not the use case descibed in the cover letter. Please clarify if
> you resubmit.
>

This was some observation at my end. If these changes don't fix the issue.
I will drop my observation and just send the missing reset controller
for ethernet

-Anand

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, back to index

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-30 10:02 [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Anand Moon
2020-07-30 10:02 ` [PATCH v1 1/3] arm64: dts: amlogic: meson-gx: add missing ethernet reset ID Anand Moon
2020-07-30 10:02 ` [PATCH v1 2/3] arm64: dts: amlogic: meson-axg: " Anand Moon
2020-07-30 10:02 ` [PATCH v1 3/3] arm64: dts: amlogic: meson-g12: " Anand Moon
2020-07-30 11:57 ` [PATCH v1 0/3] Add missing ethernet reset ID for Amlogic SoC Jerome Brunet
2020-07-30 12:14   ` Jerome Brunet
2020-07-30 13:46     ` Anand Moon
2020-07-30 13:55       ` Jerome Brunet
2020-07-30 18:04         ` Anand Moon

Linux-Amlogic Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-amlogic/0 linux-amlogic/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-amlogic linux-amlogic/ https://lore.kernel.org/linux-amlogic \
		linux-amlogic@lists.infradead.org
	public-inbox-index linux-amlogic

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.infradead.lists.linux-amlogic


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git