All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] sync SCFW defines with SCFW kit v1.13.0
@ 2022-09-15 18:18 ` Viorel Suman (OSS)
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

The latest available SCFW kit is version 1.13.0, may be found at the address below:
https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
The patchset updates the kernel defines so they are in sync with the latest SCFW kit.
"rsrc.h" patch include also the defines required for backward compatibility in the same
fashion as in SCFW kit.

ViorelSuman (2):
  dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  clk: imx8qm: sync with SCFW kit v1.13.0

 drivers/clk/imx/clk-imx8qm-rsrc.c       |   4 +-
 include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
 2 files changed, 205 insertions(+), 98 deletions(-)

-- 
2.37.2


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH 0/2] sync SCFW defines with SCFW kit v1.13.0
@ 2022-09-15 18:18 ` Viorel Suman (OSS)
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

The latest available SCFW kit is version 1.13.0, may be found at the address below:
https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
The patchset updates the kernel defines so they are in sync with the latest SCFW kit.
"rsrc.h" patch include also the defines required for backward compatibility in the same
fashion as in SCFW kit.

ViorelSuman (2):
  dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  clk: imx8qm: sync with SCFW kit v1.13.0

 drivers/clk/imx/clk-imx8qm-rsrc.c       |   4 +-
 include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
 2 files changed, 205 insertions(+), 98 deletions(-)

-- 
2.37.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-15 18:18 ` Viorel Suman (OSS)
@ 2022-09-15 18:18   ` Viorel Suman (OSS)
  -1 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

Sync defines with the latest available SCFW kit version 1.13.0,
may be found at the address below:

https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license

Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
---
 include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
 1 file changed, 203 insertions(+), 96 deletions(-)

diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
index 43885056557c..a4c68f394986 100644
--- a/include/dt-bindings/firmware/imx/rsrc.h
+++ b/include/dt-bindings/firmware/imx/rsrc.h
@@ -13,34 +13,38 @@
  * never be changed or removed (only added to at the end of the list).
  */
 
-#define IMX_SC_R_A53			0
-#define IMX_SC_R_A53_0			1
-#define IMX_SC_R_A53_1			2
-#define IMX_SC_R_A53_2			3
-#define IMX_SC_R_A53_3			4
-#define IMX_SC_R_A72			5
-#define IMX_SC_R_A72_0			6
-#define IMX_SC_R_A72_1			7
-#define IMX_SC_R_A72_2			8
-#define IMX_SC_R_A72_3			9
+#define IMX_SC_R_AP_0			0
+#define IMX_SC_R_AP_0_0			1
+#define IMX_SC_R_AP_0_1			2
+#define IMX_SC_R_AP_0_2			3
+#define IMX_SC_R_AP_0_3			4
+#define IMX_SC_R_AP_1			5
+#define IMX_SC_R_AP_1_0			6
+#define IMX_SC_R_AP_1_1			7
+#define IMX_SC_R_AP_1_2			8
+#define IMX_SC_R_AP_1_3			9
 #define IMX_SC_R_CCI			10
 #define IMX_SC_R_DB			11
 #define IMX_SC_R_DRC_0			12
 #define IMX_SC_R_DRC_1			13
 #define IMX_SC_R_GIC_SMMU		14
-#define IMX_SC_R_IRQSTR_M4_0		15
-#define IMX_SC_R_IRQSTR_M4_1		16
-#define IMX_SC_R_SMMU			17
-#define IMX_SC_R_GIC			18
+#define IMX_SC_R_IRQSTR_MCU_0		15
+#define IMX_SC_R_IRQSTR_MCU_1		16
+#define IMX_SC_R_SMMU_0			17
+#define IMX_SC_R_GIC_0			18
 #define IMX_SC_R_DC_0_BLIT0		19
 #define IMX_SC_R_DC_0_BLIT1		20
 #define IMX_SC_R_DC_0_BLIT2		21
 #define IMX_SC_R_DC_0_BLIT_OUT		22
-#define IMX_SC_R_PERF			23
+#define IMX_SC_R_PERF_0			23
+#define IMX_SC_R_USB_1_PHY		24
 #define IMX_SC_R_DC_0_WARP		25
+#define IMX_SC_R_V2X_MU_0		26
+#define IMX_SC_R_V2X_MU_1		27
 #define IMX_SC_R_DC_0_VIDEO0		28
 #define IMX_SC_R_DC_0_VIDEO1		29
 #define IMX_SC_R_DC_0_FRAC0		30
+#define IMX_SC_R_V2X_MU_2		31
 #define IMX_SC_R_DC_0			32
 #define IMX_SC_R_GPU_2_PID0		33
 #define IMX_SC_R_DC_0_PLL_0		34
@@ -49,11 +53,17 @@
 #define IMX_SC_R_DC_1_BLIT1		37
 #define IMX_SC_R_DC_1_BLIT2		38
 #define IMX_SC_R_DC_1_BLIT_OUT		39
+#define IMX_SC_R_V2X_MU_3		40
+#define IMX_SC_R_V2X_MU_4		41
 #define IMX_SC_R_DC_1_WARP		42
+#define IMX_SC_R_STM			43
+#define IMX_SC_R_SECVIO			44
 #define IMX_SC_R_DC_1_VIDEO0		45
 #define IMX_SC_R_DC_1_VIDEO1		46
 #define IMX_SC_R_DC_1_FRAC0		47
+#define IMX_SC_R_V2X			48
 #define IMX_SC_R_DC_1			49
+#define IMX_SC_R_UNUSED14		50
 #define IMX_SC_R_DC_1_PLL_0		51
 #define IMX_SC_R_DC_1_PLL_1		52
 #define IMX_SC_R_SPI_0			53
@@ -144,10 +154,10 @@
 #define IMX_SC_R_DMA_1_CH29		137
 #define IMX_SC_R_DMA_1_CH30		138
 #define IMX_SC_R_DMA_1_CH31		139
-#define IMX_SC_R_UNUSED1		140
-#define IMX_SC_R_UNUSED2		141
-#define IMX_SC_R_UNUSED3		142
-#define IMX_SC_R_UNUSED4		143
+#define IMX_SC_R_V2X_PID0		140
+#define IMX_SC_R_V2X_PID1		141
+#define IMX_SC_R_V2X_PID2		142
+#define IMX_SC_R_V2X_PID3		143
 #define IMX_SC_R_GPU_0_PID0		144
 #define IMX_SC_R_GPU_0_PID1		145
 #define IMX_SC_R_GPU_0_PID2		146
@@ -176,7 +186,7 @@
 #define IMX_SC_R_PCIE_B			169
 #define IMX_SC_R_SATA_0			170
 #define IMX_SC_R_SERDES_1		171
-#define IMX_SC_R_HSIO_GPIO		172
+#define IMX_SC_R_HSIO_GPIO_0		172
 #define IMX_SC_R_MATCH_15		173
 #define IMX_SC_R_MATCH_16		174
 #define IMX_SC_R_MATCH_17		175
@@ -243,15 +253,15 @@
 #define IMX_SC_R_ROM_0			236
 #define IMX_SC_R_FSPI_0			237
 #define IMX_SC_R_FSPI_1			238
-#define IMX_SC_R_IEE			239
-#define IMX_SC_R_IEE_R0			240
-#define IMX_SC_R_IEE_R1			241
-#define IMX_SC_R_IEE_R2			242
-#define IMX_SC_R_IEE_R3			243
-#define IMX_SC_R_IEE_R4			244
-#define IMX_SC_R_IEE_R5			245
-#define IMX_SC_R_IEE_R6			246
-#define IMX_SC_R_IEE_R7			247
+#define IMX_SC_R_IEE_0			239
+#define IMX_SC_R_IEE_0_R0		240
+#define IMX_SC_R_IEE_0_R1		241
+#define IMX_SC_R_IEE_0_R2		242
+#define IMX_SC_R_IEE_0_R3		243
+#define IMX_SC_R_IEE_0_R4		244
+#define IMX_SC_R_IEE_0_R5		245
+#define IMX_SC_R_IEE_0_R6		246
+#define IMX_SC_R_IEE_0_R7		247
 #define IMX_SC_R_SDHC_0			248
 #define IMX_SC_R_SDHC_1			249
 #define IMX_SC_R_SDHC_2			250
@@ -282,46 +292,50 @@
 #define IMX_SC_R_LVDS_2_PWM_0		275
 #define IMX_SC_R_LVDS_2_I2C_0		276
 #define IMX_SC_R_LVDS_2_I2C_1		277
-#define IMX_SC_R_M4_0_PID0		278
-#define IMX_SC_R_M4_0_PID1		279
-#define IMX_SC_R_M4_0_PID2		280
-#define IMX_SC_R_M4_0_PID3		281
-#define IMX_SC_R_M4_0_PID4		282
-#define IMX_SC_R_M4_0_RGPIO		283
-#define IMX_SC_R_M4_0_SEMA42		284
-#define IMX_SC_R_M4_0_TPM		285
-#define IMX_SC_R_M4_0_PIT		286
-#define IMX_SC_R_M4_0_UART		287
-#define IMX_SC_R_M4_0_I2C		288
-#define IMX_SC_R_M4_0_INTMUX		289
-#define IMX_SC_R_M4_0_MU_0B		292
-#define IMX_SC_R_M4_0_MU_0A0		293
-#define IMX_SC_R_M4_0_MU_0A1		294
-#define IMX_SC_R_M4_0_MU_0A2		295
-#define IMX_SC_R_M4_0_MU_0A3		296
-#define IMX_SC_R_M4_0_MU_1A		297
-#define IMX_SC_R_M4_1_PID0		298
-#define IMX_SC_R_M4_1_PID1		299
-#define IMX_SC_R_M4_1_PID2		300
-#define IMX_SC_R_M4_1_PID3		301
-#define IMX_SC_R_M4_1_PID4		302
-#define IMX_SC_R_M4_1_RGPIO		303
-#define IMX_SC_R_M4_1_SEMA42		304
-#define IMX_SC_R_M4_1_TPM		305
-#define IMX_SC_R_M4_1_PIT		306
-#define IMX_SC_R_M4_1_UART		307
-#define IMX_SC_R_M4_1_I2C		308
-#define IMX_SC_R_M4_1_INTMUX		309
-#define IMX_SC_R_M4_1_MU_0B		312
-#define IMX_SC_R_M4_1_MU_0A0		313
-#define IMX_SC_R_M4_1_MU_0A1		314
-#define IMX_SC_R_M4_1_MU_0A2		315
-#define IMX_SC_R_M4_1_MU_0A3		316
-#define IMX_SC_R_M4_1_MU_1A		317
+#define IMX_SC_R_MCU_0_PID0		278
+#define IMX_SC_R_MCU_0_PID1		279
+#define IMX_SC_R_MCU_0_PID2		280
+#define IMX_SC_R_MCU_0_PID3		281
+#define IMX_SC_R_MCU_0_PID4		282
+#define IMX_SC_R_MCU_0_RGPIO		283
+#define IMX_SC_R_MCU_0_SEMA42		284
+#define IMX_SC_R_MCU_0_TPM		285
+#define IMX_SC_R_MCU_0_PIT		286
+#define IMX_SC_R_MCU_0_UART		287
+#define IMX_SC_R_MCU_0_I2C		288
+#define IMX_SC_R_MCU_0_INTMUX		289
+#define IMX_SC_R_ENET_0_A0		290
+#define IMX_SC_R_ENET_0_A1		291
+#define IMX_SC_R_MCU_0_MU_0B		292
+#define IMX_SC_R_MCU_0_MU_0A0		293
+#define IMX_SC_R_MCU_0_MU_0A1		294
+#define IMX_SC_R_MCU_0_MU_0A2		295
+#define IMX_SC_R_MCU_0_MU_0A3		296
+#define IMX_SC_R_MCU_0_MU_1A		297
+#define IMX_SC_R_MCU_1_PID0		298
+#define IMX_SC_R_MCU_1_PID1		299
+#define IMX_SC_R_MCU_1_PID2		300
+#define IMX_SC_R_MCU_1_PID3		301
+#define IMX_SC_R_MCU_1_PID4		302
+#define IMX_SC_R_MCU_1_RGPIO		303
+#define IMX_SC_R_MCU_1_SEMA42		304
+#define IMX_SC_R_MCU_1_TPM		305
+#define IMX_SC_R_MCU_1_PIT		306
+#define IMX_SC_R_MCU_1_UART		307
+#define IMX_SC_R_MCU_1_I2C		308
+#define IMX_SC_R_MCU_1_INTMUX		309
+#define IMX_SC_R_UNUSED17		310
+#define IMX_SC_R_UNUSED18		311
+#define IMX_SC_R_MCU_1_MU_0B		312
+#define IMX_SC_R_MCU_1_MU_0A0		313
+#define IMX_SC_R_MCU_1_MU_0A1		314
+#define IMX_SC_R_MCU_1_MU_0A2		315
+#define IMX_SC_R_MCU_1_MU_0A3		316
+#define IMX_SC_R_MCU_1_MU_1A		317
 #define IMX_SC_R_SAI_0			318
 #define IMX_SC_R_SAI_1			319
 #define IMX_SC_R_SAI_2			320
-#define IMX_SC_R_IRQSTR_SCU2		321
+#define IMX_SC_R_IRQSTR_AP_0		321
 #define IMX_SC_R_IRQSTR_DSP		322
 #define IMX_SC_R_ELCDIF_PLL		323
 #define IMX_SC_R_OCRAM			324
@@ -366,33 +380,33 @@
 #define IMX_SC_R_VPU_PID5		363
 #define IMX_SC_R_VPU_PID6		364
 #define IMX_SC_R_VPU_PID7		365
-#define IMX_SC_R_VPU_UART		366
-#define IMX_SC_R_VPUCORE		367
-#define IMX_SC_R_VPUCORE_0		368
-#define IMX_SC_R_VPUCORE_1		369
-#define IMX_SC_R_VPUCORE_2		370
-#define IMX_SC_R_VPUCORE_3		371
+#define IMX_SC_R_ENET_0_A2		366
+#define IMX_SC_R_ENET_1_A0		367
+#define IMX_SC_R_ENET_1_A1		368
+#define IMX_SC_R_ENET_1_A2		369
+#define IMX_SC_R_ENET_1_A3		370
+#define IMX_SC_R_ENET_1_A4		371
 #define IMX_SC_R_DMA_4_CH0		372
 #define IMX_SC_R_DMA_4_CH1		373
 #define IMX_SC_R_DMA_4_CH2		374
 #define IMX_SC_R_DMA_4_CH3		375
 #define IMX_SC_R_DMA_4_CH4		376
-#define IMX_SC_R_ISI_CH0		377
-#define IMX_SC_R_ISI_CH1		378
-#define IMX_SC_R_ISI_CH2		379
-#define IMX_SC_R_ISI_CH3		380
-#define IMX_SC_R_ISI_CH4		381
-#define IMX_SC_R_ISI_CH5		382
-#define IMX_SC_R_ISI_CH6		383
-#define IMX_SC_R_ISI_CH7		384
-#define IMX_SC_R_MJPEG_DEC_S0		385
-#define IMX_SC_R_MJPEG_DEC_S1		386
-#define IMX_SC_R_MJPEG_DEC_S2		387
-#define IMX_SC_R_MJPEG_DEC_S3		388
-#define IMX_SC_R_MJPEG_ENC_S0		389
-#define IMX_SC_R_MJPEG_ENC_S1		390
-#define IMX_SC_R_MJPEG_ENC_S2		391
-#define IMX_SC_R_MJPEG_ENC_S3		392
+#define IMX_SC_R_ISI_0_CH0		377
+#define IMX_SC_R_ISI_0_CH1		378
+#define IMX_SC_R_ISI_0_CH2		379
+#define IMX_SC_R_ISI_0_CH3		380
+#define IMX_SC_R_ISI_0_CH4		381
+#define IMX_SC_R_ISI_0_CH5		382
+#define IMX_SC_R_ISI_0_CH6		383
+#define IMX_SC_R_ISI_0_CH7		384
+#define IMX_SC_R_MJPEG_0_DEC_S0		385
+#define IMX_SC_R_MJPEG_0_DEC_S1		386
+#define IMX_SC_R_MJPEG_0_DEC_S2		387
+#define IMX_SC_R_MJPEG_0_DEC_S3		388
+#define IMX_SC_R_MJPEG_0_ENC_S0		389
+#define IMX_SC_R_MJPEG_0_ENC_S1		390
+#define IMX_SC_R_MJPEG_0_ENC_S2		391
+#define IMX_SC_R_MJPEG_0_ENC_S3		392
 #define IMX_SC_R_MIPI_0			393
 #define IMX_SC_R_MIPI_0_PWM_0		394
 #define IMX_SC_R_MIPI_0_I2C_0		395
@@ -507,11 +521,11 @@
 #define IMX_SC_R_SECO_MU_3		504
 #define IMX_SC_R_SECO_MU_4		505
 #define IMX_SC_R_HDMI_RX_PWM_0		506
-#define IMX_SC_R_A35			507
-#define IMX_SC_R_A35_0			508
-#define IMX_SC_R_A35_1			509
-#define IMX_SC_R_A35_2			510
-#define IMX_SC_R_A35_3			511
+#define IMX_SC_R_AP_2			507
+#define IMX_SC_R_AP_2_0			508
+#define IMX_SC_R_AP_2_1			509
+#define IMX_SC_R_AP_2_2			510
+#define IMX_SC_R_AP_2_3			511
 #define IMX_SC_R_DSP			512
 #define IMX_SC_R_DSP_RAM		513
 #define IMX_SC_R_CAAM_JR1_OUT		514
@@ -532,8 +546,8 @@
 #define IMX_SC_R_BOARD_R5		529
 #define IMX_SC_R_BOARD_R6		530
 #define IMX_SC_R_BOARD_R7		531
-#define IMX_SC_R_MJPEG_DEC_MP		532
-#define IMX_SC_R_MJPEG_ENC_MP		533
+#define IMX_SC_R_MJPEG_0_DEC_MP		532
+#define IMX_SC_R_MJPEG_0_ENC_MP		533
 #define IMX_SC_R_VPU_TS_0		534
 #define IMX_SC_R_VPU_MU_0		535
 #define IMX_SC_R_VPU_MU_1		536
