mcux-sdk-ng: drivers: irqsteer_1: fix build errors

biwenli · mmahadevan108 · commit abc6247189a0 · 2025-07-28T09:28:45.000-05:00
Fix build errors,
  - drivers/irqsteer_1/fsl_irqsteer.h:187:56: error:
    pointer of type void * used in arithmetic
    [-Werror=pointer-arith]

Signed-off-by: Biwen Li &lt;biwen.li@nxp.com&gt;
diff --git a/mcux/mcux-sdk-ng/drivers/irqsteer_1/CMakeLists.txt b/mcux/mcux-sdk-ng/drivers/irqsteer_1/CMakeLists.txt
@@ -1,9 +1,9 @@
-# Copyright 2024 NXP
+# Copyright 2024-2025 NXP
 #
 # SPDX-License-Identifier: BSD-3-Clause
 
 if(CONFIG_MCUX_COMPONENT_driver.irqsteer_1)
-    mcux_component_version(2.0.1)
+    mcux_component_version(2.0.2)
 
     mcux_add_source(SOURCES fsl_irqsteer.h fsl_irqsteer.c)
 
diff --git a/mcux/mcux-sdk-ng/drivers/irqsteer_1/fsl_irqsteer.c b/mcux/mcux-sdk-ng/drivers/irqsteer_1/fsl_irqsteer.c
@@ -78,7 +78,7 @@ void IRQSTEER_Init(int32_t instIdx)
         /* Mask all interrupts. */
         for (i = 0; i < data->regNum; i++)
         {
-            *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, i, data->regNum)) = 0U;
+            *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, i, data->regNum)) = 0U;
         }
 
         for (i = 0; i < data->intGrpNum; i++)
@@ -135,7 +135,7 @@ void IRQSTEER_EnableInterrupt(int32_t instIdx, IRQn_Type irq)
     regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);
     bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;
 
-    *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) |= (1U << bitOffset);
+    *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) |= (1U << bitOffset);
 }
 
 /*! @} */
@@ -161,7 +161,7 @@ void IRQSTEER_DisableInterrupt(int32_t instIdx, IRQn_Type irq)
     regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);
     bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;
 
-    *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);
+    *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);
 }
 
 /**
@@ -266,7 +266,7 @@ IRQn_Type IRQSTEER_GetMasterNextInterrupt(int32_t instIdx, int32_t outputChanIdx
         regIdx = IRQSTEER_GetRegIdx(instIdx, outputChanIdx, i);
 
         /* Get register's value */
-        chanStatus = *(uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));
+        chanStatus = *(uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));
 
         for (j = 0; j < IRQSTEER_INT_SRC_REG_WIDTH; j++)
         {
@@ -283,7 +283,7 @@ IRQn_Type IRQSTEER_GetMasterNextInterrupt(int32_t instIdx, int32_t outputChanIdx
                  */
                  idx = (data->regNum - 1) - regIdx;
                  inputIdx = idx * IRQSTEER_INT_SRC_REG_WIDTH + bitOffset;
-                 irqNum = inputIdx + FSL_FEATURE_IRQSTEER_IRQ_START_INDEX;
+                 irqNum = (IRQn_Type)(inputIdx + FSL_FEATURE_IRQSTEER_IRQ_START_INDEX);
                  return irqNum;
             }
 
@@ -315,7 +315,7 @@ uint64_t IRQSTEER_GetMasterInterruptsStatus(int32_t instIdx, int32_t outputChanI
     for (i = 0; i < sliceNum; i++) {
         regIdx = IRQSTEER_GetRegIdx(instIdx, outputChanIdx, i);
 
-        chanStatus = *(uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));
+        chanStatus = *(uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));
 
         interrupts |= ((uint64_t)chanStatus << (IRQSTEER_INT_SRC_REG_WIDTH * i));
     }
diff --git a/mcux/mcux-sdk-ng/drivers/irqsteer_1/fsl_irqsteer.h b/mcux/mcux-sdk-ng/drivers/irqsteer_1/fsl_irqsteer.h
@@ -21,8 +21,8 @@
 
 /*! @name Driver version */
 /*! @{ */
-/*! Version 2.0.1. */
-#define FSL_IRQSTEER_DRIVER_VERSION (MAKE_VERSION(2, 0, 1))
+/*! Version 2.0.2. */
+#define FSL_IRQSTEER_DRIVER_VERSION (MAKE_VERSION(2, 0, 2))
 /*! @} */
 
 /*
@@ -184,7 +184,7 @@ static inline bool IRQSTEER_InterruptIsEnabled(int32_t instIdx, IRQn_Type irq)
     regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);
     bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;
 
-    return ((*(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= (1U << bitOffset)) != 0U);
+    return ((*(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= (1U << bitOffset)) != 0U);
 }
 
 /*!
@@ -210,11 +210,11 @@ static inline void IRQSTEER_SetInterrupt(int32_t instIdx, IRQn_Type irq, bool se
 
     if (set)
     {
-        *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) |= (1U << bitOffset);
+        *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) |= (1U << bitOffset);
     }
     else
     {
-        *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);
+        *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);
     }
 }
 
@@ -233,7 +233,7 @@ static inline void IRQSTEER_SetInterrupt(int32_t instIdx, IRQn_Type irq, bool se
 static inline void IRQSTEER_EnableMasterInterrupt(int32_t instIdx, int32_t outputChanIdx)
 {
     irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);
-    *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) &= ~(1U << outputChanIdx);
+    *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) &= ~(1U << outputChanIdx);
 }
 
 /*!
@@ -251,7 +251,7 @@ static inline void IRQSTEER_EnableMasterInterrupt(int32_t instIdx, int32_t outpu
 static inline void IRQSTEER_DisableMasterInterrupt(int32_t instIdx, int32_t outputChanIdx)
 {
     irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);
-    *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) |= (1U << outputChanIdx);
+    *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) |= (1U << outputChanIdx);
 }
 
 /*! @} */
@@ -286,7 +286,7 @@ static inline bool IRQSTEER_IsInterruptSet(int32_t instIdx, IRQn_Type irq)
     regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);
     bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;
 
-    return *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum)) & (1U << bitOffset);
+    return *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum)) & (1U << bitOffset);
 }
 
 /*!
@@ -301,7 +301,7 @@ static inline bool IRQSTEER_IsMasterInterruptSet(int32_t instIdx)
 {
     irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);
 
-    return *(volatile uint32_t *)(data->infoPtr->reg + IRQSTEER_CHAN_MSTRSTAT(data->infoPtr->irqChanIdx, data->regNum));
+    return *(volatile uint32_t *)((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MSTRSTAT(data->infoPtr->irqChanIdx, data->regNum));
 }
 
 /*!

Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,9 @@`
`1`		`-# Copyright 2024 NXP`
	`1`	`+# Copyright 2024-2025 NXP`
`2`	`2`	`#`
`3`	`3`	`# SPDX-License-Identifier: BSD-3-Clause`
`4`	`4`
`5`	`5`	`if(CONFIG_MCUX_COMPONENT_driver.irqsteer_1)`
`6`		`- mcux_component_version(2.0.1)`
	`6`	`+ mcux_component_version(2.0.2)`
`7`	`7`
`8`	`8`	`mcux_add_source(SOURCES fsl_irqsteer.h fsl_irqsteer.c)`
`9`	`9`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ void IRQSTEER_Init(int32_t instIdx)`
`78`	`78`	`/* Mask all interrupts. */`
`79`	`79`	`for (i = 0; i < data->regNum; i++)`
`80`	`80`	`{`
`81`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, i, data->regNum)) = 0U;`
	`81`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, i, data->regNum)) = 0U;`
`82`	`82`	`}`
`83`	`83`
`84`	`84`	`for (i = 0; i < data->intGrpNum; i++)`
`@@ -135,7 +135,7 @@ void IRQSTEER_EnableInterrupt(int32_t instIdx, IRQn_Type irq)`
`135`	`135`	`regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);`
`136`	`136`	`bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;`
`137`	`137`
`138`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) \|= (1U << bitOffset);`
	`138`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) \|= (1U << bitOffset);`
`139`	`139`	`}`
`140`	`140`
`141`	`141`	`/! @} /`
`@@ -161,7 +161,7 @@ void IRQSTEER_DisableInterrupt(int32_t instIdx, IRQn_Type irq)`
`161`	`161`	`regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);`
`162`	`162`	`bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;`
`163`	`163`
`164`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);`
	`164`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`/**`
`@@ -266,7 +266,7 @@ IRQn_Type IRQSTEER_GetMasterNextInterrupt(int32_t instIdx, int32_t outputChanIdx`
`266`	`266`	`regIdx = IRQSTEER_GetRegIdx(instIdx, outputChanIdx, i);`
`267`	`267`
`268`	`268`	`/* Get register's value */`
`269`		`- chanStatus = (uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));`
	`269`	`+ chanStatus = (uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));`
`270`	`270`
`271`	`271`	`for (j = 0; j < IRQSTEER_INT_SRC_REG_WIDTH; j++)`
`272`	`272`	`{`
`@@ -283,7 +283,7 @@ IRQn_Type IRQSTEER_GetMasterNextInterrupt(int32_t instIdx, int32_t outputChanIdx`
`283`	`283`	`*/`
`284`	`284`	`idx = (data->regNum - 1) - regIdx;`
`285`	`285`	`inputIdx = idx * IRQSTEER_INT_SRC_REG_WIDTH + bitOffset;`
`286`		`- irqNum = inputIdx + FSL_FEATURE_IRQSTEER_IRQ_START_INDEX;`
	`286`	`+ irqNum = (IRQn_Type)(inputIdx + FSL_FEATURE_IRQSTEER_IRQ_START_INDEX);`
