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=-8.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS, URIBL_BLOCKED,USER_AGENT_MUTT 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 13DE7C43381 for ; Fri, 22 Mar 2019 01:32:29 +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 D842120700 for ; Fri, 22 Mar 2019 01:32:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="QkpnnrOg"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="GvX9gfUV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D842120700 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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:In-Reply-To:MIME-Version:References: 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=nqtBytO7v3Pm8iWExAiIt9SyZnfnIRejPoXbdrc1wuI=; b=QkpnnrOgjAHL5Y bGJ4guOvriePkiwmjKzdn0+WszquoeGDvQbR88+RVDLV9vpl2B5++NfXIzlqueacLlgQbp33f5BbD o5MpS0jy0C+g4DTO1TG1JcAhXirEk7RZPTHcYendPA2teug24ypkK7fntlMowyRfbUEcixtoyTYmr Dy7Y5cjTeTeCSn73QS3qtL5qsc1/IkVMRNBJzZvrKx8414b2NGg780zdoNS/QbnR3vpOiuT6n7Paz ttzEYrjuOjtJftIgLbVYQfy0h8ERqgU0xZPqfkcBt4ZZKWJ4jFdGb607Uz5N8GPFAZMqDyM9GJxao tuI+hffQaEiLlgqUr30Q==; 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 1h792o-0002j7-AS; Fri, 22 Mar 2019 01:32:26 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h792l-0002ik-7X for linux-arm-kernel@lists.infradead.org; Fri, 22 Mar 2019 01:32:25 +0000 Received: from dragon (98.142.130.235.16clouds.com [98.142.130.235]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4DB8F20700; Fri, 22 Mar 2019 01:32:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1553218342; bh=OVzh/0wg9uiQK9IOT3dYhmzsMv8mvSaRvQykIFKAvOQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GvX9gfUVIGB0S4RsdR15TI4rn6XgLZPIWHWJpKlD/PjrDHP7nglL7yO97YYVWY7+g 3guSwX6WT9wImkPuCG2DY/4yegDZ5q/ZA1P2flsV2BN1SK0nmYuMxpmphko2cjeMwc 007wx4LmkZLZGSnmFeXc6jiExCcdf+0NF8SmDL1o= Date: Fri, 22 Mar 2019 09:31:53 +0800 From: Shawn Guo To: Jonathan =?iso-8859-1?Q?Neusch=E4fer?= Subject: Re: [PATCH 2/5] ARM: dts: imx50: Add Kobo Aura DTS Message-ID: <20190322013151.GR12513@dragon> References: <20190319152421.16179-1-j.neuschaefer@gmx.net> <20190319152421.16179-3-j.neuschaefer@gmx.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20190319152421.16179-3-j.neuschaefer@gmx.net> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190321_183223_598391_A421890B X-CRM114-Status: GOOD ( 22.77 ) 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: Mark Rutland , devicetree@vger.kernel.org, Heiko Stuebner , Martin Blumenstingl , Kevin Hilman , Sascha Hauer , Johan Hovold , linux-kernel@vger.kernel.org, Rob Herring , NXP Linux Team , Pengutronix Kernel Team , Manivannan Sadhasivam , Thierry Reding , Fabio Estevam , linux-arm-kernel@lists.infradead.org, Jagan Teki Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Mar 19, 2019 at 04:24:17PM +0100, Jonathan Neusch=E4fer wrote: > The Kobo Aura is an e-book reader released in 2013. > = > With the devicetree in its current state, the kernel will boot and run > for about ten seconds. To solve this, the embedded controller needs to > be told that the system should stay powered on. This will be done in a > later patchset. > = > - The IOMUXC mode bits for the SD interfaces were taken from the > vendor's U-Boot fork. > - The bus width of the eMMC is 4 bits in the vendor kernel, but I > achieved better performance with 8 bits. > - The SDIO clock frequency for the WiFi chip is 25MHz in the vendor > kernel, but the WiFi chip (BCM43362) supports 50MHz, which works > reliably on this board and gives slightly better performance. > - The I2C pins' IOMUXC settings come from the vendor's U-Boot fork. > = > Signed-off-by: Jonathan Neusch=E4fer > --- > arch/arm/boot/dts/Makefile | 3 +- > arch/arm/boot/dts/imx50-kobo-aura.dts | 245 ++++++++++++++++++++++++++ > 2 files changed, 247 insertions(+), 1 deletion(-) > create mode 100644 arch/arm/boot/dts/imx50-kobo-aura.dts > = > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f4f5aeaf3298..0c85156c552b 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -363,7 +363,8 @@ dtb-$(CONFIG_SOC_IMX35) +=3D \ > imx35-eukrea-mbimxsd35-baseboard.dtb \ > imx35-pdk.dtb > dtb-$(CONFIG_SOC_IMX50) +=3D \ > - imx50-evk.dtb > + imx50-evk.dtb \ > + imx50-kobo-aura.dtb > dtb-$(CONFIG_SOC_IMX51) +=3D \ > imx51-apf51.dtb \ > imx51-apf51dev.dtb \ > diff --git a/arch/arm/boot/dts/imx50-kobo-aura.dts b/arch/arm/boot/dts/im= x50-kobo-aura.dts > new file mode 100644 > index 000000000000..e778a677c752 > --- /dev/null > +++ b/arch/arm/boot/dts/imx50-kobo-aura.dts > @@ -0,0 +1,245 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +// Copyright 2019 Jonathan Neusch=E4fer > +// > +// The Kobo Aura e-book reader, model N514. The mainboard is marked as E= 606F0B. > + > +/dts-v1/; > +#include "imx50.dtsi" > +#include > + > +/ { > + model =3D "Kobo Aura (N514)"; > + compatible =3D "kobo,aura", "fsl,imx50"; Board compatible should also be documented. > + > + chosen { > + stdout-path =3D "serial1:115200n8"; > + }; > + > + memory@70000000 { > + device_type =3D "memory"; > + reg =3D <0x70000000 0x10000000>; > + }; > + > + gpio-leds { > + compatible =3D "gpio-leds"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_leds>; > + > + on { > + label =3D "kobo_aura:orange:on"; > + gpios =3D <&gpio6 24 GPIO_ACTIVE_LOW>; > + panic-indicator; > + }; > + }; > + > + gpio-keys { > + compatible =3D "gpio-keys"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_gpiokeys>; > + > + power { > + label =3D "Power Button"; > + gpios =3D <&gpio4 10 GPIO_ACTIVE_LOW>; > + linux,code =3D ; > + }; > + > + hallsensor { > + label =3D "Hallsensor"; > + gpios =3D <&gpio5 15 GPIO_ACTIVE_LOW>; > + linux,code =3D <0>; Use define KEY_RESERVED? > + linux,input-type =3D <5>; Use define EV_SW? > + }; > + > + frontlight { > + label =3D "Frontlight"; > + gpios =3D <&gpio4 1 GPIO_ACTIVE_LOW>; > + linux,code =3D ; > + }; > + }; > + > + sd2_pwrseq: pwrseq { > + compatible =3D "mmc-pwrseq-simple"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_sd2_reset>; > + Please do not have random newlines. > + reset-gpios =3D <&gpio4 17 GPIO_ACTIVE_LOW>; > + }; > + > + sd2_vmmc: gpio-regulator { > + compatible =3D "regulator-gpio"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_sd2_vmmc>; > + > + regulator-name =3D "vmmc"; > + states =3D <3300000 0>; > + regulator-min-microvolt =3D <3300000>; > + regulator-max-microvolt =3D <3300000>; > + enable-gpio =3D <&gpio4 12 GPIO_ACTIVE_LOW>; > + startup-delay-us =3D <100000>; > + }; > +}; > + > +&iomuxc { > + pinctrl_uart2: uart2 { > + fsl,pins =3D < > + MX50_PAD_UART2_TXD__UART2_TXD_MUX 0x1e4 > + MX50_PAD_UART2_RXD__UART2_RXD_MUX 0x1e4 > + >; > + }; > + > + pinctrl_i2c1: i2c1 { Please sort these pinctrl nodes alphabetically. > + fsl,pins =3D < > + MX50_PAD_I2C1_SCL__I2C1_SCL 0x400001fd > + MX50_PAD_I2C1_SDA__I2C1_SDA 0x400001fd > + >; > + }; > + > + pinctrl_i2c2: i2c2 { > + fsl,pins =3D < > + MX50_PAD_I2C2_SCL__I2C2_SCL 0x400001fd > + MX50_PAD_I2C2_SDA__I2C2_SDA 0x400001fd > + >; > + }; > + > + pinctrl_i2c3: i2c3 { > + fsl,pins =3D < > + MX50_PAD_I2C3_SCL__I2C3_SCL 0x400001fd > + MX50_PAD_I2C3_SDA__I2C3_SDA 0x400001fd > + >; > + }; > + > + pinctrl_leds: leds { > + fsl,pins =3D < > + MX50_PAD_PWM1__GPIO6_24 0x0 > + >; > + }; > + > + pinctrl_gpiokeys: gpiokeys { > + fsl,pins =3D < > + MX50_PAD_CSPI_MISO__GPIO4_10 0x0 > + MX50_PAD_SD2_D7__GPIO5_15 0x0 > + MX50_PAD_KEY_ROW0__GPIO4_1 0x0 > + >; > + }; > + > + pinctrl_sd1: sd1 { > + fsl,pins =3D < > + MX50_PAD_SD1_CMD__ESDHC1_CMD 0x1e4 > + MX50_PAD_SD1_CLK__ESDHC1_CLK 0xd4 > + MX50_PAD_SD1_D0__ESDHC1_DAT0 0x1d4 > + MX50_PAD_SD1_D1__ESDHC1_DAT1 0x1d4 > + MX50_PAD_SD1_D2__ESDHC1_DAT2 0x1d4 > + MX50_PAD_SD1_D3__ESDHC1_DAT3 0x1d4 > + > + MX50_PAD_SD2_CD__GPIO5_17 0x0 > + >; > + }; > + > + pinctrl_sd2: sd2 { > + fsl,pins =3D < > + MX50_PAD_SD2_CMD__ESDHC2_CMD 0x1e4 > + MX50_PAD_SD2_CLK__ESDHC2_CLK 0xd4 > + MX50_PAD_SD2_D0__ESDHC2_DAT0 0x1d4 > + MX50_PAD_SD2_D1__ESDHC2_DAT1 0x1d4 > + MX50_PAD_SD2_D2__ESDHC2_DAT2 0x1d4 > + MX50_PAD_SD2_D3__ESDHC2_DAT3 0x1d4 > + >; > + }; > + > + pinctrl_sd2_reset: sd2-reset { > + fsl,pins =3D < > + MX50_PAD_ECSPI2_MOSI__GPIO4_17 0x0 > + >; > + }; > + > + pinctrl_sd2_vmmc: sd2-vmmc { > + fsl,pins =3D < > + MX50_PAD_ECSPI1_SCLK__GPIO4_12 0x0 > + >; > + }; > + > + pinctrl_sd3: sd3 { > + fsl,pins =3D < > + MX50_PAD_SD3_CMD__ESDHC3_CMD 0x1e4 > + MX50_PAD_SD3_CLK__ESDHC3_CLK 0xd4 > + MX50_PAD_SD3_D0__ESDHC3_DAT0 0x1d4 > + MX50_PAD_SD3_D1__ESDHC3_DAT1 0x1d4 > + MX50_PAD_SD3_D2__ESDHC3_DAT2 0x1d4 > + MX50_PAD_SD3_D3__ESDHC3_DAT3 0x1d4 > + MX50_PAD_SD3_D4__ESDHC3_DAT4 0x1d4 > + MX50_PAD_SD3_D5__ESDHC3_DAT5 0x1d4 > + MX50_PAD_SD3_D6__ESDHC3_DAT6 0x1d4 > + MX50_PAD_SD3_D7__ESDHC3_DAT7 0x1d4 > + >; > + }; > +}; > + > +&uart2 { > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_uart2>; > + status =3D "okay"; > +}; > + > +&i2c1 { Please sort these labeled nodes alphabetically. > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_i2c1>; > + status =3D "okay"; > + > + /* TODO: ektf2132 touch controller at 0x15 */ > +}; > + > +&i2c2 { > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_i2c2>; > + status =3D "okay"; > + > + /* TODO: TPS65185 PMIC for E Ink at 0x68 */ > +}; > + > +&i2c3 { > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_i2c3>; > + status =3D "okay"; > + > + /* TODO: embedded controller at 0x43 */ > +}; > + > +&esdhc1 { > + status =3D "okay"; We usually end property list with 'status'. Shawn > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_sd1>; > + > + max-frequency =3D <50000000>; > + bus-width =3D <4>; > + cd-gpios =3D <&gpio5 17 GPIO_ACTIVE_LOW>; > + disable-wp; > + > + /* External =B5SD card */ > +}; > + > +&esdhc2 { > + status =3D "okay"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_sd2>; > + > + bus-width =3D <4>; > + max-frequency =3D <50000000>; > + disable-wp; > + mmc-pwrseq =3D <&sd2_pwrseq>; > + vmmc-supply =3D <&sd2_vmmc>; > + > + /* CyberTan WC121 SDIO WiFi (BCM43362) */ > +}; > + > +&esdhc3 { > + status =3D "okay"; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_sd3>; > + > + bus-width =3D <8>; > + non-removable; > + max-frequency =3D <50000000>; > + disable-wp; > + > + /* Internal eMMC */ > +}; > -- > 2.20.1 > = _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel