diff options
| author | YingKun Meng <mengyingkun@loongson.cn> | 2023-06-19 15:46:49 +0800 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-06-19 12:59:33 +0100 |
| commit | 1650e8a8818d516219b2c0cbc203f53cc6cd77a0 (patch) | |
| tree | 20d0e3f23a2387ed7b6968b52883b2fd42c88205 | |
| parent | 3e4ecd6c4e14e1eff8f52bd89240399e7dac881c (diff) | |
| download | lwn-1650e8a8818d516219b2c0cbc203f53cc6cd77a0.tar.gz lwn-1650e8a8818d516219b2c0cbc203f53cc6cd77a0.zip | |
ASoC: loongson: change the type of variable irq to int
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>
| -rw-r--r-- | sound/soc/loongson/loongson_i2s.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/loongson/loongson_i2s.h b/sound/soc/loongson/loongson_i2s.h index 52788f6a94ad..89492eebf834 100644 --- a/sound/soc/loongson/loongson_i2s.h +++ b/sound/soc/loongson/loongson_i2s.h @@ -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 { |