-----Original Message-----
From: Pavel Machek <pavel@...>
Sent: Sunday, October 20, 2019 10:49 AM
To: Biju Das <biju.das@...>
Cc: cip-dev@...; Nobuhiro Iwamatsu
<nobuhiro1.iwamatsu@...>; Pavel Machek <pavel@...>;
Chris Paterson <Chris.Paterson2@...>; Fabrizio Castro
<fabrizio.castro@...>
Subject: Re: [PATCH 4.19.y-cip 19/57] ASoC: rsnd: use 32bit TDM width as
default

On Thu 2019-10-17 08:04:51, Biju Das wrote:

From: Kuninori Morimoto <kuninori.morimoto.gx@...>

commit 82ab7e9a4d3fcec27f745be04063e17da1881dda upstream.

commit fb2815f44a9e ("ASoC: rsnd: add support for 16/24 bit slot
widths") added TDM width check, and return error if it was not 16/24/32

bit.

But it is too strict. This patch uses 32bit same as default.

@@ -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 this
function. Can the caller be fixed, instead?