[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAL_Jsq+YMHAv312PQoSCRpgGedhg3OYJnDC9=YHWg6nJkQSaLQ@mail.gmail.com>
Date: Thu, 1 Aug 2024 10:12:34 -0600
From: Rob Herring <robh@...nel.org>
To: Pawel Dembicki <paweldembicki@...il.com>
Cc: linuxppc-dev@...ts.ozlabs.org, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Michael Ellerman <mpe@...erman.id.au>,
Nicholas Piggin <npiggin@...il.com>, Christophe Leroy <christophe.leroy@...roup.eu>,
Naveen N Rao <naveen@...nel.org>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] powerpc: dtc: update P2020RDB dts
On Wed, Jul 31, 2024 at 11:14 PM Pawel Dembicki <paweldembicki@...il.com> wrote:
>
On the subject, every patch is an 'update'. Please make it more
specific. If you have a hard time coming up with something specific,
that's a sign your patch is making too many separate changes.
> P2020RDB contains multiple peripherals, which isn't added to
> devicetree:
> - Switch: Microchip VSC7385
> - PMIC: Renesas ZL2006
> - Temperature sensor: Analog Devices ADT7461
> - Two eeproms: 24C256 and 24C01
> - GPIO expander: NXP PCA9557
> - reset gpios of Ethernet PHYs
>
> This commit adds it.
>
> Some refreshments was done:
> - fixed link in ethernet-node
> - platform drivers nodes names
> - added 'gpio0' label in pq3-gpio-0.dtsi
>
> Signed-off-by: Pawel Dembicki <paweldembicki@...il.com>
> ---
> arch/powerpc/boot/dts/fsl/p2020rdb.dts | 85 +++++++++++++++++++++--
> arch/powerpc/boot/dts/fsl/pq3-gpio-0.dtsi | 2 +-
> 2 files changed, 81 insertions(+), 6 deletions(-)
>
> diff --git a/arch/powerpc/boot/dts/fsl/p2020rdb.dts b/arch/powerpc/boot/dts/fsl/p2020rdb.dts
> index 3acd3890b397..d563d37b91f1 100644
> --- a/arch/powerpc/boot/dts/fsl/p2020rdb.dts
> +++ b/arch/powerpc/boot/dts/fsl/p2020rdb.dts
> @@ -6,6 +6,7 @@
> */
>
> /include/ "p2020si-pre.dtsi"
> +#include <dt-bindings/gpio/gpio.h>
>
> / {
> model = "fsl,P2020RDB";
> @@ -33,7 +34,7 @@ lbc: localbus@...05000 {
> 0x1 0x0 0x0 0xffa00000 0x00040000
> 0x2 0x0 0x0 0xffb00000 0x00020000>;
>
> - nor@0,0 {
> + nor@0 {
Make these clean-ups a separate patch.
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "cfi-flash";
> @@ -79,7 +80,7 @@ partition@...000 {
> };
> };
>
> - nand@1,0 {
> + nand@1 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "fsl,p2020-fcm-nand",
> @@ -128,11 +129,49 @@ partition@...0000 {
> };
> };
>
> - L2switch@2,0 {
> + ethernet-switch@2 {
> #address-cells = <1>;
> #size-cells = <1>;
> - compatible = "vitesse-7385";
> + compatible = "vitesse,vsc7385";
There are 7 occurrences of this. Please fix them all. (And again,
separate patch).
> reg = <0x2 0x0 0x20000>;
> + reset-gpios = <&gpio0 12 GPIO_ACTIVE_LOW>;
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@1 {
> + reg = <1>;
> + label = "lan1";
> + };
> + port@2 {
> + reg = <2>;
> + label = "lan2";
> + };
> + port@3 {
> + reg = <3>;
> + label = "lan3";
> + };
> + port@4 {
> + reg = <4>;
> + label = "lan4";
> + };
> + vsc: port@6 {
> + reg = <6>;
> + label = "cpu";
> + ethernet = <&enet0>;
> + phy-mode = "rgmii";
> + rx-internal-delay-ps = <1400>;
> + tx-internal-delay-ps = <2000>;
> +
> + fixed-link {
> + speed = <1000>;
> + full-duplex;
> + pause;
> + };
> + };
> + };
> +
> };
>
> };
> @@ -141,12 +180,39 @@ soc: soc@...00000 {
> ranges = <0x0 0x0 0xffe00000 0x100000>;
>
> i2c@...0 {
> + temperature-sensor@4c {
> + compatible = "adi,adt7461";
> + reg = <0x4c>;
> + };
> +
> + eeprom@50 {
> + compatible = "atmel,24c256";
> + reg = <0x50>;
> + };
> +
> rtc@68 {
> compatible = "dallas,ds1339";
> reg = <0x68>;
> };
> };
>
> + i2c@...0 {
> + pmic@11 {
> + compatible = "zl2006";
Missing vendor prefix.
> + reg = <0x11>;
> + };
> +
> + gpio@18 {
> + compatible = "nxp,pca9557";
> + reg = <0x18>;
> + };
> +
> + eeprom@52 {
> + compatible = "atmel,24c01";
> + reg = <0x52>;
> + };
> + };
> +
> spi@...0 {
> flash@0 {
> #address-cells = <1>;
> @@ -200,11 +266,15 @@ mdio@...20 {
> phy0: ethernet-phy@0 {
> interrupts = <3 1 0 0>;
> reg = <0x0>;
> + reset-gpios = <&gpio0 14 GPIO_ACTIVE_LOW>;
> };
> +
> phy1: ethernet-phy@1 {
> interrupts = <3 1 0 0>;
> reg = <0x1>;
> + reset-gpios = <&gpio0 6 GPIO_ACTIVE_LOW>;
> };
> +
> tbi-phy@2 {
> device_type = "tbi-phy";
> reg = <0x2>;
> @@ -232,8 +302,13 @@ ptp_clock@...00 {
> };
>
> enet0: ethernet@...00 {
> - fixed-link = <1 1 1000 0 0>;
> phy-connection-type = "rgmii-id";
> +
> + fixed-link {
> + speed = <1000>;
> + full-duplex;
> + pause;
> + };
> };
>
> enet1: ethernet@...00 {
> diff --git a/arch/powerpc/boot/dts/fsl/pq3-gpio-0.dtsi b/arch/powerpc/boot/dts/fsl/pq3-gpio-0.dtsi
> index a1b48546b02d..5181117ea6b5 100644
> --- a/arch/powerpc/boot/dts/fsl/pq3-gpio-0.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/pq3-gpio-0.dtsi
> @@ -32,7 +32,7 @@
> * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> */
>
> -gpio-controller@...0 {
> +gpio0: gpio-controller@...0 {
> #gpio-cells = <2>;
> compatible = "fsl,pq3-gpio";
> reg = <0xfc00 0x100>;
> --
> 2.34.1
>
Powered by blists - more mailing lists