From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 6AA8918132D; Wed, 10 Apr 2024 17:48:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712771290; cv=none; b=mmQD06dOWCAB1VFh4HP/zaMdDBE1tfBFMebvZHaNOsYzCF/xcFoAAFx7qDl2exagYYZUU8O5RRFKj40Pj2NXe97x+pA6L38et2uUGmxiFw+goVwSuKfFqoloJ8Auv1sooZp1Q0BFm4pXOHBZ1qlDpVwLUnIxqPSFLSehA6KBwIQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712771290; c=relaxed/simple; bh=hKQowqjPoLWgjXPoVrz9/IkynZj3MiQDAbq+Y6Y7fKE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=seTAiNY/XaNcVk0EOXSDqY+nk9q0CVSLOplDW8/lFXQhtH59qgipx5G86/7jnw1jRno1GoVd9BgCVK/gF36s2r1KusUyiaefCnpHBghEvTO2fxt4fO5y0fRM9z9CnnjSRVwd0e0tDMPtA1l7ZMomE/qEAsMo14boupTU/ZOapmo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=lepQksBy; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="lepQksBy" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9768FC433C7; Wed, 10 Apr 2024 17:48:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1712771288; bh=hKQowqjPoLWgjXPoVrz9/IkynZj3MiQDAbq+Y6Y7fKE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lepQksByl+PqTl3mRAc+aU9f8MqHehaCQQUGPgi+fsm/qm+Be9N4lTABH+eKhoJZV 8F2jEpXDfBuD6xNddLKnY19o5QJ2wOCTjt1x1gNK90WtGYnuRhs7Hk0o992m1td7nZ cGM68YnxAloIz2GlE0UdK9bEI0b4OW0ouWerwovGLUKzEOXiGhetoA3l1zDKle9CDK +scXV17wnr3hhMkmXlEwC7N1ipnwX51LP9m32vDCoKY7jmGYEoeJFnapp1xblRClco 5zm4ZYkE0ymijFG865AyXtxZvS2fE7/hsirvObNrVtTMjuwv4dHlQXMRIT4+MqGG+n mYHv/CtiOmqow== Date: Wed, 10 Apr 2024 12:48:06 -0500 From: Rob Herring To: Krzysztof Kozlowski Cc: Bjorn Helgaas , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Krzysztof Kozlowski , Conor Dooley , Hector Martin , Sven Peter , Alyssa Rosenzweig , Ray Jui , Scott Branden , Broadcom internal kernel review list , Jim Quinlan , Nicolas Saenz Julienne , Florian Fainelli , Will Deacon , Linus Walleij , Srikanth Thokala , Ryder Lee , Jianjun Wang , Sergio Paracuellos , Matthias Brugger , AngeloGioacchino Del Regno , Daire McNamara , Bjorn Andersson , Konrad Dybcio , Marek Vasut , Yoshihiro Shimoda , Shawn Lin , Heiko Stuebner , Jingoo Han , Gustavo Pimentel , Manivannan Sadhasivam , Bharat Kumar Gogada , Michal Simek , Geert Uytterhoeven , Magnus Damm , Neil Armstrong , Mark Kettenis , Tom Joseph , Ahmad Zainie , Jiaxun Yang , Kishon Vijay Abraham I , Thippeswamy Havalige , linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: Re: [PATCH 3/3] dt-bindings: PCI: host-bridges: switch from deprecated pci-bus.yaml Message-ID: <20240410174806.GA788199-robh@kernel.org> References: <20240407102000.37213-1-krzysztof.kozlowski@linaro.org> <20240407102000.37213-3-krzysztof.kozlowski@linaro.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240407102000.37213-3-krzysztof.kozlowski@linaro.org> On Sun, Apr 07, 2024 at 12:20:00PM +0200, Krzysztof Kozlowski wrote: > dtschema package with core schemas deprecated pci-bus.yaml schema in > favor of pci-host-bridge.yaml. Update all bindings to use the latter > one. > > The difference between pci-bus.yaml and pci-host-bridge.yaml is only in > lack of "reg" property defined by the latter, which should not have any > effect here, because all these bindings define the "reg". > > The change is therefore quite trivial, except mediatek,mt7621-pcie.yaml > binding which have children nodes being also host bridges, apparently. > > Signed-off-by: Krzysztof Kozlowski > --- > Documentation/devicetree/bindings/pci/amlogic,axg-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/apple,pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/brcm,iproc-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/cdns-pcie-host.yaml | 2 +- > Documentation/devicetree/bindings/pci/faraday,ftpci100.yaml | 2 +- > Documentation/devicetree/bindings/pci/host-generic-pci.yaml | 2 +- > Documentation/devicetree/bindings/pci/intel,ixp4xx-pci.yaml | 2 +- > Documentation/devicetree/bindings/pci/intel,keembay-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/loongson.yaml | 2 +- > .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 4 ++-- > Documentation/devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 2 +- > .../devicetree/bindings/pci/microchip,pcie-host.yaml | 2 +- > Documentation/devicetree/bindings/pci/qcom,pcie-common.yaml | 2 +- > Documentation/devicetree/bindings/pci/qcom,pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/rcar-pci-host.yaml | 2 +- > .../devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml | 2 +- > .../devicetree/bindings/pci/rockchip,rk3399-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/ti,am65-pci-host.yaml | 2 +- > Documentation/devicetree/bindings/pci/versatile.yaml | 2 +- > Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml | 2 +- > Documentation/devicetree/bindings/pci/xlnx,axi-pcie-host.yaml | 2 +- > Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml | 2 +- > Documentation/devicetree/bindings/pci/xlnx,xdma-host.yaml | 2 +- > 25 files changed, 26 insertions(+), 26 deletions(-) > diff --git a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml > index 61d027239910..5bbb4a3f3dbd 100644 > --- a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml > +++ b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml > @@ -14,7 +14,7 @@ description: |+ > with 3 Root Ports. Each Root Port supports a Gen1 1-lane Link > > allOf: > - - $ref: /schemas/pci/pci-bus.yaml# > + - $ref: /schemas/pci/pci-host-bridge.yaml# > > properties: > compatible: > @@ -33,7 +33,7 @@ properties: > patternProperties: > '^pcie@[0-2],0$': > type: object > - $ref: /schemas/pci/pci-bus.yaml# > + $ref: /schemas/pci/pci-host-bridge.yaml# I think this one should be pci-pci-bridge.yaml instead since it says these are root ports and based on the unit-address here. Rob