Re: [PATCH 4/5] dt-bindings: clock: Add USB OTG clock for X1000.

From: Paul Cercueil
Date: Wed Nov 27 2019 - 12:19:19 EST


Hi Zhou,


Le mer., nov. 27, 2019 at 11:32, Zhou Yanjie <zhouyanjie@xxxxxxxx> a écrit :
Add the USB OTC clock bindings for the X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@xxxxxxxx>
---
include/dt-bindings/clock/x1000-cgu.h | 23 ++++++++++++-----------
1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/include/dt-bindings/clock/x1000-cgu.h b/include/dt-bindings/clock/x1000-cgu.h
index bbaebaf..c401fce 100644
--- a/include/dt-bindings/clock/x1000-cgu.h
+++ b/include/dt-bindings/clock/x1000-cgu.h
@@ -29,16 +29,17 @@
#define X1000_CLK_MSCMUX 14
#define X1000_CLK_MSC0 15
#define X1000_CLK_MSC1 16
-#define X1000_CLK_SSIPLL 17
-#define X1000_CLK_SSIMUX 18
-#define X1000_CLK_SFC 19
-#define X1000_CLK_I2C0 20
-#define X1000_CLK_I2C1 21
-#define X1000_CLK_I2C2 22
-#define X1000_CLK_UART0 23
-#define X1000_CLK_UART1 24
-#define X1000_CLK_UART2 25
-#define X1000_CLK_SSI 26
-#define X1000_CLK_PDMA 27

You can't do that. These macros are ABI now, since they are used in the devicetree. Just use the next valid number for your OTG clock.

Cheers,
-Paul

+#define X1000_CLK_OTG 17
+#define X1000_CLK_SSIPLL 18
+#define X1000_CLK_SSIMUX 19
+#define X1000_CLK_SFC 20
+#define X1000_CLK_I2C0 21
+#define X1000_CLK_I2C1 22
+#define X1000_CLK_I2C2 23
+#define X1000_CLK_UART0 24
+#define X1000_CLK_UART1 25
+#define X1000_CLK_UART2 26
+#define X1000_CLK_SSI 27
+#define X1000_CLK_PDMA 28

#endif /* __DT_BINDINGS_CLOCK_X1000_CGU_H__ */
--
2.7.4