Unverified Commit f5036db3 authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Mark Brown
Browse files

ASoC: fsl: mpc8610: remove useless assignment 



cppcheck warning:

sound/soc/fsl/mpc8610_hpcd.c:333:6: style: Redundant initialization
for 'ret'. The initialized value is overwritten before it is
read. [redundantInitialization]
 ret = fsl_asoc_get_dma_channel(np, "fsl,playback-dma",
     ^
sound/soc/fsl/mpc8610_hpcd.c:193:10: note: ret is initialized
 int ret = -ENODEV;
         ^
sound/soc/fsl/mpc8610_hpcd.c:333:6: note: ret is overwritten
 ret = fsl_asoc_get_dma_channel(np, "fsl,playback-dma",
     ^

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210219232937.6440-9-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 91ab7743

sound/soc/fsl/mpc8610_hpcd.c

+1 −1
Original line number Diff line number Diff line
@@ -190,7 +190,7 @@ static int mpc8610_hpcd_probe(struct platform_device *pdev) 
	struct device_node *codec_np = NULL;

	struct mpc8610_hpcd_data *machine_data;

	struct snd_soc_dai_link_component *comp;

	int ret = -ENODEV;

	int ret;

	const char *sprop;

	const u32 *iprop;