Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

dt-bindings: xilinx: Deprecate header with firmware constants

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").

Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Acked-by: Vinod Koul <vkoul@kernel.org>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Michal Simek <michal.simek@amd.com>
Link: https://lore.kernel.org/r/2a6f0229522327939e6893565e540b75f854a37b.1738600745.git.michal.simek@amd.com

+15 -15
+1 -3
Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml
··· 163 163 164 164 examples: 165 165 - | 166 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 167 166 #include <dt-bindings/interrupt-controller/irq.h> 168 167 #include <dt-bindings/power/xlnx-zynqmp-power.h> 169 168 #include <dt-bindings/reset/xlnx-zynqmp-resets.h> 170 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 171 169 #include <dt-bindings/phy/phy.h> 172 170 173 171 sata: ahci@fd0c0000 { ··· 173 175 reg = <0xfd0c0000 0x200>; 174 176 interrupt-parent = <&gic>; 175 177 interrupts = <0 133 IRQ_TYPE_LEVEL_HIGH>; 176 - clocks = <&zynqmp_clk SATA_REF>; 178 + clocks = <&zynqmp_clk 22>; 177 179 ceva,p0-cominit-params = /bits/ 8 <0x0F 0x25 0x18 0x29>; 178 180 ceva,p0-comwake-params = /bits/ 8 <0x04 0x0B 0x08 0x0F>; 179 181 ceva,p0-burst-params = /bits/ 8 <0x0A 0x08 0x4A 0x06>;
+1 -2
Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml
··· 75 75 76 76 examples: 77 77 - | 78 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 79 78 80 79 fpd_dma_chan1: dma-controller@fd500000 { 81 80 compatible = "xlnx,zynqmp-dma-1.0"; ··· 83 84 interrupts = <0 117 0x4>; 84 85 #dma-cells = <1>; 85 86 clock-names = "clk_main", "clk_apb"; 86 - clocks = <&zynqmp_clk GDMA_REF>, <&zynqmp_clk LPD_LSBUS>; 87 + clocks = <&zynqmp_clk 19>, <&zynqmp_clk 31>; 87 88 xlnx,bus-width = <128>; 88 89 dma-coherent; 89 90 };
+1 -2
Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml
··· 193 193 194 194 examples: 195 195 - | 196 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 197 196 198 197 bus { 199 198 #address-cells = <2>; ··· 203 204 interrupt-parent = <&gic>; 204 205 interrupts = <0 56 4>; 205 206 reg = <0x0 0xffa50000 0x0 0x800>; 206 - clocks = <&zynqmp_clk AMS_REF>; 207 + clocks = <&zynqmp_clk 70>; 207 208 #address-cells = <1>; 208 209 #size-cells = <1>; 209 210 #io-channel-cells = <1>;
+3 -4
Documentation/devicetree/bindings/net/cdns,macb.yaml
··· 197 197 }; 198 198 199 199 - | 200 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 201 200 #include <dt-bindings/power/xlnx-zynqmp-power.h> 202 201 #include <dt-bindings/reset/xlnx-zynqmp-resets.h> 203 202 #include <dt-bindings/phy/phy.h> ··· 209 210 interrupt-parent = <&gic>; 210 211 interrupts = <0 59 4>, <0 59 4>; 211 212 reg = <0x0 0xff0c0000 0x0 0x1000>; 212 - clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM1_REF>, 213 - <&zynqmp_clk GEM1_TX>, <&zynqmp_clk GEM1_RX>, 214 - <&zynqmp_clk GEM_TSU>; 213 + clocks = <&zynqmp_clk 31>, <&zynqmp_clk 105>, 214 + <&zynqmp_clk 51>, <&zynqmp_clk 50>, 215 + <&zynqmp_clk 44>; 215 216 clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk"; 216 217 #address-cells = <1>; 217 218 #size-cells = <0>;
+1 -2
Documentation/devicetree/bindings/spi/spi-zynqmp-qspi.yaml
··· 65 65 66 66 examples: 67 67 - | 68 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 69 68 soc { 70 69 #address-cells = <2>; 71 70 #size-cells = <2>; 72 71 73 72 qspi: spi@ff0f0000 { 74 73 compatible = "xlnx,zynqmp-qspi-1.0"; 75 - clocks = <&zynqmp_clk QSPI_REF>, <&zynqmp_clk LPD_LSBUS>; 74 + clocks = <&zynqmp_clk 53>, <&zynqmp_clk 82>; 76 75 clock-names = "ref_clk", "pclk"; 77 76 interrupts = <0 15 4>; 78 77 interrupt-parent = <&gic>;
+1 -2
Documentation/devicetree/bindings/usb/dwc3-xilinx.yaml
··· 101 101 #include <dt-bindings/dma/xlnx-zynqmp-dpdma.h> 102 102 #include <dt-bindings/power/xlnx-zynqmp-power.h> 103 103 #include <dt-bindings/reset/xlnx-zynqmp-resets.h> 104 - #include <dt-bindings/clock/xlnx-zynqmp-clk.h> 105 104 #include <dt-bindings/reset/xlnx-zynqmp-resets.h> 106 105 #include <dt-bindings/phy/phy.h> 107 106 axi { ··· 112 113 #size-cells = <0x2>; 113 114 compatible = "xlnx,zynqmp-dwc3"; 114 115 reg = <0x0 0xff9d0000 0x0 0x100>; 115 - clocks = <&zynqmp_clk USB0_BUS_REF>, <&zynqmp_clk USB3_DUAL_REF>; 116 + clocks = <&zynqmp_clk 32>, <&zynqmp_clk 34>; 116 117 clock-names = "bus_clk", "ref_clk"; 117 118 power-domains = <&zynqmp_firmware PD_USB_0>; 118 119 resets = <&zynqmp_reset ZYNQMP_RESET_USB1_CORERESET>,
+7
include/dt-bindings/clock/xlnx-zynqmp-clk.h
··· 9 9 #ifndef _DT_BINDINGS_CLK_ZYNQMP_H 10 10 #define _DT_BINDINGS_CLK_ZYNQMP_H 11 11 12 + /* 13 + * These bindings are deprecated, because they do not match the actual 14 + * concept of bindings but rather contain pure firmware values. 15 + * Instead include the header in the DTS source directory. 16 + */ 17 + #warning "These bindings are deprecated. Instead use the header in the DTS source directory." 18 + 12 19 #define IOPLL 0 13 20 #define RPLL 1 14 21 #define APLL 2