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 0CABBC02193 for ; Tue, 4 Feb 2025 21:31:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: Content-Transfer-Encoding:Content-Type:MIME-Version:References:In-Reply-To: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=a60Oy+E7kZ+sKIoJEIdZPtBpu9wL5oPJ1nkC69stlKY=; b=ou4u5akzOfMiLU n45vaEaVtkZAldQRd8irXQ3gvSUUhWdCwDnh/OTrQ+GT7hhkaVVwhMJRX0u/zNll9YLEUdslul0BR HD2/zjiJxBDUrMgqJ9/6BMfDbT/R+AtPJeu3yOz8UEBvKa5zbqL10K3AzTh0Lqay5E1XvumX7ON1P vW0P+bDUdu8IIz6wSIUxj4nD2br0z08HgY4h+V6Z4TyB4bQiNoBNdOWl9oxH7awYPizn1Le7h6Rfx NN5985eL5QBEwNWnHtOG+y7aizmiyQdJ5iGluQRgCFfkB+G2znbIoD9Wu9fSYivJXCiBgluGUOEpS eZbZvMNYhckTwRtE8nsA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tfQVg-00000001bQJ-2MyO; Tue, 04 Feb 2025 21:31:08 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tfPAS-00000001UG0-3Biw for linux-arm-kernel@bombadil.infradead.org; Tue, 04 Feb 2025 20:05:08 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=Content-Transfer-Encoding:Content-Type :MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date: Sender:Reply-To:Content-ID:Content-Description; bh=a60Oy+E7kZ+sKIoJEIdZPtBpu9wL5oPJ1nkC69stlKY=; b=Cv4QLFcPtBL4b5StWQu68TJR4a hLcAMjM16M1VlVIEJtl0aMFkOb7w/k0wECrszkPlNV6bwVEF86CzmhtfucfBNdahzSxozqNTl6vKC K/OOKpC1kiTgk5mVfXx6uDYnNE+qk96sbEbTJf34DE/eOsADpuaXMvxdZvuls2O1s5uqmhLM9Mtza noSycUCECIyB1nTC8u5KmucdmG9cHDKeL2sagDisFXg4J3RD/Tj+HMHQuJrqDrcEcWdyNE/9gBVOT JHxwBxTvelzr4GfurSWaLcZPacgljk9eaMRiqhXnmE3XnT/xSH8RTsZRug5s2E33eedUQ89Y1Pfvt ps3wNxdA==; Received: from nyc.source.kernel.org ([2604:1380:45d1:ec00::3]) by desiato.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tfPAO-0000000GQEy-0Nzg for linux-arm-kernel@lists.infradead.org; Tue, 04 Feb 2025 20:05:07 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id ADE42A42D74; Tue, 4 Feb 2025 20:03:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3CF3FC4CEE2; Tue, 4 Feb 2025 20:04:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738699499; bh=6y3E8+OpE5ZmqjX7Om9uPkD+ZQQXyu1GMUrqnd3TjQk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=ATqxfuIih0h8cjA2h0bfzeW8/14cKwjo0aYg/EDyMiNfuHEuMgbIUArMXwpxbH8tQ Da4cVQL4P8+pftQUNXCfpHfx7R/c8T5y9fXpFVdrQBkPwgdoDHoEsw5o2N/iwNbzt8 snOmZS5hwa+n4hlL3owOGHovG/7gC5BR91zkSxLubTB/2goLFqeTbHt28F5FnmdKY4 hCflsbEgNgW4l3dpav/XeOnh8UfB9jdhSJ/P2ICbj8kqOcpTDEmCzBb1u2EZeyPCm/ PCiZB/g/1ohi/zf1NFeXq1TtDI49Zm/hGspFF4kYcp0temKz7YcZ9Fx3qzuE1DJICk QRfqRdleBGwGQ== Date: Tue, 4 Feb 2025 20:04:46 +0000 From: Jonathan Cameron To: Michal Simek Subject: Re: [PATCH 2/2] dt-bindings: xilinx: Deprecate header with firmware constants Message-ID: <20250204200446.32bfb3f8@jic23-huawei> In-Reply-To: <2a6f0229522327939e6893565e540b75f854a37b.1738600745.git.michal.simek@amd.com> References: <2a6f0229522327939e6893565e540b75f854a37b.1738600745.git.michal.simek@amd.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.48; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250204_200504_484344_BF2CC39C X-CRM114-Status: GOOD ( 21.76 ) 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: , Cc: "open list:XILINX AMS DRIVER" , Michael Turquette , linux-kernel@vger.kernel.org, Niklas Cassel , Eric Dumazet , Claudiu Beznea , "open list:COMMON CLK FRAMEWORK" , Shyam Pandey , Michael Tretter , Rob Herring , Lars-Peter Clausen , michal.simek@xilinx.com, Jakub Kicinski , Paolo Abeni , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Conor Dooley , Anand Ashok Dumbre , Mark Brown , git@xilinx.com, "moderated list:ARM/ZYNQ ARCHITECTURE" , Harini Katakam , monstr@monstr.eu, "open list:LIBATA SUBSYSTEM \(Serial and Parallel ATA drivers\)" , Stephen Boyd , Greg Kroah-Hartman , "open list:USB SUBSYSTEM" , "open list:SPI SUBSYSTEM" , Andrew Lunn , Vinod Koul , Damien Le Moal , "open list:NETWORKING DRIVERS" , "open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM" , Krzysztof Kozlowski , "David S. Miller" , Mubin Sayyed Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 3 Feb 2025 17:39:11 +0100 Michal Simek wrote: > Firmware contants do not fit the purpose of bindings because they are not > independent IDs for abstractions. They are more or less just contants which > better to wire via header with DT which is using it. > That's why add deprecated message to dt binding header and also update > existing dt bindings not to use macros from the header and replace them by > it's value. Actually value is not relevant because it is only example. > > The similar changes have been done by commit 9d9292576810 ("dt-bindings: > pinctrl: samsung: deprecate header with register constants"). > > Signed-off-by: Michal Simek For IIO Acked-by: Jonathan Cameron > --- > > Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml | 4 +--- > .../bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml | 3 +-- > .../devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml | 3 +-- > Documentation/devicetree/bindings/net/cdns,macb.yaml | 7 +++---- > Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml | 3 +-- > Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml | 3 +-- > include/dt-bindings/clock/xlnx-zynqmp-clk.h | 7 +++++++ > 7 files changed, 15 insertions(+), 15 deletions(-) > > diff --git a/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml b/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml > index 9952e0ef7767..6ad78429dc74 100644 > --- a/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml > +++ b/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml > @@ -163,11 +163,9 @@ additionalProperties: false > > examples: > - | > - #include > #include > #include > #include > - #include > #include > > sata: ahci@fd0c0000 { > @@ -175,7 +173,7 @@ examples: > reg = <0xfd0c0000 0x200>; > interrupt-parent = <&gic>; > interrupts = <0 133 IRQ_TYPE_LEVEL_HIGH>; > - clocks = <&zynqmp_clk SATA_REF>; > + clocks = <&zynqmp_clk 22>; > ceva,p0-cominit-params = /bits/ 8 <0x0F 0x25 0x18 0x29>; > ceva,p0-comwake-params = /bits/ 8 <0x04 0x0B 0x08 0x0F>; > ceva,p0-burst-params = /bits/ 8 <0x0A 0x08 0x4A 0x06>; > diff --git a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml > index ac3198953b8e..b5399c65a731 100644 > --- a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml > +++ b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml > @@ -75,7 +75,6 @@ additionalProperties: false > > examples: > - | > - #include > > fpd_dma_chan1: dma-controller@fd500000 { > compatible = "xlnx,zynqmp-dma-1.0"; > @@ -84,7 +83,7 @@ examples: > interrupts = <0 117 0x4>; > #dma-cells = <1>; > clock-names = "clk_main", "clk_apb"; > - clocks = <&zynqmp_clk GDMA_REF>, <&zynqmp_clk LPD_LSBUS>; > + clocks = <&zynqmp_clk 19>, <&zynqmp_clk 31>; > xlnx,bus-width = <128>; > dma-coherent; > }; > diff --git a/Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml b/Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml > index 8cbad7e792b6..a403392fb263 100644 > --- a/Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml > @@ -193,7 +193,6 @@ additionalProperties: false > > examples: > - | > - #include > > bus { > #address-cells = <2>; > @@ -204,7 +203,7 @@ examples: > interrupt-parent = <&gic>; > interrupts = <0 56 4>; > reg = <0x0 0xffa50000 0x0 0x800>; > - clocks = <&zynqmp_clk AMS_REF>; > + clocks = <&zynqmp_clk 70>; > #address-cells = <1>; > #size-cells = <1>; > #io-channel-cells = <1>; > diff --git a/Documentation/devicetree/bindings/net/cdns,macb.yaml b/Documentation/devicetree/bindings/net/cdns,macb.yaml > index 3c30dd23cd4e..8d69846b2e09 100644 > --- a/Documentation/devicetree/bindings/net/cdns,macb.yaml > +++ b/Documentation/devicetree/bindings/net/cdns,macb.yaml > @@ -197,7 +197,6 @@ examples: > }; > > - | > - #include > #include > #include > #include > @@ -210,9 +209,9 @@ examples: > interrupt-parent = <&gic>; > interrupts = <0 59 4>, <0 59 4>; > reg = <0x0 0xff0c0000 0x0 0x1000>; > - clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM1_REF>, > - <&zynqmp_clk GEM1_TX>, <&zynqmp_clk GEM1_RX>, > - <&zynqmp_clk GEM_TSU>; > + clocks = <&zynqmp_clk 31>, <&zynqmp_clk 105>, > + <&zynqmp_clk 51>, <&zynqmp_clk 50>, > + <&zynqmp_clk 44>; > clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk"; > #address-cells = <1>; > #size-cells = <0>; > diff --git a/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml b/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml > index 04d4d3b4916d..02cf1314367b 100644 > --- a/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml > +++ b/Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml > @@ -65,14 +65,13 @@ allOf: > > examples: > - | > - #include > soc { > #address-cells = <2>; > #size-cells = <2>; > > qspi: spi@ff0f0000 { > compatible = "xlnx,zynqmp-qspi-1.0"; > - clocks = <&zynqmp_clk QSPI_REF>, <&zynqmp_clk LPD_LSBUS>; > + clocks = <&zynqmp_clk 53>, <&zynqmp_clk 82>; > clock-names = "ref_clk", "pclk"; > interrupts = <0 15 4>; > interrupt-parent = <&gic>; > diff --git a/Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml b/Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml > index 00f87a558c7d..b5843f4d17d8 100644 > --- a/Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml > +++ b/Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml > @@ -101,7 +101,6 @@ examples: > #include > #include > #include > - #include > #include > #include > axi { > @@ -113,7 +112,7 @@ examples: > #size-cells = <0x2>; > compatible = "xlnx,zynqmp-dwc3"; > reg = <0x0 0xff9d0000 0x0 0x100>; > - clocks = <&zynqmp_clk USB0_BUS_REF>, <&zynqmp_clk USB3_DUAL_REF>; > + clocks = <&zynqmp_clk 32>, <&zynqmp_clk 34>; > clock-names = "bus_clk", "ref_clk"; > power-domains = <&zynqmp_firmware PD_USB_0>; > resets = <&zynqmp_reset ZYNQMP_RESET_USB1_CORERESET>, > diff --git a/include/dt-bindings/clock/xlnx-zynqmp-clk.h b/include/dt-bindings/clock/xlnx-zynqmp-clk.h > index cdc4c0b9a374..f0f7ddd3dcbd 100644 > --- a/include/dt-bindings/clock/xlnx-zynqmp-clk.h > +++ b/include/dt-bindings/clock/xlnx-zynqmp-clk.h > @@ -9,6 +9,13 @@ > #ifndef _DT_BINDINGS_CLK_ZYNQMP_H > #define _DT_BINDINGS_CLK_ZYNQMP_H > > +/* > + * These bindings are deprecated, because they do not match the actual > + * concept of bindings but rather contain pure firmware values. > + * Instead include the header in the DTS source directory. > + */ > +#warning "These bindings are deprecated. Instead use the header in the DTS source directory." > + > #define IOPLL 0 > #define RPLL 1 > #define APLL 2