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 6E14AC4332F for ; Thu, 14 Dec 2023 02:18:10 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=tS7rd7Fm6Uot8OGWOafdK61pOnBR92RkFmgmQ2fkYJ4=; b=I09hGsqB5j6CGf QQKesKnRvSoRS+clQfcLelTYU0QdzitbZdWWTqgvCudrnKwuL87cSEvONwBW3IIV3qhBapZr1eCDq 5tfozFTfo9+3wWX0HrGFcA6n9REU6mss+OjWE9Nb+UJDt+Y+ZzvocyDyY4V5cK9YX0uT6xbDMZOP0 FQ/5wY17pgX4PbggCLRo3sUzyp/idCQ7kMGvYe3V7fYkr/9Gan9F1Ob7CGgJPj2mT8zh0/UXL/XAg euavRx4mLEYdhgvotesd+8ms4gqWoJl6Y1RLTtSt3uWL2AK1eKww6zQJ17y2I8kAiOeb2WeJVDT7f xjzazMKiKCp4sPU6WHNQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rDbII-00GUt3-1w; Thu, 14 Dec 2023 02:17:46 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rDbIE-00GUsL-36 for linux-arm-kernel@lists.infradead.org; Thu, 14 Dec 2023 02:17:44 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id 5B8D9B82033; Thu, 14 Dec 2023 02:17:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1B12DC433C7; Thu, 14 Dec 2023 02:17:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702520260; bh=RR9Mj0aVnA/eW7GuuUubuStZFpU0qgBFK0cuRAVhtT8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UQd0f7b1xYdCRUpe1EXA20dabRv5gz5PEXB1g3O4jwKmhux7+xgzXlxFG2Hy27cTN RVwZH3l6sTynCBDOVtpdhWBykbDlTjK19Ili4qt2Hn5Aq7Ts61aRLOKNmzxqeIa8lz sjEtkjgkUh3zB0eokO2QZI78cmsus1mVTczbPRHUuxT5QeIpDnAOxcHd+Kq5MukDb2 +W2SVa1T2AxEyz4D3NjujeTAoU+EcEaLx3H81GbAckuCMyuPdUcUtJ5KVoOftCTSMI GtZbjBAG9K9GznckzisWhDTpVfGUG2MOHpqLJCR/RXri2rnLd0fCjm7Yf2yIknlIEt M8ath7j2TslbA== Date: Thu, 14 Dec 2023 10:17:34 +0800 From: Shawn Guo To: Dario Binacchi Cc: linux-kernel@vger.kernel.org, Amarula patchwork , michael@amarulasolutions.com, Conor Dooley , Fabio Estevam , Krzysztof Kozlowski , NXP Linux Team , Peng Fan , Pengutronix Kernel Team , Rob Herring , Sascha Hauer , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v5 10/10] arm64: dts: imx8mn-bsh-smm-s2/pro: add display setup Message-ID: <20231214021734.GX270430@dragon> References: <20231207141723.108004-1-dario.binacchi@amarulasolutions.com> <20231207141723.108004-11-dario.binacchi@amarulasolutions.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231207141723.108004-11-dario.binacchi@amarulasolutions.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231213_181743_286570_B18D6E1C X-CRM114-Status: GOOD ( 21.98 ) 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-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 On Thu, Dec 07, 2023 at 03:16:39PM +0100, Dario Binacchi wrote: > From: Michael Trimarchi > > Add the display and nodes required for its operation. > > Signed-off-by: Michael Trimarchi > Signed-off-by: Dario Binacchi > > --- > > (no changes since v3) > > Changes in v3: > - Replace "synaptics,r63353" compatible with "syna,r63353", as > required by vendor-prefixes.yaml. > - Squash patch [09/11] dt-bindings: ili9805: add compatible string for Tianma TM041XDHG01 > into [07/11] dt-bindings: display: panel: Add Ilitek ili9805 panel controller. > > Changes in v2: > - Adjust the mipi_dsi node based on the latest patches merged into > the mainline in the dtsi files it includes. > - Added to the series the following patches: > - 0001 drm/bridge: Fix bridge disable logic > - 0002 drm/bridge: Fix a use case in the bridge disable logic > - 0003 samsung-dsim: enter display mode in the enable() callback > - 0004 drm: bridge: samsung-dsim: complete the CLKLANE_STOP setting > > .../freescale/imx8mn-bsh-smm-s2-common.dtsi | 1 + > .../freescale/imx8mn-bsh-smm-s2-display.dtsi | 121 ++++++++++++++++++ > 2 files changed, 122 insertions(+) > create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-display.dtsi > > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi > index 22a754d438f1..bbb07c650da9 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi > @@ -7,6 +7,7 @@ > /dts-v1/; > > #include "imx8mn.dtsi" > +#include "imx8mn-bsh-smm-s2-display.dtsi" > > / { > chosen { > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-display.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-display.dtsi > new file mode 100644 > index 000000000000..f0a924cbe548 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-display.dtsi > @@ -0,0 +1,121 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright 2021 BSH > + */ > + > +/ { > + backlight: backlight { > + compatible = "pwm-backlight"; > + pwms = <&pwm1 0 700000 0>; /* 700000 ns = 1337Hz */ > + brightness-levels = <0 100>; > + num-interpolated-steps = <100>; > + default-brightness-level = <50>; > + status = "okay"; > + }; > + > + reg_3v3_dvdd: regulator-3v3-O3 { > + compatible = "regulator-fixed"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_dvdd>; > + regulator-name = "3v3-dvdd-supply"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + gpio = <&gpio1 7 GPIO_ACTIVE_LOW>; > + }; > + > + reg_v3v3_avdd: regulator-3v3-O2 { > + compatible = "regulator-fixed"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_avdd>; > + regulator-name = "3v3-avdd-supply"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + gpio = <&gpio1 5 GPIO_ACTIVE_LOW>; > + }; > +}; > + > +&pwm1 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_bl>; We usually end property list with 'status'. > +}; > + > +&lcdif { > + status = "okay"; Ditto > + assigned-clocks = <&clk IMX8MN_VIDEO_PLL1>; > + assigned-clock-rates = <594000000>; > +}; > + > +&pgc_dispmix { > + assigned-clocks = <&clk IMX8MN_CLK_DISP_AXI>, <&clk IMX8MN_CLK_DISP_APB>; > + assigned-clock-parents = <&clk IMX8MN_SYS_PLL2_1000M>, <&clk IMX8MN_SYS_PLL1_800M>; > + assigned-clock-rates = <500000000>, <200000000>; > +}; > + > +&mipi_dsi { > + #address-cells = <1>; > + #size-cells = <0>; > + status = "okay"; Ditto > + samsung,esc-clock-frequency = <20000000>; > + samsung,pll-clock-frequency = <12000000>; > + > + panel@0 { > + compatible = "sharp,ls068b3sx02", "syna,r63353"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_panel>; > + reg = <0>; > + > + backlight = <&backlight>; > + dvdd-supply = <®_3v3_dvdd>; > + avdd-supply = <®_v3v3_avdd>; > + reset-gpios = <&gpio4 29 GPIO_ACTIVE_HIGH>; > + > + port { > + panel_in: endpoint { > + remote-endpoint = <&mipi_dsi_out>; > + }; > + }; > + > + }; > + > + ports { > + port@1 { > + reg = <1>; Have a newline between properties and child node. > + mipi_dsi_out: endpoint { > + remote-endpoint = <&panel_in>; > + }; > + }; > + }; > +}; > + > +&gpu { > + status = "okay"; > +}; > + > +&iomuxc { > + > + /* This is for both PWM and voltage regulators for display */ > + pinctrl_bl: pwm1grp { > + fsl,pins = < > + MX8MN_IOMUXC_GPIO1_IO01_PWM1_OUT 0x16 > + >; > + }; > + > + pinctrl_panel: panelgrp { > + fsl,pins = < > + MX8MN_IOMUXC_SAI3_RXC_GPIO4_IO29 0x16 /* panel reset */ > + >; > + }; > + > + pinctrl_dvdd: dvddgrp { > + fsl,pins = < > + MX8MN_IOMUXC_GPIO1_IO07_GPIO1_IO7 0x16 /* VDD 3V3_VO3 */ > + >; > + }; > + > + pinctrl_avdd: avddgrp { Can we sort the pinctrl node alphabetically? Shawn > + fsl,pins = < > + MX8MN_IOMUXC_GPIO1_IO05_GPIO1_IO5 0x16 /* VDD 3V3_VO2 */ > + >; > + }; > +}; > -- > 2.43.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel