Unverified Commit 99a4b91a authored by Charles Keepax's avatar Charles Keepax Committed by Mark Brown
Browse files

ASoC: cs4270: Remove redundant big endian formats 



The CODEC already provides the endianness flag on its
snd_soc_component_driver structure, specifying it is ambivalent
to endian. The core will expand the formats to cover both
endian types, as such remove the redundant specification of both
endians.

Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220504170905.332415-5-ckeepax@opensource.cirrus.com


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

sound/soc/codecs/cs4270.c

+3 −12
Original line number Diff line number Diff line
@@ -32,18 +32,9 @@ 
#include <linux/gpio/consumer.h>

#include <linux/of_device.h>



/*

 * The codec isn't really big-endian or little-endian, since the I2S

 * interface requires data to be sent serially with the MSbit first.

 * However, to support BE and LE I2S devices, we specify both here.  That

 * way, ALSA will always match the bit patterns.

 */

#define CS4270_FORMATS (SNDRV_PCM_FMTBIT_S8      | \

			SNDRV_PCM_FMTBIT_S16_LE  | SNDRV_PCM_FMTBIT_S16_BE  | \

			SNDRV_PCM_FMTBIT_S18_3LE | SNDRV_PCM_FMTBIT_S18_3BE | \

			SNDRV_PCM_FMTBIT_S20_3LE | SNDRV_PCM_FMTBIT_S20_3BE | \

			SNDRV_PCM_FMTBIT_S24_3LE | SNDRV_PCM_FMTBIT_S24_3BE | \

			SNDRV_PCM_FMTBIT_S24_LE  | SNDRV_PCM_FMTBIT_S24_BE)

#define CS4270_FORMATS (SNDRV_PCM_FMTBIT_S8      | SNDRV_PCM_FMTBIT_S16_LE  | \

			SNDRV_PCM_FMTBIT_S18_3LE | SNDRV_PCM_FMTBIT_S20_3LE | \

			SNDRV_PCM_FMTBIT_S24_3LE | SNDRV_PCM_FMTBIT_S24_LE)



/* CS4270 registers addresses */

#define CS4270_CHIPID	0x01	/* Chip ID */