Unverified Commit fd843051 authored by Cristian Ciocaltea's avatar Cristian Ciocaltea Committed by Mark Brown
Browse files

ASoC: nau8821: Simplify conditional in nau8821_get_osr() 



Get rid of the unnecessary branch to address checkpatch complaint:

  WARNING: else is not generally useful after a break or return

Signed-off-by: default avatarCristian Ciocaltea <cristian.ciocaltea@collabora.com>
Link: https://patch.msgid.link/20251231-nau8821-cleanup-v1-5-6b0b76cbbb64@collabora.com


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

sound/soc/codecs/nau8821.c

+10 −6
Original line number Diff line number Diff line
@@ -806,17 +806,21 @@ nau8821_get_osr(struct nau8821 *nau8821, int stream) 
	if (stream == SNDRV_PCM_STREAM_PLAYBACK) {

		regmap_read(nau8821->regmap, NAU8821_R2C_DAC_CTRL1, &osr);

		osr &= NAU8821_DAC_OVERSAMPLE_MASK;



		if (osr >= ARRAY_SIZE(osr_dac_sel))

			return NULL;



		return &osr_dac_sel[osr];

	} else {

	}



	regmap_read(nau8821->regmap, NAU8821_R2B_ADC_RATE, &osr);

	osr &= NAU8821_ADC_SYNC_DOWN_MASK;



	if (osr >= ARRAY_SIZE(osr_adc_sel))

		return NULL;



	return &osr_adc_sel[osr];

}

}



static int nau8821_dai_startup(struct snd_pcm_substream *substream,

			       struct snd_soc_dai *dai)