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 89B8EC8302A for ; Thu, 29 Aug 2024 10:03:15 +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=xthEvjklyi0tldf0RzO+cBlxZDiIDe0pmHoD56I97AY=; b=v3wwf46093IUURQyTO2GcGzg77 MZr7pD8tYGuTHdZSKEOKMEIwrl962OTiYKj38DoOJO4/VOHxd1hSFauvLCHmt9XCPbTt12Ta8RJLg SwX7aGiBHISYGa5Nu92Bxn2reZgRBE//dvnJNe+hPDncQAKBE8I0n3PtIw2cBwr/2Q7AOiKOe9BL1 6+rlXn7DzJPNG0SkQjFBPbDkanoI6BmrJWdlmI9E/RJh6fKYfVgC2WJOUciB25PFHNxq4bTrDDC4I XqGdG7JPmbbQEVhjpTKTpKxdPliyyW46AD6Zc+H12skADmPCnfNItFuyGzIh0oohmeGhgdAs4MmZD vDfRS/6w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sjbzZ-00000001UF1-1zFV; Thu, 29 Aug 2024 10:03:01 +0000 Received: from phobos.denx.de ([2a01:238:438b:c500:173d:9f52:ddab:ee01]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sjbyh-00000001Tzp-194o for linux-arm-kernel@lists.infradead.org; Thu, 29 Aug 2024 10:02:09 +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 B5D1C87DC7; Thu, 29 Aug 2024 12:02:02 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1724925723; bh=xthEvjklyi0tldf0RzO+cBlxZDiIDe0pmHoD56I97AY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=elVgCjnUjFPI4y42KRlcYzWDBiojtmbyFpMeHoHSMPrAw6iFw/qn8MuTPdPXJExrd KyNXT8wU2YaDGSUUuHj6OAFnqQ7ACyRxkQP4sVHy8U2HiTe98XnnGsj2yzKT8da/Yk 9Hp8uCl3qfDKgKbpMRsN1IAa6739MMXWUzR6AvApz1T7izpZVvFN8l/r+OsMPv/zOY JxgdaeCt1xWPutL3hW11YJDG5BZ9vcbzklbJA1lTzukCiU64cB+Ql0PxW+OU5w9nNH /jC0SaGxtueGfEaYeKbf6WTYj5+M/918R6DvWSEKo2YNIgNcBqsNuP4SEPzoYtUVvE KoPFqb1lSAfHA== Date: Thu, 29 Aug 2024 12:02:01 +0200 From: Lukasz Majewski To: Rob Herring Cc: Pengutronix Kernel Team , Conor Dooley , imx@lists.linux.dev, Shawn Guo , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Fabio Estevam , Sascha Hauer , devicetree@vger.kernel.org, Krzysztof Kozlowski Subject: Re: [PATCH] dts: nxp: mxs: Add descriptions for imx287 based btt3-[012] devices Message-ID: <20240829120201.1c6b97a8@wsk> In-Reply-To: References: <20240828112139.2665814-1-lukma@denx.de> <172484936571.3462707.10762753590627002293.robh@kernel.org> <20240828160553.53e164c3@wsk> 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_/ggFtdIXYD1S_KQtz.lDs_tT"; 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-20240829_030207_629564_9220F109 X-CRM114-Status: GOOD ( 46.14 ) 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_/ggFtdIXYD1S_KQtz.lDs_tT Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi Rob, > On Wed, Aug 28, 2024 at 9:12=E2=80=AFAM Lukasz Majewski w= rote: > > > > Hi Rob, > > =20 > > > On Wed, 28 Aug 2024 13:21:39 +0200, Lukasz Majewski wrote: =20 > > > > The btt3 device' HW revisions from 0 to 2 use imx287 SoC and > > > > are to some extend similar to already upstreamed XEA devices, > > > > hence are using common imx28-lwe.dtsi file. > > > > > > > > New, btt3.dtsi has been added to embrace common DTS properties > > > > for different HW revisions for this device. > > > > > > > > As a result - changes introduced in btt3-[012].dts are minimal. > > > > > > > > Signed-off-by: Lukasz Majewski > > > > --- > > > > arch/arm/boot/dts/nxp/mxs/Makefile | 3 + > > > > arch/arm/boot/dts/nxp/mxs/btt3-0.dts | 12 + > > > > arch/arm/boot/dts/nxp/mxs/btt3-1.dts | 8 + > > > > arch/arm/boot/dts/nxp/mxs/btt3-2.dts | 12 + > > > > arch/arm/boot/dts/nxp/mxs/btt3.dtsi | 320 > > > > +++++++++++++++++++++++++++ 5 files changed, 355 insertions(+) > > > > create mode 100644 arch/arm/boot/dts/nxp/mxs/btt3-0.dts > > > > create mode 100644 arch/arm/boot/dts/nxp/mxs/btt3-1.dts > > > > create mode 100644 arch/arm/boot/dts/nxp/mxs/btt3-2.dts > > > > create mode 100644 arch/arm/boot/dts/nxp/mxs/btt3.dtsi > > > > =20 > > > > > > > > > My bot found new DTB warnings on the .dts files added or changed > > > in this series. > > > > > > Some warnings may be from an existing SoC .dtsi. Or perhaps the > > > warnings are fixed by another series. Ultimately, it is up to the > > > platform maintainer whether these warnings are acceptable or not. > > > No need to reply unless the platform maintainer has comments. > > > =20 > > > > There a few things to clarify after: > > CROSS_COMPILE=3D"arm-none-linux-gnueabihf-" > > ARCH=3Darm make -j4 CHECK_DTBS=3Dy nxp/mxs/btt3-0.dtb > > > > 1. mxs-spi.yaml vs mxs-mmc.yaml -> both refer to the same set of HW > > - as those IP blocks can work as both mmc and SPI. =20 >=20 > Sounds like there should be 1 binding then. I also think so - one binding with two compatibles. >=20 > > > > 2. For the sound codec: > > codec@1a: compatible: ['wlf,wm8940', 'wlf,wm8974'] is too long > > > > This is a bit problematic as well - as WM8974 doesn't have ANY ID > > register, so it needs to be probed manually and decided which module > > with sound codec needs to be inserted. The WM8940 is the pin > > compatible replacement. =20 >=20 > So if the OS has 2 drivers for these 2 devices, which one does it > pick? (For now) user space assess if the correct codec is inserted as a module. > The answer for Linux is it is undefined. The expectation is if > there are multiple options, the firmware figures out which one is > present and adjusts the DT. So then, only the newest one would be used, and then I would move detection to u-boot and use overlays. >=20 > > 3. The simple-bus.yaml seems to be missing in the Linux kernel tree > > - problem is that imx28 has 'apbx' bus. =20 >=20 > simple-bus.yaml lives in dtschema repo. >=20 Yes, I've just noticed that. > Node names should be generic, not named for the implementation/type. > The binding is fairly flexible in allowing ".*-bus". Fabio has just prepared and shared proper patch. >=20 > > 4. The fsl,lcdif.yaml doesn't allow the situation when one specify > > by hand (in DTS) display parameters (as different ones can be used > > by the manufacturing site). =20 >=20 > Shrug. At the end of the day, the binding and dts have to match. One > (or both) has to be adjusted. Ok. >=20 > Rob 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_/ggFtdIXYD1S_KQtz.lDs_tT Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEgAyFJ+N6uu6+XupJAR8vZIA0zr0FAmbQRxkACgkQAR8vZIA0 zr0QzwgAwJdeGp91k9muym0KIPV9AcUJwVRNO8xcaTnGoTlhtepdDSVIOXuxGvqu 8rdTTMTqkxcZrdRpoe8F8cMfhAJLie+x9/E+OP3M/b02AdpYSZQEQ3IUYkckt5ra 1xFA8dJXvj/j+QP8v+k6IUBXmAdawntgEiGDlJeb7v9fUf9DMcb2bDDBPJPKLA+b SuyAAToBp0yKMkXvS33XO+uTfyGbSf5c06/cormBdhbcJZg+dBjCFdcPpDS+J+RI BiAiiwv1HWms2QKfE24mqEvi5Tubs6LRX4RzoMnCH1qEr+OJ+hud6/SwZAh+sTH3 rNtLOdm94OAKngc92Z6xg5G5FdkK5w== =zHNm -----END PGP SIGNATURE----- --Sig_/ggFtdIXYD1S_KQtz.lDs_tT--