Commit 219af251 authored by Peter Ujfalusi's avatar Peter Ujfalusi Committed by Mark Brown

ASoC: Intel: Skylake: Correct the ssp rate discovery in skl_get_ssp_clks()

The present flag is only set once when one rate has been found to be saved.
This will effectively going to ignore any rate discovered at later time and
based on the code, this is not the intention.

Fixes: bc2bd45b ("ASoC: Intel: Skylake: Parse nhlt and register clock device")
Signed-off-by: default avatarPeter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: default avatarCezary Rojewski <cezary.rojewski@intel.com>
Link: https://lore.kernel.org/r/20220630065638.11183-2-peter.ujfalusi@linux.intel.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 051dade3
...@@ -99,7 +99,6 @@ static void skl_get_ssp_clks(struct skl_dev *skl, struct skl_ssp_clk *ssp_clks, ...@@ -99,7 +99,6 @@ static void skl_get_ssp_clks(struct skl_dev *skl, struct skl_ssp_clk *ssp_clks,
struct nhlt_fmt_cfg *fmt_cfg; struct nhlt_fmt_cfg *fmt_cfg;
struct wav_fmt_ext *wav_fmt; struct wav_fmt_ext *wav_fmt;
unsigned long rate; unsigned long rate;
bool present = false;
int rate_index = 0; int rate_index = 0;
u16 channels, bps; u16 channels, bps;
u8 clk_src; u8 clk_src;
...@@ -113,6 +112,8 @@ static void skl_get_ssp_clks(struct skl_dev *skl, struct skl_ssp_clk *ssp_clks, ...@@ -113,6 +112,8 @@ static void skl_get_ssp_clks(struct skl_dev *skl, struct skl_ssp_clk *ssp_clks,
return; return;
for (i = 0; i < fmt->fmt_count; i++) { for (i = 0; i < fmt->fmt_count; i++) {
bool present = false;
fmt_cfg = &fmt->fmt_config[i]; fmt_cfg = &fmt->fmt_config[i];
wav_fmt = &fmt_cfg->fmt_ext; wav_fmt = &fmt_cfg->fmt_ext;
......
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