lkml.org 
[lkml]   [2017]   [Nov]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 094/193] ALSA: hda - Fix yet remaining issue with vmaster 0dB initialization
    Date
    4.14-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Takashi Iwai <tiwai@suse.de>

    commit d6c0615f510bc1ee26cfb2b9a3343ac99b9c46fb upstream.

    The previous fix for addressing the breakage in vmaster slave
    initialization, commit a91d66129fb9 ("ALSA: hda - Fix incorrect TLV
    callback check introduced during set_fs() removal"), introduced a new
    helper to process over each slave kctl. However, this helper passes
    only the original kctl, not the virtual slave kctl. As a result,
    HD-audio driver (which is the only user so far) couldn't initialize
    the slave correctly because it's trying to update the value directly
    with the original kctl, not with the mapped kctl.

    This patch fixes the situation again by passing both the mapped slaved
    and original slave kctls to the function. Luckily there is a single
    caller as of now, so changing the call signature is no big matter.

    Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=197959
    Fixes: a91d66129fb9 ("ALSA: hda - Fix incorrect TLV callback check introduced during set_fs() removal")
    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    include/sound/control.h | 4 +++-
    sound/core/vmaster.c | 6 ++++--
    sound/pci/hda/hda_codec.c | 10 +++++++---
    3 files changed, 14 insertions(+), 6 deletions(-)

    --- a/include/sound/control.h
    +++ b/include/sound/control.h
    @@ -249,7 +249,9 @@ int snd_ctl_add_vmaster_hook(struct snd_
    void snd_ctl_sync_vmaster(struct snd_kcontrol *kctl, bool hook_only);
    #define snd_ctl_sync_vmaster_hook(kctl) snd_ctl_sync_vmaster(kctl, true)
    int snd_ctl_apply_vmaster_slaves(struct snd_kcontrol *kctl,
    - int (*func)(struct snd_kcontrol *, void *),
    + int (*func)(struct snd_kcontrol *vslave,
    + struct snd_kcontrol *slave,
    + void *arg),
    void *arg);

    /*
    --- a/sound/core/vmaster.c
    +++ b/sound/core/vmaster.c
    @@ -495,7 +495,9 @@ EXPORT_SYMBOL_GPL(snd_ctl_sync_vmaster);
    * Returns 0 if successful, or a negative error code.
    */
    int snd_ctl_apply_vmaster_slaves(struct snd_kcontrol *kctl,
    - int (*func)(struct snd_kcontrol *, void *),
    + int (*func)(struct snd_kcontrol *vslave,
    + struct snd_kcontrol *slave,
    + void *arg),
    void *arg)
    {
    struct link_master *master;
    @@ -507,7 +509,7 @@ int snd_ctl_apply_vmaster_slaves(struct
    if (err < 0)
    return err;
    list_for_each_entry(slave, &master->slaves, list) {
    - err = func(&slave->slave, arg);
    + err = func(slave->kctl, &slave->slave, arg);
    if (err < 0)
    return err;
    }
    --- a/sound/pci/hda/hda_codec.c
    +++ b/sound/pci/hda/hda_codec.c
    @@ -1823,7 +1823,9 @@ struct slave_init_arg {
    };

    /* initialize the slave volume with 0dB via snd_ctl_apply_vmaster_slaves() */
    -static int init_slave_0dB(struct snd_kcontrol *kctl, void *_arg)
    +static int init_slave_0dB(struct snd_kcontrol *slave,
    + struct snd_kcontrol *kctl,
    + void *_arg)
    {
    struct slave_init_arg *arg = _arg;
    int _tlv[4];
    @@ -1860,7 +1862,7 @@ static int init_slave_0dB(struct snd_kco
    arg->step = step;
    val = -tlv[2] / step;
    if (val > 0) {
    - put_kctl_with_value(kctl, val);
    + put_kctl_with_value(slave, val);
    return val;
    }

    @@ -1868,7 +1870,9 @@ static int init_slave_0dB(struct snd_kco
    }

    /* unmute the slave via snd_ctl_apply_vmaster_slaves() */
    -static int init_slave_unmute(struct snd_kcontrol *slave, void *_arg)
    +static int init_slave_unmute(struct snd_kcontrol *slave,
    + struct snd_kcontrol *kctl,
    + void *_arg)
    {
    return put_kctl_with_value(slave, 1);
    }

    \
     
     \ /
      Last update: 2017-11-28 12:16    [W:4.918 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site