From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from www537.your-server.de (www537.your-server.de [188.40.3.216]) (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 4E4323B7B79; Thu, 13 Aug 2026 09:26:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=188.40.3.216 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786613213; cv=none; b=PrO56EjQViz9kZxswrEh3ge8uKnteBm0EAgsn/uyF1h8PBhb7FvhHE9OaUwGJTaKD7p/NISzxfSK5DciWT1kOVBtcI6IuBh0DZx45QXPZXKCeKkxwJPH6906e76CNBBWW1SIhRQyYydFVRaQ+BcB3gX29FYjxdqR2eX2Zz+5hNY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786613213; c=relaxed/simple; bh=Z6v38tsb1CcOWA/5O8PCVvA+QbpJ3yaT73L5yIVeGoI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=qPP7ZKozt4APAoIGdZS8bAzinwvmDjPYSxlBeZCB801QKUT6akSOultWMBpHjJCFP3yFbXdplLvzozVmM1uM/pYvR7I8Cn8AtYsi5xXIzOckQx3e33GeRK1tOsG6vR3ZITx9U/nlUC8ho1L0Wja0+vXG8EI9j1PvuI3d4mvEWc4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ew.tq-group.com; spf=pass smtp.mailfrom=ew.tq-group.com; dkim=pass (2048-bit key) header.d=ew.tq-group.com header.i=@ew.tq-group.com header.b=nKP72OPT; arc=none smtp.client-ip=188.40.3.216 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine 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=ew.tq-group.com header.i=@ew.tq-group.com header.b="nKP72OPT" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=ew.tq-group.com; s=default2602; h=Content-Type:Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID; bh=IPz+pRNv6M2Bl3l0XU+3yKO/cuTqG+5OunpOwazZoio=; b=nKP72OPTQMRqcB57kmWSrGWTPj esmryEFUibscPV2WL4LQE/LUfZ7EGKiutgMKBrrTAw5Pq3Uc8DXXtwU79wtvhDpTm1AA/nnzyw/RE 1qNnrFopYMghltvOCStmG6zSNtEts0rTgQXJtWJICX88j4x0JJt9Cao1vKCerX8ilg7PVrjDlqs6B 3Y5xoCiRLZ75fZWuKzkUE0an5ELxFy1MgnIYvk1CwTA9oWDP3JbCFVZ8VzIbsbS/Fr8cBq0enY7QK i904iLYNM0Gv0aQ7CXWk6fUBbSX8pjk5tiIhU1Q5ajJ2wPMXxXJR3dd9SJg805E52gv57qEfwGwLX Us7ErV7g==; Received: from sslproxy01.your-server.de ([78.46.139.224]) by www537.your-server.de with esmtpsa (TLS1.3) tls TLS_AES_256_GCM_SHA384 (Exim 4.96.2) (envelope-from ) id 1wuROI-000EhZ-0C; Thu, 13 Aug 2026 11:06:22 +0200 Received: from localhost ([127.0.0.1]) by sslproxy01.your-server.de with esmtpsa (TLS1.3) tls TLS_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wuROH-000CPn-1b; Thu, 13 Aug 2026 11:06:21 +0200 From: Alexander Stein To: Francesco Dolcini , Krzysztof Kozlowski Cc: Marek Vasut , Stefan Agner , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Francesco Dolcini , 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 v3 1/4] dt-bindings: lcdif: Add endpoint bus-width property Date: Thu, 13 Aug 2026 11:06:20 +0200 Message-ID: <4870648.tdWV9SEqCh@steina-w> Organization: TQ-Systems GmbH In-Reply-To: References: <20260812132954.148571-1-francesco@dolcini.it> <7661119.LvFx2qVVIh@steina-w> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" X-Virus-Scanned: Clear (ClamAV 1.4.3/28091/Thu Aug 13 08:25:54 2026) Am Donnerstag, 13. August 2026, 11:03:06 CEST schrieb Krzysztof Kozlowski: > On 13/08/2026 10:56, Alexander Stein wrote: > > Am Donnerstag, 13. August 2026, 10:34:02 CEST schrieb Francesco Dolcini: > >> On Thu, Aug 13, 2026 at 10:10:43AM +0200, Krzysztof Kozlowski wrote: > >>> On Wed, Aug 12, 2026 at 03:29:49PM +0200, Francesco Dolcini wrote: > >>>> From: Francesco Dolcini > >>>> > >>>> The DPI output of the LCD interface can be configured with a bus wid= th > >>>> of 16, 18, or 24 bits. Introduce a bus-width property to the output > >>>> endpoint, specifying the number of parallel RGB bits connected. > >>>> > >>>> Signed-off-by: Francesco Dolcini > >>>> --- > >>>> v3: use video-interfaces.yaml > >>>> v2: use the common bus-width property instead of the legacy interfac= e-pix-fmt > >>>> --- > >>>> Documentation/devicetree/bindings/display/fsl,lcdif.yaml | 9 ++++++= +++ > >>>> 1 file changed, 9 insertions(+) > >>>> > >>>> diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yam= l b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >>>> index 2dd0411ec651..1f3daf002707 100644 > >>>> --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >>>> +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >>>> @@ -71,6 +71,15 @@ properties: > >>>> $ref: /schemas/graph.yaml#/properties/port > >>>> description: The LCDIF output port > >>>> =20 > >>>> + properties: > >>>> + endpoint: > >>>> + $ref: /schemas/media/video-interfaces.yaml > >>> > >>> This won't work, you need to test your DTS. The port schema does not > >>> allow video-interfaces properties. You need port-base, like in every > >>> other case. > >> > >> Thanks for the review Krzysztof > >> > >> Given my testing was not ok, and I am not sure what I did wrong exactly > >> on my tests, is this the change I should do? >=20 > Yes >=20 > >> > >> diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml = b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >> index 1f3daf002707..89cea0c14cf1 100644 > >> --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >> +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > >> @@ -68,7 +68,8 @@ properties: > >> maxItems: 1 > >> =20 > >> port: > >> - $ref: /schemas/graph.yaml#/properties/port > >> + $ref: /schemas/graph.yaml#/$defs/port-base > >> + unevaluatedProperties: false > >> description: The LCDIF output port > >=20 > > With that change: > > Tested-by: Alexander Stein >=20 > Please do not. Build testing is not testing (in the meaning of > "Tested-by" tag), otherwise where is Rob's tested-by tag for all the > patches he build tested? Well, I tested this on actual hardware, see patch 2. But then this is irrelevant for for dt-bindings. So let's go with: Acked-by: Alexander Stein Thanks and best regards, Alexander =2D-=20 TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/