2 Jun
2022
2 Jun
'22
6:26 p.m.
On Thu, Jun 02, 2022 at 11:29:20AM +0200, Mark Brown wrote:
Currently snd_soc_info_volsw_sx() is implemented indirectly, wrapping snd_soc_info_volsw() and modifying the values it sets up rather than directly setting up the values reported to userspace. This makes it much harder to follow what the intended behaviour of these controls is. Let's rewrite the function to be self contained with a clarifying comment at the top in an effort to help maintainability.
Signed-off-by: Mark Brown broonie@kernel.org
Reviewed-by: Charles Keepax ckeepax@opensource.cirrus.com
I didn't quite get around to testing this today but I will hopefully find a quick minute to test tomorrow/over the weekend. That said it looks good to me.
Thanks, Charles