Commit 31dbb503 authored by Stefan Binding's avatar Stefan Binding Committed by Takashi Iwai
Browse files

ALSA: hda: cs35l41: Fix endian conversions 



Found during static analysis, ensure variables are correct
types before endian conversion.

Signed-off-by: default avatarStefan Binding <sbinding@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230606103436.455348-3-sbinding@opensource.cirrus.com


Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 448425f0

sound/pci/hda/cs35l41_hda.c

+4 −4
Original line number Diff line number Diff line
@@ -308,8 +308,8 @@ static int cs35l41_request_firmware_files(struct cs35l41_hda *cs35l41, 
}



#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 @@ static int cs35l41_runtime_resume(struct device *dev) 


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;

	}