From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx1.tq-group.com (mx1.tq-group.com [93.104.207.81]) (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 2CAF11E0E16; Tue, 7 Jan 2025 10:44:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=93.104.207.81 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736246667; cv=none; b=fFJrulGnTnmYxVIBp6T4/UcfsHE6zUdAPD+QOpZSUNB8gd9iKLPKegZZXbv950EyJtqNprbsBw5QBYtHnwKyRQvje7SSwYnp4D5QN+P1ieIIw39+ffvmGpJmFYjtc1kLA4Lp6I01fDqOjL1e3W9wmRuXx1j6kVPIw40T8iX3GNs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736246667; c=relaxed/simple; bh=9QeiEwFgSO54/qEv1gNWfGViuzafH1D2ITwxAK1wYL4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=GdNye2FusjHZ/iVg6zZkSEpwlNHpJIdzLC5TkqZLhYRxuWBpN5v2vrSLcvOtCmEO6x53fcamgvxJYpZO2ulWp5nfzWNDDwVzcgPz0wqFp2OYeSy56lAzMmd8UWo8oGn2vIdZX+tS1mHdBdgf6ymm6nJNOjLZ/TJno4bdAxyFzEI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ew.tq-group.com; spf=pass smtp.mailfrom=ew.tq-group.com; dkim=pass (2048-bit key) header.d=tq-group.com header.i=@tq-group.com header.b=WU8KthIk; dkim=fail (0-bit key) header.d=ew.tq-group.com header.i=@ew.tq-group.com header.b=g5GU7kqq reason="key not found in DNS"; arc=none smtp.client-ip=93.104.207.81 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ew.tq-group.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ew.tq-group.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=tq-group.com header.i=@tq-group.com header.b="WU8KthIk"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=ew.tq-group.com header.i=@ew.tq-group.com header.b="g5GU7kqq" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1736246663; x=1767782663; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=y+jHg0ZRQrFyhampVjN9ER8cthsZkPVauZNM9jlIGhs=; b=WU8KthIkvA/bOT0wnQ3hyn8YVOkbHGLKGxkfo5KuIAN9zSqOTSizp2fs jNY88VAkUJYuAEWQILxzfLruBF92D6TIPCyWWsQRnu5w26S9BBgjqC8Hy Im5b9gXBEdU5Qtg8tbG+GYMndeqo8/B7CsBEYH/DRrpjJTiSM6mPzi1oZ jwvmdiUnqX/m40oYU/8EMTL9Z7NhWBl2CwezQMUJYzeo56GZtgoxpVb1w NLsSbseworsOlAuuMHB/djPvL7jhsydNH4nSELh+UtQBjFV8yah/Wc3n/ dIQ88qdLhTpV1NQIre3ENcr/sE+VisSb4TQj+8nmwsYxjSKBjpkvoxbVL w==; X-CSE-ConnectionGUID: Ou6ReafJR7SkqcE1EYK60w== X-CSE-MsgGUID: tKisNOsmRJWh6tH7qXJrIQ== X-IronPort-AV: E=Sophos;i="6.12,295,1728943200"; d="scan'208";a="40890871" Received: from vmailcow01.tq-net.de ([10.150.86.48]) by mx1.tq-group.com with ESMTP; 07 Jan 2025 11:44:20 +0100 X-CheckPoint: {677D0584-E-7C22D67C-DB53DE17} X-MAIL-CPID: A3034B88B8AF551AC765663C245DBFCB_3 X-Control-Analysis: str=0001.0A682F24.677D0585.0019,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 80EDA163815; Tue, 7 Jan 2025 11:44:12 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ew.tq-group.com; s=dkim; t=1736246655; 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=y+jHg0ZRQrFyhampVjN9ER8cthsZkPVauZNM9jlIGhs=; b=g5GU7kqqoZap7uAn4UMBWajpOzprQFaxD556BOi1CgZY6rHJNyHUk9jGerT8Wb4sf8lln9 pheiYjByt1f2wMFLy9QBUC8+1IR3FSRFikQexnlwfzwcJOi0idLSAjld7IS2L8LX7vFly4 7eSj9sVnb59W1uAaYipSUotv7IFuEVeFf3A8XLF19tAmnuZZZmDmuuJ1bOKnVsiXryu3i2 delK0r0S/xJ11fn9Jc1DjCX1cpijQ8EgYsFz1W6EdMhBsS/aujrc13r7Np1ASoi+f51G/8 hyQC62sqORa4OhKIA80cMODoOPGd23KHBB03EBfhxmN+JKKveO5R5WPngkyq9A== From: Alexander Stein To: Fabio Estevam Cc: Inki Dae , Jagan Teki , Marek Szyprowski , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 2/2] dt-bindings: samsung, mipi-dsim: Add imx7d specific compatible Date: Tue, 07 Jan 2025 11:44:10 +0100 Message-ID: <3358169.aeNJFYEL58@steina-w> Organization: TQ-Systems GmbH In-Reply-To: References: <20250107094943.518474-1-alexander.stein@ew.tq-group.com> <20250107094943.518474-3-alexander.stein@ew.tq-group.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" X-Last-TLS-Session-Version: TLSv1.3 Hi Fabio, Am Dienstag, 7. Januar 2025, 11:35:48 CET schrieb Fabio Estevam: > Hi Alexander, >=20 > On Tue, Jan 7, 2025 at 6:50=E2=80=AFAM Alexander Stein > wrote: > > > > This add a imx7(d) specific compatible which is compatible to imx8mm. > > This silences the dtbs_check warning: > > arch/arm/boot/dts/nxp/imx/imx7s-mba7.dtb: dsi@30760000: compatible: 'on= eOf' conditional failed, one must be fixed: > > ['fsl,imx7d-mipi-dsim', 'fsl,imx8mm-mipi-dsim'] is too long >=20 > The warning here is about using fsl,imx8mm-mipi-dsim. I'm not sure what you exactly mean. The warning isn't about using fsl,imx8m= m-mipi-dsim, instead fsl,imx7d-mipi-dsim does not match any compatibles listed in the schema. > > Signed-off-by: Alexander Stein > > --- > > .../devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/display/bridge/samsung,m= ipi-dsim.yaml b/Documentation/devicetree/bindings/display/bridge/samsung,mi= pi-dsim.yaml > > index 4ed7a799ba26b..e43fec5609417 100644 > > --- a/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsi= m.yaml > > +++ b/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsi= m.yaml > > @@ -27,7 +27,9 @@ properties: > > - fsl,imx8mm-mipi-dsim > > - fsl,imx8mp-mipi-dsim > > - items: > > - - const: fsl,imx8mn-mipi-dsim > > + - enum: > > + - fsl,imx7d-mipi-dsim > > + - fsl,imx8mn-mipi-dsim >=20 > But you added a fallback to fsl,imx8mn-mipi-dsim, so this does not look r= ight. Admittedly I'm having a hard time on DT schemas at times, but this change a= llows imx7d OR imx8mn to be used together with imx8mm. > I have already sent the following fix to address this issue: >=20 > https://lore.kernel.org/all/20241217131431.1464983-1-festevam@gmail.com/T/ This patch essentially does the same but add a new entry to the oneOf. If you add fsl,imx8mn-mipi-dsim to that enum as we end up with the same solution. Best regards, Alexander =2D-=20 TQ-Systems GmbH | M=C3=BChlstra=C3=9Fe 2, Gut Delling | 82229 Seefeld, Germ= any Amtsgericht M=C3=BCnchen, HRB 105018 Gesch=C3=A4ftsf=C3=BChrer: Detlef Schneider, R=C3=BCdiger Stahl, Stefan Sch= neider http://www.tq-group.com/