ASoC: SOF: Intel: hda-dai: use lowest channels

bardliao · bardliao · commit 5b7f7fa4ede5 · 2024-01-26T18:08:44.000+08:00
The ch_mask in a link indicates the locations of the channels.
However, the channels used in each stream are the lowest channels.
So convert ch_mask to GENMASK(hweight_long(ch_mask) - 1, 0).

Signed-off-by: Bard Liao &lt;yung-chuan.liao@linux.intel.com&gt;
diff --git a/sound/soc/sof/intel/hda-dai.c b/sound/soc/sof/intel/hda-dai.c
@@ -510,6 +510,7 @@ int sdw_hda_dai_hw_params(struct snd_pcm_substream *substream,
 		if (ch_maps->cpu == cpu_dai_id)
 			ch_mask |= ch_maps->ch_mask;
 	}
+	ch_mask = GENMASK(hweight_long(ch_mask) - 1, 0);
 
 	ret = hdac_bus_eml_sdw_map_stream_ch(sof_to_bus(sdev), link_id, cpu_dai->id,
 					     ch_mask,

Original file line number	Diff line number	Diff line change
`@@ -510,6 +510,7 @@ int sdw_hda_dai_hw_params(struct snd_pcm_substream *substream,`
`510`	`510`	`if (ch_maps->cpu == cpu_dai_id)`
`511`	`511`	`ch_mask \|= ch_maps->ch_mask;`
`512`	`512`	`}`
	`513`	`+ ch_mask = GENMASK(hweight_long(ch_mask) - 1, 0);`
`513`	`514`
`514`	`515`	`ret = hdac_bus_eml_sdw_map_stream_ch(sof_to_bus(sdev), link_id, cpu_dai->id,`
`515`	`516`	`ch_mask,`