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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 668EBC001DB for ; Sun, 13 Aug 2023 16:20:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-Id:Cc:To :Subject:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=1plzhQFsGjL7gXR4smTivQ9KLgjnWShN3LuQYSoKSzg=; b=Yz3jhzmkqkA8YgEIWhU8yHbPz1 pc5kQNzlljXvPOtD03Roub91DF1hFbywVOIfNPZcZ5lb4MSQtN6vfgoxWOQkjtIZrU+647yvuFCIn l0oy6BfBC4mG/He2sH82pO3LdqGGQk9askU+ziWxVGQVe1LZUPOPPa8hSmCT2RnWfr8zoI1hgqMY+ sZBtcFZRqzQMsiLCrYkuKUJ5u/yuNrAFsrRNkg2WIGad1AxLzilBHDLDMJFIHI6nd3PH4Cy0sORfh omFAnXK0CJ55DaACUe7QOL3RU2BopTReQ4YOsDyCpDPpAKZdX61cNak0fDbKembz78EovQcDQ5N74 3j8IJYBw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qVDpH-00FR94-1r; Sun, 13 Aug 2023 16:20:23 +0000 Received: from relay04.th.seeweb.it ([2001:4b7a:2000:18::165]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qVDpC-00FR7A-2g for linux-arm-kernel@lists.infradead.org; Sun, 13 Aug 2023 16:20:21 +0000 Received: from [192.168.2.144] (bband-dyn221.178-41-211.t-com.sk [178.41.211.221]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by m-r1.th.seeweb.it (Postfix) with ESMTPSA id 02D361F55A; Sun, 13 Aug 2023 18:20:06 +0200 (CEST) Date: Sun, 13 Aug 2023 18:20:00 +0200 From: Martin Botka Subject: Re: [PATCH v4 3/4] arm64: dts: allwinner: h616: Add BigTreeTech CB1 SoM & boards support To: Jernej =?iso-8859-2?q?=A9krabec?= Cc: Martin Botka , Martin Botka , Konrad Dybcio , AngeloGioacchino Del Regno , Marijn Suijten , Jami Kettunen , Paul Bouchara , Andre Przywara , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Samuel Holland , Ludwig Kormann , Icenowy Zheng , Andrew Lunn , Heiko Stuebner , Shawn Guo , Bjorn Andersson , Chris Morgan , Jagan Teki , Maxime Ripard , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org Message-Id: In-Reply-To: <3249535.44csPzL39Z@jernej-laptop> References: <20230807145349.2220490-1-martin@biqu3d.com> <3017957701F1CF4A+20230807145349.2220490-4-martin@biqu3d.com> <3249535.44csPzL39Z@jernej-laptop> X-Mailer: geary/43.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230813_092019_269364_56013A74 X-CRM114-Status: GOOD ( 21.72 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-2"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, Aug 13 2023 at 05:52:10 PM +02:00:00, Jernej =A9krabec = wrote: > Hi Martin, > = > since this will be obviously delayed to 6.7 cycle due to mfd patch = > not being > merged in time, I have few nits below. > = > Dne ponedeljek, 07. avgust 2023 ob 16:53:23 CEST je Martin Botka = > napisal(a): >> From: Martin Botka >> = >> CB1 is Compute Module style board that plugs into Rpi board style = >> adapter or >> Manta 3D printer boards (M4P/M8P). >> = >> The SoM features: >> - H616 SoC >> - 1GiB of RAM >> - AXP313A PMIC >> - RTL8189FTV WiFi >> = >> Boards feature: >> - 4x USB via USB2 hub (usb1 on SoM). >> - SDcard slot for loading images. >> - Ethernet port wired to the internal PHY. (100M) >> - 2x HDMI 2.0. (Only 1 usable on CB1) >> - Power and Status LEDs. (Only Status LED usable on CB1) >> - 40 pin GPIO header >> = >> Currently working: >> - Booting >> - USB >> - UART >> - MMC >> - Status LED >> - WiFi (RTL8189FS via out of tree driver) >> = >> I didnt want to duplicate things so the manta DTS can also be used = >> on BTT >> pi4b adapter. CB1 SoM has its own DTSI file in case other boards = >> shows up >> that accept this SoM. >> = >> Signed-off-by: Martin Botka >> Reviewed-by: Andre Przywara >> --- >> Changes in V2: >> - Fixed whitespace errors >> - Move UART into carrier boards and BTT Pi >> - Remove usb1-vbus regulator >> - Fix ranges and naming of AXP313A rails >> - Add comment specifying why broken-cd in mmc0 is needed >> - Rename sdio_wifi to wifi >> - Specify in commit description that USB-OTG doesnt work >> Changes in V3: >> - Add missed semicolons >> - Move model string from dtsi to board dts >> - Add cb1 compatible >> - Remove extra empty line >> Changed in V4: >> - Extend the range of vcc-dram to 1.5V (1.35V max caused issues = >> with >> booting up >> = >> arch/arm64/boot/dts/allwinner/Makefile | 1 + >> .../sun50i-h616-bigtreetech-cb1-manta.dts | 35 +++++ >> .../sun50i-h616-bigtreetech-cb1.dtsi | 140 = >> ++++++++++++++++++ >> 3 files changed, 176 insertions(+) >> create mode 100644 >> arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1-manta.dts = >> create >> mode 100644 = >> arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1.dtsi >> = >> diff --git a/arch/arm64/boot/dts/allwinner/Makefile >> b/arch/arm64/boot/dts/allwinner/Makefile index = >> 6a96494a2e0a..7b386428510b >> 100644 >> --- a/arch/arm64/boot/dts/allwinner/Makefile >> +++ b/arch/arm64/boot/dts/allwinner/Makefile >> @@ -38,5 +38,6 @@ dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h6-pine-h64.dtb >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h6-pine-h64-model-b.dtb >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h6-tanix-tx6.dtb >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h6-tanix-tx6-mini.dtb >> +dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h616-bigtreetech-cb1-manta.dtb >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h616-orangepi-zero2.dtb >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h616-x96-mate.dtb >> diff --git >> = >> a/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1-manta.dts >> = >> b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1-manta.dts = >> new >> file mode 100644 >> index 000000000000..dbce61b355d6 >> --- /dev/null >> +++ = >> b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1-manta.dts >> @@ -0,0 +1,35 @@ >> +// SPDX-License-Identifier: (GPL-2.0+ or MIT) >> +/* >> + * Copyright (C) 2023 Martin Botka . >> + */ >> + >> +/dts-v1/; >> + >> +#include "sun50i-h616-bigtreetech-cb1.dtsi" >> + >> +/ { >> + model =3D "BigTreeTech CB1"; >> + compatible =3D "bigtreetech,cb1-manta", "bigtreetech,cb1", >> "allwinner,sun50i-h616"; + >> + aliases { >> + serial0 =3D &uart0; >> + }; >> + >> + chosen { >> + stdout-path =3D "serial0:115200n8"; >> + }; >> +}; >> + >> +&ehci1 { >> + status =3D "okay"; >> +}; >> + >> +&ohci1 { >> + status =3D "okay"; >> +}; >> + >> +&uart0 { >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&uart0_ph_pins>; >> + status =3D "okay"; >> +}; >> diff --git = >> a/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1.dtsi >> b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1.dtsi = >> new file >> mode 100644 >> index 000000000000..5e756f217813 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-cb1.dtsi >> @@ -0,0 +1,140 @@ >> +// SPDX-License-Identifier: (GPL-2.0+ or MIT) >> +/* >> + * Copyright (C) 2023 Martin Botka . >> + */ >> + >> +/dts-v1/; >> + >> +#include "sun50i-h616.dtsi" >> + >> +#include >> +#include >> +#include >> + >> +/ { >> + aliases { >> + ethernet0 =3D &rtl8189ftv; >> + }; >> + >> + leds { >> + compatible =3D "gpio-leds"; >> + >> + led-0 { >> + function =3D LED_FUNCTION_STATUS; >> + color =3D ; >> + gpios =3D <&pio 7 5 GPIO_ACTIVE_HIGH>; /* PH5 > */ >> + }; >> + }; >> + >> + reg_vcc5v: regulator-vcc5v { >> + /* board wide 5V supply from carrier boards */ >> + compatible =3D "regulator-fixed"; >> + regulator-name =3D "vcc-5v"; >> + regulator-min-microvolt =3D <5000000>; >> + regulator-max-microvolt =3D <5000000>; >> + regulator-always-on; >> + }; >> + >> + reg_vcc33_wifi: vcc33-wifi { >> + /* Always on 3.3V regulator for WiFi */ > = > Please drop the comment. It's pretty obvious from properties. > = >> + compatible =3D "regulator-fixed"; >> + regulator-name =3D "vcc33-wifi"; >> + regulator-min-microvolt =3D <3300000>; >> + regulator-max-microvolt =3D <3300000>; >> + regulator-always-on; >> + vin-supply =3D <®_vcc5v>; >> + }; >> + >> + reg_vcc_wifi_io: vcc-wifi-io { >> + /* Always on 1.8V/300mA regulator for WiFi */ > = > Ditto. > = > Once fixed, you can add: > Reviewed-by: Jernej Skrabec > = > Best regards, > Jernej Got it for both comments. Will send V5 tomorrow. Cheers, Martin > = >> + compatible =3D "regulator-fixed"; >> + regulator-name =3D "vcc-wifi-io"; >> + regulator-min-microvolt =3D <1800000>; >> + regulator-max-microvolt =3D <1800000>; >> + regulator-always-on; >> + vin-supply =3D <®_vcc33_wifi>; >> + }; >> + >> + wifi_pwrseq: wifi-pwrseq { >> + compatible =3D "mmc-pwrseq-simple"; >> + clocks =3D <&rtc 1>; >> + clock-names =3D "ext_clock"; >> + reset-gpios =3D <&pio 6 18 GPIO_ACTIVE_LOW>; /* PG18 */ >> + post-power-on-delay-ms =3D <200>; >> + }; >> +}; >> + >> +&mmc0 { >> + vmmc-supply =3D <®_dldo1>; >> + /* Card detection pin is not connected */ >> + broken-cd; >> + bus-width =3D <4>; >> + status =3D "okay"; >> +}; >> + >> +&mmc1 { >> + vmmc-supply =3D <®_vcc33_wifi>; >> + vqmmc-supply =3D <®_vcc_wifi_io>; >> + mmc-pwrseq =3D <&wifi_pwrseq>; >> + bus-width =3D <4>; >> + non-removable; >> + mmc-ddr-1_8v; >> + status =3D "okay"; >> + >> + rtl8189ftv: wifi@1 { >> + reg =3D <1>; >> + }; >> +}; >> + >> +&r_i2c { >> + status =3D "okay"; >> + >> + axp313a: pmic@36 { >> + compatible =3D "x-powers,axp313a"; >> + reg =3D <0x36>; >> + interrupt-controller; >> + #interrupt-cells =3D <1>; >> + >> + regulators{ >> + reg_dcdc1: dcdc1 { >> + regulator-name =3D "vdd-gpu-sys"; >> + regulator-min-microvolt =3D > <810000>; >> + regulator-max-microvolt =3D > <990000>; >> + regulator-always-on; >> + }; >> + >> + reg_dcdc2: dcdc2 { >> + regulator-name =3D "vdd-cpu"; >> + regulator-min-microvolt =3D > <810000>; >> + regulator-max-microvolt =3D > <1100000>; >> + regulator-ramp-delay =3D <200>; >> + regulator-always-on; >> + }; >> + >> + reg_dcdc3: dcdc3 { >> + regulator-name =3D "vcc-dram"; >> + regulator-min-microvolt =3D > <1350000>; >> + regulator-max-microvolt =3D > <1500000>; >> + regulator-always-on; >> + }; >> + >> + reg_aldo1: aldo1 { >> + regulator-name =3D "vcc-1v8-pll"; >> + regulator-min-microvolt =3D > <1800000>; >> + regulator-max-microvolt =3D > <1800000>; >> + regulator-always-on; >> + }; >> + >> + reg_dldo1: dldo1 { >> + regulator-name =3D "vcc-3v3-io"; >> + regulator-min-microvolt =3D > <3300000>; >> + regulator-max-microvolt =3D > <3300000>; >> + regulator-always-on; >> + }; >> + }; >> + }; >> +}; >> + >> +&usbphy { >> + status =3D "okay"; >> +}; > = > = > = > = _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel