dmaengine: pl08x: constify pointer to char in filter function

krzk · vinodkoul · commit 16374aba8249 · 2024-02-23T12:44:08.000+05:30
The opaque argument chan_id passed to filter function is actually pointer to const memory, so make that obvious in the filter for code readability and safety. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Link: https://lore.kernel.org/r/20240208202742.631307-1-krzysztof.kozlowski@linaro.org Signed-off-by: Vinod Koul <vkoul@kernel.org>
diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
@@ -2239,7 +2239,7 @@ static int pl08x_resume(struct dma_chan *chan)
 bool pl08x_filter_id(struct dma_chan *chan, void *chan_id)
 {
 	struct pl08x_dma_chan *plchan;
-	char *name = chan_id;
+	const char *name = chan_id;
 
 	/* Reject channels for devices not bound to this driver */
 	if (chan->device->dev->driver != &pl08x_amba_driver.drv)

Original file line number	Diff line number	Diff line change
`@@ -2239,7 +2239,7 @@ static int pl08x_resume(struct dma_chan *chan)`
`2239`	`2239`	`bool pl08x_filter_id(struct dma_chan chan, void chan_id)`
`2240`	`2240`	`{`
`2241`	`2241`	`struct pl08x_dma_chan *plchan;`
`2242`		`- char *name = chan_id;`
	`2242`	`+ const char *name = chan_id;`
`2243`	`2243`
`2244`	`2244`	`/* Reject channels for devices not bound to this driver */`
`2245`	`2245`	`if (chan->device->dev->driver != &pl08x_amba_driver.drv)`