lkml.org 
[lkml]   [2022]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.19 0918/1157] ASoC: soc-core.c: fixup snd_soc_of_get_dai_link_cpus()
    Date
    From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

    [ Upstream commit 586fb2641371cf7f23a401ab1c79b17e3ec457f4 ]

    commit 900dedd7e47cc3f ("ASoC: Introduce snd_soc_of_get_dai_link_cpus")
    adds new snd_soc_of_get_dai_link_cpus(), but it is using
    "codec" everywhere. It is very strange, and is issue when error case.
    It should call cpu instead of codec in error case.
    This patch tidyup it.

    Fixes: 900dedd7e47cc3f ("ASoC: Introduce snd_soc_of_get_dai_link_cpus")
    Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
    Reviewed-by: Martin Povišer <povik+lin@cutebit.org>
    Link: https://lore.kernel.org/r/87zgi5p7k1.wl-kuninori.morimoto.gx@renesas.com
    Signed-off-by: Mark Brown <broonie@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    sound/soc/soc-core.c | 18 +++++++++---------
    1 file changed, 9 insertions(+), 9 deletions(-)

    diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
    index 9574f86dd4de..46f0e8eb79b3 100644
    --- a/sound/soc/soc-core.c
    +++ b/sound/soc/soc-core.c
    @@ -3433,26 +3433,26 @@ int snd_soc_of_get_dai_link_cpus(struct device *dev,
    struct of_phandle_args args;
    struct snd_soc_dai_link_component *component;
    char *name;
    - int index, num_codecs, ret;
    + int index, num_cpus, ret;

    - /* Count the number of CODECs */
    + /* Count the number of CPUs */
    name = "sound-dai";
    - num_codecs = of_count_phandle_with_args(of_node, name,
    + num_cpus = of_count_phandle_with_args(of_node, name,
    "#sound-dai-cells");
    - if (num_codecs <= 0) {
    - if (num_codecs == -ENOENT)
    + if (num_cpus <= 0) {
    + if (num_cpus == -ENOENT)
    dev_err(dev, "No 'sound-dai' property\n");
    else
    dev_err(dev, "Bad phandle in 'sound-dai'\n");
    - return num_codecs;
    + return num_cpus;
    }
    component = devm_kcalloc(dev,
    - num_codecs, sizeof(*component),
    + num_cpus, sizeof(*component),
    GFP_KERNEL);
    if (!component)
    return -ENOMEM;
    dai_link->cpus = component;
    - dai_link->num_cpus = num_codecs;
    + dai_link->num_cpus = num_cpus;

    /* Parse the list */
    for_each_link_cpus(dai_link, index, component) {
    @@ -3468,7 +3468,7 @@ int snd_soc_of_get_dai_link_cpus(struct device *dev,
    }
    return 0;
    err:
    - snd_soc_of_put_dai_link_codecs(dai_link);
    + snd_soc_of_put_dai_link_cpus(dai_link);
    dai_link->cpus = NULL;
    dai_link->num_cpus = 0;
    return ret;
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-08-16 02:44    [W:2.323 / U:0.736 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site