drivers: dma: fix off-by-one error in silabs LDMA

kartben · danieldegrasse · commit bc03f23d5069 · 2025-06-16T10:43:00.000-04:00
Channel numbers are 0-based so a channel number equal to the number of
channels is invalid.

Signed-off-by: Benjamin Cabé &lt;benjamin@zephyrproject.org&gt;
diff --git a/drivers/dma/dma_silabs_ldma.c b/drivers/dma/dma_silabs_ldma.c
@@ -329,7 +329,7 @@ static int dma_silabs_configure(const struct device *dev, uint32_t channel,
 	LDMA_TransferCfg_t *xfer_config = &chan_conf->xfer_config;
 	int ret;
 
-	if (channel > data->dma_ctx.dma_channels) {
+	if (channel >= data->dma_ctx.dma_channels) {
 		return -EINVAL;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,7 @@ static int dma_silabs_configure(const struct device *dev, uint32_t channel,`
`329`	`329`	`LDMA_TransferCfg_t *xfer_config = &chan_conf->xfer_config;`
`330`	`330`	`int ret;`
`331`	`331`
`332`		`- if (channel > data->dma_ctx.dma_channels) {`
	`332`	`+ if (channel >= data->dma_ctx.dma_channels) {`
`333`	`333`	`return -EINVAL;`
`334`	`334`	`}`
`335`	`335`