From: wangdicheng <wangdicheng@kylinos.cn>
This series fixes three compiler warnings in the sound subsystem,
covering both logical redundancies and a potential runtime bug.
Two categories of warnings are addressed:
1. "possible condition with no effect (if == else)" -- redundant
conditional branches that produce identical code.
2. "do_div() does a 64-by-32 division" -- use of do_div() with
a 64-bit divisor, which silently truncates the upper 32 bits.
Changes in v2:
- Dropped "ASoC: tlv320aic32x4: Use div64_ul for division by unsigned long".
Per David's review, parent_rate is bounded by AIC32X4_MAX_PLL_CLKIN
(20MHz) and will never exceed 32 bits, so do_div() is fine there.
- Updated patch 3 (fsl_easrc) commit message with concrete numerical
proof that val1 can exceed 32 bits after >> 12, and clarified that
the truncation happens when val1 is used as a divisor.
wangdicheng (3):
ASoC: mediatek: mt8189: Remove redundant else-if branch with identical
body
ALSA: sparc/dbri: Fix "possible condition with no effect" warning
ASoC: fsl_easrc: Use div64_u64 for 64-by-64 division
sound/soc/fsl/fsl_easrc.c | 2 +-
sound/soc/mediatek/mt8189/mt8189-dai-adda.c | 2 --
sound/sparc/dbri.c | 2 ++
3 files changed, 3 insertions(+), 3 deletions(-)
--
2.25.1