2019-11-06 15:54:56

by Ioana Ciornei

[permalink] [raw]
Subject: [PATCH 0/2] arm64: dts: lx2160a: add EMDIO1 and phy nodes

This patch set adds the External MDIO1 node and the two
RGMII PHYs connected to it.

Ioana Ciornei (2):
arm64: dts: lx2160a: add emdio1 node
arm64: dts: lx2160a: add RGMII phy nodes

arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts | 27 +++++++++++++++++++++++
arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 11 +++++++++
2 files changed, 38 insertions(+)

--
1.9.1


2019-11-06 15:55:27

by Ioana Ciornei

[permalink] [raw]
Subject: [PATCH 1/2] arm64: dts: lx2160a: add emdio1 node

Add the External MDIO1 device node found in the WRIOP global memory
region. This is needed for management of external PHYs.

Signed-off-by: Ioana Ciornei <[email protected]>
---
arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 11 +++++++++++
1 file changed, 11 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
index 80268c6ed5fb..8b87a0122b54 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
@@ -478,6 +478,17 @@
little-endian;
};

+ /* WRIOP0: 0x8b8_0000, E-MDIO1: 0x1_6000 */
+ emdio1: mdio@8b96000 {
+ compatible = "fsl,fman-memac-mdio";
+ reg = <0x0 0x8b96000 0x0 0x1000>;
+ interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ little-endian;
+ status = "disabled";
+ };
+
i2c0: i2c@2000000 {
compatible = "fsl,vf610-i2c";
#address-cells = <1>;
--
1.9.1

2019-11-21 13:00:58

by Ioana Ciornei

[permalink] [raw]
Subject: RE: [PATCH 0/2] arm64: dts: lx2160a: add EMDIO1 and phy nodes


> Subject: [PATCH 0/2] arm64: dts: lx2160a: add EMDIO1 and phy nodes
>
> This patch set adds the External MDIO1 node and the two RGMII PHYs
> connected to it.
>
> Ioana Ciornei (2):
> arm64: dts: lx2160a: add emdio1 node
> arm64: dts: lx2160a: add RGMII phy nodes
>
> arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts | 27
> +++++++++++++++++++++++
> arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 11 +++++++++
> 2 files changed, 38 insertions(+)
>

Hi Shawn,

Could you please take a look at this patch set?

Thanks,
Ioana


2019-12-04 06:58:50

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: lx2160a: add emdio1 node

On Wed, Nov 06, 2019 at 05:52:15PM +0200, Ioana Ciornei wrote:
> Add the External MDIO1 device node found in the WRIOP global memory
> region. This is needed for management of external PHYs.
>
> Signed-off-by: Ioana Ciornei <[email protected]>
> ---
> arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> index 80268c6ed5fb..8b87a0122b54 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> @@ -478,6 +478,17 @@
> little-endian;
> };
>
> + /* WRIOP0: 0x8b8_0000, E-MDIO1: 0x1_6000 */
> + emdio1: mdio@8b96000 {

Please find the place for the node in order of unit-address.

Shawn

> + compatible = "fsl,fman-memac-mdio";
> + reg = <0x0 0x8b96000 0x0 0x1000>;
> + interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + little-endian;
> + status = "disabled";
> + };
> +
> i2c0: i2c@2000000 {
> compatible = "fsl,vf610-i2c";
> #address-cells = <1>;
> --
> 1.9.1
>