@@ -738,8 +738,8 @@ static int rsnd_soc_set_dai_tdm_slot(struct snd_soc_dai *dai, case 32: break; default:- dev_err(dev, "unsupported slot width value: %d\n", slot_width);- return -EINVAL;+ /* use default */+ slot_width = 32; } switch (slots) {I don't think I like this. People should not be passing strange values to thisfunction. Can the caller be fixed, instead?Morimoto San, Do you agree to Pavel's comment?
@@ -738,8 +738,8 @@ static int rsnd_soc_set_dai_tdm_slot(struct snd_soc_dai *dai, case 32: break; default:- dev_err(dev, "unsupported slot width value: %d\n", slot_width);- return -EINVAL;+ /* use default */+ slot_width = 32; } switch (slots) {I don't think I like this. People should not be passing strange values to thisfunction. Can the caller be fixed, instead?
@@ -738,8 +738,8 @@ static int rsnd_soc_set_dai_tdm_slot(struct snd_soc_dai *dai, case 32: break; default:- dev_err(dev, "unsupported slot width value: %d\n", slot_width);- return -EINVAL;+ /* use default */+ slot_width = 32; } switch (slots) {
© 2023 Groups.io