Re: [PATCH v2 2/9] ASoC: codecs: framer-codec: don't use array if single pattarn

[Herve Codina <herve.codina@bootlin.com>]

Hi Kuninori

On Mon, 8 Jun 2026 00:57:53 +0000
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote:

Still the typo in commit title
s/pattarn/pattern/

Best regards,
Hervé

> Because it is confusable during debugging ASoC FW update, tidyup
> auto format style not to use array if single pattern case.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  sound/soc/codecs/framer-codec.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/sound/soc/codecs/framer-codec.c b/sound/soc/codecs/framer-codec.c
> index 6f57a3aeecc89..a87a78390172d 100644
> --- a/sound/soc/codecs/framer-codec.c
> +++ b/sound/soc/codecs/framer-codec.c
> @@ -238,15 +238,13 @@ static int framer_dai_startup(struct snd_pcm_substream *substream,
>  	return 0;
>  }
>  
> -static const u64 framer_dai_formats[] = {
> -	SND_SOC_POSSIBLE_DAIFMT_DSP_B,
> -};
> +static const u64 framer_dai_formats = SND_SOC_POSSIBLE_DAIFMT_DSP_B;
>  
>  static const struct snd_soc_dai_ops framer_dai_ops = {
>  	.startup	= framer_dai_startup,
>  	.set_tdm_slot	= framer_dai_set_tdm_slot,
> -	.auto_selectable_formats     = framer_dai_formats,
> -	.num_auto_selectable_formats = ARRAY_SIZE(framer_dai_formats),
> +	.auto_selectable_formats     = &framer_dai_formats,
> +	.num_auto_selectable_formats = 1,
>  };
>  
>  static struct snd_soc_dai_driver framer_dai_driver = {



-- 
Hervé Codina, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com