ASoC: amd: acp: Fix initialization of ext_intr_stat1 in i2s_irq_handler()
Subsystems (3)
PATCH 1/1 ASoC: amd: acp: Fix initialization of ext_intr_stat1 in i2s_irq_handler()
Clang warns:
../sound/soc/amd/acp/acp-platform.c:117:19: error: variable 'ext_intr_stat1' is uninitialized when used here [-Werror,-Wuninitialized]
if (stream && (ext_intr_stat1 & stream->irq_bit)) {
^~~~~~~~~~~~~~
../sound/soc/amd/acp/acp-platform.c:97:35: note: initialize the variable 'ext_intr_stat1' to silence this warning
u32 ext_intr_stat, ext_intr_stat1, i;
^
= 01 error generated.
The variable was not properly renamed, correct it to resolve the warning.
--- sound/soc/amd/acp/acp-platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
sound/soc/amd/acp/acp-platform.c +1 -1
@@ -100,7 +100,7 @@ static irqreturn_t i2s_irq_handler(int irq, void *data)return IRQ_NONE ;if (adata->rsrc->no_of_ctrls ==2 ) - ext_intr_stat =readl (ACP_EXTERNAL_INTR_STAT (adata, (rsrc->irqp_used -1 ))); + ext_intr_stat1 =readl (ACP_EXTERNAL_INTR_STAT (adata, (rsrc->irqp_used -1 ))); ext_intr_stat =readl (ACP_EXTERNAL_INTR_STAT (adata, rsrc->irqp_used));
base-commit: be0aa8d4b0fcb4532bf7973141e911998ab39508
Signature · 1 line
2.37.1
Quoted 11 lines · Nathan Chancellor
Clang warns:
../sound/soc/amd/acp/acp-platform.c:117:19: error: variable 'ext_intr_stat1' is uninitialized when used here [-Werror,-Wuninitialized] if (stream && (ext_intr_stat1 & stream->irq_bit)) { ^~~~~~~~~~~~~~ ../sound/soc/amd/acp/acp-platform.c:97:35: note: initialize the variable 'ext_intr_stat1' to silence this warning u32 ext_intr_stat, ext_intr_stat1, i; ^ = 0 1 error generated.
Quoted 6 lines
The variable was not properly renamed, correct it to resolve the warning.
Fixes 93f53881473c ("ASoC: amd: acp: Modify local variables name to generic")Signed-off-by Nathan Chancellor <[email protected]>
Quoted 19 lines
--- sound/soc/amd/acp/acp-platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)sound/soc/amd/acp/acp-platform.c +1
-1@@ -100,7 +100,7 @@ static irqreturn_t i2s_irq_handler(int irq, void *data)return IRQ_NONE ;if (adata->rsrc->no_of_ctrls ==2 ) - ext_intr_stat =readl (ACP_EXTERNAL_INTR_STAT (adata, (rsrc->irqp_used -1 ))); + ext_intr_stat1 =readl (ACP_EXTERNAL_INTR_STAT (adata, (rsrc->irqp_used -1 ))); ext_intr_stat =readl (ACP_EXTERNAL_INTR_STAT (adata, rsrc->irqp_used));base-commit:
be0aa8d4b0fcb4532bf7973141e911998ab39508
Mark Brown
maintainerQuoted 9 lines · Nathan Chancellor
Clang warns:
../sound/soc/amd/acp/acp-platform.c:117:19: error: variable 'ext_intr_stat1' is uninitialized when used here [-Werror,-Wuninitialized]if (stream && (ext_intr_stat1 & stream->irq_bit)) { ^~~~~~~~~~~~~~ ../sound/soc/amd/acp/acp-platform.c:97:35: note: initialize the variable 'ext_intr_stat1' to silence this warning u32 ext_intr_stat, ext_intr_stat1, i; ^ = 0
Quoted 3 lines
1 error generated.
[...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/1] ASoC: amd: acp: Fix initialization of ext_intr_stat1 in i2s_irq_handler()
commit: d81677410f172c2b946393c09b46ff9e8dc1b6ec
All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying to this mail.
Thanks,
Mark