@@ -565,6 +579,95 @@
 #define IMX_SC_PM_CLK_PLL		4	/* PLL */
 #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
 
+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35			IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53			IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
+#define IMX_SC_R_A72			IMX_SC_R_AP_1
+#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
+#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
+#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
+#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
+#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
+#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
+#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
+#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
+#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
+#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
+#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
+#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
+#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
+#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
+#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
+#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
+#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
+#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
+#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
+#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
+#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
+#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
+#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
+#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
+#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
+#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
+#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
+#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
+#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
+#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
+#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
+#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
+#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
+#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
+#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
+#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
+#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
+#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
+#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
+#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
+#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
+#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
+#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
+#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
+#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
+#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
+#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
+#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
+#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
+#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
+#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
+#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
+#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
+#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
+#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
+#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
+#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
+#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
+#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
+#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
+#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
+#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
+#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
+#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
+#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
+#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
+#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
+#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
+#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
+#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
+#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
+#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
+#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
+#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
+
 /*
  * Defines for SC CONTROL
  */
@@ -630,6 +733,10 @@
 #define IMX_SC_C_INTF_SEL			59
 #define IMX_SC_C_RXC_DLY			60
 #define IMX_SC_C_TIMER_SEL			61
-#define IMX_SC_C_LAST				62
+#define IMX_SC_C_MISC0				62
+#define IMX_SC_C_MISC1				63
+#define IMX_SC_C_MISC2				64
+#define IMX_SC_C_MISC3				65
+#define IMX_SC_C_LAST				66
 
 #endif /* __DT_BINDINGS_RSCRC_IMX_H */
-- 
2.37.2


^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-15 18:18   ` Viorel Suman (OSS)
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

Sync defines with the latest available SCFW kit version 1.13.0,
may be found at the address below:

https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license

Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
---
 include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
 1 file changed, 203 insertions(+), 96 deletions(-)

diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
index 43885056557c..a4c68f394986 100644
--- a/include/dt-bindings/firmware/imx/rsrc.h
+++ b/include/dt-bindings/firmware/imx/rsrc.h
@@ -13,34 +13,38 @@
  * never be changed or removed (only added to at the end of the list).
  */
 
-#define IMX_SC_R_A53			0
-#define IMX_SC_R_A53_0			1
-#define IMX_SC_R_A53_1			2
-#define IMX_SC_R_A53_2			3
-#define IMX_SC_R_A53_3			4
-#define IMX_SC_R_A72			5
-#define IMX_SC_R_A72_0			6
-#define IMX_SC_R_A72_1			7
-#define IMX_SC_R_A72_2			8
-#define IMX_SC_R_A72_3			9
+#define IMX_SC_R_AP_0			0
+#define IMX_SC_R_AP_0_0			1
+#define IMX_SC_R_AP_0_1			2
+#define IMX_SC_R_AP_0_2			3
+#define IMX_SC_R_AP_0_3			4
+#define IMX_SC_R_AP_1			5
+#define IMX_SC_R_AP_1_0			6
+#define IMX_SC_R_AP_1_1			7
+#define IMX_SC_R_AP_1_2			8
+#define IMX_SC_R_AP_1_3			9
 #define IMX_SC_R_CCI			10
 #define IMX_SC_R_DB			11
 #define IMX_SC_R_DRC_0			12
 #define IMX_SC_R_DRC_1			13
 #define IMX_SC_R_GIC_SMMU		14
-#define IMX_SC_R_IRQSTR_M4_0		15
-#define IMX_SC_R_IRQSTR_M4_1		16
-#define IMX_SC_R_SMMU			17
-#define IMX_SC_R_GIC			18
+#define IMX_SC_R_IRQSTR_MCU_0		15
+#define IMX_SC_R_IRQSTR_MCU_1		16
+#define IMX_SC_R_SMMU_0			17
+#define IMX_SC_R_GIC_0			18
 #define IMX_SC_R_DC_0_BLIT0		19
 #define IMX_SC_R_DC_0_BLIT1		20
 #define IMX_SC_R_DC_0_BLIT2		21
 #define IMX_SC_R_DC_0_BLIT_OUT		22
-#define IMX_SC_R_PERF			23
+#define IMX_SC_R_PERF_0			23
+#define IMX_SC_R_USB_1_PHY		24
 #define IMX_SC_R_DC_0_WARP		25
+#define IMX_SC_R_V2X_MU_0		26
+#define IMX_SC_R_V2X_MU_1		27
 #define IMX_SC_R_DC_0_VIDEO0		28
 #define IMX_SC_R_DC_0_VIDEO1		29
 #define IMX_SC_R_DC_0_FRAC0		30
+#define IMX_SC_R_V2X_MU_2		31
 #define IMX_SC_R_DC_0			32
 #define IMX_SC_R_GPU_2_PID0		33
 #define IMX_SC_R_DC_0_PLL_0		34
@@ -49,11 +53,17 @@
 #define IMX_SC_R_DC_1_BLIT1		37
 #define IMX_SC_R_DC_1_BLIT2		38
 #define IMX_SC_R_DC_1_BLIT_OUT		39
+#define IMX_SC_R_V2X_MU_3		40
+#define IMX_SC_R_V2X_MU_4		41
 #define IMX_SC_R_DC_1_WARP		42
+#define IMX_SC_R_STM			43
+#define IMX_SC_R_SECVIO			44
 #define IMX_SC_R_DC_1_VIDEO0		45
 #define IMX_SC_R_DC_1_VIDEO1		46
 #define IMX_SC_R_DC_1_FRAC0		47
+#define IMX_SC_R_V2X			48
 #define IMX_SC_R_DC_1			49
+#define IMX_SC_R_UNUSED14		50
 #define IMX_SC_R_DC_1_PLL_0		51
 #define IMX_SC_R_DC_1_PLL_1		52
 #define IMX_SC_R_SPI_0			53
@@ -144,10 +154,10 @@
 #define IMX_SC_R_DMA_1_CH29		137
 #define IMX_SC_R_DMA_1_CH30		138
 #define IMX_SC_R_DMA_1_CH31		139
-#define IMX_SC_R_UNUSED1		140
-#define IMX_SC_R_UNUSED2		141
-#define IMX_SC_R_UNUSED3		142
-#define IMX_SC_R_UNUSED4		143
+#define IMX_SC_R_V2X_PID0		140
+#define IMX_SC_R_V2X_PID1		141
+#define IMX_SC_R_V2X_PID2		142
+#define IMX_SC_R_V2X_PID3		143
 #define IMX_SC_R_GPU_0_PID0		144
 #define IMX_SC_R_GPU_0_PID1		145
 #define IMX_SC_R_GPU_0_PID2		146
@@ -176,7 +186,7 @@
 #define IMX_SC_R_PCIE_B			169
 #define IMX_SC_R_SATA_0			170
 #define IMX_SC_R_SERDES_1		171
-#define IMX_SC_R_HSIO_GPIO		172
+#define IMX_SC_R_HSIO_GPIO_0		172
 #define IMX_SC_R_MATCH_15		173
 #define IMX_SC_R_MATCH_16		174
 #define IMX_SC_R_MATCH_17		175
@@ -243,15 +253,15 @@
 #define IMX_SC_R_ROM_0			236
 #define IMX_SC_R_FSPI_0			237
 #define IMX_SC_R_FSPI_1			238
-#define IMX_SC_R_IEE			239
-#define IMX_SC_R_IEE_R0			240
-#define IMX_SC_R_IEE_R1			241
-#define IMX_SC_R_IEE_R2			242
-#define IMX_SC_R_IEE_R3			243
-#define IMX_SC_R_IEE_R4			244
-#define IMX_SC_R_IEE_R5			245
-#define IMX_SC_R_IEE_R6			246
-#define IMX_SC_R_IEE_R7			247
+#define IMX_SC_R_IEE_0			239
+#define IMX_SC_R_IEE_0_R0		240
+#define IMX_SC_R_IEE_0_R1		241
+#define IMX_SC_R_IEE_0_R2		242
+#define IMX_SC_R_IEE_0_R3		243
+#define IMX_SC_R_IEE_0_R4		244
+#define IMX_SC_R_IEE_0_R5		245
+#define IMX_SC_R_IEE_0_R6		246
+#define IMX_SC_R_IEE_0_R7		247
 #define IMX_SC_R_SDHC_0			248
 #define IMX_SC_R_SDHC_1			249
 #define IMX_SC_R_SDHC_2			250
@@ -282,46 +292,50 @@
 #define IMX_SC_R_LVDS_2_PWM_0		275
 #define IMX_SC_R_LVDS_2_I2C_0		276
 #define IMX_SC_R_LVDS_2_I2C_1		277
-#define IMX_SC_R_M4_0_PID0		278
-#define IMX_SC_R_M4_0_PID1		279
-#define IMX_SC_R_M4_0_PID2		280
-#define IMX_SC_R_M4_0_PID3		281
-#define IMX_SC_R_M4_0_PID4		282
-#define IMX_SC_R_M4_0_RGPIO		283
-#define IMX_SC_R_M4_0_SEMA42		284
-#define IMX_SC_R_M4_0_TPM		285
-#define IMX_SC_R_M4_0_PIT		286
-#define IMX_SC_R_M4_0_UART		287
-#define IMX_SC_R_M4_0_I2C		288
-#define IMX_SC_R_M4_0_INTMUX		289
-#define IMX_SC_R_M4_0_MU_0B		292
-#define IMX_SC_R_M4_0_MU_0A0		293
-#define IMX_SC_R_M4_0_MU_0A1		294
-#define IMX_SC_R_M4_0_MU_0A2		295
-#define IMX_SC_R_M4_0_MU_0A3		296
-#define IMX_SC_R_M4_0_MU_1A		297
-#define IMX_SC_R_M4_1_PID0		298
-#define IMX_SC_R_M4_1_PID1		299
-#define IMX_SC_R_M4_1_PID2		300
-#define IMX_SC_R_M4_1_PID3		301
-#define IMX_SC_R_M4_1_PID4		302
-#define IMX_SC_R_M4_1_RGPIO		303
-#define IMX_SC_R_M4_1_SEMA42		304
-#define IMX_SC_R_M4_1_TPM		305
-#define IMX_SC_R_M4_1_PIT		306
-#define IMX_SC_R_M4_1_UART		307
-#define IMX_SC_R_M4_1_I2C		308
-#define IMX_SC_R_M4_1_INTMUX		309
-#define IMX_SC_R_M4_1_MU_0B		312
-#define IMX_SC_R_M4_1_MU_0A0		313
-#define IMX_SC_R_M4_1_MU_0A1		314
-#define IMX_SC_R_M4_1_MU_0A2		315
-#define IMX_SC_R_M4_1_MU_0A3		316
-#define IMX_SC_R_M4_1_MU_1A		317
+#define IMX_SC_R_MCU_0_PID0		278
+#define IMX_SC_R_MCU_0_PID1		279
+#define IMX_SC_R_MCU_0_PID2		280
+#define IMX_SC_R_MCU_0_PID3		281
+#define IMX_SC_R_MCU_0_PID4		282
+#define IMX_SC_R_MCU_0_RGPIO		283
+#define IMX_SC_R_MCU_0_SEMA42		284
+#define IMX_SC_R_MCU_0_TPM		285
+#define IMX_SC_R_MCU_0_PIT		286
+#define IMX_SC_R_MCU_0_UART		287
+#define IMX_SC_R_MCU_0_I2C		288
+#define IMX_SC_R_MCU_0_INTMUX		289
+#define IMX_SC_R_ENET_0_A0		290
+#define IMX_SC_R_ENET_0_A1		291
+#define IMX_SC_R_MCU_0_MU_0B		292
+#define IMX_SC_R_MCU_0_MU_0A0		293
+#define IMX_SC_R_MCU_0_MU_0A1		294
+#define IMX_SC_R_MCU_0_MU_0A2		295
+#define IMX_SC_R_MCU_0_MU_0A3		296
+#define IMX_SC_R_MCU_0_MU_1A		297
+#define IMX_SC_R_MCU_1_PID0		298
+#define IMX_SC_R_MCU_1_PID1		299
+#define IMX_SC_R_MCU_1_PID2		300
+#define IMX_SC_R_MCU_1_PID3		301
+#define IMX_SC_R_MCU_1_PID4		302
+#define IMX_SC_R_MCU_1_RGPIO		303
+#define IMX_SC_R_MCU_1_SEMA42		304
+#define IMX_SC_R_MCU_1_TPM		305
+#define IMX_SC_R_MCU_1_PIT		306
+#define IMX_SC_R_MCU_1_UART		307
+#define IMX_SC_R_MCU_1_I2C		308
+#define IMX_SC_R_MCU_1_INTMUX		309
+#define IMX_SC_R_UNUSED17		310
+#define IMX_SC_R_UNUSED18		311
+#define IMX_SC_R_MCU_1_MU_0B		312
+#define IMX_SC_R_MCU_1_MU_0A0		313
+#define IMX_SC_R_MCU_1_MU_0A1		314
+#define IMX_SC_R_MCU_1_MU_0A2		315
+#define IMX_SC_R_MCU_1_MU_0A3		316
+#define IMX_SC_R_MCU_1_MU_1A		317
 #define IMX_SC_R_SAI_0			318
 #define IMX_SC_R_SAI_1			319
 #define IMX_SC_R_SAI_2			320
-#define IMX_SC_R_IRQSTR_SCU2		321
+#define IMX_SC_R_IRQSTR_AP_0		321
 #define IMX_SC_R_IRQSTR_DSP		322
 #define IMX_SC_R_ELCDIF_PLL		323
 #define IMX_SC_R_OCRAM			324
@@ -366,33 +380,33 @@
 #define IMX_SC_R_VPU_PID5		363
 #define IMX_SC_R_VPU_PID6		364
 #define IMX_SC_R_VPU_PID7		365
-#define IMX_SC_R_VPU_UART		366
-#define IMX_SC_R_VPUCORE		367
-#define IMX_SC_R_VPUCORE_0		368
-#define IMX_SC_R_VPUCORE_1		369
-#define IMX_SC_R_VPUCORE_2		370
-#define IMX_SC_R_VPUCORE_3		371
+#define IMX_SC_R_ENET_0_A2		366
+#define IMX_SC_R_ENET_1_A0		367
+#define IMX_SC_R_ENET_1_A1		368
+#define IMX_SC_R_ENET_1_A2		369
+#define IMX_SC_R_ENET_1_A3		370
+#define IMX_SC_R_ENET_1_A4		371
 #define IMX_SC_R_DMA_4_CH0		372
 #define IMX_SC_R_DMA_4_CH1		373
 #define IMX_SC_R_DMA_4_CH2		374
 #define IMX_SC_R_DMA_4_CH3		375
 #define IMX_SC_R_DMA_4_CH4		376
-#define IMX_SC_R_ISI_CH0		377
-#define IMX_SC_R_ISI_CH1		378
-#define IMX_SC_R_ISI_CH2		379
-#define IMX_SC_R_ISI_CH3		380
-#define IMX_SC_R_ISI_CH4		381
-#define IMX_SC_R_ISI_CH5		382
-#define IMX_SC_R_ISI_CH6		383
-#define IMX_SC_R_ISI_CH7		384
-#define IMX_SC_R_MJPEG_DEC_S0		385
-#define IMX_SC_R_MJPEG_DEC_S1		386
-#define IMX_SC_R_MJPEG_DEC_S2		387
-#define IMX_SC_R_MJPEG_DEC_S3		388
-#define IMX_SC_R_MJPEG_ENC_S0		389
-#define IMX_SC_R_MJPEG_ENC_S1		390
-#define IMX_SC_R_MJPEG_ENC_S2		391
-#define IMX_SC_R_MJPEG_ENC_S3		392
+#define IMX_SC_R_ISI_0_CH0		377
+#define IMX_SC_R_ISI_0_CH1		378
+#define IMX_SC_R_ISI_0_CH2		379
+#define IMX_SC_R_ISI_0_CH3		380
+#define IMX_SC_R_ISI_0_CH4		381
+#define IMX_SC_R_ISI_0_CH5		382
+#define IMX_SC_R_ISI_0_CH6		383
+#define IMX_SC_R_ISI_0_CH7		384
+#define IMX_SC_R_MJPEG_0_DEC_S0		385
+#define IMX_SC_R_MJPEG_0_DEC_S1		386
+#define IMX_SC_R_MJPEG_0_DEC_S2		387
+#define IMX_SC_R_MJPEG_0_DEC_S3		388
+#define IMX_SC_R_MJPEG_0_ENC_S0		389
+#define IMX_SC_R_MJPEG_0_ENC_S1		390
+#define IMX_SC_R_MJPEG_0_ENC_S2		391
+#define IMX_SC_R_MJPEG_0_ENC_S3		392
 #define IMX_SC_R_MIPI_0			393
 #define IMX_SC_R_MIPI_0_PWM_0		394
 #define IMX_SC_R_MIPI_0_I2C_0		395
@@ -507,11 +521,11 @@
 #define IMX_SC_R_SECO_MU_3		504
 #define IMX_SC_R_SECO_MU_4		505
 #define IMX_SC_R_HDMI_RX_PWM_0		506
-#define IMX_SC_R_A35			507
-#define IMX_SC_R_A35_0			508
-#define IMX_SC_R_A35_1			509
-#define IMX_SC_R_A35_2			510
-#define IMX_SC_R_A35_3			511
+#define IMX_SC_R_AP_2			507
+#define IMX_SC_R_AP_2_0			508
+#define IMX_SC_R_AP_2_1			509
+#define IMX_SC_R_AP_2_2			510
+#define IMX_SC_R_AP_2_3			511
 #define IMX_SC_R_DSP			512
 #define IMX_SC_R_DSP_RAM		513
 #define IMX_SC_R_CAAM_JR1_OUT		514
@@ -532,8 +546,8 @@
 #define IMX_SC_R_BOARD_R5		529
 #define IMX_SC_R_BOARD_R6		530
 #define IMX_SC_R_BOARD_R7		531
-#define IMX_SC_R_MJPEG_DEC_MP		532
-#define IMX_SC_R_MJPEG_ENC_MP		533
+#define IMX_SC_R_MJPEG_0_DEC_MP		532
+#define IMX_SC_R_MJPEG_0_ENC_MP		533
 #define IMX_SC_R_VPU_TS_0		534
 #define IMX_SC_R_VPU_MU_0		535
 #define IMX_SC_R_VPU_MU_1		536
@@ -565,6 +579,95 @@
 #define IMX_SC_PM_CLK_PLL		4	/* PLL */
 #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
 
+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35			IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53			IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
+#define IMX_SC_R_A72			IMX_SC_R_AP_1
+#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
+#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
+#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
+#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
+#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
+#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
+#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
+#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
+#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
+#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
+#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
+#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
+#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
+#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
+#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
+#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
+#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
+#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
+#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
+#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
+#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
+#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
+#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
+#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
+#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
+#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
+#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
+#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
+#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
+#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
+#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
+#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
+#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
+#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
+#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
+#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
+#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
+#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
+#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
+#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
+#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
+#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
+#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
+#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
+#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
+#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
+#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
+#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
+#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
+#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
+#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
+#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
+#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
+#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
+#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
+#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
+#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
+#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
+#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
+#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
+#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
+#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
+#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
+#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
+#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
+#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
+#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
+#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
+#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
+#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
+#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
+#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
+#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
+#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
+
 /*
  * Defines for SC CONTROL
  */
@@ -630,6 +733,10 @@
 #define IMX_SC_C_INTF_SEL			59
 #define IMX_SC_C_RXC_DLY			60
 #define IMX_SC_C_TIMER_SEL			61
-#define IMX_SC_C_LAST				62
+#define IMX_SC_C_MISC0				62
+#define IMX_SC_C_MISC1				63
+#define IMX_SC_C_MISC2				64
+#define IMX_SC_C_MISC3				65
+#define IMX_SC_C_LAST				66
 
 #endif /* __DT_BINDINGS_RSCRC_IMX_H */
-- 
2.37.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH 2/2] clk: imx8qm: sync with SCFW kit v1.13.0
  2022-09-15 18:18 ` Viorel Suman (OSS)
@ 2022-09-15 18:18   ` Viorel Suman (OSS)
  -1 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

Sync clock resources with the latest available SCFW kit version 1.13.0,
may be found at the address below:

https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license

Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
---
 drivers/clk/imx/clk-imx8qm-rsrc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/clk/imx/clk-imx8qm-rsrc.c b/drivers/clk/imx/clk-imx8qm-rsrc.c
index 87e0b6ac027e..ee01acd1a181 100644
--- a/drivers/clk/imx/clk-imx8qm-rsrc.c
+++ b/drivers/clk/imx/clk-imx8qm-rsrc.c
@@ -79,8 +79,8 @@ static const u32 imx8qm_clk_scu_rsrc_table[] = {
 	IMX_SC_R_M4_0_I2C,
 	IMX_SC_R_M4_1_I2C,
 	IMX_SC_R_AUDIO_PLL_0,
-	IMX_SC_R_VPU_UART,
-	IMX_SC_R_VPUCORE,
+	IMX_SC_R_ENET_0_A2,
+	IMX_SC_R_ENET_1_A0,
 	IMX_SC_R_MIPI_0,
 	IMX_SC_R_MIPI_0_PWM_0,
 	IMX_SC_R_MIPI_0_I2C_0,
-- 
2.37.2


^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH 2/2] clk: imx8qm: sync with SCFW kit v1.13.0
@ 2022-09-15 18:18   ` Viorel Suman (OSS)
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman (OSS) @ 2022-09-15 18:18 UTC (permalink / raw)
  To: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

From: Viorel Suman <viorel.suman@nxp.com>

Sync clock resources with the latest available SCFW kit version 1.13.0,
may be found at the address below:

https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license

Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
---
 drivers/clk/imx/clk-imx8qm-rsrc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/clk/imx/clk-imx8qm-rsrc.c b/drivers/clk/imx/clk-imx8qm-rsrc.c
index 87e0b6ac027e..ee01acd1a181 100644
--- a/drivers/clk/imx/clk-imx8qm-rsrc.c
+++ b/drivers/clk/imx/clk-imx8qm-rsrc.c
@@ -79,8 +79,8 @@ static const u32 imx8qm_clk_scu_rsrc_table[] = {
 	IMX_SC_R_M4_0_I2C,
 	IMX_SC_R_M4_1_I2C,
 	IMX_SC_R_AUDIO_PLL_0,
-	IMX_SC_R_VPU_UART,
-	IMX_SC_R_VPUCORE,
+	IMX_SC_R_ENET_0_A2,
+	IMX_SC_R_ENET_1_A0,
 	IMX_SC_R_MIPI_0,
 	IMX_SC_R_MIPI_0_PWM_0,
 	IMX_SC_R_MIPI_0_I2C_0,
-- 
2.37.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-15 18:18   ` Viorel Suman (OSS)
@ 2022-09-16  6:30     ` Alexander Stein
  -1 siblings, 0 replies; 20+ messages in thread
From: Alexander Stein @ 2022-09-16  6:30 UTC (permalink / raw)
  To: Viorel Suman (OSS)
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Am Donnerstag, 15. September 2022, 20:18:04 CEST schrieb Viorel Suman (OSS):
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&ap
> pType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)

This is not bisectable and breaks compilation, as this patch removes symbols 
which are still used in drivers/clk/imx/clk-imx8qm-rsrc.c (addressed in 2nd 
patch). IMHO this series should be squashed into one patch.

Best regards,
Alexander

> diff --git a/include/dt-bindings/firmware/imx/rsrc.h
> b/include/dt-bindings/firmware/imx/rsrc.h index 43885056557c..a4c68f394986
> 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).
>   */
> 
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1
> -#define IMX_SC_R_A53_1			2
> -#define IMX_SC_R_A53_2			3
> -#define IMX_SC_R_A53_3			4
> -#define IMX_SC_R_A72			5
> -#define IMX_SC_R_A72_0			6
> -#define IMX_SC_R_A72_1			7
> -#define IMX_SC_R_A72_2			8
> -#define IMX_SC_R_A72_3			9
> +#define IMX_SC_R_AP_0			0
> +#define IMX_SC_R_AP_0_0			1
> +#define IMX_SC_R_AP_0_1			2
> +#define IMX_SC_R_AP_0_2			3
> +#define IMX_SC_R_AP_0_3			4
> +#define IMX_SC_R_AP_1			5
> +#define IMX_SC_R_AP_1_0			6
> +#define IMX_SC_R_AP_1_1			7
> +#define IMX_SC_R_AP_1_2			8
> +#define IMX_SC_R_AP_1_3			9
>  #define IMX_SC_R_CCI			10
>  #define IMX_SC_R_DB			11
>  #define IMX_SC_R_DRC_0			12
>  #define IMX_SC_R_DRC_1			13
>  #define IMX_SC_R_GIC_SMMU		14
> -#define IMX_SC_R_IRQSTR_M4_0		15
> -#define IMX_SC_R_IRQSTR_M4_1		16
> -#define IMX_SC_R_SMMU			17
> -#define IMX_SC_R_GIC			18
> +#define IMX_SC_R_IRQSTR_MCU_0		15
> +#define IMX_SC_R_IRQSTR_MCU_1		16
> +#define IMX_SC_R_SMMU_0			17
> +#define IMX_SC_R_GIC_0			18
>  #define IMX_SC_R_DC_0_BLIT0		19
>  #define IMX_SC_R_DC_0_BLIT1		20
>  #define IMX_SC_R_DC_0_BLIT2		21
>  #define IMX_SC_R_DC_0_BLIT_OUT		22
> -#define IMX_SC_R_PERF			23
> +#define IMX_SC_R_PERF_0			23
> +#define IMX_SC_R_USB_1_PHY		24
>  #define IMX_SC_R_DC_0_WARP		25
> +#define IMX_SC_R_V2X_MU_0		26
> +#define IMX_SC_R_V2X_MU_1		27
>  #define IMX_SC_R_DC_0_VIDEO0		28
>  #define IMX_SC_R_DC_0_VIDEO1		29
>  #define IMX_SC_R_DC_0_FRAC0		30
> +#define IMX_SC_R_V2X_MU_2		31
>  #define IMX_SC_R_DC_0			32
>  #define IMX_SC_R_GPU_2_PID0		33
>  #define IMX_SC_R_DC_0_PLL_0		34
> @@ -49,11 +53,17 @@
>  #define IMX_SC_R_DC_1_BLIT1		37
>  #define IMX_SC_R_DC_1_BLIT2		38
>  #define IMX_SC_R_DC_1_BLIT_OUT		39
> +#define IMX_SC_R_V2X_MU_3		40
> +#define IMX_SC_R_V2X_MU_4		41
>  #define IMX_SC_R_DC_1_WARP		42
> +#define IMX_SC_R_STM			43
> +#define IMX_SC_R_SECVIO			44
>  #define IMX_SC_R_DC_1_VIDEO0		45
>  #define IMX_SC_R_DC_1_VIDEO1		46
>  #define IMX_SC_R_DC_1_FRAC0		47
> +#define IMX_SC_R_V2X			48
>  #define IMX_SC_R_DC_1			49
> +#define IMX_SC_R_UNUSED14		50
>  #define IMX_SC_R_DC_1_PLL_0		51
>  #define IMX_SC_R_DC_1_PLL_1		52
>  #define IMX_SC_R_SPI_0			53
> @@ -144,10 +154,10 @@
>  #define IMX_SC_R_DMA_1_CH29		137
>  #define IMX_SC_R_DMA_1_CH30		138
>  #define IMX_SC_R_DMA_1_CH31		139
> -#define IMX_SC_R_UNUSED1		140
> -#define IMX_SC_R_UNUSED2		141
> -#define IMX_SC_R_UNUSED3		142
> -#define IMX_SC_R_UNUSED4		143
> +#define IMX_SC_R_V2X_PID0		140
> +#define IMX_SC_R_V2X_PID1		141
> +#define IMX_SC_R_V2X_PID2		142
> +#define IMX_SC_R_V2X_PID3		143
>  #define IMX_SC_R_GPU_0_PID0		144
>  #define IMX_SC_R_GPU_0_PID1		145
>  #define IMX_SC_R_GPU_0_PID2		146
> @@ -176,7 +186,7 @@
>  #define IMX_SC_R_PCIE_B			169
>  #define IMX_SC_R_SATA_0			170
>  #define IMX_SC_R_SERDES_1		171
> -#define IMX_SC_R_HSIO_GPIO		172
> +#define IMX_SC_R_HSIO_GPIO_0		172
>  #define IMX_SC_R_MATCH_15		173
>  #define IMX_SC_R_MATCH_16		174
>  #define IMX_SC_R_MATCH_17		175
> @@ -243,15 +253,15 @@
>  #define IMX_SC_R_ROM_0			236
>  #define IMX_SC_R_FSPI_0			237
>  #define IMX_SC_R_FSPI_1			238
> -#define IMX_SC_R_IEE			239
> -#define IMX_SC_R_IEE_R0			240
> -#define IMX_SC_R_IEE_R1			241
> -#define IMX_SC_R_IEE_R2			242
> -#define IMX_SC_R_IEE_R3			243
> -#define IMX_SC_R_IEE_R4			244
> -#define IMX_SC_R_IEE_R5			245
> -#define IMX_SC_R_IEE_R6			246
> -#define IMX_SC_R_IEE_R7			247
> +#define IMX_SC_R_IEE_0			239
> +#define IMX_SC_R_IEE_0_R0		240
> +#define IMX_SC_R_IEE_0_R1		241
> +#define IMX_SC_R_IEE_0_R2		242
> +#define IMX_SC_R_IEE_0_R3		243
> +#define IMX_SC_R_IEE_0_R4		244
> +#define IMX_SC_R_IEE_0_R5		245
> +#define IMX_SC_R_IEE_0_R6		246
> +#define IMX_SC_R_IEE_0_R7		247
>  #define IMX_SC_R_SDHC_0			248
>  #define IMX_SC_R_SDHC_1			249
>  #define IMX_SC_R_SDHC_2			250
> @@ -282,46 +292,50 @@
>  #define IMX_SC_R_LVDS_2_PWM_0		275
>  #define IMX_SC_R_LVDS_2_I2C_0		276
>  #define IMX_SC_R_LVDS_2_I2C_1		277
> -#define IMX_SC_R_M4_0_PID0		278
> -#define IMX_SC_R_M4_0_PID1		279
> -#define IMX_SC_R_M4_0_PID2		280
> -#define IMX_SC_R_M4_0_PID3		281
> -#define IMX_SC_R_M4_0_PID4		282
> -#define IMX_SC_R_M4_0_RGPIO		283
> -#define IMX_SC_R_M4_0_SEMA42		284
> -#define IMX_SC_R_M4_0_TPM		285
> -#define IMX_SC_R_M4_0_PIT		286
> -#define IMX_SC_R_M4_0_UART		287
> -#define IMX_SC_R_M4_0_I2C		288
> -#define IMX_SC_R_M4_0_INTMUX		289
> -#define IMX_SC_R_M4_0_MU_0B		292
> -#define IMX_SC_R_M4_0_MU_0A0		293
> -#define IMX_SC_R_M4_0_MU_0A1		294
> -#define IMX_SC_R_M4_0_MU_0A2		295
> -#define IMX_SC_R_M4_0_MU_0A3		296
> -#define IMX_SC_R_M4_0_MU_1A		297
> -#define IMX_SC_R_M4_1_PID0		298
> -#define IMX_SC_R_M4_1_PID1		299
> -#define IMX_SC_R_M4_1_PID2		300
> -#define IMX_SC_R_M4_1_PID3		301
> -#define IMX_SC_R_M4_1_PID4		302
> -#define IMX_SC_R_M4_1_RGPIO		303
> -#define IMX_SC_R_M4_1_SEMA42		304
> -#define IMX_SC_R_M4_1_TPM		305
> -#define IMX_SC_R_M4_1_PIT		306
> -#define IMX_SC_R_M4_1_UART		307
> -#define IMX_SC_R_M4_1_I2C		308
> -#define IMX_SC_R_M4_1_INTMUX		309
> -#define IMX_SC_R_M4_1_MU_0B		312
> -#define IMX_SC_R_M4_1_MU_0A0		313
> -#define IMX_SC_R_M4_1_MU_0A1		314
> -#define IMX_SC_R_M4_1_MU_0A2		315
> -#define IMX_SC_R_M4_1_MU_0A3		316
> -#define IMX_SC_R_M4_1_MU_1A		317
> +#define IMX_SC_R_MCU_0_PID0		278
> +#define IMX_SC_R_MCU_0_PID1		279
> +#define IMX_SC_R_MCU_0_PID2		280
> +#define IMX_SC_R_MCU_0_PID3		281
> +#define IMX_SC_R_MCU_0_PID4		282
> +#define IMX_SC_R_MCU_0_RGPIO		283
> +#define IMX_SC_R_MCU_0_SEMA42		284
> +#define IMX_SC_R_MCU_0_TPM		285
> +#define IMX_SC_R_MCU_0_PIT		286
> +#define IMX_SC_R_MCU_0_UART		287
> +#define IMX_SC_R_MCU_0_I2C		288
> +#define IMX_SC_R_MCU_0_INTMUX		289
> +#define IMX_SC_R_ENET_0_A0		290
> +#define IMX_SC_R_ENET_0_A1		291
> +#define IMX_SC_R_MCU_0_MU_0B		292
> +#define IMX_SC_R_MCU_0_MU_0A0		293
> +#define IMX_SC_R_MCU_0_MU_0A1		294
> +#define IMX_SC_R_MCU_0_MU_0A2		295
> +#define IMX_SC_R_MCU_0_MU_0A3		296
> +#define IMX_SC_R_MCU_0_MU_1A		297
> +#define IMX_SC_R_MCU_1_PID0		298
> +#define IMX_SC_R_MCU_1_PID1		299
> +#define IMX_SC_R_MCU_1_PID2		300
> +#define IMX_SC_R_MCU_1_PID3		301
> +#define IMX_SC_R_MCU_1_PID4		302
> +#define IMX_SC_R_MCU_1_RGPIO		303
> +#define IMX_SC_R_MCU_1_SEMA42		304
> +#define IMX_SC_R_MCU_1_TPM		305
> +#define IMX_SC_R_MCU_1_PIT		306
> +#define IMX_SC_R_MCU_1_UART		307
> +#define IMX_SC_R_MCU_1_I2C		308
> +#define IMX_SC_R_MCU_1_INTMUX		309
> +#define IMX_SC_R_UNUSED17		310
> +#define IMX_SC_R_UNUSED18		311
> +#define IMX_SC_R_MCU_1_MU_0B		312
> +#define IMX_SC_R_MCU_1_MU_0A0		313
> +#define IMX_SC_R_MCU_1_MU_0A1		314
> +#define IMX_SC_R_MCU_1_MU_0A2		315
> +#define IMX_SC_R_MCU_1_MU_0A3		316
> +#define IMX_SC_R_MCU_1_MU_1A		317
>  #define IMX_SC_R_SAI_0			318
>  #define IMX_SC_R_SAI_1			319
>  #define IMX_SC_R_SAI_2			320
> -#define IMX_SC_R_IRQSTR_SCU2		321
> +#define IMX_SC_R_IRQSTR_AP_0		321
>  #define IMX_SC_R_IRQSTR_DSP		322
>  #define IMX_SC_R_ELCDIF_PLL		323
>  #define IMX_SC_R_OCRAM			324
> @@ -366,33 +380,33 @@
>  #define IMX_SC_R_VPU_PID5		363
>  #define IMX_SC_R_VPU_PID6		364
>  #define IMX_SC_R_VPU_PID7		365
> -#define IMX_SC_R_VPU_UART		366
> -#define IMX_SC_R_VPUCORE		367
> -#define IMX_SC_R_VPUCORE_0		368
> -#define IMX_SC_R_VPUCORE_1		369
> -#define IMX_SC_R_VPUCORE_2		370
> -#define IMX_SC_R_VPUCORE_3		371
> +#define IMX_SC_R_ENET_0_A2		366
> +#define IMX_SC_R_ENET_1_A0		367
> +#define IMX_SC_R_ENET_1_A1		368
> +#define IMX_SC_R_ENET_1_A2		369
> +#define IMX_SC_R_ENET_1_A3		370
> +#define IMX_SC_R_ENET_1_A4		371
>  #define IMX_SC_R_DMA_4_CH0		372
>  #define IMX_SC_R_DMA_4_CH1		373
>  #define IMX_SC_R_DMA_4_CH2		374
>  #define IMX_SC_R_DMA_4_CH3		375
>  #define IMX_SC_R_DMA_4_CH4		376
> -#define IMX_SC_R_ISI_CH0		377
> -#define IMX_SC_R_ISI_CH1		378
> -#define IMX_SC_R_ISI_CH2		379
> -#define IMX_SC_R_ISI_CH3		380
> -#define IMX_SC_R_ISI_CH4		381
> -#define IMX_SC_R_ISI_CH5		382
> -#define IMX_SC_R_ISI_CH6		383
> -#define IMX_SC_R_ISI_CH7		384
> -#define IMX_SC_R_MJPEG_DEC_S0		385
> -#define IMX_SC_R_MJPEG_DEC_S1		386
> -#define IMX_SC_R_MJPEG_DEC_S2		387
> -#define IMX_SC_R_MJPEG_DEC_S3		388
> -#define IMX_SC_R_MJPEG_ENC_S0		389
> -#define IMX_SC_R_MJPEG_ENC_S1		390
> -#define IMX_SC_R_MJPEG_ENC_S2		391
> -#define IMX_SC_R_MJPEG_ENC_S3		392
> +#define IMX_SC_R_ISI_0_CH0		377
> +#define IMX_SC_R_ISI_0_CH1		378
> +#define IMX_SC_R_ISI_0_CH2		379
> +#define IMX_SC_R_ISI_0_CH3		380
> +#define IMX_SC_R_ISI_0_CH4		381
> +#define IMX_SC_R_ISI_0_CH5		382
> +#define IMX_SC_R_ISI_0_CH6		383
> +#define IMX_SC_R_ISI_0_CH7		384
> +#define IMX_SC_R_MJPEG_0_DEC_S0		385
> +#define IMX_SC_R_MJPEG_0_DEC_S1		386
> +#define IMX_SC_R_MJPEG_0_DEC_S2		387
> +#define IMX_SC_R_MJPEG_0_DEC_S3		388
> +#define IMX_SC_R_MJPEG_0_ENC_S0		389
> +#define IMX_SC_R_MJPEG_0_ENC_S1		390
> +#define IMX_SC_R_MJPEG_0_ENC_S2		391
> +#define IMX_SC_R_MJPEG_0_ENC_S3		392
>  #define IMX_SC_R_MIPI_0			393
>  #define IMX_SC_R_MIPI_0_PWM_0		394
>  #define IMX_SC_R_MIPI_0_I2C_0		395
> @@ -507,11 +521,11 @@
>  #define IMX_SC_R_SECO_MU_3		504
>  #define IMX_SC_R_SECO_MU_4		505
>  #define IMX_SC_R_HDMI_RX_PWM_0		506
> -#define IMX_SC_R_A35			507
> -#define IMX_SC_R_A35_0			508
> -#define IMX_SC_R_A35_1			509
> -#define IMX_SC_R_A35_2			510
> -#define IMX_SC_R_A35_3			511
> +#define IMX_SC_R_AP_2			507
> +#define IMX_SC_R_AP_2_0			508
> +#define IMX_SC_R_AP_2_1			509
> +#define IMX_SC_R_AP_2_2			510
> +#define IMX_SC_R_AP_2_3			511
>  #define IMX_SC_R_DSP			512
>  #define IMX_SC_R_DSP_RAM		513
>  #define IMX_SC_R_CAAM_JR1_OUT		514
> @@ -532,8 +546,8 @@
>  #define IMX_SC_R_BOARD_R5		529
>  #define IMX_SC_R_BOARD_R6		530
>  #define IMX_SC_R_BOARD_R7		531
> -#define IMX_SC_R_MJPEG_DEC_MP		532
> -#define IMX_SC_R_MJPEG_ENC_MP		533
> +#define IMX_SC_R_MJPEG_0_DEC_MP		532
> +#define IMX_SC_R_MJPEG_0_ENC_MP		533
>  #define IMX_SC_R_VPU_TS_0		534
>  #define IMX_SC_R_VPU_MU_0		535
>  #define IMX_SC_R_VPU_MU_1		536
> @@ -565,6 +579,95 @@
>  #define IMX_SC_PM_CLK_PLL		4	/* PLL */
>  #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
> 
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35			IMX_SC_R_AP_2
> +#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
> +#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
> +#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
> +#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
> +#define IMX_SC_R_A53			IMX_SC_R_AP_0
> +#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
> +#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
> +#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
> +#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
> +#define IMX_SC_R_A72			IMX_SC_R_AP_1
> +#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
> +#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
> +#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
> +#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
> +#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
> +#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
> +#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
> +#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
> +#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
> +#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
> +#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
> +#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
> +#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
> +#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
> +#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
> +#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
> +#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
> +#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
> +#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
> +#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
> +#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
> +#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
> +#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
> +#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
> +#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
> +#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
> +#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
> +#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
> +#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
> +#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
> +#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
> +#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
> +#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
> +#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
> +#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
> +#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
> +#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
> +#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
> +#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
> +#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
> +#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
> +#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
> +#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
> +#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
> +#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
> +#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
> +#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
> +#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
> +#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
> +#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
> +#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
> +#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
> +#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
> +#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
> +#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
> +#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
> +#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
> +#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
> +#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
> +#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
> +#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
> +#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
> +#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
> +#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
> +#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
> +#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
> +#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
> +#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
> +#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
> +#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
> +#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
> +#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
> +#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
> +#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
> +
>  /*
>   * Defines for SC CONTROL
>   */
> @@ -630,6 +733,10 @@
>  #define IMX_SC_C_INTF_SEL			59
>  #define IMX_SC_C_RXC_DLY			60
>  #define IMX_SC_C_TIMER_SEL			61
> -#define IMX_SC_C_LAST				62
> +#define IMX_SC_C_MISC0				62
> +#define IMX_SC_C_MISC1				63
> +#define IMX_SC_C_MISC2				64
> +#define IMX_SC_C_MISC3				65
> +#define IMX_SC_C_LAST				66
> 
>  #endif /* __DT_BINDINGS_RSCRC_IMX_H */





^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16  6:30     ` Alexander Stein
  0 siblings, 0 replies; 20+ messages in thread
From: Alexander Stein @ 2022-09-16  6:30 UTC (permalink / raw)
  To: Viorel Suman (OSS)
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Am Donnerstag, 15. September 2022, 20:18:04 CEST schrieb Viorel Suman (OSS):
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&ap
> pType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)

This is not bisectable and breaks compilation, as this patch removes symbols 
which are still used in drivers/clk/imx/clk-imx8qm-rsrc.c (addressed in 2nd 
patch). IMHO this series should be squashed into one patch.

Best regards,
Alexander

> diff --git a/include/dt-bindings/firmware/imx/rsrc.h
> b/include/dt-bindings/firmware/imx/rsrc.h index 43885056557c..a4c68f394986
> 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).
>   */
> 
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1
> -#define IMX_SC_R_A53_1			2
> -#define IMX_SC_R_A53_2			3
> -#define IMX_SC_R_A53_3			4
> -#define IMX_SC_R_A72			5
> -#define IMX_SC_R_A72_0			6
> -#define IMX_SC_R_A72_1			7
> -#define IMX_SC_R_A72_2			8
> -#define IMX_SC_R_A72_3			9
> +#define IMX_SC_R_AP_0			0
> +#define IMX_SC_R_AP_0_0			1
> +#define IMX_SC_R_AP_0_1			2
> +#define IMX_SC_R_AP_0_2			3
> +#define IMX_SC_R_AP_0_3			4
> +#define IMX_SC_R_AP_1			5
> +#define IMX_SC_R_AP_1_0			6
> +#define IMX_SC_R_AP_1_1			7
> +#define IMX_SC_R_AP_1_2			8
> +#define IMX_SC_R_AP_1_3			9
>  #define IMX_SC_R_CCI			10
>  #define IMX_SC_R_DB			11
>  #define IMX_SC_R_DRC_0			12
>  #define IMX_SC_R_DRC_1			13
>  #define IMX_SC_R_GIC_SMMU		14
> -#define IMX_SC_R_IRQSTR_M4_0		15
> -#define IMX_SC_R_IRQSTR_M4_1		16
> -#define IMX_SC_R_SMMU			17
> -#define IMX_SC_R_GIC			18
> +#define IMX_SC_R_IRQSTR_MCU_0		15
> +#define IMX_SC_R_IRQSTR_MCU_1		16
> +#define IMX_SC_R_SMMU_0			17
> +#define IMX_SC_R_GIC_0			18
>  #define IMX_SC_R_DC_0_BLIT0		19
>  #define IMX_SC_R_DC_0_BLIT1		20
>  #define IMX_SC_R_DC_0_BLIT2		21
>  #define IMX_SC_R_DC_0_BLIT_OUT		22
> -#define IMX_SC_R_PERF			23
> +#define IMX_SC_R_PERF_0			23
> +#define IMX_SC_R_USB_1_PHY		24
>  #define IMX_SC_R_DC_0_WARP		25
> +#define IMX_SC_R_V2X_MU_0		26
> +#define IMX_SC_R_V2X_MU_1		27
>  #define IMX_SC_R_DC_0_VIDEO0		28
>  #define IMX_SC_R_DC_0_VIDEO1		29
>  #define IMX_SC_R_DC_0_FRAC0		30
> +#define IMX_SC_R_V2X_MU_2		31
>  #define IMX_SC_R_DC_0			32
>  #define IMX_SC_R_GPU_2_PID0		33
>  #define IMX_SC_R_DC_0_PLL_0		34
> @@ -49,11 +53,17 @@
>  #define IMX_SC_R_DC_1_BLIT1		37
>  #define IMX_SC_R_DC_1_BLIT2		38
>  #define IMX_SC_R_DC_1_BLIT_OUT		39
> +#define IMX_SC_R_V2X_MU_3		40
> +#define IMX_SC_R_V2X_MU_4		41
>  #define IMX_SC_R_DC_1_WARP		42
> +#define IMX_SC_R_STM			43
> +#define IMX_SC_R_SECVIO			44
>  #define IMX_SC_R_DC_1_VIDEO0		45
>  #define IMX_SC_R_DC_1_VIDEO1		46
>  #define IMX_SC_R_DC_1_FRAC0		47
> +#define IMX_SC_R_V2X			48
>  #define IMX_SC_R_DC_1			49
> +#define IMX_SC_R_UNUSED14		50
>  #define IMX_SC_R_DC_1_PLL_0		51
>  #define IMX_SC_R_DC_1_PLL_1		52
>  #define IMX_SC_R_SPI_0			53
> @@ -144,10 +154,10 @@
>  #define IMX_SC_R_DMA_1_CH29		137
>  #define IMX_SC_R_DMA_1_CH30		138
>  #define IMX_SC_R_DMA_1_CH31		139
> -#define IMX_SC_R_UNUSED1		140
> -#define IMX_SC_R_UNUSED2		141
> -#define IMX_SC_R_UNUSED3		142
> -#define IMX_SC_R_UNUSED4		143
> +#define IMX_SC_R_V2X_PID0		140
> +#define IMX_SC_R_V2X_PID1		141
> +#define IMX_SC_R_V2X_PID2		142
> +#define IMX_SC_R_V2X_PID3		143
>  #define IMX_SC_R_GPU_0_PID0		144
>  #define IMX_SC_R_GPU_0_PID1		145
>  #define IMX_SC_R_GPU_0_PID2		146
> @@ -176,7 +186,7 @@
>  #define IMX_SC_R_PCIE_B			169
>  #define IMX_SC_R_SATA_0			170
>  #define IMX_SC_R_SERDES_1		171
> -#define IMX_SC_R_HSIO_GPIO		172
> +#define IMX_SC_R_HSIO_GPIO_0		172
>  #define IMX_SC_R_MATCH_15		173
>  #define IMX_SC_R_MATCH_16		174
>  #define IMX_SC_R_MATCH_17		175
> @@ -243,15 +253,15 @@
>  #define IMX_SC_R_ROM_0			236
>  #define IMX_SC_R_FSPI_0			237
>  #define IMX_SC_R_FSPI_1			238
> -#define IMX_SC_R_IEE			239
> -#define IMX_SC_R_IEE_R0			240
> -#define IMX_SC_R_IEE_R1			241
> -#define IMX_SC_R_IEE_R2			242
> -#define IMX_SC_R_IEE_R3			243
> -#define IMX_SC_R_IEE_R4			244
> -#define IMX_SC_R_IEE_R5			245
> -#define IMX_SC_R_IEE_R6			246
> -#define IMX_SC_R_IEE_R7			247
> +#define IMX_SC_R_IEE_0			239
> +#define IMX_SC_R_IEE_0_R0		240
> +#define IMX_SC_R_IEE_0_R1		241
> +#define IMX_SC_R_IEE_0_R2		242
> +#define IMX_SC_R_IEE_0_R3		243
> +#define IMX_SC_R_IEE_0_R4		244
> +#define IMX_SC_R_IEE_0_R5		245
> +#define IMX_SC_R_IEE_0_R6		246
> +#define IMX_SC_R_IEE_0_R7		247
>  #define IMX_SC_R_SDHC_0			248
>  #define IMX_SC_R_SDHC_1			249
>  #define IMX_SC_R_SDHC_2			250
> @@ -282,46 +292,50 @@
>  #define IMX_SC_R_LVDS_2_PWM_0		275
>  #define IMX_SC_R_LVDS_2_I2C_0		276
>  #define IMX_SC_R_LVDS_2_I2C_1		277
> -#define IMX_SC_R_M4_0_PID0		278
> -#define IMX_SC_R_M4_0_PID1		279
> -#define IMX_SC_R_M4_0_PID2		280
> -#define IMX_SC_R_M4_0_PID3		281
> -#define IMX_SC_R_M4_0_PID4		282
> -#define IMX_SC_R_M4_0_RGPIO		283
> -#define IMX_SC_R_M4_0_SEMA42		284
> -#define IMX_SC_R_M4_0_TPM		285
> -#define IMX_SC_R_M4_0_PIT		286
> -#define IMX_SC_R_M4_0_UART		287
> -#define IMX_SC_R_M4_0_I2C		288
> -#define IMX_SC_R_M4_0_INTMUX		289
> -#define IMX_SC_R_M4_0_MU_0B		292
> -#define IMX_SC_R_M4_0_MU_0A0		293
> -#define IMX_SC_R_M4_0_MU_0A1		294
> -#define IMX_SC_R_M4_0_MU_0A2		295
> -#define IMX_SC_R_M4_0_MU_0A3		296
> -#define IMX_SC_R_M4_0_MU_1A		297
> -#define IMX_SC_R_M4_1_PID0		298
> -#define IMX_SC_R_M4_1_PID1		299
> -#define IMX_SC_R_M4_1_PID2		300
> -#define IMX_SC_R_M4_1_PID3		301
> -#define IMX_SC_R_M4_1_PID4		302
> -#define IMX_SC_R_M4_1_RGPIO		303
> -#define IMX_SC_R_M4_1_SEMA42		304
> -#define IMX_SC_R_M4_1_TPM		305
> -#define IMX_SC_R_M4_1_PIT		306
> -#define IMX_SC_R_M4_1_UART		307
> -#define IMX_SC_R_M4_1_I2C		308
> -#define IMX_SC_R_M4_1_INTMUX		309
> -#define IMX_SC_R_M4_1_MU_0B		312
> -#define IMX_SC_R_M4_1_MU_0A0		313
> -#define IMX_SC_R_M4_1_MU_0A1		314
> -#define IMX_SC_R_M4_1_MU_0A2		315
> -#define IMX_SC_R_M4_1_MU_0A3		316
> -#define IMX_SC_R_M4_1_MU_1A		317
> +#define IMX_SC_R_MCU_0_PID0		278
> +#define IMX_SC_R_MCU_0_PID1		279
> +#define IMX_SC_R_MCU_0_PID2		280
> +#define IMX_SC_R_MCU_0_PID3		281
> +#define IMX_SC_R_MCU_0_PID4		282
> +#define IMX_SC_R_MCU_0_RGPIO		283
> +#define IMX_SC_R_MCU_0_SEMA42		284
> +#define IMX_SC_R_MCU_0_TPM		285
> +#define IMX_SC_R_MCU_0_PIT		286
> +#define IMX_SC_R_MCU_0_UART		287
> +#define IMX_SC_R_MCU_0_I2C		288
> +#define IMX_SC_R_MCU_0_INTMUX		289
> +#define IMX_SC_R_ENET_0_A0		290
> +#define IMX_SC_R_ENET_0_A1		291
> +#define IMX_SC_R_MCU_0_MU_0B		292
> +#define IMX_SC_R_MCU_0_MU_0A0		293
> +#define IMX_SC_R_MCU_0_MU_0A1		294
> +#define IMX_SC_R_MCU_0_MU_0A2		295
> +#define IMX_SC_R_MCU_0_MU_0A3		296
> +#define IMX_SC_R_MCU_0_MU_1A		297
> +#define IMX_SC_R_MCU_1_PID0		298
> +#define IMX_SC_R_MCU_1_PID1		299
> +#define IMX_SC_R_MCU_1_PID2		300
> +#define IMX_SC_R_MCU_1_PID3		301
> +#define IMX_SC_R_MCU_1_PID4		302
> +#define IMX_SC_R_MCU_1_RGPIO		303
> +#define IMX_SC_R_MCU_1_SEMA42		304
> +#define IMX_SC_R_MCU_1_TPM		305
> +#define IMX_SC_R_MCU_1_PIT		306
> +#define IMX_SC_R_MCU_1_UART		307
> +#define IMX_SC_R_MCU_1_I2C		308
> +#define IMX_SC_R_MCU_1_INTMUX		309
> +#define IMX_SC_R_UNUSED17		310
> +#define IMX_SC_R_UNUSED18		311
> +#define IMX_SC_R_MCU_1_MU_0B		312
> +#define IMX_SC_R_MCU_1_MU_0A0		313
> +#define IMX_SC_R_MCU_1_MU_0A1		314
> +#define IMX_SC_R_MCU_1_MU_0A2		315
> +#define IMX_SC_R_MCU_1_MU_0A3		316
> +#define IMX_SC_R_MCU_1_MU_1A		317
>  #define IMX_SC_R_SAI_0			318
>  #define IMX_SC_R_SAI_1			319
>  #define IMX_SC_R_SAI_2			320
> -#define IMX_SC_R_IRQSTR_SCU2		321
> +#define IMX_SC_R_IRQSTR_AP_0		321
>  #define IMX_SC_R_IRQSTR_DSP		322
>  #define IMX_SC_R_ELCDIF_PLL		323
>  #define IMX_SC_R_OCRAM			324
> @@ -366,33 +380,33 @@
>  #define IMX_SC_R_VPU_PID5		363
>  #define IMX_SC_R_VPU_PID6		364
>  #define IMX_SC_R_VPU_PID7		365
> -#define IMX_SC_R_VPU_UART		366
> -#define IMX_SC_R_VPUCORE		367
> -#define IMX_SC_R_VPUCORE_0		368
> -#define IMX_SC_R_VPUCORE_1		369
> -#define IMX_SC_R_VPUCORE_2		370
> -#define IMX_SC_R_VPUCORE_3		371
> +#define IMX_SC_R_ENET_0_A2		366
> +#define IMX_SC_R_ENET_1_A0		367
> +#define IMX_SC_R_ENET_1_A1		368
> +#define IMX_SC_R_ENET_1_A2		369
> +#define IMX_SC_R_ENET_1_A3		370
> +#define IMX_SC_R_ENET_1_A4		371
>  #define IMX_SC_R_DMA_4_CH0		372
>  #define IMX_SC_R_DMA_4_CH1		373
>  #define IMX_SC_R_DMA_4_CH2		374
>  #define IMX_SC_R_DMA_4_CH3		375
>  #define IMX_SC_R_DMA_4_CH4		376
> -#define IMX_SC_R_ISI_CH0		377
> -#define IMX_SC_R_ISI_CH1		378
> -#define IMX_SC_R_ISI_CH2		379
> -#define IMX_SC_R_ISI_CH3		380
> -#define IMX_SC_R_ISI_CH4		381
> -#define IMX_SC_R_ISI_CH5		382
> -#define IMX_SC_R_ISI_CH6		383
> -#define IMX_SC_R_ISI_CH7		384
> -#define IMX_SC_R_MJPEG_DEC_S0		385
> -#define IMX_SC_R_MJPEG_DEC_S1		386
> -#define IMX_SC_R_MJPEG_DEC_S2		387
> -#define IMX_SC_R_MJPEG_DEC_S3		388
> -#define IMX_SC_R_MJPEG_ENC_S0		389
> -#define IMX_SC_R_MJPEG_ENC_S1		390
> -#define IMX_SC_R_MJPEG_ENC_S2		391
> -#define IMX_SC_R_MJPEG_ENC_S3		392
> +#define IMX_SC_R_ISI_0_CH0		377
> +#define IMX_SC_R_ISI_0_CH1		378
> +#define IMX_SC_R_ISI_0_CH2		379
> +#define IMX_SC_R_ISI_0_CH3		380
> +#define IMX_SC_R_ISI_0_CH4		381
> +#define IMX_SC_R_ISI_0_CH5		382
> +#define IMX_SC_R_ISI_0_CH6		383
> +#define IMX_SC_R_ISI_0_CH7		384
> +#define IMX_SC_R_MJPEG_0_DEC_S0		385
> +#define IMX_SC_R_MJPEG_0_DEC_S1		386
> +#define IMX_SC_R_MJPEG_0_DEC_S2		387
> +#define IMX_SC_R_MJPEG_0_DEC_S3		388
> +#define IMX_SC_R_MJPEG_0_ENC_S0		389
> +#define IMX_SC_R_MJPEG_0_ENC_S1		390
> +#define IMX_SC_R_MJPEG_0_ENC_S2		391
> +#define IMX_SC_R_MJPEG_0_ENC_S3		392
>  #define IMX_SC_R_MIPI_0			393
>  #define IMX_SC_R_MIPI_0_PWM_0		394
>  #define IMX_SC_R_MIPI_0_I2C_0		395
> @@ -507,11 +521,11 @@
>  #define IMX_SC_R_SECO_MU_3		504
>  #define IMX_SC_R_SECO_MU_4		505
>  #define IMX_SC_R_HDMI_RX_PWM_0		506
> -#define IMX_SC_R_A35			507
> -#define IMX_SC_R_A35_0			508
> -#define IMX_SC_R_A35_1			509
> -#define IMX_SC_R_A35_2			510
> -#define IMX_SC_R_A35_3			511
> +#define IMX_SC_R_AP_2			507
> +#define IMX_SC_R_AP_2_0			508
> +#define IMX_SC_R_AP_2_1			509
> +#define IMX_SC_R_AP_2_2			510
> +#define IMX_SC_R_AP_2_3			511
>  #define IMX_SC_R_DSP			512
>  #define IMX_SC_R_DSP_RAM		513
>  #define IMX_SC_R_CAAM_JR1_OUT		514
> @@ -532,8 +546,8 @@
>  #define IMX_SC_R_BOARD_R5		529
>  #define IMX_SC_R_BOARD_R6		530
>  #define IMX_SC_R_BOARD_R7		531
> -#define IMX_SC_R_MJPEG_DEC_MP		532
> -#define IMX_SC_R_MJPEG_ENC_MP		533
> +#define IMX_SC_R_MJPEG_0_DEC_MP		532
> +#define IMX_SC_R_MJPEG_0_ENC_MP		533
>  #define IMX_SC_R_VPU_TS_0		534
>  #define IMX_SC_R_VPU_MU_0		535
>  #define IMX_SC_R_VPU_MU_1		536
> @@ -565,6 +579,95 @@
>  #define IMX_SC_PM_CLK_PLL		4	/* PLL */
>  #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
> 
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35			IMX_SC_R_AP_2
> +#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
> +#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
> +#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
> +#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
> +#define IMX_SC_R_A53			IMX_SC_R_AP_0
> +#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
> +#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
> +#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
> +#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
> +#define IMX_SC_R_A72			IMX_SC_R_AP_1
> +#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
> +#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
> +#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
> +#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
> +#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
> +#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
> +#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
> +#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
> +#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
> +#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
> +#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
> +#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
> +#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
> +#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
> +#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
> +#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
> +#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
> +#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
> +#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
> +#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
> +#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
> +#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
> +#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
> +#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
> +#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
> +#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
> +#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
> +#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
> +#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
> +#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
> +#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
> +#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
> +#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
> +#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
> +#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
> +#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
> +#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
> +#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
> +#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
> +#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
> +#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
> +#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
> +#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
> +#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
> +#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
> +#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
> +#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
> +#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
> +#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
> +#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
> +#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
> +#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
> +#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
> +#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
> +#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
> +#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
> +#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
> +#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
> +#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
> +#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
> +#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
> +#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
> +#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
> +#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
> +#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
> +#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
> +#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
> +#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
> +#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
> +#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
> +#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
> +#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
> +#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
> +#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
> +
>  /*
>   * Defines for SC CONTROL
>   */
> @@ -630,6 +733,10 @@
>  #define IMX_SC_C_INTF_SEL			59
>  #define IMX_SC_C_RXC_DLY			60
>  #define IMX_SC_C_TIMER_SEL			61
> -#define IMX_SC_C_LAST				62
> +#define IMX_SC_C_MISC0				62
> +#define IMX_SC_C_MISC1				63
> +#define IMX_SC_C_MISC2				64
> +#define IMX_SC_C_MISC3				65
> +#define IMX_SC_C_LAST				66
> 
>  #endif /* __DT_BINDINGS_RSCRC_IMX_H */





_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-15 18:18   ` Viorel Suman (OSS)
@ 2022-09-16  7:43     ` Ahmad Fatoum
  -1 siblings, 0 replies; 20+ messages in thread
From: Ahmad Fatoum @ 2022-09-16  7:43 UTC (permalink / raw)
  To: Viorel Suman (OSS),
	Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Hello,

On 15.09.22 20:18, Viorel Suman (OSS) wrote:
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)
> 
> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> index 43885056557c..a4c68f394986 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).

Above comment literally says that you should never remove defines below.
Please provide a better justification why you deem breaking DT backward
compatibility acceptable.

Thanks,
Ahmad

>   */
>  
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1
> -#define IMX_SC_R_A53_1			2
> -#define IMX_SC_R_A53_2			3
> -#define IMX_SC_R_A53_3			4
> -#define IMX_SC_R_A72			5
> -#define IMX_SC_R_A72_0			6
> -#define IMX_SC_R_A72_1			7
> -#define IMX_SC_R_A72_2			8
> -#define IMX_SC_R_A72_3			9
> +#define IMX_SC_R_AP_0			0
> +#define IMX_SC_R_AP_0_0			1
> +#define IMX_SC_R_AP_0_1			2
> +#define IMX_SC_R_AP_0_2			3
> +#define IMX_SC_R_AP_0_3			4
> +#define IMX_SC_R_AP_1			5
> +#define IMX_SC_R_AP_1_0			6
> +#define IMX_SC_R_AP_1_1			7
> +#define IMX_SC_R_AP_1_2			8
> +#define IMX_SC_R_AP_1_3			9
>  #define IMX_SC_R_CCI			10
>  #define IMX_SC_R_DB			11
>  #define IMX_SC_R_DRC_0			12
>  #define IMX_SC_R_DRC_1			13
>  #define IMX_SC_R_GIC_SMMU		14
> -#define IMX_SC_R_IRQSTR_M4_0		15
> -#define IMX_SC_R_IRQSTR_M4_1		16
> -#define IMX_SC_R_SMMU			17
> -#define IMX_SC_R_GIC			18
> +#define IMX_SC_R_IRQSTR_MCU_0		15
> +#define IMX_SC_R_IRQSTR_MCU_1		16
> +#define IMX_SC_R_SMMU_0			17
> +#define IMX_SC_R_GIC_0			18
>  #define IMX_SC_R_DC_0_BLIT0		19
>  #define IMX_SC_R_DC_0_BLIT1		20
>  #define IMX_SC_R_DC_0_BLIT2		21
>  #define IMX_SC_R_DC_0_BLIT_OUT		22
> -#define IMX_SC_R_PERF			23
> +#define IMX_SC_R_PERF_0			23
> +#define IMX_SC_R_USB_1_PHY		24
>  #define IMX_SC_R_DC_0_WARP		25
> +#define IMX_SC_R_V2X_MU_0		26
> +#define IMX_SC_R_V2X_MU_1		27
>  #define IMX_SC_R_DC_0_VIDEO0		28
>  #define IMX_SC_R_DC_0_VIDEO1		29
>  #define IMX_SC_R_DC_0_FRAC0		30
> +#define IMX_SC_R_V2X_MU_2		31
>  #define IMX_SC_R_DC_0			32
>  #define IMX_SC_R_GPU_2_PID0		33
>  #define IMX_SC_R_DC_0_PLL_0		34
> @@ -49,11 +53,17 @@
>  #define IMX_SC_R_DC_1_BLIT1		37
>  #define IMX_SC_R_DC_1_BLIT2		38
>  #define IMX_SC_R_DC_1_BLIT_OUT		39
> +#define IMX_SC_R_V2X_MU_3		40
> +#define IMX_SC_R_V2X_MU_4		41
>  #define IMX_SC_R_DC_1_WARP		42
> +#define IMX_SC_R_STM			43
> +#define IMX_SC_R_SECVIO			44
>  #define IMX_SC_R_DC_1_VIDEO0		45
>  #define IMX_SC_R_DC_1_VIDEO1		46
>  #define IMX_SC_R_DC_1_FRAC0		47
> +#define IMX_SC_R_V2X			48
>  #define IMX_SC_R_DC_1			49
> +#define IMX_SC_R_UNUSED14		50
>  #define IMX_SC_R_DC_1_PLL_0		51
>  #define IMX_SC_R_DC_1_PLL_1		52
>  #define IMX_SC_R_SPI_0			53
> @@ -144,10 +154,10 @@
>  #define IMX_SC_R_DMA_1_CH29		137
>  #define IMX_SC_R_DMA_1_CH30		138
>  #define IMX_SC_R_DMA_1_CH31		139
> -#define IMX_SC_R_UNUSED1		140
> -#define IMX_SC_R_UNUSED2		141
> -#define IMX_SC_R_UNUSED3		142
> -#define IMX_SC_R_UNUSED4		143
> +#define IMX_SC_R_V2X_PID0		140
> +#define IMX_SC_R_V2X_PID1		141
> +#define IMX_SC_R_V2X_PID2		142
> +#define IMX_SC_R_V2X_PID3		143
>  #define IMX_SC_R_GPU_0_PID0		144
>  #define IMX_SC_R_GPU_0_PID1		145
>  #define IMX_SC_R_GPU_0_PID2		146
> @@ -176,7 +186,7 @@
>  #define IMX_SC_R_PCIE_B			169
>  #define IMX_SC_R_SATA_0			170
>  #define IMX_SC_R_SERDES_1		171
> -#define IMX_SC_R_HSIO_GPIO		172
> +#define IMX_SC_R_HSIO_GPIO_0		172
>  #define IMX_SC_R_MATCH_15		173
>  #define IMX_SC_R_MATCH_16		174
>  #define IMX_SC_R_MATCH_17		175
> @@ -243,15 +253,15 @@
>  #define IMX_SC_R_ROM_0			236
>  #define IMX_SC_R_FSPI_0			237
>  #define IMX_SC_R_FSPI_1			238
> -#define IMX_SC_R_IEE			239
> -#define IMX_SC_R_IEE_R0			240
> -#define IMX_SC_R_IEE_R1			241
> -#define IMX_SC_R_IEE_R2			242
> -#define IMX_SC_R_IEE_R3			243
> -#define IMX_SC_R_IEE_R4			244
> -#define IMX_SC_R_IEE_R5			245
> -#define IMX_SC_R_IEE_R6			246
> -#define IMX_SC_R_IEE_R7			247
> +#define IMX_SC_R_IEE_0			239
> +#define IMX_SC_R_IEE_0_R0		240
> +#define IMX_SC_R_IEE_0_R1		241
> +#define IMX_SC_R_IEE_0_R2		242
> +#define IMX_SC_R_IEE_0_R3		243
> +#define IMX_SC_R_IEE_0_R4		244
> +#define IMX_SC_R_IEE_0_R5		245
> +#define IMX_SC_R_IEE_0_R6		246
> +#define IMX_SC_R_IEE_0_R7		247
>  #define IMX_SC_R_SDHC_0			248
>  #define IMX_SC_R_SDHC_1			249
>  #define IMX_SC_R_SDHC_2			250
> @@ -282,46 +292,50 @@
>  #define IMX_SC_R_LVDS_2_PWM_0		275
>  #define IMX_SC_R_LVDS_2_I2C_0		276
>  #define IMX_SC_R_LVDS_2_I2C_1		277
> -#define IMX_SC_R_M4_0_PID0		278
> -#define IMX_SC_R_M4_0_PID1		279
> -#define IMX_SC_R_M4_0_PID2		280
> -#define IMX_SC_R_M4_0_PID3		281
> -#define IMX_SC_R_M4_0_PID4		282
> -#define IMX_SC_R_M4_0_RGPIO		283
> -#define IMX_SC_R_M4_0_SEMA42		284
> -#define IMX_SC_R_M4_0_TPM		285
> -#define IMX_SC_R_M4_0_PIT		286
> -#define IMX_SC_R_M4_0_UART		287
> -#define IMX_SC_R_M4_0_I2C		288
> -#define IMX_SC_R_M4_0_INTMUX		289
> -#define IMX_SC_R_M4_0_MU_0B		292
> -#define IMX_SC_R_M4_0_MU_0A0		293
> -#define IMX_SC_R_M4_0_MU_0A1		294
> -#define IMX_SC_R_M4_0_MU_0A2		295
> -#define IMX_SC_R_M4_0_MU_0A3		296
> -#define IMX_SC_R_M4_0_MU_1A		297
> -#define IMX_SC_R_M4_1_PID0		298
> -#define IMX_SC_R_M4_1_PID1		299
> -#define IMX_SC_R_M4_1_PID2		300
> -#define IMX_SC_R_M4_1_PID3		301
> -#define IMX_SC_R_M4_1_PID4		302
> -#define IMX_SC_R_M4_1_RGPIO		303
> -#define IMX_SC_R_M4_1_SEMA42		304
> -#define IMX_SC_R_M4_1_TPM		305
> -#define IMX_SC_R_M4_1_PIT		306
> -#define IMX_SC_R_M4_1_UART		307
> -#define IMX_SC_R_M4_1_I2C		308
> -#define IMX_SC_R_M4_1_INTMUX		309
> -#define IMX_SC_R_M4_1_MU_0B		312
> -#define IMX_SC_R_M4_1_MU_0A0		313
> -#define IMX_SC_R_M4_1_MU_0A1		314
> -#define IMX_SC_R_M4_1_MU_0A2		315
> -#define IMX_SC_R_M4_1_MU_0A3		316
> -#define IMX_SC_R_M4_1_MU_1A		317
> +#define IMX_SC_R_MCU_0_PID0		278
> +#define IMX_SC_R_MCU_0_PID1		279
> +#define IMX_SC_R_MCU_0_PID2		280
> +#define IMX_SC_R_MCU_0_PID3		281
> +#define IMX_SC_R_MCU_0_PID4		282
> +#define IMX_SC_R_MCU_0_RGPIO		283
> +#define IMX_SC_R_MCU_0_SEMA42		284
> +#define IMX_SC_R_MCU_0_TPM		285
> +#define IMX_SC_R_MCU_0_PIT		286
> +#define IMX_SC_R_MCU_0_UART		287
> +#define IMX_SC_R_MCU_0_I2C		288
> +#define IMX_SC_R_MCU_0_INTMUX		289
> +#define IMX_SC_R_ENET_0_A0		290
> +#define IMX_SC_R_ENET_0_A1		291
> +#define IMX_SC_R_MCU_0_MU_0B		292
> +#define IMX_SC_R_MCU_0_MU_0A0		293
> +#define IMX_SC_R_MCU_0_MU_0A1		294
> +#define IMX_SC_R_MCU_0_MU_0A2		295
> +#define IMX_SC_R_MCU_0_MU_0A3		296
> +#define IMX_SC_R_MCU_0_MU_1A		297
> +#define IMX_SC_R_MCU_1_PID0		298
> +#define IMX_SC_R_MCU_1_PID1		299
> +#define IMX_SC_R_MCU_1_PID2		300
> +#define IMX_SC_R_MCU_1_PID3		301
> +#define IMX_SC_R_MCU_1_PID4		302
> +#define IMX_SC_R_MCU_1_RGPIO		303
> +#define IMX_SC_R_MCU_1_SEMA42		304
> +#define IMX_SC_R_MCU_1_TPM		305
> +#define IMX_SC_R_MCU_1_PIT		306
> +#define IMX_SC_R_MCU_1_UART		307
> +#define IMX_SC_R_MCU_1_I2C		308
> +#define IMX_SC_R_MCU_1_INTMUX		309
> +#define IMX_SC_R_UNUSED17		310
> +#define IMX_SC_R_UNUSED18		311
> +#define IMX_SC_R_MCU_1_MU_0B		312
> +#define IMX_SC_R_MCU_1_MU_0A0		313
> +#define IMX_SC_R_MCU_1_MU_0A1		314
> +#define IMX_SC_R_MCU_1_MU_0A2		315
> +#define IMX_SC_R_MCU_1_MU_0A3		316
> +#define IMX_SC_R_MCU_1_MU_1A		317
>  #define IMX_SC_R_SAI_0			318
>  #define IMX_SC_R_SAI_1			319
>  #define IMX_SC_R_SAI_2			320
> -#define IMX_SC_R_IRQSTR_SCU2		321
> +#define IMX_SC_R_IRQSTR_AP_0		321
>  #define IMX_SC_R_IRQSTR_DSP		322
>  #define IMX_SC_R_ELCDIF_PLL		323
>  #define IMX_SC_R_OCRAM			324
> @@ -366,33 +380,33 @@
>  #define IMX_SC_R_VPU_PID5		363
>  #define IMX_SC_R_VPU_PID6		364
>  #define IMX_SC_R_VPU_PID7		365
> -#define IMX_SC_R_VPU_UART		366
> -#define IMX_SC_R_VPUCORE		367
> -#define IMX_SC_R_VPUCORE_0		368
> -#define IMX_SC_R_VPUCORE_1		369
> -#define IMX_SC_R_VPUCORE_2		370
> -#define IMX_SC_R_VPUCORE_3		371
> +#define IMX_SC_R_ENET_0_A2		366
> +#define IMX_SC_R_ENET_1_A0		367
> +#define IMX_SC_R_ENET_1_A1		368
> +#define IMX_SC_R_ENET_1_A2		369
> +#define IMX_SC_R_ENET_1_A3		370
> +#define IMX_SC_R_ENET_1_A4		371
>  #define IMX_SC_R_DMA_4_CH0		372
>  #define IMX_SC_R_DMA_4_CH1		373
>  #define IMX_SC_R_DMA_4_CH2		374
>  #define IMX_SC_R_DMA_4_CH3		375
>  #define IMX_SC_R_DMA_4_CH4		376
> -#define IMX_SC_R_ISI_CH0		377
> -#define IMX_SC_R_ISI_CH1		378
> -#define IMX_SC_R_ISI_CH2		379
> -#define IMX_SC_R_ISI_CH3		380
> -#define IMX_SC_R_ISI_CH4		381
> -#define IMX_SC_R_ISI_CH5		382
> -#define IMX_SC_R_ISI_CH6		383
> -#define IMX_SC_R_ISI_CH7		384
> -#define IMX_SC_R_MJPEG_DEC_S0		385
> -#define IMX_SC_R_MJPEG_DEC_S1		386
> -#define IMX_SC_R_MJPEG_DEC_S2		387
> -#define IMX_SC_R_MJPEG_DEC_S3		388
> -#define IMX_SC_R_MJPEG_ENC_S0		389
> -#define IMX_SC_R_MJPEG_ENC_S1		390
> -#define IMX_SC_R_MJPEG_ENC_S2		391
> -#define IMX_SC_R_MJPEG_ENC_S3		392
> +#define IMX_SC_R_ISI_0_CH0		377
> +#define IMX_SC_R_ISI_0_CH1		378
> +#define IMX_SC_R_ISI_0_CH2		379
> +#define IMX_SC_R_ISI_0_CH3		380
> +#define IMX_SC_R_ISI_0_CH4		381
> +#define IMX_SC_R_ISI_0_CH5		382
> +#define IMX_SC_R_ISI_0_CH6		383
> +#define IMX_SC_R_ISI_0_CH7		384
> +#define IMX_SC_R_MJPEG_0_DEC_S0		385
> +#define IMX_SC_R_MJPEG_0_DEC_S1		386
> +#define IMX_SC_R_MJPEG_0_DEC_S2		387
> +#define IMX_SC_R_MJPEG_0_DEC_S3		388
> +#define IMX_SC_R_MJPEG_0_ENC_S0		389
> +#define IMX_SC_R_MJPEG_0_ENC_S1		390
> +#define IMX_SC_R_MJPEG_0_ENC_S2		391
> +#define IMX_SC_R_MJPEG_0_ENC_S3		392
>  #define IMX_SC_R_MIPI_0			393
>  #define IMX_SC_R_MIPI_0_PWM_0		394
>  #define IMX_SC_R_MIPI_0_I2C_0		395
> @@ -507,11 +521,11 @@
>  #define IMX_SC_R_SECO_MU_3		504
>  #define IMX_SC_R_SECO_MU_4		505
>  #define IMX_SC_R_HDMI_RX_PWM_0		506
> -#define IMX_SC_R_A35			507
> -#define IMX_SC_R_A35_0			508
> -#define IMX_SC_R_A35_1			509
> -#define IMX_SC_R_A35_2			510
> -#define IMX_SC_R_A35_3			511
> +#define IMX_SC_R_AP_2			507
> +#define IMX_SC_R_AP_2_0			508
> +#define IMX_SC_R_AP_2_1			509
> +#define IMX_SC_R_AP_2_2			510
> +#define IMX_SC_R_AP_2_3			511
>  #define IMX_SC_R_DSP			512
>  #define IMX_SC_R_DSP_RAM		513
>  #define IMX_SC_R_CAAM_JR1_OUT		514
> @@ -532,8 +546,8 @@
>  #define IMX_SC_R_BOARD_R5		529
>  #define IMX_SC_R_BOARD_R6		530
>  #define IMX_SC_R_BOARD_R7		531
> -#define IMX_SC_R_MJPEG_DEC_MP		532
> -#define IMX_SC_R_MJPEG_ENC_MP		533
> +#define IMX_SC_R_MJPEG_0_DEC_MP		532
> +#define IMX_SC_R_MJPEG_0_ENC_MP		533
>  #define IMX_SC_R_VPU_TS_0		534
>  #define IMX_SC_R_VPU_MU_0		535
>  #define IMX_SC_R_VPU_MU_1		536
> @@ -565,6 +579,95 @@
>  #define IMX_SC_PM_CLK_PLL		4	/* PLL */
>  #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
>  
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35			IMX_SC_R_AP_2
> +#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
> +#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
> +#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
> +#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
> +#define IMX_SC_R_A53			IMX_SC_R_AP_0
> +#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
> +#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
> +#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
> +#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
> +#define IMX_SC_R_A72			IMX_SC_R_AP_1
> +#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
> +#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
> +#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
> +#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
> +#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
> +#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
> +#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
> +#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
> +#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
> +#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
> +#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
> +#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
> +#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
> +#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
> +#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
> +#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
> +#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
> +#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
> +#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
> +#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
> +#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
> +#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
> +#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
> +#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
> +#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
> +#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
> +#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
> +#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
> +#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
> +#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
> +#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
> +#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
> +#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
> +#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
> +#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
> +#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
> +#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
> +#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
> +#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
> +#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
> +#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
> +#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
> +#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
> +#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
> +#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
> +#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
> +#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
> +#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
> +#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
> +#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
> +#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
> +#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
> +#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
> +#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
> +#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
> +#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
> +#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
> +#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
> +#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
> +#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
> +#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
> +#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
> +#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
> +#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
> +#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
> +#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
> +#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
> +#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
> +#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
> +#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
> +#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
> +#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
> +#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
> +#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
> +
>  /*
>   * Defines for SC CONTROL
>   */
> @@ -630,6 +733,10 @@
>  #define IMX_SC_C_INTF_SEL			59
>  #define IMX_SC_C_RXC_DLY			60
>  #define IMX_SC_C_TIMER_SEL			61
> -#define IMX_SC_C_LAST				62
> +#define IMX_SC_C_MISC0				62
> +#define IMX_SC_C_MISC1				63
> +#define IMX_SC_C_MISC2				64
> +#define IMX_SC_C_MISC3				65
> +#define IMX_SC_C_LAST				66
>  
>  #endif /* __DT_BINDINGS_RSCRC_IMX_H */


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16  7:43     ` Ahmad Fatoum
  0 siblings, 0 replies; 20+ messages in thread
