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 39BACC52D7C for ; Fri, 9 Aug 2024 18:11:01 +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:In-Reply-To:Content-Type: 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=hWX41+3ky5fSuzJkKiEBC0ZQsdiOk6Fi7DGWICXoitw=; b=LvYU2XePl4QBMQ0ilwbYSwXcji Ay2JVmLlwyv0aZJoLwNLGUpp8bYVusVAb8pEpmPan2C2Y9udLY+C4B9sm3qoFpW0RWojcFYNiMEzA p1+X/y2kAtHxgEcNsTg71aIZl7ZU/spSxky0eBrn6KP5wVokheUpc001OHy3p4iuhBnYOaXo/gd3B G2oH0/1QLbuRxxAM48oNWyxjFrzZEPFnW8NZVgYFv755JkLPIM7SqVHU14LYJsD8soFvmKB6fTof4 IbhYzEBQ7OJpBMNAGooxWgi8e01OFImnGQwHzN+4Bhn5AYkQYJgl3mJyavtdV3FFsqvTMl5vZMHWv +MDBSDWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1scU4e-0000000CALf-2c5Q; Fri, 09 Aug 2024 18:10:48 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1scU42-0000000CAA8-2LyZ for linux-arm-kernel@lists.infradead.org; Fri, 09 Aug 2024 18:10:12 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id F1D2461757; Fri, 9 Aug 2024 18:10:09 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4122AC32782; Fri, 9 Aug 2024 18:10:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1723227009; bh=ps6unyElBa8V3B369PXGF09PFDZoznk+uwxeku8zQRg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Gy8bqaINlNhZfAv6ihJGwo7i1eC0/Pum9J0j8PcRxdxX7NU17seqO6TBsKSZn4uWs UCzGjpOlvDqXU/tSp3q2ildCLBO8WIHigzX0o5KDMf2eJWk6S/thINKdEfK/JttYW7 VKx7e7gh/YHi25D/+CEDGXD5lvIHOM2TOrJT0A63j9Rbf5sKxuPkKyZKFUh4OPyPyy JlGdTEXuOUMd8l583sT/a8LbwiVTOGHJCiycAbzCfiD6Yczvu17Cz8Jj8nnDmWZOMg aLc7K0hGu9mqHt02kdxLmwk7kZEiq9YYLOFIVf56VnZO20q1vSNbPQele6UPpTZBfA HHhIwcbi4gzAg== Date: Fri, 9 Aug 2024 12:10:07 -0600 From: Rob Herring To: Thippeswamy Havalige Cc: lpieralisi@kernel.org, kw@linux.com, bhelgaas@google.com, krzk+dt@kernel.org, conor+dt@kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-pci@vger.kernel.org, thippeswamy.havalige@amd.com, linux-arm-kernel@lists.infradead.org, michal.simek@amd.com Subject: Re: [PATCH v3 1/2] dt-bindings: PCI: xilinx-xdma: Add schemas for Xilinx QDMA PCIe Root Port Bridge Message-ID: <20240809181007.GA967711-robh@kernel.org> References: <20240809060955.1982335-1-thippesw@amd.com> <20240809060955.1982335-2-thippesw@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240809060955.1982335-2-thippesw@amd.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240809_111010_669180_BF907DFF X-CRM114-Status: GOOD ( 16.29 ) 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 On Fri, Aug 09, 2024 at 11:39:54AM +0530, Thippeswamy Havalige wrote: > Add YAML devicetree schemas for Xilinx QDMA Soft IP PCIe Root Port > Bridge. > > Signed-off-by: Thippeswamy Havalige > --- > .../devicetree/bindings/pci/xlnx,xdma-host.yaml | 36 ++++++++++++++++++++-- > 1 file changed, 34 insertions(+), 2 deletions(-) > --- > changes in v3 > - constrain the new entry to only the new compatible. > - Remove example. > > changes in v2 > - update dt node label with pcie. > --- > diff --git a/Documentation/devicetree/bindings/pci/xlnx,xdma-host.yaml b/Documentation/devicetree/bindings/pci/xlnx,xdma-host.yaml > index 2f59b3a..f1efd919 100644 > --- a/Documentation/devicetree/bindings/pci/xlnx,xdma-host.yaml > +++ b/Documentation/devicetree/bindings/pci/xlnx,xdma-host.yaml > @@ -14,10 +14,21 @@ allOf: > > properties: > compatible: > - const: xlnx,xdma-host-3.00 > + enum: > + - xlnx,xdma-host-3.00 > + - xlnx,qdma-host-3.00 Kind of odd that both IP have the exact same version number. Please explain in the commit message where it comes from. If you just copied it from the previous one, then nak. > > reg: > - maxItems: 1 > + items: > + - description: configuration region and XDMA bridge register. > + - description: QDMA bridge register. > + minItems: 1 > + > + reg-names: > + items: > + - const: cfg > + - const: breg > + minItems: 1 > > ranges: > maxItems: 2 > @@ -76,6 +87,27 @@ required: > - "#interrupt-cells" > - interrupt-controller > > +if: > + properties: > + compatible: > + contains: > + enum: > + - xlnx,qdma-host-3.00 > +then: > + properties: > + reg: > + minItems: 2 > + reg-names: > + minItems: 2 > + required: > + - reg-names > +else: > + properties: > + reg: > + maxItems: 1 > + reg-names: > + maxItems: 1 > + > unevaluatedProperties: false > > examples: > -- > 1.8.3.1 >