`287`	`287`	`return irqNum;`
`288`	`288`	`}`
`289`	`289`
`@@ -315,7 +315,7 @@ uint64_t IRQSTEER_GetMasterInterruptsStatus(int32_t instIdx, int32_t outputChanI`
`315`	`315`	`for (i = 0; i < sliceNum; i++) {`
`316`	`316`	`regIdx = IRQSTEER_GetRegIdx(instIdx, outputChanIdx, i);`
`317`	`317`
`318`		`- chanStatus = (uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));`
	`318`	`+ chanStatus = (uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum));`
`319`	`319`
`320`	`320`	`interrupts \|= ((uint64_t)chanStatus << (IRQSTEER_INT_SRC_REG_WIDTH * i));`
`321`	`321`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,8 +21,8 @@`
`21`	`21`
`22`	`22`	`/! @name Driver version /`
`23`	`23`	`/! @{ /`
`24`		`-/! Version 2.0.1. /`
`25`		`-#define FSL_IRQSTEER_DRIVER_VERSION (MAKE_VERSION(2, 0, 1))`
	`24`	`+/! Version 2.0.2. /`
	`25`	`+#define FSL_IRQSTEER_DRIVER_VERSION (MAKE_VERSION(2, 0, 2))`
`26`	`26`	`/! @} /`
`27`	`27`
`28`	`28`	`/*`
`@@ -184,7 +184,7 @@ static inline bool IRQSTEER_InterruptIsEnabled(int32_t instIdx, IRQn_Type irq)`
`184`	`184`	`regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);`
`185`	`185`	`bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;`
`186`	`186`
`187`		`- return (((volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= (1U << bitOffset)) != 0U);`
	`187`	`+ return (((volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MASK(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= (1U << bitOffset)) != 0U);`
`188`	`188`	`}`
`189`	`189`
`190`	`190`	`/*!`
`@@ -210,11 +210,11 @@ static inline void IRQSTEER_SetInterrupt(int32_t instIdx, IRQn_Type irq, bool se`
`210`	`210`
`211`	`211`	`if (set)`
`212`	`212`	`{`
`213`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) \|= (1U << bitOffset);`
	`213`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) \|= (1U << bitOffset);`
`214`	`214`	`}`
`215`	`215`	`else`
`216`	`216`	`{`
`217`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);`
	`217`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_SET(data->infoPtr->irqChanIdx, regIdx, data->regNum)) &= ~(1U << bitOffset);`
`218`	`218`	`}`
`219`	`219`	`}`
`220`	`220`
`@@ -233,7 +233,7 @@ static inline void IRQSTEER_SetInterrupt(int32_t instIdx, IRQn_Type irq, bool se`
`233`	`233`	`static inline void IRQSTEER_EnableMasterInterrupt(int32_t instIdx, int32_t outputChanIdx)`
`234`	`234`	`{`
`235`	`235`	`irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);`
`236`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) &= ~(1U << outputChanIdx);`
	`236`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) &= ~(1U << outputChanIdx);`
`237`	`237`	`}`
`238`	`238`
`239`	`239`	`/*!`
`@@ -251,7 +251,7 @@ static inline void IRQSTEER_EnableMasterInterrupt(int32_t instIdx, int32_t outpu`
`251`	`251`	`static inline void IRQSTEER_DisableMasterInterrupt(int32_t instIdx, int32_t outputChanIdx)`
`252`	`252`	`{`
`253`	`253`	`irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);`
`254`		`- (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) \|= (1U << outputChanIdx);`
	`254`	`+ (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MINTDIS(data->infoPtr->irqChanIdx, data->regNum)) \|= (1U << outputChanIdx);`
`255`	`255`	`}`
`256`	`256`
`257`	`257`	`/! @} /`
`@@ -286,7 +286,7 @@ static inline bool IRQSTEER_IsInterruptSet(int32_t instIdx, IRQn_Type irq)`
`286`	`286`	`regIdx = IRQSTEER_GEN_REG_IDX(data->regNum, inputIdx);`
`287`	`287`	`bitOffset = inputIdx % IRQSTEER_INT_SRC_REG_WIDTH;`
`288`	`288`
`289`		`- return (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum)) & (1U << bitOffset);`
	`289`	`+ return (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_STATUS(data->infoPtr->irqChanIdx, regIdx, data->regNum)) & (1U << bitOffset);`
`290`	`290`	`}`
`291`	`291`
`292`	`292`	`/*!`
`@@ -301,7 +301,7 @@ static inline bool IRQSTEER_IsMasterInterruptSet(int32_t instIdx)`
`301`	`301`	`{`
`302`	`302`	`irqsteer_data_t *data = IRQSTEER_GetIrqsteerData(instIdx);`
`303`	`303`
`304`		`- return (volatile uint32_t )(data->infoPtr->reg + IRQSTEER_CHAN_MSTRSTAT(data->infoPtr->irqChanIdx, data->regNum));`
	`304`	`+ return (volatile uint32_t )((uint32_t)data->infoPtr->reg + IRQSTEER_CHAN_MSTRSTAT(data->infoPtr->irqChanIdx, data->regNum));`
`305`	`305`	`}`
`306`	`306`
`307`	`307`	`/*!`