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 E8231CA100D for ; Tue, 5 Sep 2023 20:49:51 +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=8GH/P/Hw+oV5JluOOEd5LeD5uaaTadxwCJqvKmIP28o=; b=SrKbAfXK0YWBqYBs4ygLnGSMIx bVSBTlHBxKFxkteZCDRkc+EFr1SDL0Dm4LE+XfhLdQOvYRR7htMGJZ6mzw5WVbYKw37HzeJVFiJ6e XZ8Crv33yyGXkONlcT5L5Ij1j1h22SzbhbeGbTAkQ720xfjo0bCPKUNaplLevnsqdZv5jNF/pqG0o g0DpZnp0CgUzEIHB7q/svQFUKusBX4CTl8okAEx/tMmlcH4+Boh8/RoBTsb9/8NFh8FWUPnPsN8rB hrYaZGLoBbZrp2C7jdjc7fDXOncTme8NChtTX8czOgPm0DpzowCGkrMHY4vWrwcIr4mGt/7EtdKs+ TJSwhulQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qdczJ-006iDV-1S; Tue, 05 Sep 2023 20:49:29 +0000 Received: from relay01.th.seeweb.it ([5.144.164.162]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qdczE-006iBK-1R for linux-arm-kernel@lists.infradead.org; Tue, 05 Sep 2023 20:49:27 +0000 Received: from [192.168.2.144] (bband-dyn191.178-41-225.t-com.sk [178.41.225.191]) (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 6D07C1FA74; Tue, 5 Sep 2023 22:49:09 +0200 (CEST) Date: Tue, 05 Sep 2023 22:49:02 +0200 From: Martin Botka Subject: Re: [PATCH v4 4/4] arm64: dts: allwinner: h616: Add BigTreeTech Pi 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 , Martin Botka , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Samuel Holland , Ludwig Kormann , Andrew Lunn , Icenowy Zheng , 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: <1909632.taCxCBeP46@archlinux> References: <20230807145349.2220490-1-martin@biqu3d.com> <1909632.taCxCBeP46@archlinux> 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-20230905_134924_819099_AAEEE5EB X-CRM114-Status: GOOD ( 40.62 ) 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 Tue, Sep 5 2023 at 10:45:27 PM +02:00:00, Jernej =A9krabec = wrote: > On Thursday, August 24, 2023 10:48:20 PM CEST Martin Botka wrote: >> On Mon, Aug 14 2023 at 12:47:59 PM +02:00:00, Martin Botka >> = >> wrote: >> > On Mon, Aug 14 2023 at 12:26:07 PM +02:00:00, Jernej =A9krabec >> > >> > wrote: >> >> Dne nedelja, 13. avgust 2023 ob 18:22:49 CEST je Martin Botka >> >> >> >> =7Fnapisal(a): >> >>> On Sun, Aug 13 2023 at 05:55:35 PM +02:00:00, Jernej =A9krabec >> >>> >> >>> wrote: >> >>> > Dne ponedeljek, 07. avgust 2023 ob 16:53:24 CEST je Martin = >> Botka >> >>> > >> >>> > napisal(a): >> >>> >> The BigTreeTech Pi is an H616 based board based on CB1. >> >>> >> Just in Rpi format board. >> >>> >> >> >>> >> It features the same internals as BTT CB1 but adds: >> >>> >> - Fan port >> >>> >> - IR receiver >> >>> >> - ADXL345 Accelerometer connector via SPI >> >>> >> - 24V DC power supply via terminal plugs >> >>> >> - USB to CAN module connector (The actual USB to CAN >> >>> >> >>> =7F=7Fhappens on >> >>> >> >>> >> the >> >>> >> >> >>> >> external module) >> >>> >> >> >>> >> List of currently working things is same as BTT CB1 but = >> also: >> >>> >> - IR receiver >> >>> >> - ADXL345 connector >> >>> >> >> >>> >> Signed-off-by: Martin Botka >> >>> >> Reviewed-by: Andre Przywara >> >>> >> --- >> >>> >> >> >>> >> Changes in V2: >> >>> >> - Add UART alongside aliases and chosen for it >> >>> >> - Add model string >> >>> >> - Enable IR receiver >> >>> >> - Enable SPI0 for ADXL345 connector >> >>> >> >> >>> >> Changes in V3: >> >>> >> - Add missing semicolons >> >>> >> - Add pinctrl for SPI0 >> >>> >> >> >>> >> arch/arm64/boot/dts/allwinner/Makefile | 1 + >> >>> >> .../allwinner/sun50i-h616-bigtreetech-pi.dts | 70 >> >>> >> >> >>> >> +++++++++++++++++++ >> >>> >> >> >>> >> 2 files changed, 71 insertions(+) >> >>> >> create mode 100644 >> >>> >> >> >>> >> = >> arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts >> >>> >> >> >>> >> diff --git a/arch/arm64/boot/dts/allwinner/Makefile >> >>> >> b/arch/arm64/boot/dts/allwinner/Makefile index >> >>> >> >> >>> >> 7b386428510b..0b6232a7f328 >> >>> >> >> >>> >> 100644 >> >>> >> --- a/arch/arm64/boot/dts/allwinner/Makefile >> >>> >> +++ b/arch/arm64/boot/dts/allwinner/Makefile >> >>> >> @@ -39,5 +39,6 @@ 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 >> >>> >> >>> =7F=7Fsun50i-h616-bigtreetech-cb1-manta.dtb >> >>> >> >>> >> +dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-h616-bigtreetech-pi.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-pi.dts >> >>> >> >> >>> >> = >> b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts >> >>> >> >>> =7F=7Fnew >> >>> >> >>> >> file >> >>> >> >> >>> >> mode 100644 >> >>> >> index 000000000000..b0d0386e8f13 >> >>> >> --- /dev/null >> >>> >> +++ >> >>> >> >>> =7F=7Fb/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts >> >>> >> >>> >> @@ -0,0 +1,70 @@ >> >>> >> +// SPDX-License-Identifier: (GPL-2.0+ or MIT) >> >>> >> +/* >> >>> >> + * Copyright (C) 2023 Martin Botka . >> >>> >> + */ >> >>> >> + >> >>> >> +/dts-v1/; >> >>> >> + >> >>> >> +#include "sun50i-h616-bigtreetech-cb1.dtsi" >> >>> >> + >> >>> >> +/ { >> >>> >> + model =3D "BigTreeTech Pi"; >> >>> >> + compatible =3D "bigtreetech,pi", "allwinner,sun50i-h616"; >> >>> >> + >> >>> >> + aliases { >> >>> >> + serial0 =3D &uart0; >> >>> >> + }; >> >>> >> + >> >>> >> + chosen { >> >>> >> + stdout-path =3D "serial0:115200n8"; >> >>> >> + }; >> >>> >> +}; >> >>> >> + >> >>> >> +&ehci0 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ehci1 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ehci2 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ehci3 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ir { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ohci0 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ohci1 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ohci2 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&ohci3 { >> >>> >> + status =3D "okay"; >> >>> >> +}; >> >>> >> + >> >>> >> +&spi0 { >> >>> >> + /* SPI connection for onboard connector for ADXL345 >> >>> >> >>> =7F=7Faccelerometer >> >>> >> >>> > */ >> >>> > >> >>> >> + status =3D "okay"; >> >>> >> + pinctrl-names =3D "default"; >> >>> >> + pinctrl-0 =3D <&spi0_pins>, <&spi0_cs0_pin>; >> >>> > >> >>> > Driver and compatible for ADXL345 already exists, why don't = >> you >> >>> >> >>> =7F=7Fadd >> >>> >> >>> > child node >> >>> > for it? >> >>> > >> >>> > Best regards, >> >>> > Jernej >> >>> >> >>> Ah. So the ADXL345 actually wont be driven by kernel. >> >> >> >> DT is hardware description, it's not concerned what is done on >> >> =7Fsoftware side, >> >> either kernel or user space. >> > >> > Im aware. But this is not a device that is on the board. Its = >> simply a >> > connector for the device. >> > Like Rpi has connectors for camera module :) > = > Actually it matters only if this other board is firmly connected with = > base > board. If it is not, then SPI node should be dropped and be handled = > with DT > overlays. Got it. Will drop it then. > = >> > >> >>> The SPI connection is enabled so that klipper (3d printer >> >>> >> >>> firmware) =7F=7Fcan >> >>> >> >>> be told to look for ADXL345 at this SPI and use it on its own. >> >>> >> >>> Klipper will initialize and communicate with the ADXL on its = >> own. >> >> >> >> What do you mean by firmware? User space app? In this case I = >> suppose >> >> =7Fyou'll use >> >> direct SPI commands from user space? AFAIK that's less and less >> >> =7Fsupported by >> >> kernel (in contrast to I2C). >> > >> > Firmware as in 3d printer firmware. Klipper runs on the board = >> (CB1 or >> > BTT Pi) and is indeed an userspace app. >> > And indeed uses direct SPI commands to the device. >> > >> > The reason for this is the flexibility. >> > If Klipper read the values from kernel or well from the files the >> > ADXL driver would create >> > then it would be unable to communicate with ADXL that is on = >> toolhead >> > board. Or would have to have >> > direct initialization either way for those. Thus it just controls = >> the >> > ADXL itself :) >> > >> > I understand that this may be bit confusing. If there is still >> > something not clear im more then happy to explain in >> > full detail how the userspace and 3D printer communicate :) > = > As I said, DT doesn't care about implementation. DT is HW = > description, so > either if it's fixed connection (soldered wires), then describe in = > full, > otherwise it's considered addon board and thus non-essential, so it = > should be > handled with DT overlays. OK > = > Can be CB1 bought separately from 3D printer package and thus used = > without > ADXL sensor? Yes > = >> > >> > Cheers, >> > Martin. >> = >> Hello, >> Jernej any comments on this ? >> I would like to resolve this conversation :) >> We do have a bunch of time before the new cycle but never hurts to = >> get >> it figured out ahead of time :) > = > Sorry, not enough free time over summer. No worries :) Cheers, Martin > = > Best regards, > Jernej > = >> = >> Cheers, >> Martin >> = >> >> Best regards, >> >> Jernej >> >> >> >>> >> +}; >> >>> >> + >> >>> >> +&uart0 { >> >>> >> + pinctrl-names =3D "default"; >> >>> >> + pinctrl-0 =3D <&uart0_ph_pins>; >> >>> >> + status =3D "okay"; >> >>> >> +}; > = > = > = > = _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel