ASoC: loongson: change the type of variable irq to int
authorYingKun Meng <mengyingkun@loongson.cn>
Mon, 19 Jun 2023 07:46:49 +0000 (15:46 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 19 Jun 2023 11:59:33 +0000 (12:59 +0100)
We use variable 'irq' to store the return value of fwnode_get_irq_byname().
A negative value indicates that the operation failed. If the type of 'irq'
is unsigned int, we never know if the operation failed.

Reported-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Closes: https://lore.kernel.org/loongarch/325dd825-6fa5-0ebc-4b7e-7acf2d2840e4@loongson.cn/
Signed-off-by: YingKun Meng <mengyingkun@loongson.cn>
Link: https://lore.kernel.org/r/20230619074649.3608726-1-mengyingkun@loongson.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/loongson/loongson_i2s.h

index 52788f6..89492ee 100644 (file)
@@ -45,7 +45,7 @@
 struct loongson_dma_data {
        dma_addr_t dev_addr;            /* device physical address for DMA */
        void __iomem *order_addr;       /* DMA order register */
-       u32 irq;                        /* DMA irq */
+       int irq;                        /* DMA irq */
 };
 
 struct loongson_i2s {