From: Ahmad Fatoum @ 2022-09-16  7:43 UTC (permalink / raw)
  To: Viorel Suman (OSS),
	Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Hello,

On 15.09.22 20:18, Viorel Suman (OSS) wrote:
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)
> 
> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> index 43885056557c..a4c68f394986 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).

Above comment literally says that you should never remove defines below.
Please provide a better justification why you deem breaking DT backward
compatibility acceptable.

Thanks,
Ahmad

>   */
>  
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1
> -#define IMX_SC_R_A53_1			2
> -#define IMX_SC_R_A53_2			3
> -#define IMX_SC_R_A53_3			4
> -#define IMX_SC_R_A72			5
> -#define IMX_SC_R_A72_0			6
> -#define IMX_SC_R_A72_1			7
> -#define IMX_SC_R_A72_2			8
> -#define IMX_SC_R_A72_3			9
> +#define IMX_SC_R_AP_0			0
> +#define IMX_SC_R_AP_0_0			1
> +#define IMX_SC_R_AP_0_1			2
> +#define IMX_SC_R_AP_0_2			3
> +#define IMX_SC_R_AP_0_3			4
> +#define IMX_SC_R_AP_1			5
> +#define IMX_SC_R_AP_1_0			6
> +#define IMX_SC_R_AP_1_1			7
> +#define IMX_SC_R_AP_1_2			8
> +#define IMX_SC_R_AP_1_3			9
>  #define IMX_SC_R_CCI			10
>  #define IMX_SC_R_DB			11
>  #define IMX_SC_R_DRC_0			12
>  #define IMX_SC_R_DRC_1			13
>  #define IMX_SC_R_GIC_SMMU		14
> -#define IMX_SC_R_IRQSTR_M4_0		15
> -#define IMX_SC_R_IRQSTR_M4_1		16
> -#define IMX_SC_R_SMMU			17
> -#define IMX_SC_R_GIC			18
> +#define IMX_SC_R_IRQSTR_MCU_0		15
> +#define IMX_SC_R_IRQSTR_MCU_1		16
> +#define IMX_SC_R_SMMU_0			17
> +#define IMX_SC_R_GIC_0			18
>  #define IMX_SC_R_DC_0_BLIT0		19
>  #define IMX_SC_R_DC_0_BLIT1		20
>  #define IMX_SC_R_DC_0_BLIT2		21
>  #define IMX_SC_R_DC_0_BLIT_OUT		22
> -#define IMX_SC_R_PERF			23
> +#define IMX_SC_R_PERF_0			23
> +#define IMX_SC_R_USB_1_PHY		24
>  #define IMX_SC_R_DC_0_WARP		25
> +#define IMX_SC_R_V2X_MU_0		26
> +#define IMX_SC_R_V2X_MU_1		27
>  #define IMX_SC_R_DC_0_VIDEO0		28
>  #define IMX_SC_R_DC_0_VIDEO1		29
>  #define IMX_SC_R_DC_0_FRAC0		30
> +#define IMX_SC_R_V2X_MU_2		31
>  #define IMX_SC_R_DC_0			32
>  #define IMX_SC_R_GPU_2_PID0		33
>  #define IMX_SC_R_DC_0_PLL_0		34
> @@ -49,11 +53,17 @@
>  #define IMX_SC_R_DC_1_BLIT1		37
>  #define IMX_SC_R_DC_1_BLIT2		38
>  #define IMX_SC_R_DC_1_BLIT_OUT		39
> +#define IMX_SC_R_V2X_MU_3		40
> +#define IMX_SC_R_V2X_MU_4		41
>  #define IMX_SC_R_DC_1_WARP		42
> +#define IMX_SC_R_STM			43
> +#define IMX_SC_R_SECVIO			44
>  #define IMX_SC_R_DC_1_VIDEO0		45
>  #define IMX_SC_R_DC_1_VIDEO1		46
>  #define IMX_SC_R_DC_1_FRAC0		47
> +#define IMX_SC_R_V2X			48
>  #define IMX_SC_R_DC_1			49
> +#define IMX_SC_R_UNUSED14		50
>  #define IMX_SC_R_DC_1_PLL_0		51
>  #define IMX_SC_R_DC_1_PLL_1		52
>  #define IMX_SC_R_SPI_0			53
> @@ -144,10 +154,10 @@
>  #define IMX_SC_R_DMA_1_CH29		137
>  #define IMX_SC_R_DMA_1_CH30		138
>  #define IMX_SC_R_DMA_1_CH31		139
> -#define IMX_SC_R_UNUSED1		140
> -#define IMX_SC_R_UNUSED2		141
> -#define IMX_SC_R_UNUSED3		142
> -#define IMX_SC_R_UNUSED4		143
> +#define IMX_SC_R_V2X_PID0		140
> +#define IMX_SC_R_V2X_PID1		141
> +#define IMX_SC_R_V2X_PID2		142
> +#define IMX_SC_R_V2X_PID3		143
>  #define IMX_SC_R_GPU_0_PID0		144
>  #define IMX_SC_R_GPU_0_PID1		145
>  #define IMX_SC_R_GPU_0_PID2		146
> @@ -176,7 +186,7 @@
>  #define IMX_SC_R_PCIE_B			169
>  #define IMX_SC_R_SATA_0			170
>  #define IMX_SC_R_SERDES_1		171
> -#define IMX_SC_R_HSIO_GPIO		172
> +#define IMX_SC_R_HSIO_GPIO_0		172
>  #define IMX_SC_R_MATCH_15		173
>  #define IMX_SC_R_MATCH_16		174
>  #define IMX_SC_R_MATCH_17		175
> @@ -243,15 +253,15 @@
>  #define IMX_SC_R_ROM_0			236
>  #define IMX_SC_R_FSPI_0			237
>  #define IMX_SC_R_FSPI_1			238
> -#define IMX_SC_R_IEE			239
> -#define IMX_SC_R_IEE_R0			240
> -#define IMX_SC_R_IEE_R1			241
> -#define IMX_SC_R_IEE_R2			242
> -#define IMX_SC_R_IEE_R3			243
> -#define IMX_SC_R_IEE_R4			244
> -#define IMX_SC_R_IEE_R5			245
> -#define IMX_SC_R_IEE_R6			246
> -#define IMX_SC_R_IEE_R7			247
> +#define IMX_SC_R_IEE_0			239
> +#define IMX_SC_R_IEE_0_R0		240
> +#define IMX_SC_R_IEE_0_R1		241
> +#define IMX_SC_R_IEE_0_R2		242
> +#define IMX_SC_R_IEE_0_R3		243
> +#define IMX_SC_R_IEE_0_R4		244
> +#define IMX_SC_R_IEE_0_R5		245
> +#define IMX_SC_R_IEE_0_R6		246
> +#define IMX_SC_R_IEE_0_R7		247
>  #define IMX_SC_R_SDHC_0			248
>  #define IMX_SC_R_SDHC_1			249
>  #define IMX_SC_R_SDHC_2			250
> @@ -282,46 +292,50 @@
>  #define IMX_SC_R_LVDS_2_PWM_0		275
>  #define IMX_SC_R_LVDS_2_I2C_0		276
>  #define IMX_SC_R_LVDS_2_I2C_1		277
> -#define IMX_SC_R_M4_0_PID0		278
> -#define IMX_SC_R_M4_0_PID1		279
> -#define IMX_SC_R_M4_0_PID2		280
> -#define IMX_SC_R_M4_0_PID3		281
> -#define IMX_SC_R_M4_0_PID4		282
> -#define IMX_SC_R_M4_0_RGPIO		283
> -#define IMX_SC_R_M4_0_SEMA42		284
> -#define IMX_SC_R_M4_0_TPM		285
> -#define IMX_SC_R_M4_0_PIT		286
> -#define IMX_SC_R_M4_0_UART		287
> -#define IMX_SC_R_M4_0_I2C		288
> -#define IMX_SC_R_M4_0_INTMUX		289
> -#define IMX_SC_R_M4_0_MU_0B		292
> -#define IMX_SC_R_M4_0_MU_0A0		293
> -#define IMX_SC_R_M4_0_MU_0A1		294
> -#define IMX_SC_R_M4_0_MU_0A2		295
> -#define IMX_SC_R_M4_0_MU_0A3		296
> -#define IMX_SC_R_M4_0_MU_1A		297
> -#define IMX_SC_R_M4_1_PID0		298
> -#define IMX_SC_R_M4_1_PID1		299
> -#define IMX_SC_R_M4_1_PID2		300
> -#define IMX_SC_R_M4_1_PID3		301
> -#define IMX_SC_R_M4_1_PID4		302
> -#define IMX_SC_R_M4_1_RGPIO		303
> -#define IMX_SC_R_M4_1_SEMA42		304
> -#define IMX_SC_R_M4_1_TPM		305
> -#define IMX_SC_R_M4_1_PIT		306
> -#define IMX_SC_R_M4_1_UART		307
> -#define IMX_SC_R_M4_1_I2C		308
> -#define IMX_SC_R_M4_1_INTMUX		309
> -#define IMX_SC_R_M4_1_MU_0B		312
> -#define IMX_SC_R_M4_1_MU_0A0		313
> -#define IMX_SC_R_M4_1_MU_0A1		314
> -#define IMX_SC_R_M4_1_MU_0A2		315
> -#define IMX_SC_R_M4_1_MU_0A3		316
> -#define IMX_SC_R_M4_1_MU_1A		317
> +#define IMX_SC_R_MCU_0_PID0		278
> +#define IMX_SC_R_MCU_0_PID1		279
> +#define IMX_SC_R_MCU_0_PID2		280
> +#define IMX_SC_R_MCU_0_PID3		281
> +#define IMX_SC_R_MCU_0_PID4		282
> +#define IMX_SC_R_MCU_0_RGPIO		283
> +#define IMX_SC_R_MCU_0_SEMA42		284
> +#define IMX_SC_R_MCU_0_TPM		285
> +#define IMX_SC_R_MCU_0_PIT		286
> +#define IMX_SC_R_MCU_0_UART		287
> +#define IMX_SC_R_MCU_0_I2C		288
> +#define IMX_SC_R_MCU_0_INTMUX		289
> +#define IMX_SC_R_ENET_0_A0		290
> +#define IMX_SC_R_ENET_0_A1		291
> +#define IMX_SC_R_MCU_0_MU_0B		292
> +#define IMX_SC_R_MCU_0_MU_0A0		293
> +#define IMX_SC_R_MCU_0_MU_0A1		294
> +#define IMX_SC_R_MCU_0_MU_0A2		295
> +#define IMX_SC_R_MCU_0_MU_0A3		296
> +#define IMX_SC_R_MCU_0_MU_1A		297
> +#define IMX_SC_R_MCU_1_PID0		298
> +#define IMX_SC_R_MCU_1_PID1		299
> +#define IMX_SC_R_MCU_1_PID2		300
> +#define IMX_SC_R_MCU_1_PID3		301
> +#define IMX_SC_R_MCU_1_PID4		302
> +#define IMX_SC_R_MCU_1_RGPIO		303
> +#define IMX_SC_R_MCU_1_SEMA42		304
> +#define IMX_SC_R_MCU_1_TPM		305
> +#define IMX_SC_R_MCU_1_PIT		306
> +#define IMX_SC_R_MCU_1_UART		307
> +#define IMX_SC_R_MCU_1_I2C		308
> +#define IMX_SC_R_MCU_1_INTMUX		309
> +#define IMX_SC_R_UNUSED17		310
> +#define IMX_SC_R_UNUSED18		311
> +#define IMX_SC_R_MCU_1_MU_0B		312
> +#define IMX_SC_R_MCU_1_MU_0A0		313
> +#define IMX_SC_R_MCU_1_MU_0A1		314
> +#define IMX_SC_R_MCU_1_MU_0A2		315
> +#define IMX_SC_R_MCU_1_MU_0A3		316
> +#define IMX_SC_R_MCU_1_MU_1A		317
>  #define IMX_SC_R_SAI_0			318
>  #define IMX_SC_R_SAI_1			319
>  #define IMX_SC_R_SAI_2			320
> -#define IMX_SC_R_IRQSTR_SCU2		321
> +#define IMX_SC_R_IRQSTR_AP_0		321
>  #define IMX_SC_R_IRQSTR_DSP		322
>  #define IMX_SC_R_ELCDIF_PLL		323
>  #define IMX_SC_R_OCRAM			324
> @@ -366,33 +380,33 @@
>  #define IMX_SC_R_VPU_PID5		363
>  #define IMX_SC_R_VPU_PID6		364
>  #define IMX_SC_R_VPU_PID7		365
> -#define IMX_SC_R_VPU_UART		366
> -#define IMX_SC_R_VPUCORE		367
> -#define IMX_SC_R_VPUCORE_0		368
> -#define IMX_SC_R_VPUCORE_1		369
> -#define IMX_SC_R_VPUCORE_2		370
> -#define IMX_SC_R_VPUCORE_3		371
> +#define IMX_SC_R_ENET_0_A2		366
> +#define IMX_SC_R_ENET_1_A0		367
> +#define IMX_SC_R_ENET_1_A1		368
> +#define IMX_SC_R_ENET_1_A2		369
> +#define IMX_SC_R_ENET_1_A3		370
> +#define IMX_SC_R_ENET_1_A4		371
>  #define IMX_SC_R_DMA_4_CH0		372
>  #define IMX_SC_R_DMA_4_CH1		373
>  #define IMX_SC_R_DMA_4_CH2		374
>  #define IMX_SC_R_DMA_4_CH3		375
>  #define IMX_SC_R_DMA_4_CH4		376
> -#define IMX_SC_R_ISI_CH0		377
> -#define IMX_SC_R_ISI_CH1		378
> -#define IMX_SC_R_ISI_CH2		379
> -#define IMX_SC_R_ISI_CH3		380
> -#define IMX_SC_R_ISI_CH4		381
> -#define IMX_SC_R_ISI_CH5		382
> -#define IMX_SC_R_ISI_CH6		383
> -#define IMX_SC_R_ISI_CH7		384
> -#define IMX_SC_R_MJPEG_DEC_S0		385
> -#define IMX_SC_R_MJPEG_DEC_S1		386
> -#define IMX_SC_R_MJPEG_DEC_S2		387
> -#define IMX_SC_R_MJPEG_DEC_S3		388
> -#define IMX_SC_R_MJPEG_ENC_S0		389
> -#define IMX_SC_R_MJPEG_ENC_S1		390
> -#define IMX_SC_R_MJPEG_ENC_S2		391
> -#define IMX_SC_R_MJPEG_ENC_S3		392
> +#define IMX_SC_R_ISI_0_CH0		377
> +#define IMX_SC_R_ISI_0_CH1		378
> +#define IMX_SC_R_ISI_0_CH2		379
> +#define IMX_SC_R_ISI_0_CH3		380
> +#define IMX_SC_R_ISI_0_CH4		381
> +#define IMX_SC_R_ISI_0_CH5		382
> +#define IMX_SC_R_ISI_0_CH6		383
> +#define IMX_SC_R_ISI_0_CH7		384
> +#define IMX_SC_R_MJPEG_0_DEC_S0		385
> +#define IMX_SC_R_MJPEG_0_DEC_S1		386
> +#define IMX_SC_R_MJPEG_0_DEC_S2		387
> +#define IMX_SC_R_MJPEG_0_DEC_S3		388
> +#define IMX_SC_R_MJPEG_0_ENC_S0		389
> +#define IMX_SC_R_MJPEG_0_ENC_S1		390
> +#define IMX_SC_R_MJPEG_0_ENC_S2		391
> +#define IMX_SC_R_MJPEG_0_ENC_S3		392
>  #define IMX_SC_R_MIPI_0			393
>  #define IMX_SC_R_MIPI_0_PWM_0		394
>  #define IMX_SC_R_MIPI_0_I2C_0		395
> @@ -507,11 +521,11 @@
>  #define IMX_SC_R_SECO_MU_3		504
>  #define IMX_SC_R_SECO_MU_4		505
>  #define IMX_SC_R_HDMI_RX_PWM_0		506
> -#define IMX_SC_R_A35			507
> -#define IMX_SC_R_A35_0			508
> -#define IMX_SC_R_A35_1			509
> -#define IMX_SC_R_A35_2			510
> -#define IMX_SC_R_A35_3			511
> +#define IMX_SC_R_AP_2			507
> +#define IMX_SC_R_AP_2_0			508
> +#define IMX_SC_R_AP_2_1			509
> +#define IMX_SC_R_AP_2_2			510
> +#define IMX_SC_R_AP_2_3			511
>  #define IMX_SC_R_DSP			512
>  #define IMX_SC_R_DSP_RAM		513
>  #define IMX_SC_R_CAAM_JR1_OUT		514
> @@ -532,8 +546,8 @@
>  #define IMX_SC_R_BOARD_R5		529
>  #define IMX_SC_R_BOARD_R6		530
>  #define IMX_SC_R_BOARD_R7		531
> -#define IMX_SC_R_MJPEG_DEC_MP		532
> -#define IMX_SC_R_MJPEG_ENC_MP		533
> +#define IMX_SC_R_MJPEG_0_DEC_MP		532
> +#define IMX_SC_R_MJPEG_0_ENC_MP		533
>  #define IMX_SC_R_VPU_TS_0		534
>  #define IMX_SC_R_VPU_MU_0		535
>  #define IMX_SC_R_VPU_MU_1		536
> @@ -565,6 +579,95 @@
>  #define IMX_SC_PM_CLK_PLL		4	/* PLL */
>  #define IMX_SC_PM_CLK_BYPASS		4	/* Bypass clock */
>  
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35			IMX_SC_R_AP_2
> +#define IMX_SC_R_A35_0			IMX_SC_R_AP_2_0
> +#define IMX_SC_R_A35_1			IMX_SC_R_AP_2_1
> +#define IMX_SC_R_A35_2			IMX_SC_R_AP_2_2
> +#define IMX_SC_R_A35_3			IMX_SC_R_AP_2_3
> +#define IMX_SC_R_A53			IMX_SC_R_AP_0
> +#define IMX_SC_R_A53_0			IMX_SC_R_AP_0_0
> +#define IMX_SC_R_A53_1			IMX_SC_R_AP_0_1
> +#define IMX_SC_R_A53_2			IMX_SC_R_AP_0_2
> +#define IMX_SC_R_A53_3			IMX_SC_R_AP_0_3
> +#define IMX_SC_R_A72			IMX_SC_R_AP_1
> +#define IMX_SC_R_A72_0			IMX_SC_R_AP_1_0
> +#define IMX_SC_R_A72_1			IMX_SC_R_AP_1_1
> +#define IMX_SC_R_A72_2			IMX_SC_R_AP_1_2
> +#define IMX_SC_R_A72_3			IMX_SC_R_AP_1_3
> +#define IMX_SC_R_GIC			IMX_SC_R_GIC_0
> +#define IMX_SC_R_HSIO_GPIO		IMX_SC_R_HSIO_GPIO_0
> +#define IMX_SC_R_IEE			IMX_SC_R_IEE_0
> +#define IMX_SC_R_IEE_R0			IMX_SC_R_IEE_0_R0
> +#define IMX_SC_R_IEE_R1			IMX_SC_R_IEE_0_R1
> +#define IMX_SC_R_IEE_R2			IMX_SC_R_IEE_0_R2
> +#define IMX_SC_R_IEE_R3			IMX_SC_R_IEE_0_R3
> +#define IMX_SC_R_IEE_R4			IMX_SC_R_IEE_0_R4
> +#define IMX_SC_R_IEE_R5			IMX_SC_R_IEE_0_R5
> +#define IMX_SC_R_IEE_R6			IMX_SC_R_IEE_0_R6
> +#define IMX_SC_R_IEE_R7			IMX_SC_R_IEE_0_R7
> +#define IMX_SC_R_IRQSTR_M4_0		IMX_SC_R_IRQSTR_MCU_0
> +#define IMX_SC_R_IRQSTR_M4_1		IMX_SC_R_IRQSTR_MCU_1
> +#define IMX_SC_R_IRQSTR_SCU2		IMX_SC_R_IRQSTR_AP_0
> +#define IMX_SC_R_ISI_CH0		IMX_SC_R_ISI_0_CH0
> +#define IMX_SC_R_ISI_CH1		IMX_SC_R_ISI_0_CH1
> +#define IMX_SC_R_ISI_CH2		IMX_SC_R_ISI_0_CH2
> +#define IMX_SC_R_ISI_CH3		IMX_SC_R_ISI_0_CH3
> +#define IMX_SC_R_ISI_CH4		IMX_SC_R_ISI_0_CH4
> +#define IMX_SC_R_ISI_CH5		IMX_SC_R_ISI_0_CH5
> +#define IMX_SC_R_ISI_CH6		IMX_SC_R_ISI_0_CH6
> +#define IMX_SC_R_ISI_CH7		IMX_SC_R_ISI_0_CH7
> +#define IMX_SC_R_M4_0_I2C		IMX_SC_R_MCU_0_I2C
> +#define IMX_SC_R_M4_0_INTMUX		IMX_SC_R_MCU_0_INTMUX
> +#define IMX_SC_R_M4_0_MU_0A0		IMX_SC_R_MCU_0_MU_0A0
> +#define IMX_SC_R_M4_0_MU_0A1		IMX_SC_R_MCU_0_MU_0A1
> +#define IMX_SC_R_M4_0_MU_0A2		IMX_SC_R_MCU_0_MU_0A2
> +#define IMX_SC_R_M4_0_MU_0A3		IMX_SC_R_MCU_0_MU_0A3
> +#define IMX_SC_R_M4_0_MU_0B		IMX_SC_R_MCU_0_MU_0B
> +#define IMX_SC_R_M4_0_MU_1A		IMX_SC_R_MCU_0_MU_1A
> +#define IMX_SC_R_M4_0_PID0		IMX_SC_R_MCU_0_PID0
> +#define IMX_SC_R_M4_0_PID1		IMX_SC_R_MCU_0_PID1
> +#define IMX_SC_R_M4_0_PID2		IMX_SC_R_MCU_0_PID2
> +#define IMX_SC_R_M4_0_PID3		IMX_SC_R_MCU_0_PID3
> +#define IMX_SC_R_M4_0_PID4		IMX_SC_R_MCU_0_PID4
> +#define IMX_SC_R_M4_0_PIT		IMX_SC_R_MCU_0_PIT
> +#define IMX_SC_R_M4_0_RGPIO		IMX_SC_R_MCU_0_RGPIO
> +#define IMX_SC_R_M4_0_SEMA42		IMX_SC_R_MCU_0_SEMA42
> +#define IMX_SC_R_M4_0_TPM		IMX_SC_R_MCU_0_TPM
> +#define IMX_SC_R_M4_0_UART		IMX_SC_R_MCU_0_UART
> +#define IMX_SC_R_M4_1_I2C		IMX_SC_R_MCU_1_I2C
> +#define IMX_SC_R_M4_1_INTMUX		IMX_SC_R_MCU_1_INTMUX
> +#define IMX_SC_R_M4_1_MU_0A0		IMX_SC_R_MCU_1_MU_0A0
> +#define IMX_SC_R_M4_1_MU_0A1		IMX_SC_R_MCU_1_MU_0A1
> +#define IMX_SC_R_M4_1_MU_0A2		IMX_SC_R_MCU_1_MU_0A2
> +#define IMX_SC_R_M4_1_MU_0A3		IMX_SC_R_MCU_1_MU_0A3
> +#define IMX_SC_R_M4_1_MU_0B		IMX_SC_R_MCU_1_MU_0B
> +#define IMX_SC_R_M4_1_MU_1A		IMX_SC_R_MCU_1_MU_1A
> +#define IMX_SC_R_M4_1_PID0		IMX_SC_R_MCU_1_PID0
> +#define IMX_SC_R_M4_1_PID1		IMX_SC_R_MCU_1_PID1
> +#define IMX_SC_R_M4_1_PID2		IMX_SC_R_MCU_1_PID2
> +#define IMX_SC_R_M4_1_PID3		IMX_SC_R_MCU_1_PID3
> +#define IMX_SC_R_M4_1_PID4		IMX_SC_R_MCU_1_PID4
> +#define IMX_SC_R_M4_1_PIT		IMX_SC_R_MCU_1_PIT
> +#define IMX_SC_R_M4_1_RGPIO		IMX_SC_R_MCU_1_RGPIO
> +#define IMX_SC_R_M4_1_SEMA42		IMX_SC_R_MCU_1_SEMA42
> +#define IMX_SC_R_M4_1_TPM		IMX_SC_R_MCU_1_TPM
> +#define IMX_SC_R_M4_1_UART		IMX_SC_R_MCU_1_UART
> +#define IMX_SC_R_MJPEG_DEC_MP		IMX_SC_R_MJPEG_0_DEC_MP
> +#define IMX_SC_R_MJPEG_DEC_S0		IMX_SC_R_MJPEG_0_DEC_S0
> +#define IMX_SC_R_MJPEG_DEC_S1		IMX_SC_R_MJPEG_0_DEC_S1
> +#define IMX_SC_R_MJPEG_DEC_S2		IMX_SC_R_MJPEG_0_DEC_S2
> +#define IMX_SC_R_MJPEG_DEC_S3		IMX_SC_R_MJPEG_0_DEC_S3
> +#define IMX_SC_R_MJPEG_ENC_MP		IMX_SC_R_MJPEG_0_ENC_MP
> +#define IMX_SC_R_MJPEG_ENC_S0		IMX_SC_R_MJPEG_0_ENC_S0
> +#define IMX_SC_R_MJPEG_ENC_S1		IMX_SC_R_MJPEG_0_ENC_S1
> +#define IMX_SC_R_MJPEG_ENC_S2		IMX_SC_R_MJPEG_0_ENC_S2
> +#define IMX_SC_R_MJPEG_ENC_S3		IMX_SC_R_MJPEG_0_ENC_S3
> +#define IMX_SC_R_PERF			IMX_SC_R_PERF_0
> +#define IMX_SC_R_SMMU			IMX_SC_R_SMMU_0
> +
>  /*
>   * Defines for SC CONTROL
>   */
> @@ -630,6 +733,10 @@
>  #define IMX_SC_C_INTF_SEL			59
>  #define IMX_SC_C_RXC_DLY			60
>  #define IMX_SC_C_TIMER_SEL			61
> -#define IMX_SC_C_LAST				62
> +#define IMX_SC_C_MISC0				62
> +#define IMX_SC_C_MISC1				63
> +#define IMX_SC_C_MISC2				64
> +#define IMX_SC_C_MISC3				65
> +#define IMX_SC_C_LAST				66
>  
>  #endif /* __DT_BINDINGS_RSCRC_IMX_H */


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-15 18:18   ` Viorel Suman (OSS)
@ 2022-09-16  9:38     ` Krzysztof Kozlowski
  -1 siblings, 0 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-16  9:38 UTC (permalink / raw)
  To: Viorel Suman (OSS),
	Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)
