[PATCH] ARM64: dts: meson-gxl: add internal ethernet PHY irq

[PATCH] ARM64: dts: meson-gxl: add internal ethernet PHY irq

[Jerome Brunet]

Add the interrupt of the internal ethernet PHY

Signed-off-by: Jerome Brunet <jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
Hi Kevin,

This changes depends on the net-next changes adding interrupt support [0].
It is really important that this change is not merged before its
dependency.

If it is merged before, instead of polling, the phy would wait for an
interrupt which has not been configured and will never fire.

Tested on the libretech-cc and khadas VIM.

Cheers
Jerome

https://lkml.kernel.org/r/20171218094446.31912-7-jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org

 arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
index 4a316a11a00e..8bc404508a4f 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
@@ -631,6 +631,7 @@
 
 			internal_phy: ethernet-phy@8 {
 				compatible = "ethernet-phy-id0181.4400", "ethernet-phy-ieee802.3-c22";
+				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
 				reg = <8>;
 				max-speed = <100>;
 			};
-- 
2.14.3

--
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

Re: [PATCH] ARM64: dts: meson-gxl: add internal ethernet PHY irq

[Kevin Hilman]

Jerome Brunet <jbrunet@baylibre.com> writes:

> Add the interrupt of the internal ethernet PHY
>
> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
> ---
> Hi Kevin,
>
> This changes depends on the net-next changes adding interrupt support [0].
> It is really important that this change is not merged before its
> dependency.
>
> If it is merged before, instead of polling, the phy would wait for an
> interrupt which has not been configured and will never fire.

Thanks for the warning. I'll wait and merge these after -rc1 to be sure
of all dependencies.

Kevin