diff options
author | Stefan Binding <sbinding@opensource.cirrus.com> | 2023-06-06 11:34:35 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2023-06-06 14:32:05 +0200 |
commit | 31dbb503f07a059419f16773e47df4a31093ba31 (patch) | |
tree | 699cfd00e15f3c9c348482766836b6b238b995d6 /sound | |
parent | 448425f05b16f124290ad82b836c04da63b63035 (diff) | |
download | linux-stable-31dbb503f07a059419f16773e47df4a31093ba31.tar.gz linux-stable-31dbb503f07a059419f16773e47df4a31093ba31.tar.bz2 linux-stable-31dbb503f07a059419f16773e47df4a31093ba31.zip |
ALSA: hda: cs35l41: Fix endian conversions
Found during static analysis, ensure variables are correct
types before endian conversion.
Signed-off-by: Stefan Binding <sbinding@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230606103436.455348-3-sbinding@opensource.cirrus.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/pci/hda/cs35l41_hda.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sound/pci/hda/cs35l41_hda.c b/sound/pci/hda/cs35l41_hda.c index d100189e15b8..ce5faa620517 100644 --- a/sound/pci/hda/cs35l41_hda.c +++ b/sound/pci/hda/cs35l41_hda.c @@ -308,8 +308,8 @@ out: } #if IS_ENABLED(CONFIG_EFI) -static int cs35l41_apply_calibration(struct cs35l41_hda *cs35l41, unsigned int ambient, - unsigned int r0, unsigned int status, unsigned int checksum) +static int cs35l41_apply_calibration(struct cs35l41_hda *cs35l41, __be32 ambient, __be32 r0, + __be32 status, __be32 checksum) { int ret; @@ -745,7 +745,7 @@ err: static int cs35l41_smart_amp(struct cs35l41_hda *cs35l41) { - int halo_sts; + __be32 halo_sts; int ret; ret = cs35l41_init_dsp(cs35l41); @@ -773,7 +773,7 @@ static int cs35l41_smart_amp(struct cs35l41_hda *cs35l41) &halo_sts, sizeof(halo_sts)); if (ret) { - dev_err(cs35l41->dev, "Timeout waiting for HALO Core to start. State: %d\n", + dev_err(cs35l41->dev, "Timeout waiting for HALO Core to start. State: %u\n", halo_sts); goto clean_dsp; } |