> 
> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> index 43885056557c..a4c68f394986 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).
>   */
>  
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1

You cannot change binding constants... Aren't you breaking all possible
boards and users?


Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16  9:38     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-16  9:38 UTC (permalink / raw)
  To: Viorel Suman (OSS),
	Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
> From: Viorel Suman <viorel.suman@nxp.com>
> 
> Sync defines with the latest available SCFW kit version 1.13.0,
> may be found at the address below:
> 
> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> 
> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> ---
>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>  1 file changed, 203 insertions(+), 96 deletions(-)
> 
> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> index 43885056557c..a4c68f394986 100644
> --- a/include/dt-bindings/firmware/imx/rsrc.h
> +++ b/include/dt-bindings/firmware/imx/rsrc.h
> @@ -13,34 +13,38 @@
>   * never be changed or removed (only added to at the end of the list).
>   */
>  
> -#define IMX_SC_R_A53			0
> -#define IMX_SC_R_A53_0			1

You cannot change binding constants... Aren't you breaking all possible
boards and users?


Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-16  9:38     ` Krzysztof Kozlowski
@ 2022-09-16 10:17       ` Viorel Suman
  -1 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman @ 2022-09-16 10:17 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 22-09-16 10:38:30, Krzysztof Kozlowski wrote:
> On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
> > From: Viorel Suman <viorel.suman@nxp.com>
> > 
> > Sync defines with the latest available SCFW kit version 1.13.0,
> > may be found at the address below:
> > 
> > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> > 
> > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > ---
> >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> > index 43885056557c..a4c68f394986 100644
> > --- a/include/dt-bindings/firmware/imx/rsrc.h
> > +++ b/include/dt-bindings/firmware/imx/rsrc.h
> > @@ -13,34 +13,38 @@
> >   * never be changed or removed (only added to at the end of the list).
> >   */
> >  
> > -#define IMX_SC_R_A53			0
> > -#define IMX_SC_R_A53_0			1
> 
> You cannot change binding constants... Aren't you breaking all possible
> boards and users?

