| Message ID | 20190620134708.28311-1-enric.balletbo@collabora.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ASoC: rk3399_gru_sound: Support 32, 44.1 and 88.2 kHz sample rates | expand |
On Thu, Jun 20, 2019 at 03:47:08PM +0200, Enric Balletbo i Serra wrote: > According to the datasheet the max98357a also supports 32, 44.1 and > 88.2 kHz sample rate. This support was also introduced recently by > commit fdf34366d324 ("ASoC: max98357a: add missing supported rates"). > This patch adds support for these rates also for the machine driver so > we get rid of the errors like the below and we are able to play files > using these sample rates. Does the machine actually need to validate this at all? The component drivers can all apply whatever constraints are needed and do their own validation, the machine driver is just getting in the way here.
Hi Mark, Missatge de Mark Brown <broonie@kernel.org> del dia dj., 20 de juny 2019 a les 17:42: > > On Thu, Jun 20, 2019 at 03:47:08PM +0200, Enric Balletbo i Serra wrote: > > According to the datasheet the max98357a also supports 32, 44.1 and > > 88.2 kHz sample rate. This support was also introduced recently by > > commit fdf34366d324 ("ASoC: max98357a: add missing supported rates"). > > This patch adds support for these rates also for the machine driver so > > we get rid of the errors like the below and we are able to play files > > using these sample rates. > > Does the machine actually need to validate this at all? The component > drivers can all apply whatever constraints are needed and do their own > validation, the machine driver is just getting in the way here. I think you have reason, I'll test by removing these validation and respin the patch. Thanks, ~ Enric > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip
diff --git a/sound/soc/rockchip/rk3399_gru_sound.c b/sound/soc/rockchip/rk3399_gru_sound.c index 3d0cc6e90d7b..8dfe1a560e42 100644 --- a/sound/soc/rockchip/rk3399_gru_sound.c +++ b/sound/soc/rockchip/rk3399_gru_sound.c @@ -59,7 +59,10 @@ static int rockchip_sound_max98357a_hw_params(struct snd_pcm_substream *substrea switch (params_rate(params)) { case 8000: case 16000: + case 32000: + case 44100: case 48000: + case 88200: case 96000: mclk = params_rate(params) * SOUND_FS; break;
According to the datasheet the max98357a also supports 32, 44.1 and 88.2 kHz sample rate. This support was also introduced recently by commit fdf34366d324 ("ASoC: max98357a: add missing supported rates"). This patch adds support for these rates also for the machine driver so we get rid of the errors like the below and we are able to play files using these sample rates. rk3399-gru-sound sound: rockchip_sound_max98357a_hw_params() doesn't support this sample rate: 44100 rk3399-gru-sound sound: ASoC: machine hw_params failed: -22 Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com> --- sound/soc/rockchip/rk3399_gru_sound.c | 3 +++ 1 file changed, 3 insertions(+)