]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ASoC: audio-graph-card2: Drop warning for manually selected DAI formats
authorAlvin Šipraga <alvin.sipraga@analog.com>
Wed, 17 Jun 2026 14:55:08 +0000 (16:55 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 18 Jun 2026 12:51:18 +0000 (13:51 +0100)
Sometimes the DAI format must be specified in the audio-graph-card2
device tree, so emitting a warning can be misleading. Revert back to
emitting no warning.

A few examples where automatic format selection might not be applicable:

- For DPCM, where the other side of the DAI link is not apparent, no
  proper selection can actually be made. This can lead to disagreeing
  formats.

- Due to hardware peculiarities, some ostensibly supported formats might
  not work in practice.

In either case, the only correct solution is for the sound card to set
the format

Link: https://lore.kernel.org/all/87ik7s36k2.wl-kuninori.morimoto.gx@renesas.com/
Signed-off-by: Alvin Šipraga <alvin.sipraga@analog.com>
Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://patch.msgid.link/20260617145508.327213-1-alvin@pqrs.dk
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/generic/audio-graph-card2.c

index 6894bb936cfd2cadd13ee8bdcf94d16376889e93..0202ed0ee78e87ffe614327ecc396e437cd6164c 100644 (file)
@@ -778,18 +778,6 @@ static void graph_link_init(struct simple_util_priv *priv,
        graph_parse_daifmt(ports_cpu,   &daifmt);
        graph_parse_daifmt(ports_codec, &daifmt);
        graph_parse_daifmt(lnk,         &daifmt);
-       if (daifmt) {
-               struct device *dev = simple_priv_to_dev(priv);
-
-               /*
-                * Recommend to use Auto Select by using .auto_selectable_formats.
-                * linux/sound/soc/renesas/rcar/core.c can be good sample for it.
-                *
-                * One note is that Audio Graph Card2 still keeps compatible to set
-                * DAI format via DT.
-                */
-               dev_warn_once(dev, "use .auto_selectable_formats on each corresponding CPU/Codec driver");
-       }
 
        graph_util_parse_link_direction(lnk,            &playback_only, &capture_only);
        graph_util_parse_link_direction(ports_cpu,      &playback_only, &capture_only);