Hi Krzysztof,

There is a backward compatibility section added in the end of the patch,
it follows the same approach as in SCFW kit v1.13.0:

+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35                   IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0                 IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1                 IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2                 IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3                 IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53                   IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0                 IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1                 IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2                 IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3                 IMX_SC_R_AP_0_3

Regards,
Viorel


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16 10:17       ` Viorel Suman
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman @ 2022-09-16 10:17 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 22-09-16 10:38:30, Krzysztof Kozlowski wrote:
> On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
> > From: Viorel Suman <viorel.suman@nxp.com>
> > 
> > Sync defines with the latest available SCFW kit version 1.13.0,
> > may be found at the address below:
> > 
> > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> > 
> > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > ---
> >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> > index 43885056557c..a4c68f394986 100644
> > --- a/include/dt-bindings/firmware/imx/rsrc.h
> > +++ b/include/dt-bindings/firmware/imx/rsrc.h
> > @@ -13,34 +13,38 @@
> >   * never be changed or removed (only added to at the end of the list).
> >   */
> >  
> > -#define IMX_SC_R_A53			0
> > -#define IMX_SC_R_A53_0			1
> 
> You cannot change binding constants... Aren't you breaking all possible
> boards and users?

Hi Krzysztof,

There is a backward compatibility section added in the end of the patch,
it follows the same approach as in SCFW kit v1.13.0:

+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35                   IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0                 IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1                 IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2                 IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3                 IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53                   IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0                 IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1                 IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2                 IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3                 IMX_SC_R_AP_0_3

Regards,
Viorel


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-16 10:17       ` Viorel Suman
@ 2022-09-16 10:30         ` Krzysztof Kozlowski
  -1 siblings, 0 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-16 10:30 UTC (permalink / raw)
  To: Viorel Suman
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 16/09/2022 11:17, Viorel Suman wrote:
> On 22-09-16 10:38:30, Krzysztof Kozlowski wrote:
>> On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
>>> From: Viorel Suman <viorel.suman@nxp.com>
>>>
>>> Sync defines with the latest available SCFW kit version 1.13.0,
>>> may be found at the address below:
>>>
>>> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
>>>
>>> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
>>> ---
>>>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>>>  1 file changed, 203 insertions(+), 96 deletions(-)
>>>
>>> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
>>> index 43885056557c..a4c68f394986 100644
>>> --- a/include/dt-bindings/firmware/imx/rsrc.h
>>> +++ b/include/dt-bindings/firmware/imx/rsrc.h
>>> @@ -13,34 +13,38 @@
>>>   * never be changed or removed (only added to at the end of the list).
>>>   */
>>>  
>>> -#define IMX_SC_R_A53			0
>>> -#define IMX_SC_R_A53_0			1
>>
>> You cannot change binding constants... Aren't you breaking all possible
>> boards and users?
> 
> Hi Krzysztof,
> 
> There is a backward compatibility section added in the end of the patch,
> it follows the same approach as in SCFW kit v1.13.0:
> 
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35                   IMX_SC_R_AP_2

Then it looks fine, except someone would need to compare/test the numbers...


Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16 10:30         ` Krzysztof Kozlowski
  0 siblings, 0 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-16 10:30 UTC (permalink / raw)
  To: Viorel Suman
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 16/09/2022 11:17, Viorel Suman wrote:
> On 22-09-16 10:38:30, Krzysztof Kozlowski wrote:
>> On 15/09/2022 19:18, Viorel Suman (OSS) wrote:
>>> From: Viorel Suman <viorel.suman@nxp.com>
>>>
>>> Sync defines with the latest available SCFW kit version 1.13.0,
>>> may be found at the address below:
>>>
>>> https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
>>>
>>> Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
>>> ---
>>>  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
>>>  1 file changed, 203 insertions(+), 96 deletions(-)
>>>
>>> diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
>>> index 43885056557c..a4c68f394986 100644
>>> --- a/include/dt-bindings/firmware/imx/rsrc.h
>>> +++ b/include/dt-bindings/firmware/imx/rsrc.h
>>> @@ -13,34 +13,38 @@
>>>   * never be changed or removed (only added to at the end of the list).
>>>   */
>>>  
>>> -#define IMX_SC_R_A53			0
>>> -#define IMX_SC_R_A53_0			1
>>
>> You cannot change binding constants... Aren't you breaking all possible
>> boards and users?
> 
> Hi Krzysztof,
> 
> There is a backward compatibility section added in the end of the patch,
> it follows the same approach as in SCFW kit v1.13.0:
> 
> +/*
> + * Compatibility defines for sc_rsrc_t
> + */
> +#define IMX_SC_R_A35                   IMX_SC_R_AP_2

