[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170314070732.GH3618@dragon>
Date: Tue, 14 Mar 2017 15:07:34 +0800
From: Shawn Guo <shawnguo@...nel.org>
To: Madalin Bucur <madalin.bucur@....com>
Cc: robh+dt@...nel.org, mark.rutland@....com,
devicetree@...r.kernel.org, catalin.marinas@....com,
will.deacon@....com, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 1/2] dts: arm64: add LS1043A DPAA support
On Wed, Mar 01, 2017 at 05:35:04PM +0200, Madalin Bucur wrote:
> Signed-off-by: Madalin Bucur <madalin.bucur@....com>
Empty commit log is generally not welcome, especially when the commit
adds so many files.
> ---
> arch/arm64/boot/dts/freescale/fsl-ls1043-post.dtsi | 41 +++++++++++
> arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 2 +
> arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts | 75 ++++++++++++++++++++
> arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 73 +++++++++++++++++++-
> .../boot/dts/freescale/qoriq-bman1-portals.dtsi | 67 ++++++++++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-10g-0.dtsi | 42 ++++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi | 41 +++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-1.dtsi | 41 +++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-2.dtsi | 41 +++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-3.dtsi | 41 +++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-4.dtsi | 41 +++++++++++
> .../boot/dts/freescale/qoriq-fman3-0-1g-5.dtsi | 41 +++++++++++
> arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi | 80 ++++++++++++++++++++++
> .../boot/dts/freescale/qoriq-qman1-portals.dtsi | 76 ++++++++++++++++++++
> 14 files changed, 701 insertions(+), 1 deletion(-)
> create mode 100644 arch/arm64/boot/dts/freescale/fsl-ls1043-post.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-bman1-portals.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-10g-0.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-1.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-2.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-3.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-4.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-5.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> create mode 100644 arch/arm64/boot/dts/freescale/qoriq-qman1-portals.dtsi
I'm not comfortable with so many and complex include level. Can you
please elaborate it a bit and help us understand the necessity of doing
that?
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043-post.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043-post.dtsi
> new file mode 100644
> index 0000000..bf443d2
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043-post.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 device tree nodes for ls1043
> + *
> + * Copyright 2015-2016 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
It's still controversial whether we should use SPDX tag.
https://lkml.org/lkml/2017/2/28/750
> +
> +&soc {
> +
> +/include/ "qoriq-fman3-0.dtsi"
> +/include/ "qoriq-fman3-0-1g-0.dtsi"
> +/include/ "qoriq-fman3-0-1g-1.dtsi"
> +/include/ "qoriq-fman3-0-1g-2.dtsi"
> +/include/ "qoriq-fman3-0-1g-3.dtsi"
> +/include/ "qoriq-fman3-0-1g-4.dtsi"
> +/include/ "qoriq-fman3-0-1g-5.dtsi"
> +/include/ "qoriq-fman3-0-10g-0.dtsi"
We usually put the includes at the beginning of the file, and #include
is more recommended than /include/.
> + fman@...0000 {
> + enet0: ethernet@...00 {
> + };
> +
> + enet1: ethernet@...00 {
> + };
> +
> + enet2: ethernet@...00 {
> + };
> +
> + enet3: ethernet@...00 {
> + };
> +
> + enet4: ethernet@...00 {
> + };
> +
> + enet5: ethernet@...00 {
> + };
> +
> + enet6: ethernet@...00 {
> + };
> + };
I do not quite understand why these nodes are empty.
> +};
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> index 0989d63..ee66bb2 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> @@ -181,3 +181,5 @@
> reg = <0>;
> };
> };
> +
> +/include/ "fsl-ls1043-post.dtsi"
Move it to header of the file.
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> index c37110b..d94f003 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> @@ -139,3 +139,78 @@
> &duart1 {
> status = "okay";
> };
> +
> +/include/ "fsl-ls1043-post.dtsi"
> +
Ditto
> +&soc {
> + fman@...0000 {
> + ethernet@...00 {
You defined enet0 label. Why don't you use it?
> + phy-handle = <&qsgmii_phy1>;
> + phy-connection-type = "qsgmii";
> + };
> +
> + ethernet@...00 {
> + phy-handle = <&qsgmii_phy2>;
> + phy-connection-type = "qsgmii";
> + };
> +
> + ethernet@...00 {
> + phy-handle = <&rgmii_phy1>;
> + phy-connection-type = "rgmii";
> + };
> +
> + ethernet@...00 {
> + phy-handle = <&rgmii_phy2>;
> + phy-connection-type = "rgmii";
> + };
> +
> + ethernet@...00 {
> + phy-handle = <&qsgmii_phy3>;
> + phy-connection-type = "qsgmii";
> + };
> +
> + ethernet@...00 {
> + phy-handle = <&qsgmii_phy4>;
> + phy-connection-type = "qsgmii";
> + };
> +
> + ethernet@...00 { /* 10GEC1 */
> + phy-handle = <&aqr105_phy>;
> + phy-connection-type = "xgmii";
> + };
> +
> + mdio@...00 {
Use label reference the node you already define, so that you do not have
to maintain the device tree hierarchy for referencing the node.
> + rgmii_phy1: ethernet-phy@1 {
> + reg = <0x1>;
> + };
> +
> + rgmii_phy2: ethernet-phy@2 {
> + reg = <0x2>;
> + };
> +
> + qsgmii_phy1: ethernet-phy@3 {
> + reg = <0x4>;
The unit address in the node name should match 'reg' value.
> + };
> +
> + qsgmii_phy2: ethernet-phy@4 {
> + reg = <0x5>;
> + };
> +
> + qsgmii_phy3: ethernet-phy@5 {
> + reg = <0x6>;
> + };
> +
> + qsgmii_phy4: ethernet-phy@6 {
> + reg = <0x7>;
> + };
> + };
> +
> + mdio@...00 {
> + aqr105_phy: ethernet-phy@c {
> + compatible = "ethernet-phy-ieee802.3-c45";
> + interrupts = <0 132 4>;
> + reg = <0x1>;
> + };
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> index ec13a6e..ac1e0af 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> @@ -52,6 +52,17 @@
> #address-cells = <2>;
> #size-cells = <2>;
>
> + aliases {
> + fman0 = &fman0;
> + ethernet0 = &enet0;
> + ethernet1 = &enet1;
> + ethernet2 = &enet2;
> + ethernet3 = &enet3;
> + ethernet4 = &enet4;
> + ethernet5 = &enet5;
> + ethernet6 = &enet6;
> + };
> +
> cpus {
> #address-cells = <1>;
> #size-cells = <0>;
> @@ -106,6 +117,36 @@
> /* DRAM space 1, size: 2GiB DRAM */
> };
>
> + reserved-memory {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + bman_fbpr: bman-fbpr {
> + size = <0 0x1000000>;
> + alignment = <0 0x1000000>;
> + };
> +
> + qman_fqd: qman-fqd {
> + size = <0 0x400000>;
> + alignment = <0 0x400000>;
> + };
> +
> + qman_pfdr: qman-pfdr {
> + size = <0 0x2000000>;
> + alignment = <0 0x2000000>;
> + };
> + };
> +
> + bportals: bman-portals@...000000 {
> + ranges = <0x0 0x5 0x08000000 0x8000000>;
> + };
> +
> + qportals: qman-portals@...000000 {
> + ranges = <0x0 0x5 0x00000000 0x8000000>;
> + };
Why are these two devices directly under root, not soc node?
> +
> +
> sysclk: sysclk {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> @@ -152,7 +193,7 @@
> interrupts = <1 9 0xf08>;
> };
>
> - soc {
> + soc: soc {
> compatible = "simple-bus";
> #address-cells = <2>;
> #size-cells = <2>;
> @@ -333,6 +374,18 @@
> };
> };
>
> + qman: qman@...0000 {
> + compatible = "fsl,qman";
> + reg = <0x00 0x1880000 0x0 0x10000>;
> + interrupts = <0 45 0x4>;
> + };
> +
> + bman: bman@...0000 {
> + compatible = "fsl,bman";
> + reg = <0x00 0x1890000 0x0 0x10000>;
> + interrupts = <0 45 0x4>;
> + };
> +
> dspi0: dspi@...0000 {
> compatible = "fsl,ls1043a-dspi", "fsl,ls1021a-v1.0-dspi";
> #address-cells = <1>;
> @@ -686,3 +739,21 @@
> };
>
> };
> +
> +&bman_fbpr {
> + compatible = "fsl,bman-fbpr";
> + alloc-ranges = <0 0 0x10000 0>;
> +};
> +
> +&qman_fqd {
> + compatible = "fsl,qman-fqd";
> + alloc-ranges = <0 0 0x10000 0>;
> +};
> +
> +&qman_pfdr {
> + compatible = "fsl,qman-pfdr";
> + alloc-ranges = <0 0 0x10000 0>;
> +};
Why do you need to separate these from the nodes in reserved-memory?
I stop right here. Honestly, I do not like the patch as its current
form. I guess you should do something to make the reviewing of the
changes a bit easier.
Shawn
> +
> +/include/ "qoriq-qman1-portals.dtsi"
> +/include/ "qoriq-bman1-portals.dtsi"
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-bman1-portals.dtsi b/arch/arm64/boot/dts/freescale/qoriq-bman1-portals.dtsi
> new file mode 100644
> index 0000000..b007344
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-bman1-portals.dtsi
> @@ -0,0 +1,67 @@
> +/*
> + * QorIQ BMan Portals device tree
> + *
> + * Copyright 2011-2016 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +&bportals {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "simple-bus";
> +
> + bman-portal@0 {
> + compatible = "fsl,bman-portal";
> + reg = <0x0 0x4000 0x4000000 0x4000>;
> + interrupts = <0 173 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x10000 0x4000 0x4010000 0x4000>;
> + interrupts = <0 175 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x20000 0x4000 0x4020000 0x4000>;
> + interrupts = <0 177 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x30000 0x4000 0x4030000 0x4000>;
> + interrupts = <0 179 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x40000 0x4000 0x4040000 0x4000>;
> + interrupts = <0 181 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x50000 0x4000 0x4050000 0x4000>;
> + interrupts = <0 183 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x60000 0x4000 0x4060000 0x4000>;
> + interrupts = <0 185 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x70000 0x4000 0x4070000 0x4000>;
> + interrupts = <0 187 0x4>;
> + };
> +
> + bman-portal@...00 {
> + compatible = "fsl,bman-portal";
> + reg = <0x80000 0x4000 0x4080000 0x4000>;
> + interrupts = <0 189 0x4>;
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-10g-0.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-10g-0.dtsi
> new file mode 100644
> index 0000000..ecdffe7
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-10g-0.dtsi
> @@ -0,0 +1,42 @@
> +/*
> + * QorIQ FMan v3 10g port #0 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x10: port@...00 {
> + cell-index = <0x10>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x90000 0x1000>;
> + fsl,fman-10g-port;
> + };
> +
> + fman0_tx_0x30: port@...00 {
> + cell-index = <0x30>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xb0000 0x1000>;
> + fsl,fman-10g-port;
> + };
> +
> + ethernet@...00 {
> + cell-index = <0x8>;
> + compatible = "fsl,fman-memac";
> + reg = <0xf0000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x10 &fman0_tx_0x30>;
> + pcsphy-handle = <&pcsphy6>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xf1000 0x1000>;
> +
> + pcsphy6: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi
> new file mode 100644
> index 0000000..d600786
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #0 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x08: port@...00 {
> + cell-index = <0x8>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x88000 0x1000>;
> + };
> +
> + fman0_tx_0x28: port@...00 {
> + cell-index = <0x28>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xa8000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <0>;
> + compatible = "fsl,fman-memac";
> + reg = <0xe0000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x08 &fman0_tx_0x28>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy0>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xe1000 0x1000>;
> +
> + pcsphy0: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-1.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-1.dtsi
> new file mode 100644
> index 0000000..3c0b76d
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-1.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #1 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x09: port@...00 {
> + cell-index = <0x9>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x89000 0x1000>;
> + };
> +
> + fman0_tx_0x29: port@...00 {
> + cell-index = <0x29>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xa9000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <1>;
> + compatible = "fsl,fman-memac";
> + reg = <0xe2000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x09 &fman0_tx_0x29>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy1>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xe3000 0x1000>;
> +
> + pcsphy1: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-2.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-2.dtsi
> new file mode 100644
> index 0000000..89633af
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-2.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #2 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x0a: port@...00 {
> + cell-index = <0xa>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x8a000 0x1000>;
> + };
> +
> + fman0_tx_0x2a: port@...00 {
> + cell-index = <0x2a>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xaa000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <2>;
> + compatible = "fsl,fman-memac";
> + reg = <0xe4000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x0a &fman0_tx_0x2a>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy2>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xe5000 0x1000>;
> +
> + pcsphy2: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-3.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-3.dtsi
> new file mode 100644
> index 0000000..87c2b70
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-3.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #3 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x0b: port@...00 {
> + cell-index = <0xb>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x8b000 0x1000>;
> + };
> +
> + fman0_tx_0x2b: port@...00 {
> + cell-index = <0x2b>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xab000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <3>;
> + compatible = "fsl,fman-memac";
> + reg = <0xe6000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x0b &fman0_tx_0x2b>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy3>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xe7000 0x1000>;
> +
> + pcsphy3: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-4.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-4.dtsi
> new file mode 100644
> index 0000000..8f4d74b
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-4.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #4 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x0c: port@...00 {
> + cell-index = <0xc>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x8c000 0x1000>;
> + };
> +
> + fman0_tx_0x2c: port@...00 {
> + cell-index = <0x2c>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xac000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <4>;
> + compatible = "fsl,fman-memac";
> + reg = <0xe8000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x0c &fman0_tx_0x2c>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy4>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xe9000 0x1000>;
> +
> + pcsphy4: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-5.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-5.dtsi
> new file mode 100644
> index 0000000..d534f77
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-5.dtsi
> @@ -0,0 +1,41 @@
> +/*
> + * QorIQ FMan v3 1g port #5 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman@...0000 {
> + fman0_rx_0x0d: port@...00 {
> + cell-index = <0xd>;
> + compatible = "fsl,fman-v3-port-rx";
> + reg = <0x8d000 0x1000>;
> + };
> +
> + fman0_tx_0x2d: port@...00 {
> + cell-index = <0x2d>;
> + compatible = "fsl,fman-v3-port-tx";
> + reg = <0xad000 0x1000>;
> + };
> +
> + ethernet@...00 {
> + cell-index = <5>;
> + compatible = "fsl,fman-memac";
> + reg = <0xea000 0x1000>;
> + fsl,fman-ports = <&fman0_rx_0x0d &fman0_tx_0x2d>;
> + ptp-timer = <&ptp_timer0>;
> + pcsphy-handle = <&pcsphy5>;
> + };
> +
> + mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xeb000 0x1000>;
> +
> + pcsphy5: ethernet-phy@0 {
> + reg = <0x0>;
> + };
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> new file mode 100644
> index 0000000..a559441
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> @@ -0,0 +1,80 @@
> +/*
> + * QorIQ FMan v3 device tree
> + *
> + * Copyright 2012-2015 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +fman0: fman@...0000 {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + cell-index = <0>;
> + compatible = "fsl,fman";
> + ranges = <0x0 0x00 0x1a00000 0x100000>;
> + reg = <0x0 0x1a00000 0x0 0x100000>;
> + interrupts = <0 44 0x4>, <0 45 0x4>;
> + clocks = <&clockgen 3 0>;
> + clock-names = "fmanclk";
> + fsl,qman-channel-range = <0x800 0x10>;
> +
> + muram@0 {
> + compatible = "fsl,fman-muram";
> + reg = <0x0 0x60000>;
> + };
> +
> + fman0_oh_0x2: port@...00 {
> + cell-index = <0x2>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x82000 0x1000>;
> + };
> +
> + fman0_oh_0x3: port@...00 {
> + cell-index = <0x3>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x83000 0x1000>;
> + };
> +
> + fman0_oh_0x4: port@...00 {
> + cell-index = <0x4>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x84000 0x1000>;
> + };
> +
> + fman0_oh_0x5: port@...00 {
> + cell-index = <0x5>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x85000 0x1000>;
> + };
> +
> + fman0_oh_0x6: port@...00 {
> + cell-index = <0x6>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x86000 0x1000>;
> + };
> +
> + fman0_oh_0x7: port@...00 {
> + cell-index = <0x7>;
> + compatible = "fsl,fman-v3-port-oh";
> + reg = <0x87000 0x1000>;
> + };
> +
> + mdio0: mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xfc000 0x1000>;
> + };
> +
> + xmdio0: mdio@...00 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> + reg = <0xfd000 0x1000>;
> + };
> +
> + ptp_timer0: ptp-timer@...00 {
> + compatible = "fsl,fman-ptp-timer";
> + reg = <0xfe000 0x1000>;
> + };
> +};
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-qman1-portals.dtsi b/arch/arm64/boot/dts/freescale/qoriq-qman1-portals.dtsi
> new file mode 100644
> index 0000000..4baec2c
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/qoriq-qman1-portals.dtsi
> @@ -0,0 +1,76 @@
> +/*
> + * QorIQ QMan Portals device tree
> + *
> + * Copyright 2011-2016 Freescale Semiconductor Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> + */
> +
> +&qportals {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "simple-bus";
> +
> + qportal0: qman-portal@0 {
> + compatible = "fsl,qman-portal";
> + reg = <0x0 0x4000 0x4000000 0x4000>;
> + interrupts = <0 172 0x4>;
> + cell-index = <0>;
> + };
> +
> + qportal1: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x10000 0x4000 0x4010000 0x4000>;
> + interrupts = <0 174 0x4>;
> + cell-index = <1>;
> + };
> +
> + qportal2: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x20000 0x4000 0x4020000 0x4000>;
> + interrupts = <0 176 0x4>;
> + cell-index = <2>;
> + };
> +
> + qportal3: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x30000 0x4000 0x4030000 0x4000>;
> + interrupts = <0 178 0x4>;
> + cell-index = <3>;
> + };
> +
> + qportal4: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x40000 0x4000 0x4040000 0x4000>;
> + interrupts = <0 180 0x4>;
> + cell-index = <4>;
> + };
> +
> + qportal5: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x50000 0x4000 0x4050000 0x4000>;
> + interrupts = <0 182 0x4>;
> + cell-index = <5>;
> + };
> +
> + qportal6: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x60000 0x4000 0x4060000 0x4000>;
> + interrupts = <0 184 0x4>;
> + cell-index = <6>;
> + };
> +
> + qportal7: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x70000 0x4000 0x4070000 0x4000>;
> + interrupts = <0 186 0x4>;
> + cell-index = <7>;
> + };
> +
> + qportal8: qman-portal@...00 {
> + compatible = "fsl,qman-portal";
> + reg = <0x80000 0x4000 0x4080000 0x4000>;
> + interrupts = <0 188 0x4>;
> + cell-index = <8>;
> + };
> +};
> --
> 2.1.0
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Powered by blists - more mailing lists