Skip to content

Commit fbe4a94

Browse files
LaurentiuM1234dleach02
authored andcommitted
drivers: irqsteer: adjust CHn_MASK index computation
For IRQ_STEER, lower CHn_MASK register indexes are used to mask higher interrupt IDs. For instance, in the case of i.MX8MP, the mapping is as follows: CHn_MASK4 => masks interrupts [31:0] CHn_MASK3 => masks interrupts [63:32] CHn_MASK2 => masks interrupts [95:64] CHn_MASK1 => masks interrupts [127:96] CHn_MASK0 => masks interrupts [159:128] The `IRQSTEER_GetRegIndex()` function is used to fetch the CHn_MASK register index based on a given slice. The term "slice" is used to refer to an index of a CHn_MASK register in the set of CHn_MASK registers assigned to a certain master. Assuming the following partition scheme (i.MX8MP): { CHn_MASK4 } is assigned to MASTER0 { CHn_MASK3, CHn_MASK2 } is assigned to MASTER1 { CHn_MASK1, CHn_MASK0 } is assiged to MASTER2 CHn_MASK3 would be at slice (index) 0, CHn_MASK2 would be at slice 1, CHn_MASK1 would be at slice 0 and so on. To compute the CHn_MASK register index found at a given slice, `IRQSTEER_GetRegIndex()` uses a base index, which is either the lowest or the highest CHn_MASK register index in a master's partition (for instance, for MASTER1 the higher would be 3 and the lowest would be 2). For IRQ_STEER instances with an uneven number of CHn_MASK registers the base is the lowest index, while in the case of instances with an even number of CHn_MASK registers it's exactly the opposite. This is an issue because the software using this function might expect the base to either be the lowest or the highest index ALL THE TIME (since this affects the order in which the CHn_MASK register indexes are returned). As such, fix this problem by making the base the highest CHn_MASK register index. Signed-off-by: Laurentiu Mihalcea <[email protected]>
1 parent 4a4741f commit fbe4a94

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

mcux/mcux-sdk/drivers/irqsteer/fsl_irqsteer.c

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -168,15 +168,16 @@ uint32_t IRQSTEER_GetMasterIrqCount(IRQSTEER_Type *base, irqsteer_int_master_t i
168168
return count;
169169
}
170170

171-
static uint32_t IRQSTEER_GetRegIndex(irqsteer_int_master_t intMasterIndex, uint32_t slice)
171+
static uint32_t IRQSTEER_GetRegIndex(irqsteer_int_master_t intMasterIndex,
172+
uint32_t slice, uint32_t sliceNum)
172173
{
173174
uint32_t base = FSL_FEATURE_IRQSTEER_CHn_MASK_COUNT - 1 - intMasterIndex * 2;
174175

175176
if (FSL_FEATURE_IRQSTEER_CHn_MASK_COUNT % 2) {
176-
return base + slice;
177-
} else {
178-
return base - slice;
177+
base += sliceNum - 1;
179178
}
179+
180+
return base - slice;
180181
}
181182

182183
/*!
@@ -225,7 +226,7 @@ IRQn_Type IRQSTEER_GetMasterNextInterrupt(IRQSTEER_Type *base, irqsteer_int_mast
225226
bitOffset = 0;
226227

227228
/* compute the index of the register to be queried */
228-
regIndex = IRQSTEER_GetRegIndex(intMasterIndex, i);
229+
regIndex = IRQSTEER_GetRegIndex(intMasterIndex, i, sliceNum + 1);
229230

230231
/* get register's value */
231232
chanStatus = base->CHn_STATUS[regIndex];
@@ -255,7 +256,7 @@ uint64_t IRQSTEER_GetMasterInterruptsStatus(IRQSTEER_Type *base, irqsteer_int_ma
255256
sliceNum = IRQSTEER_GetMasterIrqCount(base, intMasterIndex) / 32 - 1;
256257

257258
for (i = 0; i <= sliceNum; i++) {
258-
regIndex = IRQSTEER_GetRegIndex(intMasterIndex, i);
259+
regIndex = IRQSTEER_GetRegIndex(intMasterIndex, i, sliceNum + 1);
259260

260261
chanStatus = base->CHn_STATUS[regIndex];
261262

0 commit comments

Comments
 (0)