Then it looks fine, except someone would need to compare/test the numbers...


Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
  2022-09-16  7:43     ` Ahmad Fatoum
@ 2022-09-16 10:36       ` Viorel Suman
  -1 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman @ 2022-09-16 10:36 UTC (permalink / raw)
  To: Ahmad Fatoum
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 22-09-16 09:43:51, Ahmad Fatoum wrote:
> Hello,
> 
> On 15.09.22 20:18, Viorel Suman (OSS) wrote:
> > From: Viorel Suman <viorel.suman@nxp.com>
> > 
> > Sync defines with the latest available SCFW kit version 1.13.0,
> > may be found at the address below:
> > 
> > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> > 
> > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > ---
> >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> > index 43885056557c..a4c68f394986 100644
> > --- a/include/dt-bindings/firmware/imx/rsrc.h
> > +++ b/include/dt-bindings/firmware/imx/rsrc.h
> > @@ -13,34 +13,38 @@
> >   * never be changed or removed (only added to at the end of the list).
> 
> Above comment literally says that you should never remove defines below.
> Please provide a better justification why you deem breaking DT backward
> compatibility acceptable.
> 
> Thanks,
> Ahmad
> 
> >   */
> >  
> > -#define IMX_SC_R_A53			0
> > -#define IMX_SC_R_A53_0			1
> > -#define IMX_SC_R_A53_1			2
> > -#define IMX_SC_R_A53_2			3
> > -#define IMX_SC_R_A53_3			4
...

Hi Ahmad,

The defines you mentioned are not removed, just redefined in the
backward compatibility section added in the end of the patch.
The patch follows the same approach as in SCFW kit v1.13.0, so
it's much easier to synchronize them.

+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35                   IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0                 IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1                 IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2                 IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3                 IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53                   IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0                 IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1                 IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2                 IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3                 IMX_SC_R_AP_0_3
...

The only renamed defines are below, the reason is that
VPU_UART and VPUCORE* defines were replaced with ENET defines
in SCFW kit long time ago, to quote the author:
   "The VPU UARTs haven't existed since QM A0 was supported."

> > -#define IMX_SC_R_VPU_UART		366
> > -#define IMX_SC_R_VPUCORE		367
> > -#define IMX_SC_R_VPUCORE_0		368
> > -#define IMX_SC_R_VPUCORE_1		369
> > -#define IMX_SC_R_VPUCORE_2		370
> > -#define IMX_SC_R_VPUCORE_3		371
> > +#define IMX_SC_R_ENET_0_A2		366
> > +#define IMX_SC_R_ENET_1_A0		367
> > +#define IMX_SC_R_ENET_1_A1		368
> > +#define IMX_SC_R_ENET_1_A2		369
> > +#define IMX_SC_R_ENET_1_A3		370
> > +#define IMX_SC_R_ENET_1_A4		371

Regards,
Viorel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16 10:36       ` Viorel Suman
  0 siblings, 0 replies; 20+ messages in thread
From: Viorel Suman @ 2022-09-16 10:36 UTC (permalink / raw)
  To: Ahmad Fatoum
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

On 22-09-16 09:43:51, Ahmad Fatoum wrote:
> Hello,
> 
> On 15.09.22 20:18, Viorel Suman (OSS) wrote:
> > From: Viorel Suman <viorel.suman@nxp.com>
> > 
> > Sync defines with the latest available SCFW kit version 1.13.0,
> > may be found at the address below:
> > 
> > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.0&appType=license
> > 
> > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > ---
> >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > diff --git a/include/dt-bindings/firmware/imx/rsrc.h b/include/dt-bindings/firmware/imx/rsrc.h
> > index 43885056557c..a4c68f394986 100644
> > --- a/include/dt-bindings/firmware/imx/rsrc.h
> > +++ b/include/dt-bindings/firmware/imx/rsrc.h
> > @@ -13,34 +13,38 @@
> >   * never be changed or removed (only added to at the end of the list).
> 
> Above comment literally says that you should never remove defines below.
> Please provide a better justification why you deem breaking DT backward
> compatibility acceptable.
> 
> Thanks,
> Ahmad
> 
> >   */
> >  
> > -#define IMX_SC_R_A53			0
> > -#define IMX_SC_R_A53_0			1
> > -#define IMX_SC_R_A53_1			2
> > -#define IMX_SC_R_A53_2			3
> > -#define IMX_SC_R_A53_3			4
...

Hi Ahmad,

The defines you mentioned are not removed, just redefined in the
backward compatibility section added in the end of the patch.
The patch follows the same approach as in SCFW kit v1.13.0, so
it's much easier to synchronize them.

+/*
+ * Compatibility defines for sc_rsrc_t
+ */
+#define IMX_SC_R_A35                   IMX_SC_R_AP_2
+#define IMX_SC_R_A35_0                 IMX_SC_R_AP_2_0
+#define IMX_SC_R_A35_1                 IMX_SC_R_AP_2_1
+#define IMX_SC_R_A35_2                 IMX_SC_R_AP_2_2
+#define IMX_SC_R_A35_3                 IMX_SC_R_AP_2_3
+#define IMX_SC_R_A53                   IMX_SC_R_AP_0
+#define IMX_SC_R_A53_0                 IMX_SC_R_AP_0_0
+#define IMX_SC_R_A53_1                 IMX_SC_R_AP_0_1
+#define IMX_SC_R_A53_2                 IMX_SC_R_AP_0_2
+#define IMX_SC_R_A53_3                 IMX_SC_R_AP_0_3
...

The only renamed defines are below, the reason is that
VPU_UART and VPUCORE* defines were replaced with ENET defines
in SCFW kit long time ago, to quote the author:
   "The VPU UARTs haven't existed since QM A0 was supported."

> > -#define IMX_SC_R_VPU_UART		366
> > -#define IMX_SC_R_VPUCORE		367
> > -#define IMX_SC_R_VPUCORE_0		368
> > -#define IMX_SC_R_VPUCORE_1		369
> > -#define IMX_SC_R_VPUCORE_2		370
> > -#define IMX_SC_R_VPUCORE_3		371
> > +#define IMX_SC_R_ENET_0_A2		366
> > +#define IMX_SC_R_ENET_1_A0		367
> > +#define IMX_SC_R_ENET_1_A1		368
> > +#define IMX_SC_R_ENET_1_A2		369
> > +#define IMX_SC_R_ENET_1_A3		370
> > +#define IMX_SC_R_ENET_1_A4		371

Regards,
Viorel

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
       [not found]     ` <20220916110936.jmxgjps5zrcndjxn@fsr-ub1664-116>
@ 2022-09-16 13:03         ` Alexander Stein
  0 siblings, 0 replies; 20+ messages in thread
From: Alexander Stein @ 2022-09-16 13:03 UTC (permalink / raw)
  To: Viorel Suman
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Hi Viorel,

added missing CC.

Am Freitag, 16. September 2022, 13:09:36 CEST schrieb Viorel Suman:
> On 22-09-16 08:30:46, Alexander Stein wrote:
> > Am Donnerstag, 15. September 2022, 20:18:04 CEST schrieb Viorel Suman 
(OSS):
> > > From: Viorel Suman <viorel.suman@nxp.com>
> > > 
> > > Sync defines with the latest available SCFW kit version 1.13.0,
> > > may be found at the address below:
> > > 
> > > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.
> > > 0&ap pType=license
> > > 
> > > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > > ---
> > > 
> > >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> > >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > This is not bisectable and breaks compilation, as this patch removes
> > symbols which are still used in drivers/clk/imx/clk-imx8qm-rsrc.c
> > (addressed in 2nd patch). IMHO this series should be squashed into one
> > patch.
> 
> Hi Alexander,
> 
> Thank you for review, you are right. Is sending a squashed v2 the usual
> approach in such case or shall I hope that the maintainer will squash them
> before push ? Just want to avoid unnecessary noise.

AFAIK maintainers will not squash patches. This is something you will need to 
do yourself. So sending a v2 seems right.
Also please keep all the other recipients on CC.

Best regards,
Alexander




^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: Re: [PATCH 1/2] dt-bindings: firmware: imx: sync with SCFW kit v1.13.0
@ 2022-09-16 13:03         ` Alexander Stein
  0 siblings, 0 replies; 20+ messages in thread
From: Alexander Stein @ 2022-09-16 13:03 UTC (permalink / raw)
  To: Viorel Suman
  Cc: Abel Vesa, Michael Turquette, Stephen Boyd, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Rob Herring, Krzysztof Kozlowski, Viorel Suman,
	linux-clk, linux-arm-kernel, linux-kernel, devicetree

Hi Viorel,

added missing CC.

Am Freitag, 16. September 2022, 13:09:36 CEST schrieb Viorel Suman:
> On 22-09-16 08:30:46, Alexander Stein wrote:
> > Am Donnerstag, 15. September 2022, 20:18:04 CEST schrieb Viorel Suman 
(OSS):
> > > From: Viorel Suman <viorel.suman@nxp.com>
> > > 
> > > Sync defines with the latest available SCFW kit version 1.13.0,
> > > may be found at the address below:
> > > 
> > > https://www.nxp.com/webapp/Download?colCode=L5.15.32_2.0.0_SCFWKIT-1.13.
> > > 0&ap pType=license
> > > 
> > > Signed-off-by: Viorel Suman <viorel.suman@nxp.com>
> > > ---
> > > 
> > >  include/dt-bindings/firmware/imx/rsrc.h | 299 ++++++++++++++++--------
> > >  1 file changed, 203 insertions(+), 96 deletions(-)
> > 
> > This is not bisectable and breaks compilation, as this patch removes
> > symbols which are still used in drivers/clk/imx/clk-imx8qm-rsrc.c
> > (addressed in 2nd patch). IMHO this series should be squashed into one
> > patch.
> 
> Hi Alexander,
> 
> Thank you for review, you are right. Is sending a squashed v2 the usual
> approach in such case or shall I hope that the maintainer will squash them
> before push ? Just want to avoid unnecessary noise.

AFAIK maintainers will not squash patches. This is something you will need to 
do yourself. So sending a v2 seems right.
Also please keep all the other recipients on CC.

Best regards,
Alexander




_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2022-09-16 13:05 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-15 18:18 [PATCH 0/2] sync SCFW defines with SCFW kit v1.13.0 Viorel Suman (OSS)
2022-09-15 18:18 ` Viorel Suman (OSS)
2022-09-15 18:18 ` [PATCH 1/2] dt-bindings: firmware: imx: sync " Viorel Suman (OSS)
2022-09-15 18:18   ` Viorel Suman (OSS)
2022-09-16  6:30   ` Alexander Stein
2022-09-16  6:30     ` Alexander Stein
     [not found]     ` <20220916110936.jmxgjps5zrcndjxn@fsr-ub1664-116>
2022-09-16 13:03       ` Alexander Stein
2022-09-16 13:03         ` Alexander Stein
2022-09-16  7:43   ` Ahmad Fatoum
2022-09-16  7:43     ` Ahmad Fatoum
2022-09-16 10:36     ` Viorel Suman
2022-09-16 10:36       ` Viorel Suman
2022-09-16  9:38   ` Krzysztof Kozlowski
2022-09-16  9:38     ` Krzysztof Kozlowski
2022-09-16 10:17     ` Viorel Suman
2022-09-16 10:17       ` Viorel Suman
2022-09-16 10:30       ` Krzysztof Kozlowski
2022-09-16 10:30         ` Krzysztof Kozlowski
2022-09-15 18:18 ` [PATCH 2/2] clk: imx8qm: " Viorel Suman (OSS)
2022-09-15 18:18   ` Viorel Suman (OSS)

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.