From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [217.70.183.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 698EC20C028 for ; Thu, 23 Jan 2025 09:41:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.197 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737625323; cv=none; b=G38yFnHW9eLdH6XBqNYAedjRWWoPDuuFE4/5IX3MgPHiTcvxJNR/5SQ1iyAty7So1jNoTYaxKGwHwfIW4mrVq3HsMIelyfbSLPZi+7W6nvsp2K1MEHQPTW2jDP7MyeoNlbbQt5BwtoP1XNCi1ZeOpV3IvgM2zrutYVkFR8URLIw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737625323; c=relaxed/simple; bh=R8734WZE9YzXSuKpzfSaWJTn5B895MHzZ38V+/QD+Mg=; h=Mime-Version:Content-Type:Date:Message-Id:From:To:Subject:Cc: References:In-Reply-To; b=k/KnKOXc8F7gZ8Xb02kuAZ8PQzPmPnUZHXwrRomFUpMnMLDp2AhDfF1hXDZZAPPMbxvANiyiV28Y/rCC1Vx6k7e6KZZQLRg0uQrWmRfGoSpr8RVuUJF0hShzm/IpdoRhleH1XuGUTBvgiLnO+S55pxe5mbwlV+zfctoJA0lKHoY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=UJ4zshe3; arc=none smtp.client-ip=217.70.183.197 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="UJ4zshe3" Received: by mail.gandi.net (Postfix) with ESMTPSA id DBBE31C0003; Thu, 23 Jan 2025 09:41:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1737625318; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=igDCe1x3xTNsIM45RXmz5PFqCPI/aYPG1aPW/HcNGd4=; b=UJ4zshe3EB8DFHjQ7p569pU14BPcj+YE2VPlKO9cqBCjMF07HiQG4iFVbG6osBFUxlUct6 MlMsS2Vfbucho97qjn61wEt+MWThhmloPXdlpnyZmcqzIA9K50ZrIvNBciPetNC/SEY8P2 Q1qjI/BSXLHFSesPWU+FxBbs59+QsOedCVGY9pEqwtpMsBSJGSvvL3mNAb5Fdy9hzibJlJ FIkmwF8IwIkKT8ZjODVYy0g2ES6FC9o8I9WEwKEV67R5cGp3GIh4SSwt49SRacU87xGI6E 5gfXuca16Mv2dxBAgIr53X2OqYToKT46mzPHsmRbdyl6MJX5RpiHJfKSi3hLag== Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Thu, 23 Jan 2025 10:41:57 +0100 Message-Id: From: "Antonin Godard" To: "Krzysztof Kozlowski" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Shawn Guo" , "Sascha Hauer" , "Pengutronix Kernel Team" , "Fabio Estevam" Subject: Re: [PATCH 3/3] ARM: dts: imx6ul: Add Variscite Concerto board support Cc: "Thomas Petazzoni" , , , , X-Mailer: aerc 0.18.2-100-gc2048ef30452-dirty References: <20250121-varsom6ul-concerto-dts-v1-0-eb349acf0ac6@bootlin.com> <20250121-varsom6ul-concerto-dts-v1-3-eb349acf0ac6@bootlin.com> <56e74e80-8e90-4784-b284-bee1af35e37e@kernel.org> In-Reply-To: <56e74e80-8e90-4784-b284-bee1af35e37e@kernel.org> X-GND-Sasl: antonin.godard@bootlin.com Hi Krzysztof, On Tue Jan 21, 2025 at 5:03 PM CET, Krzysztof Kozlowski wrote: > On 21/01/2025 10:33, Antonin Godard wrote: >> This patch adds support for the Variscite Concerto Carrier Board. > > Please do not use "This commit/patch/change", but imperative mood. See > longer explanation here: > https://elixir.bootlin.com/linux/v5.17.1/source/Documentation/process/sub= mitting-patches.rst#L95 Will do in v2. >> >> This Carrier-Board has the following: >> - LVDS interface for the VLCD-CAP-GLD-LVDS 7" LCD 800 x 480 touch >> display (not configured) >> - USB Host + USB OTG Connector >> - 10/100 Mbps Ethernet >> - miniPCI-Express slot >> - SD Card connector >> - Audio Headphone/Line In jack connectors >> - S-ATA >> - On-board DMIC >>=20 >> Product Page: https://www.variscite.com/product/single-board-computers/c= oncerto-board >>=20 >> This file is based on the one provided by Variscite on their own kernel, >> but adapted for mainline. >>=20 >> Signed-off-by: Antonin Godard >> --- >> arch/arm/boot/dts/nxp/imx/Makefile | 1 + >> .../boot/dts/nxp/imx/imx6ul-var-som-concerto.dts | 331 ++++++++++++++= +++++++ >> 2 files changed, 332 insertions(+) >>=20 >> diff --git a/arch/arm/boot/dts/nxp/imx/Makefile b/arch/arm/boot/dts/nxp/= imx/Makefile >> index 39a153536d2a2b8f75b5fbe4332660f89442064a..94c9bc94cc8e2daa1fb3b568= 6b0b58db1f6678b6 100644 >> --- a/arch/arm/boot/dts/nxp/imx/Makefile >> +++ b/arch/arm/boot/dts/nxp/imx/Makefile >> @@ -329,6 +329,7 @@ dtb-$(CONFIG_SOC_IMX6UL) +=3D \ >> imx6ul-tx6ul-0010.dtb \ >> imx6ul-tx6ul-0011.dtb \ >> imx6ul-tx6ul-mainboard.dtb \ >> + imx6ul-var-som-concerto.dtb \ >> imx6ull-14x14-evk.dtb \ >> imx6ull-colibri-aster.dtb \ >> imx6ull-colibri-emmc-aster.dtb \ >> diff --git a/arch/arm/boot/dts/nxp/imx/imx6ul-var-som-concerto.dts b/arc= h/arm/boot/dts/nxp/imx/imx6ul-var-som-concerto.dts >> new file mode 100644 >> index 0000000000000000000000000000000000000000..4289641d94c5a72ba985f339= 652039dbf13da40c >> --- /dev/null >> +++ b/arch/arm/boot/dts/nxp/imx/imx6ul-var-som-concerto.dts >> @@ -0,0 +1,331 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Support for Variscite MX6 Concerto Carrier board with the VAR-SOM-MX= 6UL >> + * Variscite SoM mounted on it >> + * >> + * Copyright 2019 Variscite Ltd. >> + * Copyright 2025 Bootlin >> + */ >> + >> +#include "imx6ul-var-som.dtsi" >> + >> +/ { >> + model =3D "Variscite VAR-SOM-MX6UL Concerto Board"; >> + compatible =3D "variscite,mx6concerto", "variscite,var-som-imx6ul", "f= sl,imx6ul"; >> + >> + backlight { >> + compatible =3D "pwm-backlight"; >> + pwms =3D <&pwm4 0 20000 0>; >> + brightness-levels =3D <0 4 8 16 32 64 128 255>; >> + default-brightness-level =3D <6>; >> + status =3D "okay"; > > Which file disables it? This is a mistake, I forgot to remove this when removing the parts I couldn= 't test. Will remove this node in v2. >> + }; >> + >> + chosen { >> + stdout-path =3D &uart1; >> + }; >> + >> + gpio-keys { >> + compatible =3D "gpio-keys"; >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_gpio_key_back>, <&pinctrl_gpio_key_wakeup>; >> + >> + key-back { >> + gpios =3D <&gpio4 14 GPIO_ACTIVE_LOW>; >> + linux,code =3D ; >> + }; >> + >> + key-wakeup { >> + gpios =3D <&gpio5 8 GPIO_ACTIVE_LOW>; >> + linux,code =3D ; >> + wakeup-source; >> + }; >> + }; >> + >> + leds { >> + compatible =3D "gpio-leds"; >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_gpio_leds>; >> + >> + gpled2 { > > led-0 > led-1 > led-2 Will rename this node to led-0, and set the label to "gpled2" (this is how = it's named on the schematic/datasheet). > Are there other leds here? Nothing else is obvious to me on the schematic. There is no gpled0 or gpled= 1. >> + gpios =3D <&gpio1 25 GPIO_ACTIVE_HIGH>; >> + linux,default-trigger =3D "heartbeat"; > > Missing function and color Will set the function to STATUS, color green. This is a general purpose led= that can be used for debugging purposes or as a status indicator, I think. >> + }; >> + }; >> +}; >> + >> +&can1 { >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_flexcan1>; >> + status =3D "okay"; >> +}; >> + >> +&fec1 { >> + status =3D "disabled"; >> +}; >> + >> +&fec2 { >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_enet2>, <&pinctrl_enet2_gpio>, <&pinctrl_enet2= _mdio>; >> + phy-mode =3D "rmii"; >> + phy-handle =3D <ðphy1>; >> + phy-reset-gpios =3D <&gpio5 5 GPIO_ACTIVE_LOW>; >> + phy-reset-duration =3D <100>; >> + status =3D "okay"; >> + >> + mdio { >> + #address-cells =3D <1>; >> + #size-cells =3D <0>; >> + >> + ethphy1: ethernet-phy@3 { >> + compatible =3D "ethernet-phy-ieee802.3-c22"; >> + micrel,rmii-reference-clock-select-25-mhz =3D <1>; >> + micrel,led-mode =3D <0>; >> + clocks =3D <&rmii_ref_clk>; >> + clock-names =3D "rmii-ref"; >> + reg =3D <3>; >> + }; >> + }; >> +}; >> + >> +&i2c1 { >> + clock-frequency =3D <100000>; >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_i2c1>; >> + status =3D "okay"; >> + >> + /* DS1337 RTC module */ > > Drop comment, obvious. This cannot be anything else, because node name > and compatible told that. Will do in v2. >> + rtc@68 { >> + /* >> + * To actually use this interrupt >> + * connect pins J14.8 & J14.10 on the Concerto-Board. >> + */ >> + compatible =3D "dallas,ds1337"; >> + reg =3D <0x68>; >> + pinctrl-names =3D "default"; >> + pinctrl-0 =3D <&pinctrl_rtc>; >> + interrupt-parent =3D <&gpio1>; >> + interrupts =3D <10 IRQ_TYPE_EDGE_FALLING>; >> + }; >> +}; > > > Best regards, > Krzysztof Thanks for the review, Antonin --=20 Antonin Godard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com