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 472EDCDB474 for ; Sun, 22 Oct 2023 17:39:58 +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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=em7bqUVA5B61aFw3KotbjFZFSt93VMOO7/TXUUinxtU=; b=Y5IBSXGjHDk+79lIv4uPMhekrp fAvHZ8QCY6OGjYQXprU8JKy1j4/4otLbd4kR7mnj5VRB6sDqe0IAmWzEH7taqzK5e5AhYLsTVmMEP ePdk+ULHdWQvU4dGStu5/dvrt1+3o1xIh/37ScqzP8Chomj4t5femj+h7pW47L44AWr4qx6WdLNSP dN1iQlH7qX6ubUOJeRCdqypTbG3iWNRJSLJhMtV2O3C/y1XE0pOttRzgGfgF0H/fJERgnX1EQO320 cz4atokOz04iKtfASBqRqlg6VtHTGwG2pQqOk3tBDZSUgnblvvsi2eMFk2xqKntAFZN3BLZB1VP+/ VEsiiFMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qucQE-005lSS-1T; Sun, 22 Oct 2023 17:39:30 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qucQB-005lRf-37 for linux-arm-kernel@lists.infradead.org; Sun, 22 Oct 2023 17:39:29 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id 2B23FB81214; Sun, 22 Oct 2023 17:39:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B49A0C433C8; Sun, 22 Oct 2023 17:39:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697996358; bh=V2tHPMR1TOkzDOcp6KPYSszP9A29fq1AuLXpL5Br5NM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=K6/4L4H7VtSF3BDW3Lq6DK9X1E8D3f58bD2SDrSbnNAGYVFktdLMflOBllDGlYWoC GIiHZIuL8+AfxBwjT+JTZAcPcjA7TfMFEXeaYSIy6H3i1EL3MuuGrmPLGMaGXtxcHn L2iTgbKxkiU3ufeX30VLqh1pcCWhMw6pOfvLwAQVPrLbiKRtXnh2YKFqBPsRoQrsIn YSm2UF9eJISTXVDKPKU1q5Hy1i52Ze9JT3TX3KAHhujFayx9ME/oYAVZG+q38n0Vvl lx6oMuZ6+zcugE2dq0PNpBkH6Pdv/ODEzfnZR0bMMrkk9V2ID3hKA6Z3Jxy3DTSf23 TR7830cTppCgA== Date: Sun, 22 Oct 2023 18:39:12 +0100 From: Conor Dooley To: Alexander Stein Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Fabio Estevam , Abel Vesa , Peng Fan , Michael Turquette , Stephen Boyd , Ulf Hansson , Marek Vasut , Pengutronix Kernel Team , NXP Linux Team , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux@ew.tq-group.com, linux-clk@vger.kernel.org, linux-pm@vger.kernel.org Subject: Re: [PATCH 2/5] dt-bindings: soc: imx93-media-blk-ctrl: Add LDB subnode into schema and example Message-ID: <20231022-helper-dating-a0f65a8f6f72@spud> References: <20231020130019.665853-1-alexander.stein@ew.tq-group.com> <20231020130019.665853-3-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 In-Reply-To: <20231020130019.665853-3-alexander.stein@ew.tq-group.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231022_103928_329174_F15A28A4 X-CRM114-Status: GOOD ( 18.11 ) 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: multipart/mixed; boundary="===============0180890596701757460==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============0180890596701757460== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="OjkkNCobwn9OU9NU" Content-Disposition: inline --OjkkNCobwn9OU9NU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Yo, On Fri, Oct 20, 2023 at 03:00:15PM +0200, Alexander Stein wrote: > Document the LDB bridge subnode and add the subnode into the example. > For the subnode to work, the block control must scan its subnodes and > bind drivers to them, do not misuse either simple-bus or simple-mfd > here. >=20 > Signed-off-by: Alexander Stein > --- > .../soc/imx/fsl,imx93-media-blk-ctrl.yaml | 44 +++++++++++++++++++ > 1 file changed, 44 insertions(+) >=20 > diff --git a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-bl= k-ctrl.yaml b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk= -ctrl.yaml > index b3554e7f9e76..5ba66dfb0e05 100644 > --- a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.= yaml > +++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.= yaml > @@ -24,6 +24,12 @@ properties: > reg: > maxItems: 1 > =20 > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 1 > + > '#power-domain-cells': > const: 1 > =20 > @@ -46,9 +52,16 @@ properties: > - const: csi > - const: dsi > =20 > + bridge@20: > + type: object > + $ref: /schemas/display/bridge/fsl,ldb.yaml# > + unevaluatedProperties: false > + > required: > - compatible > - reg > + - '#address-cells' > + - '#size-cells' It seems to make little sense to me that these would become required when the bridge is optional. Is it valid to have one of these media-blk-ctrls without the ldb subnode? > - power-domains > - clocks > - clock-names > @@ -77,4 +90,35 @@ examples: > clock-names =3D "apb", "axi", "nic", "disp", "cam", > "pxp", "lcdif", "isi", "csi", "dsi"; > #power-domain-cells =3D <1>; > + #address-cells =3D <1>; > + #size-cells =3D <1>; > + > + bridge@20 { > + compatible =3D "fsl,imx93-ldb"; > + reg =3D <0x20 0x4>, <0x24 0x4>; > + reg-names =3D "ldb", "lvds"; > + clocks =3D <&clk IMX93_CLK_LVDS_GATE>; > + clock-names =3D "ldb"; > + > + ports { > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + > + port@0 { > + reg =3D <0>; > + > + ldb_from_lcdif2: endpoint { > + remote-endpoint =3D <&lcdif2_to_ldb>; > + }; > + }; > + > + port@1 { > + reg =3D <1>; > + > + ldb_lvds: endpoint { > + remote-endpoint =3D <&ldb_to_panel>; > + }; > + }; > + }; > + }; > }; > --=20 > 2.34.1 >=20 --OjkkNCobwn9OU9NU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZTVeQAAKCRB4tDGHoIJi 0l9qAP0VlEaAluBRXsbW8N9Eandg3TOavmfUjpyH+1wBFkQQOwEA0yPLz0di2m0x pDXwKnhPdmOAymwdxMVUVzBgnfeRrgs= =7x7o -----END PGP SIGNATURE----- --OjkkNCobwn9OU9NU-- --===============0180890596701757460== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0180890596701757460==--