From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,T_DKIMWL_WL_HIGH,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CC6E4C04AB5 for ; Mon, 3 Jun 2019 15:12:02 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8933627B42 for ; Mon, 3 Jun 2019 15:12:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="s+G/J8jO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8933627B42 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2nwqZrGva0MIdVDcUMjVCf83Z0ngN8N87WSHpTlBNe0=; b=s+G/J8jOsNgsSs IiNZ+rEZF6i/BbxL+x9+sR67wOR46i9qKW7AOcBxhKZ28rk89ymx+7ZmWLzuOKIbl1GZNUb7fzDlv mI7OPCrYUee13/rVG3Si79oAaZB7UgwT1O47XbCBWNqE1TEElxEOvIEBtc2KcvQlZnCO/eZJ75aNu 0UOiFdhRTz6biw59CK0QO0ON5zi9TLMqApLNXSvQW+wnJo7OGBDnsludWZ//8OIpZYuyxGBkUSdTv sMslKJL//dm3HqPnVqw6JSkBNnVUJ1cuNUkVsaniPIn1Ay+NdTlKURM5xCV4Sc0UrBtGmSWIdC7F9 ENb4zIe/kYejbkzi9zuQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hXocw-0007oC-Tk; Mon, 03 Jun 2019 15:11:58 +0000 Received: from relay2-d.mail.gandi.net ([217.70.183.194]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hXocr-0007nj-S9 for linux-arm-kernel@lists.infradead.org; Mon, 03 Jun 2019 15:11:56 +0000 X-Originating-IP: 92.137.69.152 Received: from localhost (alyon-656-1-672-152.w92-137.abo.wanadoo.fr [92.137.69.152]) (Authenticated sender: gregory.clement@bootlin.com) by relay2-d.mail.gandi.net (Postfix) with ESMTPSA id BCE324001B; Mon, 3 Jun 2019 15:11:49 +0000 (UTC) From: Gregory CLEMENT To: Tomasz Maciej Nowak , Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , Rob Herring , Mark Rutland Subject: Re: [PATCH] arm64: dts: marvell: add ESPRESSObin variants In-Reply-To: <20190527111156.3539-1-tmn505@gmail.com> References: <20190527111156.3539-1-tmn505@gmail.com> Date: Mon, 03 Jun 2019 17:11:49 +0200 Message-ID: <87ef4ak8ju.fsf@FE-laptop> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190603_081154_221342_6230BAB2 X-CRM114-Status: GOOD ( 20.41 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Konstantin Porotchkin , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Tomasz Maciej Nowak, > This commit adds dts for different variants of ESPRESSObin board: > > ESPRESSObin with soldered eMMC, > > ESPRESSObin V7, compared to prior versions some passive elements changed > and ethernet ports labels positions have been reversed, > > ESPRESSObin V7 with soldered eMMC. > > Since most of elements are the same, one common dtsi is created and > referenced in each dts of particular variant. > > Signed-off-by: Tomasz Maciej Nowak With your patch "arm64: dts: armada-3720-espressobin: correct spi node" applied this one don't apply anymore. Could you rebase it? Thanks, Gregory > --- > .../marvell/armada-3720-espressobin-emmc.dts | 42 ++++ > .../armada-3720-espressobin-v7-emmc.dts | 59 ++++++ > .../marvell/armada-3720-espressobin-v7.dts | 36 ++++ > .../dts/marvell/armada-3720-espressobin.dts | 200 +----------------- > .../dts/marvell/armada-3720-espressobin.dtsi | 193 +++++++++++++++++ > 5 files changed, 331 insertions(+), 199 deletions(-) > create mode 100644 arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts > create mode 100644 arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts > create mode 100644 arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7.dts > create mode 100644 arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi > > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts > new file mode 100644 > index 000000000000..bd9ed9dc9c3e > --- /dev/null > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts > @@ -0,0 +1,42 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Device Tree file for Globalscale Marvell ESPRESSOBin Board with eMMC > + * Copyright (C) 2018 Marvell > + * > + * Romain Perier > + * Konstantin Porotchkin > + * > + */ > +/* > + * Schematic available at http://espressobin.net/wp-content/uploads/2017/08/ESPRESSObin_V5_Schematics.pdf > + */ > + > +#include "armada-3720-espressobin.dtsi" > + > +/ { > + model = "Globalscale Marvell ESPRESSOBin Board (eMMC)"; > + compatible = "globalscale,espressobin-emmc", "globalscale,espressobin", > + "marvell,armada3720", "marvell,armada3710"; > +}; > + > +/* U11 */ > +&sdhci0 { > + non-removable; > + bus-width = <8>; > + mmc-ddr-1_8v; > + mmc-hs400-1_8v; > + marvell,xenon-emmc; > + marvell,xenon-tun-count = <9>; > + marvell,pad-type = "fixed-1-8v"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc_pins>; > + status = "okay"; > + > + #address-cells = <1>; > + #size-cells = <0>; > + mmccard: mmccard@0 { > + compatible = "mmc-card"; > + reg = <0>; > + }; > +}; > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts > new file mode 100644 > index 000000000000..6e876a6d9532 > --- /dev/null > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts > @@ -0,0 +1,59 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Device Tree file for Globalscale Marvell ESPRESSOBin Board V7 with eMMC > + * Copyright (C) 2018 Marvell > + * > + * Romain Perier > + * Konstantin Porotchkin > + * > + */ > +/* > + * Schematic available at http://wiki.espressobin.net/tiki-download_file.php?fileId=200 > + */ > + > +#include "armada-3720-espressobin.dtsi" > + > +/ { > + model = "Globalscale Marvell ESPRESSOBin Board V7 (eMMC)"; > + compatible = "globalscale,espressobin-v7-emmc", "globalscale,espressobin-v7", > + "globalscale,espressobin", "marvell,armada3720", > + "marvell,armada3710"; > +}; > + > +&switch0 { > + ports { > + port@1 { > + reg = <1>; > + label = "lan1"; > + phy-handle = <&switch0phy0>; > + }; > + > + port@3 { > + reg = <3>; > + label = "wan"; > + phy-handle = <&switch0phy2>; > + }; > + }; > +}; > + > +/* U11 */ > +&sdhci0 { > + non-removable; > + bus-width = <8>; > + mmc-ddr-1_8v; > + mmc-hs400-1_8v; > + marvell,xenon-emmc; > + marvell,xenon-tun-count = <9>; > + marvell,pad-type = "fixed-1-8v"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc_pins>; > + status = "okay"; > + > + #address-cells = <1>; > + #size-cells = <0>; > + mmccard: mmccard@0 { > + compatible = "mmc-card"; > + reg = <0>; > + }; > +}; > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7.dts > new file mode 100644 > index 000000000000..0f8405d085fd > --- /dev/null > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7.dts > @@ -0,0 +1,36 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Device Tree file for Globalscale Marvell ESPRESSOBin Board V7 > + * Copyright (C) 2018 Marvell > + * > + * Romain Perier > + * Konstantin Porotchkin > + * > + */ > +/* > + * Schematic available at http://wiki.espressobin.net/tiki-download_file.php?fileId=200 > + */ > + > +#include "armada-3720-espressobin.dtsi" > + > +/ { > + model = "Globalscale Marvell ESPRESSOBin Board V7"; > + compatible = "globalscale,espressobin-v7", "globalscale,espressobin", > + "marvell,armada3720", "marvell,armada3710"; > +}; > + > +&switch0 { > + ports { > + port@1 { > + reg = <1>; > + label = "lan1"; > + phy-handle = <&switch0phy0>; > + }; > + > + port@3 { > + reg = <3>; > + label = "wan"; > + phy-handle = <&switch0phy2>; > + }; > + }; > +}; > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts > index 6be019e1888e..1542d836c090 100644 > --- a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts > @@ -12,207 +12,9 @@ > > /dts-v1/; > > -#include > -#include "armada-372x.dtsi" > +#include "armada-3720-espressobin.dtsi" > > / { > model = "Globalscale Marvell ESPRESSOBin Board"; > compatible = "globalscale,espressobin", "marvell,armada3720", "marvell,armada3710"; > - > - chosen { > - stdout-path = "serial0:115200n8"; > - }; > - > - memory@0 { > - device_type = "memory"; > - reg = <0x00000000 0x00000000 0x00000000 0x20000000>; > - }; > - > - vcc_sd_reg1: regulator { > - compatible = "regulator-gpio"; > - regulator-name = "vcc_sd1"; > - regulator-min-microvolt = <1800000>; > - regulator-max-microvolt = <3300000>; > - regulator-boot-on; > - > - gpios = <&gpionb 4 GPIO_ACTIVE_HIGH>; > - gpios-states = <0>; > - states = <1800000 0x1 > - 3300000 0x0>; > - enable-active-high; > - }; > -}; > - > -/* J9 */ > -&pcie0 { > - status = "okay"; > - phys = <&comphy1 0>; > - pinctrl-names = "default"; > - pinctrl-0 = <&pcie_reset_pins &pcie_clkreq_pins>; > -}; > - > -/* J6 */ > -&sata { > - status = "okay"; > - phys = <&comphy2 0>; > - phy-names = "sata-phy"; > -}; > - > -/* J1 */ > -&sdhci1 { > - wp-inverted; > - bus-width = <4>; > - cd-gpios = <&gpionb 3 GPIO_ACTIVE_LOW>; > - marvell,pad-type = "sd"; > - vqmmc-supply = <&vcc_sd_reg1>; > - > - pinctrl-names = "default"; > - pinctrl-0 = <&sdio_pins>; > - status = "okay"; > -}; > - > -/* U11 */ > -&sdhci0 { > - non-removable; > - bus-width = <8>; > - mmc-ddr-1_8v; > - mmc-hs400-1_8v; > - marvell,xenon-emmc; > - marvell,xenon-tun-count = <9>; > - marvell,pad-type = "fixed-1-8v"; > - > - pinctrl-names = "default"; > - pinctrl-0 = <&mmc_pins>; > -/* > - * This eMMC is not populated on all boards, so disable it by > - * default and let the bootloader enable it, if it is present > - */ > - status = "disabled"; > -}; > - > -&spi0 { > - status = "okay"; > - > - flash@0 { > - reg = <0>; > - compatible = "winbond,w25q32dw", "jedec,spi-flash"; > - spi-max-frequency = <104000000>; > - m25p,fast-read; > - > - partitions { > - compatible = "fixed-partitions"; > - #address-cells = <1>; > - #size-cells = <1>; > - > - partition@0 { > - label = "uboot"; > - reg = <0 0x180000>; > - }; > - > - partition@180000 { > - label = "ubootenv"; > - reg = <0x180000 0x10000>; > - }; > - }; > - }; > -}; > - > -/* Exported on the micro USB connector J5 through an FTDI */ > -&uart0 { > - pinctrl-names = "default"; > - pinctrl-0 = <&uart1_pins>; > - status = "okay"; > -}; > - > -/* > - * Connector J17 and J18 expose a number of different features. Some pins are > - * multiplexed. This is the case for instance for the following features: > - * - UART1 (pin 24 = RX, pin 26 = TX). See armada-3720-db.dts for an example of > - * how to enable it. Beware that the signals are 1.8V TTL. > - * - I2C > - * - SPI > - * - MMC > - */ > - > -/* J7 */ > -&usb3 { > - status = "okay"; > -}; > - > -/* J8 */ > -&usb2 { > - status = "okay"; > -}; > - > -&mdio { > - switch0: switch0@1 { > - compatible = "marvell,mv88e6085"; > - #address-cells = <1>; > - #size-cells = <0>; > - reg = <1>; > - > - dsa,member = <0 0>; > - > - ports { > - #address-cells = <1>; > - #size-cells = <0>; > - > - port@0 { > - reg = <0>; > - label = "cpu"; > - ethernet = <ð0>; > - phy-mode = "rgmii-id"; > - fixed-link { > - speed = <1000>; > - full-duplex; > - }; > - }; > - > - port@1 { > - reg = <1>; > - label = "wan"; > - phy-handle = <&switch0phy0>; > - }; > - > - port@2 { > - reg = <2>; > - label = "lan0"; > - phy-handle = <&switch0phy1>; > - }; > - > - port@3 { > - reg = <3>; > - label = "lan1"; > - phy-handle = <&switch0phy2>; > - }; > - > - }; > - > - mdio { > - #address-cells = <1>; > - #size-cells = <0>; > - > - switch0phy0: switch0phy0@11 { > - reg = <0x11>; > - }; > - switch0phy1: switch0phy1@12 { > - reg = <0x12>; > - }; > - switch0phy2: switch0phy2@13 { > - reg = <0x13>; > - }; > - }; > - }; > -}; > - > -ð0 { > - pinctrl-names = "default"; > - pinctrl-0 = <&rgmii_pins>, <&smi_pins>; > - phy-mode = "rgmii-id"; > - status = "okay"; > - > - fixed-link { > - speed = <1000>; > - full-duplex; > - }; > }; > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi > new file mode 100644 > index 000000000000..4a114db76bf9 > --- /dev/null > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi > @@ -0,0 +1,193 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Device Tree file for Globalscale Marvell ESPRESSOBin Board > + * Copyright (C) 2016 Marvell > + * > + * Romain Perier > + * > + */ > + > +/dts-v1/; > + > +#include > +#include "armada-372x.dtsi" > + > +/ { > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > + > + memory@0 { > + device_type = "memory"; > + reg = <0x00000000 0x00000000 0x00000000 0x20000000>; > + }; > + > + vcc_sd_reg1: regulator { > + compatible = "regulator-gpio"; > + regulator-name = "vcc_sd1"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + > + gpios = <&gpionb 4 GPIO_ACTIVE_HIGH>; > + gpios-states = <0>; > + states = <1800000 0x1 > + 3300000 0x0>; > + enable-active-high; > + }; > +}; > + > +/* J9 */ > +&pcie0 { > + status = "okay"; > + phys = <&comphy1 0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pcie_reset_pins &pcie_clkreq_pins>; > +}; > + > +/* J6 */ > +&sata { > + status = "okay"; > + phys = <&comphy2 0>; > + phy-names = "sata-phy"; > +}; > + > +/* J1 */ > +&sdhci1 { > + wp-inverted; > + bus-width = <4>; > + cd-gpios = <&gpionb 3 GPIO_ACTIVE_LOW>; > + marvell,pad-type = "sd"; > + vqmmc-supply = <&vcc_sd_reg1>; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&sdio_pins>; > + status = "okay"; > +}; > + > +&spi0 { > + status = "okay"; > + > + flash@0 { > + reg = <0>; > + compatible = "winbond,w25q32dw", "jedec,spi-flash"; > + spi-max-frequency = <104000000>; > + m25p,fast-read; > + > + partitions { > + compatible = "fixed-partitions"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + partition@0 { > + label = "uboot"; > + reg = <0 0x180000>; > + }; > + > + partition@180000 { > + label = "ubootenv"; > + reg = <0x180000 0x10000>; > + }; > + }; > + }; > +}; > + > +/* Exported on the micro USB connector J5 through an FTDI */ > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart1_pins>; > + status = "okay"; > +}; > + > +/* > + * Connector J17 and J18 expose a number of different features. Some pins are > + * multiplexed. This is the case for instance for the following features: > + * - UART1 (pin 24 = RX, pin 26 = TX). See armada-3720-db.dts for an example of > + * how to enable it. Beware that the signals are 1.8V TTL. > + * - I2C > + * - SPI > + * - MMC > + */ > + > +/* J7 */ > +&usb3 { > + status = "okay"; > +}; > + > +/* J8 */ > +&usb2 { > + status = "okay"; > +}; > + > +&mdio { > + switch0: switch0@1 { > + compatible = "marvell,mv88e6085"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <1>; > + > + dsa,member = <0 0>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + label = "cpu"; > + ethernet = <ð0>; > + phy-mode = "rgmii-id"; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + > + port@1 { > + reg = <1>; > + label = "wan"; > + phy-handle = <&switch0phy0>; > + }; > + > + port@2 { > + reg = <2>; > + label = "lan0"; > + phy-handle = <&switch0phy1>; > + }; > + > + port@3 { > + reg = <3>; > + label = "lan1"; > + phy-handle = <&switch0phy2>; > + }; > + > + }; > + > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + switch0phy0: switch0phy0@11 { > + reg = <0x11>; > + }; > + switch0phy1: switch0phy1@12 { > + reg = <0x12>; > + }; > + switch0phy2: switch0phy2@13 { > + reg = <0x13>; > + }; > + }; > + }; > +}; > + > +ð0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&rgmii_pins>, <&smi_pins>; > + phy-mode = "rgmii-id"; > + status = "okay"; > + > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > +}; > -- > 2.21.0 > -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel