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=-20.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham 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 5E680C433DB for ; Tue, 26 Jan 2021 18:36:21 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 10D3D2223D for ; Tue, 26 Jan 2021 18:36:21 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 10D3D2223D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kemnade.info Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=okv2dUmpadFsq9cswucJPM0T7Va8NMFKq8IEQdhtCyo=; b=2+9Ute11zbeSAJ+tmEuTqpFvD E8GRs111dR0zyZPVVcFzqz3VVzPW1RUS66Hc78DJDPZT7tXDU/aop/QYpikUjPwLcayP3uT8+ao7G 1kSYQOS0SeUTiIUeik2AVwNV5Pakl0PRrhhxhrZMgOa4Iriqx6LAeUbzmyRq0s9HIzZOnzrenv3Df dmXoAz+xjGf/qyh6bGElXbBZZ5pfe8Y/6IsKbOPRypUfxlJxfFZpudd2SFc5pXgO9naWRq+YqZN5W sX3wpjHVGoFcpqaG/hWLZZsec1How1E7OEe/bfJzRp9/CVvAc+sK62zC7GJ49Qa2mpwVMCdUDYjHV yTa2fqXvQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l4TAe-00089J-73; Tue, 26 Jan 2021 18:34:32 +0000 Received: from mail.andi.de1.cc ([2a01:238:4321:8900:456f:ecd6:43e:202c]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1l4TAa-00087t-LC for linux-arm-kernel@lists.infradead.org; Tue, 26 Jan 2021 18:34:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=kemnade.info; s=20180802; h=Content-Transfer-Encoding:Content-Type: MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=sEe7JZmLM6XUizAhFmljQXgrT2pBkysrCkaIf1qi5+A=; b=cLiUWGAfdyZayF6zjF6dU2GFdO ewzTXtNjF8uG8GGQqkHANNwicnMMc0wSTl1l3x9+QH0OXNn9Bz/4/CURL+P/w/OhAgKolJjnHfGt3 PgyirGr1F6JiT/MBCVoABSavVfF9TVtktLfHvZLtkvEB1qLnGWpGVXoPxACjuY3Laa08=; Received: from p200300ccff1467001a3da2fffebfd33a.dip0.t-ipconnect.de ([2003:cc:ff14:6700:1a3d:a2ff:febf:d33a] helo=aktux) by mail.andi.de1.cc with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1l4TAS-0006AS-Hf; Tue, 26 Jan 2021 19:34:20 +0100 Date: Tue, 26 Jan 2021 19:34:19 +0100 From: Andreas Kemnade To: Armin Preiml Subject: Re: [PATCH] dts: ARM: add kobo glo hd ebook reader Message-ID: <20210126193419.1e067639@aktux> In-Reply-To: <20210126173130.45427-1-apreiml@strohwolke.at> References: <20210126173130.45427-1-apreiml@strohwolke.at> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210126_133428_929920_52D85971 X-CRM114-Status: GOOD ( 25.89 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, linux-kernel@vger.kernel.org, Rob Herring , linux-imx@nxp.com, kernel@pengutronix.de, festevam@gmail.com, 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+linux-arm-kernel=archiver.kernel.org@lists.infradead.org [resent with some more complete address list, get_maintainer.pl provides more] Hi, On Tue, 26 Jan 2021 18:31:31 +0100 Armin Preiml wrote: > This patch adds basic support for the kobo glo hd reader. It defines CPU, > memory, UART and storage.Also add pin control settings for i2c and sdhc. > > All values where extracted from the vendor kernel and u-boot sources. > > Signed-off-by: Armin Preiml > > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/imx6sl-kobo-glo-hd.dts | 164 +++++++++++++++++++++++ > 2 files changed, 165 insertions(+) > create mode 100644 arch/arm/boot/dts/imx6sl-kobo-glo-hd.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 3d1ea0b25..ba608414e 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -598,6 +598,7 @@ dtb-$(CONFIG_SOC_IMX6Q) += \ > imx6qp-zii-rdu2.dtb > dtb-$(CONFIG_SOC_IMX6SL) += \ > imx6sl-evk.dtb \ > + imx6sl-kobo-glo-hd.dtb \ > imx6sl-tolino-shine2hd.dtb \ > imx6sl-tolino-shine3.dtb \ > imx6sl-warp.dtb > diff --git a/arch/arm/boot/dts/imx6sl-kobo-glo-hd.dts b/arch/arm/boot/dts/imx6sl-kobo-glo-hd.dts > new file mode 100644 > index 000000000..759699e9e > --- /dev/null > +++ b/arch/arm/boot/dts/imx6sl-kobo-glo-hd.dts > @@ -0,0 +1,164 @@ > +// SPDX-License-Identifier: (GPL-2.0) > +/* > + * Device tree for the Kobo Glo HD ebook reader. > + * > + * Name on mainboard is: 37NB-E60Q90+4A1 > + * Board name in uboot sources: E60Q90 > + * > + * Copyright 2021 Armin Preiml > + * based on works > + * Copyright 2015 Freescale Semiconductor, Inc. > + */ > + > +/dts-v1/; > + > +#include > +#include > +#include "imx6sl.dtsi" > + > +/ { > + model = "Kobo Glo HD"; > + compatible = "kobo,glohd", "fsl,imx6sl"; > + kobo,glohd should be added to Documentation/devicetree/bindings/arm/fsl.yaml [...] > + pinctrl_uart1: uart1grp { > + fsl,pins = < > + MX6SL_PAD_UART1_TXD__UART1_TX_DATA 0x1b0b1 > + MX6SL_PAD_UART1_RXD__UART1_RX_DATA 0x1b0b1 > + >; > + }; > + hmm, pictures (we are talking about FCC ID NOIKBN437, right?) show two uarts, next to each other. Which one is this? The other uart might be the same as in the tolino2-shinehd. > + pinctrl_usdhc1: usdhc1grp { /* 50MHZ (>50MHZ: 0x1f0f9) */ > + fsl,pins = < > + MX6SL_PAD_SD1_CLK__SD1_CLK 0x1f071 > + MX6SL_PAD_SD1_CMD__SD1_CMD 0x1f071 > + MX6SL_PAD_SD1_DAT0__SD1_DATA0 0x1f071 > + MX6SL_PAD_SD1_DAT1__SD1_DATA1 0x1f071 > + MX6SL_PAD_SD1_DAT2__SD1_DATA2 0x1f071 > + MX6SL_PAD_SD1_DAT3__SD1_DATA3 0x1f071 > + MX6SL_PAD_SD1_DAT4__SD1_DATA4 0x1f071 > + MX6SL_PAD_SD1_DAT5__SD1_DATA5 0x1f071 > + MX6SL_PAD_SD1_DAT6__SD1_DATA6 0x1f071 > + MX6SL_PAD_SD1_DAT7__SD1_DATA7 0x1f071 > + >; > + }; > + That is for emmc? And it is not populated, so probably better not enable that here. [...] > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_uart1>; > + status = "okay"; > +}; > + > +&usdhc1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usdhc1>; > + status = "okay"; > +}; > + and also not add this. I guess it might be a good idea to really compare it to the tolino2 shine hd. The boards seem to look similar, but not identical. Hmm, different number of buttons? The question is how similar they are. I think the Kobo Glo HD has a good driver situation. The main PMIC RC5T619 (the rn5t618 driver supports that) is well supported in mainline kernel. It should give regulators, RTC and battery information. The touchscreen (driver zforce_ts) should alse be supported. So you might go further forward. If you are really brave, you could add a complete devicetree on top of my branch kobo/drm-merged-5.10 of github.com/akemnade/linux. Besides of backlight, it should give full hardware support (including a drm driver for the display), so we can see what is different and whether we can use a .dtsi file to define common things. Picture of the Tolino2 internals: https://misc.andi.de1.cc/tolino2.jpg Regards, Andreas _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel