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 52518CF9C5B for ; Mon, 23 Sep 2024 09:48:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+tppFS2oLuaEwqRTzj9fnAMiRlSoieCQHR+Y3pC9ZME=; b=JJP07akLsbEtVCdcCDxI5G0P3c Ku8ZdaIkF+0R7LL8NvIUBMPNTf7byTFD93bJuBAYZzso1VDq1pY9UgaMXZGrHMvxordQyUWJ0QwSb tYhOEHJJ5vWINqFUVCe3XVs3iep1hJuLdmTzS7h7AMwxkQqZWkmTmUo5fU8BXQRkajjje3Q7WXaiD vLvOagyoHOibrEoRYvKPQmut3YV7Qnn1VrdJ7vW9YMD7sGXmzNp/BYdLotZPeXlIKCnBwwO8EerN2 baCwET/rah5+a5aeJTTDiaN5ANQ3D2XyzvQZnmLAZPjdetrgodNjRBI5pfCpba1T8LC0wjBZXlG5r Se88VVRA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1ssfgD-0000000GsHx-01hi; Mon, 23 Sep 2024 09:48:29 +0000 Received: from phobos.denx.de ([85.214.62.61]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1ssff4-0000000Gs7n-1Cj5 for linux-arm-kernel@lists.infradead.org; Mon, 23 Sep 2024 09:47:20 +0000 Received: from wsk (85-222-111-42.dynamic.chello.pl [85.222.111.42]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: lukma@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id D98378862E; Mon, 23 Sep 2024 11:47:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1727084833; bh=+tppFS2oLuaEwqRTzj9fnAMiRlSoieCQHR+Y3pC9ZME=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=bepTSk1dhA43opvHvQDdQq2JunFcLNitxrJMiNQIJvlZqAKqIvMToMQEyUbLBbV4J Nf//FDNv7PWTI1xjLkoCqdjTckEYHWmYHNzmpW8F0YauKur/OuQYbAVt51NuRaKedT Rg6mKqgZLsiJjxttUHrPHzjRih8tMSJ41vG6f21j78+coggQqW/1clpWyQrcLDFnCk yAaMgOMjuzyZrIbLZfK/B603LeSpa1ku9vMhpf2UQ/g/h3GeVyJhdExR9J8kz7Y/lz 6sRzIsnZZnKBVuePzUt9O/ZWSD0u0QleaDIMAnJisVyoktRw7yG3E/+POJb0kqJax8 Ss0YAjLW8Bivw== Date: Mon, 23 Sep 2024 11:47:11 +0200 From: Lukasz Majewski To: Fabio Estevam , Rob Herring Cc: Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 2/2] dts: nxp: mxs: Add descriptions for imx287 based btt3-[012] devices Message-ID: <20240923114711.1c294b6b@wsk> In-Reply-To: References: <20240912124825.2528984-1-lukma@denx.de> <20240912124825.2528984-2-lukma@denx.de> Organization: denx.de X-Mailer: Claws Mail 3.19.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/Ilu4b09qMfQa=FASJqP83n="; protocol="application/pgp-signature"; micalg=pgp-sha512 X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240923_024718_659603_58F5F8AA X-CRM114-Status: GOOD ( 18.27 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --Sig_/Ilu4b09qMfQa=FASJqP83n= Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi Fabio, > Hi Lukasz, >=20 > On Thu, Sep 12, 2024 at 9:48=E2=80=AFAM Lukasz Majewski w= rote: >=20 > > +&lcdif { > > + pinctrl-names =3D "default"; > > + pinctrl-0 =3D <&lcdif_24bit_pins_a>, <&lcdif_sync_pins_bttc>, > > + <&lcdif_reset_pins_bttc>; > > + lcd-supply =3D <®_3v3>; > > + display =3D <&display0>; > > + status =3D "okay"; > > + display0: display0 { > > + bits-per-pixel =3D <32>; > > + bus-width =3D <24>; > > + display-timings { > > + native-mode =3D <&timing0>; > > + timing0: timing0 { > > + clock-frequency =3D <6500000>; > > + hactive =3D <320>; > > + vactive =3D <240>; > > + hfront-porch =3D <20>; > > + hback-porch =3D <38>; > > + hsync-len =3D <30>; > > + vfront-porch =3D <4>; > > + vback-porch =3D <14>; > > + vsync-len =3D <4>; > > + hsync-active =3D <0>; > > + vsync-active =3D <0>; > > + de-active =3D <0>; > > + pixelclk-active =3D <1>; =20 >=20 > According to fsl,lcdif.yaml, a remote-endpoint to the display is > needed. >=20 > See imx28-evk.dts for an example. This file has the: panel { compatible =3D "sii,43wvf1g"; Whereas in those devices (i.e. btt3) - I don't know the names of the displays - manufacturer buys them according to the timing properties. Hence the question - how shall I proceed? IMHO the most straightforward way is to modify fsl,lcdif.yaml to not require "port" and "remote-endpoint" and instead add support for "display-timings" and "timingX" Especially that Documentation/devicetree/bindings/display/panel/display-timings.yaml are already defined and used by many imx boards (from imx25 to imx6q) - git grep -n "display-timings" Even the imx28-m28evk.dts is using the "display-timings" and not "remote-endpoint" approach. Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de --Sig_/Ilu4b09qMfQa=FASJqP83n= Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEgAyFJ+N6uu6+XupJAR8vZIA0zr0FAmbxOR8ACgkQAR8vZIA0 zr0RMAf+P+IBxVFol0awhUrlSAa6pvq0jmSf8oazCU+VunQD0ICMJEQBpxVd4jTN Iz7ov8d6CQ1A2e5uJnG+O3va0aZ8mqmwQ6B1wosg+GtUJtQVpOkadJRx7uYs/JQf L7ymVrgGQ7vC2xof5qG2mFKHK5AbjNAZNsThXUc8Nfo2wniCcene2bOMzykUfisN cav071REJJVVkZ39RAhLy5mv9R61VQaLMgBPWQ+0Ps7xEr50w+er5qspcW+FQBMP CVcfZ3VwQ8WYUGuyHzialbpTUcmJAM3MVb+9SdSxGYohjgyHsJpNhNcuHMBu4fET 1PyLx4J4PqB07xcRnKKymE7dsL5ZEg== =BLj/ -----END PGP SIGNATURE----- --Sig_/Ilu4b09qMfQa=FASJqP83n=--