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 A51EDE82CAC for ; Wed, 27 Sep 2023 15:36:38 +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=mkFXMSxaYGMsfqU9uSUvQtI2IQpcewDzJMyGl5ZrWq0=; b=DJfjo1z7+jaQyZ7DWRi7YTudes 53Oz0o+yjFfIrcl5kx1QO8U01Yi6uyzbZFnk7l8gRQJO7Nd8vgQS6UtsU0/LZwe7Qa4H5yqOH/GIP Jq8tH4WuXgVJpECSE9HdXTK5h2/V2MEgvaBqRNmg9QV0NI1SOVB/2ui1NC2L73UR/m0Bj66NP1uBM S383ucbuJwjabQDYd+99LScKuWap8KjPCgd4ZHJ5LUWpkJsG9HVZBFKBi+vUVxBnrqY/1rlcVIGvK PSnoCxP5++5y0B5/FLwe6gSuSVGHAfvpiLntwOZyZp7o39kfdbjHv4kZVYwTlzT+KJn8ufDoId1vs 1QGa4kqw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qlWa0-001G9z-25; Wed, 27 Sep 2023 15:36:00 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qlWZx-001G9I-2z for linux-arm-kernel@lists.infradead.org; Wed, 27 Sep 2023 15:35:59 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 68A7F6190D; Wed, 27 Sep 2023 15:35:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 22173C433C8; Wed, 27 Sep 2023 15:35:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695828955; bh=Vbcpyf8NR6obj9hRrWqEruzSZGutGz1yExgH+iLFr+Q=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=A9hxyKZqoQ2qZv3jGkVqTnIuauZkwqAo2+Age9iyTVmJ9qkjfNF2zLfHQdX6ZmKvq /MCMSKw1PdhwHoijPT73JL2aDQ0FRhCZBxtDXzgq1aIHrolCMSGN8PV1xGdvC9Yo/N SaKBJjDjwaY0350cc14w8W6kRTrIkWAlxuCjbMPtlk7Sdxl7Ty2d9QwCz4LEMk1iZg UG7S3AK3FzWGN3aNMkPdcjQV2IhwZ0MMXFhV7Wml8GdgtvJLTTMkFDQ5wdtO77y6go p41vvBRfxmJRvcf5Tl4fR2ff2okcVbOhz6cLpmmfAFgSGwB2nPMf9vqUlwr3ALHsQ/ 5b3WlrtnhXRAQ== Date: Wed, 27 Sep 2023 16:35:49 +0100 From: Conor Dooley To: Rob Herring Cc: Bjorn Helgaas , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Krzysztof Kozlowski , Conor Dooley , Ray Jui , Scott Branden , Broadcom internal kernel review list , Ray Jui , Scott Branden , linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] dt-bindings: PCI: brcm,iproc-pcie: Fix 'msi' child node schema Message-ID: <20230927-utmost-copied-6982126a7f00@spud> References: <20230926155351.31117-1-robh@kernel.org> <20230926155613.33904-3-robh@kernel.org> MIME-Version: 1.0 In-Reply-To: <20230926155613.33904-3-robh@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230927_083558_050783_7CB1B3FE X-CRM114-Status: GOOD ( 23.99 ) 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="===============7802231350463099557==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============7802231350463099557== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1BjGV0efq9T3tTST" Content-Disposition: inline --1BjGV0efq9T3tTST Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Sep 26, 2023 at 10:56:09AM -0500, Rob Herring wrote: > The 'msi' child node schema is missing constraints on additional properti= es. > It turns out it is incomplete and properties for it are documented in the > parent node by mistake. Move the reference to msi-controller.yaml and > the custom properties to the 'msi' node. Adding 'unevaluatedProperties' > ensures all the properties in the 'msi' node are documented. >=20 > With the schema corrected, a minimal interrupt controller node is needed > to properly decode the interrupt properties since the example has > multiple interrupt parents. I suppose this is an ABI break, but the patch just makes the binding match the example and intent. Feels like of all the patches doing the unevaluatedProperty additions, this one is the most deserving of a fixes tag, since the original binding just seems to be completely wrong? Otherwise, Acked-by: Conor Dooley Thanks, Conor. >=20 > Signed-off-by: Rob Herring > --- > .../bindings/pci/brcm,iproc-pcie.yaml | 24 ++++++++++++------- > 1 file changed, 16 insertions(+), 8 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/pci/brcm,iproc-pcie.yaml b= /Documentation/devicetree/bindings/pci/brcm,iproc-pcie.yaml > index 6730d68fedc7..0e07ab61a48d 100644 > --- a/Documentation/devicetree/bindings/pci/brcm,iproc-pcie.yaml > +++ b/Documentation/devicetree/bindings/pci/brcm,iproc-pcie.yaml > @@ -12,7 +12,6 @@ maintainers: > =20 > allOf: > - $ref: /schemas/pci/pci-bus.yaml# > - - $ref: /schemas/interrupt-controller/msi-controller.yaml# > =20 > properties: > compatible: > @@ -63,20 +62,24 @@ properties: > =20 > msi: > type: object > + $ref: /schemas/interrupt-controller/msi-controller.yaml# > + unevaluatedProperties: false > + > properties: > compatible: > items: > - const: brcm,iproc-msi > =20 > - msi-parent: true > + interrupts: > + maxItems: 4 > =20 > - msi-controller: true > + brcm,pcie-msi-inten: > + type: boolean > + description: > + Needs to be present for some older iProc platforms that requir= e the > + interrupt enable registers to be set explicitly to enable MSI > =20 > - brcm,pcie-msi-inten: > - type: boolean > - description: > > - Needs to be present for some older iProc platforms that require the > - interrupt enable registers to be set explicitly to enable MSI > + msi-parent: true > =20 > dependencies: > brcm,pcie-ob-axi-offset: ["brcm,pcie-ob"] > @@ -104,6 +107,11 @@ examples: > - | > #include > =20 > + gic: interrupt-controller { > + interrupt-controller; > + #interrupt-cells =3D <3>; > + }; > + > pcie@18012000 { > compatible =3D "brcm,iproc-pcie"; > reg =3D <0x18012000 0x1000>; > --=20 > 2.40.1 >=20 --1BjGV0efq9T3tTST Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZRRL1QAKCRB4tDGHoIJi 0vG2AQDluSVffOKOHbIEMmIpNz0g1yHoArvULfqpNTho1ylwlQEA/EmGD6/g/1ZF dKNKjfPaJCXwB3Jg7O7xQ9vpU0G2xA0= =4PdT -----END PGP SIGNATURE----- --1BjGV0efq9T3tTST-- --===============7802231350463099557== 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 --===============7802231350463099557==--