mirror of
https://github.com/followmsi/android_kernel_google_msm.git
synced 2024-11-06 23:17:41 +00:00
ALSA: hda - Move one-time init codes from generic_hdmi_init()
commit 8b8d654b55
upstream.
The codes to initialize work struct or create a proc interface should
be called only once and never although it's called many times through
the init callback. Move that stuff into patch_generic_hdmi() so that
it's called only once.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
BugLink: https://bugs.launchpad.net/bugs/1212160
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
a275776838
commit
1ddb811a54
1 changed files with 17 additions and 5 deletions
|
@ -1282,6 +1282,22 @@ static int generic_hdmi_build_controls(struct hda_codec *codec)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int generic_hdmi_init_per_pins(struct hda_codec *codec)
|
||||||
|
{
|
||||||
|
struct hdmi_spec *spec = codec->spec;
|
||||||
|
int pin_idx;
|
||||||
|
|
||||||
|
for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) {
|
||||||
|
struct hdmi_spec_per_pin *per_pin = &spec->pins[pin_idx];
|
||||||
|
struct hdmi_eld *eld = &per_pin->sink_eld;
|
||||||
|
|
||||||
|
per_pin->codec = codec;
|
||||||
|
INIT_DELAYED_WORK(&per_pin->work, hdmi_repoll_eld);
|
||||||
|
snd_hda_eld_proc_new(codec, eld, pin_idx);
|
||||||
|
}
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
static int generic_hdmi_init(struct hda_codec *codec)
|
static int generic_hdmi_init(struct hda_codec *codec)
|
||||||
{
|
{
|
||||||
struct hdmi_spec *spec = codec->spec;
|
struct hdmi_spec *spec = codec->spec;
|
||||||
|
@ -1290,14 +1306,9 @@ static int generic_hdmi_init(struct hda_codec *codec)
|
||||||
for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) {
|
for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) {
|
||||||
struct hdmi_spec_per_pin *per_pin = &spec->pins[pin_idx];
|
struct hdmi_spec_per_pin *per_pin = &spec->pins[pin_idx];
|
||||||
hda_nid_t pin_nid = per_pin->pin_nid;
|
hda_nid_t pin_nid = per_pin->pin_nid;
|
||||||
struct hdmi_eld *eld = &per_pin->sink_eld;
|
|
||||||
|
|
||||||
hdmi_init_pin(codec, pin_nid);
|
hdmi_init_pin(codec, pin_nid);
|
||||||
snd_hda_jack_detect_enable(codec, pin_nid, pin_nid);
|
snd_hda_jack_detect_enable(codec, pin_nid, pin_nid);
|
||||||
|
|
||||||
per_pin->codec = codec;
|
|
||||||
INIT_DELAYED_WORK(&per_pin->work, hdmi_repoll_eld);
|
|
||||||
snd_hda_eld_proc_new(codec, eld, pin_idx);
|
|
||||||
}
|
}
|
||||||
snd_hda_jack_report_sync(codec);
|
snd_hda_jack_report_sync(codec);
|
||||||
return 0;
|
return 0;
|
||||||
|
@ -1343,6 +1354,7 @@ static int patch_generic_hdmi(struct hda_codec *codec)
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
codec->patch_ops = generic_hdmi_patch_ops;
|
codec->patch_ops = generic_hdmi_patch_ops;
|
||||||
|
generic_hdmi_init_per_pins(codec);
|
||||||
|
|
||||||
init_channel_allocations();
|
init_channel_allocations();
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue