Commit cfacadbd authored by Charles Keepax's avatar Charles Keepax Committed by Mark Brown

ASoC: cros_ec_codec: Add endianness flag in i2s_rx_component_driver

The endianness flag is used on the CODEC side to specify an
ambivalence to endian, typically because it is lost over the hardware
link. The i2s_rx component receives audio over an I2S DAI and as such
should have endianness applied.

A fixup is also required to use the width directly rather than relying
on the format in hw_params, now both little and big endian would be
supported.
Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220504170905.332415-27-ckeepax@opensource.cirrus.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent ff69ec96
...@@ -232,11 +232,11 @@ static int i2s_rx_hw_params(struct snd_pcm_substream *substream, ...@@ -232,11 +232,11 @@ static int i2s_rx_hw_params(struct snd_pcm_substream *substream,
if (params_rate(params) != 48000) if (params_rate(params) != 48000)
return -EINVAL; return -EINVAL;
switch (params_format(params)) { switch (params_width(params)) {
case SNDRV_PCM_FORMAT_S16_LE: case 16:
depth = EC_CODEC_I2S_RX_SAMPLE_DEPTH_16; depth = EC_CODEC_I2S_RX_SAMPLE_DEPTH_16;
break; break;
case SNDRV_PCM_FORMAT_S24_LE: case 24:
depth = EC_CODEC_I2S_RX_SAMPLE_DEPTH_24; depth = EC_CODEC_I2S_RX_SAMPLE_DEPTH_24;
break; break;
default: default:
...@@ -387,6 +387,7 @@ static const struct snd_soc_component_driver i2s_rx_component_driver = { ...@@ -387,6 +387,7 @@ static const struct snd_soc_component_driver i2s_rx_component_driver = {
.num_dapm_widgets = ARRAY_SIZE(i2s_rx_dapm_widgets), .num_dapm_widgets = ARRAY_SIZE(i2s_rx_dapm_widgets),
.dapm_routes = i2s_rx_dapm_routes, .dapm_routes = i2s_rx_dapm_routes,
.num_dapm_routes = ARRAY_SIZE(i2s_rx_dapm_routes), .num_dapm_routes = ARRAY_SIZE(i2s_rx_dapm_routes),
.endianness = 1,
}; };
static void *wov_map_shm(struct cros_ec_codec_priv *priv, static void *wov_map_shm(struct cros_ec_codec_priv *priv,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment