diff options
| author | Rafael J. Wysocki <rjw@sisk.pl> | 2011-12-21 21:59:45 +0100 | 
|---|---|---|
| committer | Rafael J. Wysocki <rjw@sisk.pl> | 2011-12-21 21:59:45 +0100 | 
| commit | b00f4dc5ff022cb9cbaffd376d9454d7fa1e496f (patch) | |
| tree | 40f1b232e2f1e8ac365317a14fdcbcb331722b46 /sound/pci/hda/patch_via.c | |
| parent | 1eac8111e0763853266a171ce11214da3a347a0a (diff) | |
| parent | b9e26dfdad5a4f9cbdaacafac6998614cc9c41bc (diff) | |
| download | olio-linux-3.10-b00f4dc5ff022cb9cbaffd376d9454d7fa1e496f.tar.xz olio-linux-3.10-b00f4dc5ff022cb9cbaffd376d9454d7fa1e496f.zip  | |
Merge branch 'master' into pm-sleep
* master: (848 commits)
  SELinux: Fix RCU deref check warning in sel_netport_insert()
  binary_sysctl(): fix memory leak
  mm/vmalloc.c: remove static declaration of va from __get_vm_area_node
  ipmi_watchdog: restore settings when BMC reset
  oom: fix integer overflow of points in oom_badness
  memcg: keep root group unchanged if creation fails
  nilfs2: potential integer overflow in nilfs_ioctl_clean_segments()
  nilfs2: unbreak compat ioctl
  cpusets: stall when updating mems_allowed for mempolicy or disjoint nodemask
  evm: prevent racing during tfm allocation
  evm: key must be set once during initialization
  mmc: vub300: fix type of firmware_rom_wait_states module parameter
  Revert "mmc: enable runtime PM by default"
  mmc: sdhci: remove "state" argument from sdhci_suspend_host
  x86, dumpstack: Fix code bytes breakage due to missing KERN_CONT
  IB/qib: Correct sense on freectxts increment and decrement
  RDMA/cma: Verify private data length
  cgroups: fix a css_set not found bug in cgroup_attach_proc
  oprofile: Fix uninitialized memory access when writing to writing to oprofilefs
  Revert "xen/pv-on-hvm kexec: add xs_reset_watches to shutdown watches from old kernel"
  ...
Conflicts:
	kernel/cgroup_freezer.c
Diffstat (limited to 'sound/pci/hda/patch_via.c')
| -rw-r--r-- | sound/pci/hda/patch_via.c | 76 | 
1 files changed, 43 insertions, 33 deletions
diff --git a/sound/pci/hda/patch_via.c b/sound/pci/hda/patch_via.c index 431c0d417ee..b5137629f8e 100644 --- a/sound/pci/hda/patch_via.c +++ b/sound/pci/hda/patch_via.c @@ -208,6 +208,7 @@ struct via_spec {  	/* work to check hp jack state */  	struct hda_codec *codec;  	struct delayed_work vt1708_hp_work; +	int hp_work_active;  	int vt1708_jack_detect;  	int vt1708_hp_present; @@ -305,27 +306,35 @@ enum {  static void analog_low_current_mode(struct hda_codec *codec);  static bool is_aa_path_mute(struct hda_codec *codec); -static void vt1708_start_hp_work(struct via_spec *spec) +#define hp_detect_with_aa(codec) \ +	(snd_hda_get_bool_hint(codec, "analog_loopback_hp_detect") == 1 && \ +	 !is_aa_path_mute(codec)) + +static void vt1708_stop_hp_work(struct via_spec *spec)  {  	if (spec->codec_type != VT1708 || spec->autocfg.hp_pins[0] == 0)  		return; -	snd_hda_codec_write(spec->codec, 0x1, 0, 0xf81, -			    !spec->vt1708_jack_detect); -	if (!delayed_work_pending(&spec->vt1708_hp_work)) -		schedule_delayed_work(&spec->vt1708_hp_work, -				      msecs_to_jiffies(100)); +	if (spec->hp_work_active) { +		snd_hda_codec_write(spec->codec, 0x1, 0, 0xf81, 1); +		cancel_delayed_work_sync(&spec->vt1708_hp_work); +		spec->hp_work_active = 0; +	}  } -static void vt1708_stop_hp_work(struct via_spec *spec) +static void vt1708_update_hp_work(struct via_spec *spec)  {  	if (spec->codec_type != VT1708 || spec->autocfg.hp_pins[0] == 0)  		return; -	if (snd_hda_get_bool_hint(spec->codec, "analog_loopback_hp_detect") == 1 -	    && !is_aa_path_mute(spec->codec)) -		return; -	snd_hda_codec_write(spec->codec, 0x1, 0, 0xf81, -			    !spec->vt1708_jack_detect); -	cancel_delayed_work_sync(&spec->vt1708_hp_work); +	if (spec->vt1708_jack_detect && +	    (spec->active_streams || hp_detect_with_aa(spec->codec))) { +		if (!spec->hp_work_active) { +			snd_hda_codec_write(spec->codec, 0x1, 0, 0xf81, 0); +			schedule_delayed_work(&spec->vt1708_hp_work, +					      msecs_to_jiffies(100)); +			spec->hp_work_active = 1; +		} +	} else if (!hp_detect_with_aa(spec->codec)) +		vt1708_stop_hp_work(spec);  }  static void set_widgets_power_state(struct hda_codec *codec) @@ -343,12 +352,7 @@ static int analog_input_switch_put(struct snd_kcontrol *kcontrol,  	set_widgets_power_state(codec);  	analog_low_current_mode(snd_kcontrol_chip(kcontrol)); -	if (snd_hda_get_bool_hint(codec, "analog_loopback_hp_detect") == 1) { -		if (is_aa_path_mute(codec)) -			vt1708_start_hp_work(codec->spec); -		else -			vt1708_stop_hp_work(codec->spec); -	} +	vt1708_update_hp_work(codec->spec);  	return change;  } @@ -1154,7 +1158,7 @@ static int via_playback_multi_pcm_prepare(struct hda_pcm_stream *hinfo,  	spec->cur_dac_stream_tag = stream_tag;  	spec->cur_dac_format = format;  	mutex_unlock(&spec->config_mutex); -	vt1708_start_hp_work(spec); +	vt1708_update_hp_work(spec);  	return 0;  } @@ -1174,7 +1178,7 @@ static int via_playback_hp_pcm_prepare(struct hda_pcm_stream *hinfo,  	spec->cur_hp_stream_tag = stream_tag;  	spec->cur_hp_format = format;  	mutex_unlock(&spec->config_mutex); -	vt1708_start_hp_work(spec); +	vt1708_update_hp_work(spec);  	return 0;  } @@ -1188,7 +1192,7 @@ static int via_playback_multi_pcm_cleanup(struct hda_pcm_stream *hinfo,  	snd_hda_multi_out_analog_cleanup(codec, &spec->multiout);  	spec->active_streams &= ~STREAM_MULTI_OUT;  	mutex_unlock(&spec->config_mutex); -	vt1708_stop_hp_work(spec); +	vt1708_update_hp_work(spec);  	return 0;  } @@ -1203,7 +1207,7 @@ static int via_playback_hp_pcm_cleanup(struct hda_pcm_stream *hinfo,  		snd_hda_codec_setup_stream(codec, spec->hp_dac_nid, 0, 0, 0);  	spec->active_streams &= ~STREAM_INDEP_HP;  	mutex_unlock(&spec->config_mutex); -	vt1708_stop_hp_work(spec); +	vt1708_update_hp_work(spec);  	return 0;  } @@ -1645,7 +1649,8 @@ static void via_hp_automute(struct hda_codec *codec)  	int nums;  	struct via_spec *spec = codec->spec; -	if (!spec->hp_independent_mode && spec->autocfg.hp_pins[0]) +	if (!spec->hp_independent_mode && spec->autocfg.hp_pins[0] && +	    (spec->codec_type != VT1708 || spec->vt1708_jack_detect))  		present = snd_hda_jack_detect(codec, spec->autocfg.hp_pins[0]);  	if (spec->smart51_enabled) @@ -2612,8 +2617,6 @@ static int vt1708_jack_detect_get(struct snd_kcontrol *kcontrol,  	if (spec->codec_type != VT1708)  		return 0; -	spec->vt1708_jack_detect = -		!((snd_hda_codec_read(codec, 0x1, 0, 0xf84, 0) >> 8) & 0x1);  	ucontrol->value.integer.value[0] = spec->vt1708_jack_detect;  	return 0;  } @@ -2623,18 +2626,22 @@ static int vt1708_jack_detect_put(struct snd_kcontrol *kcontrol,  {  	struct hda_codec *codec = snd_kcontrol_chip(kcontrol);  	struct via_spec *spec = codec->spec; -	int change; +	int val;  	if (spec->codec_type != VT1708)  		return 0; -	spec->vt1708_jack_detect = ucontrol->value.integer.value[0]; -	change = (0x1 & (snd_hda_codec_read(codec, 0x1, 0, 0xf84, 0) >> 8)) -		== !spec->vt1708_jack_detect; -	if (spec->vt1708_jack_detect) { +	val = !!ucontrol->value.integer.value[0]; +	if (spec->vt1708_jack_detect == val) +		return 0; +	spec->vt1708_jack_detect = val; +	if (spec->vt1708_jack_detect && +	    snd_hda_get_bool_hint(codec, "analog_loopback_hp_detect") != 1) {  		mute_aa_path(codec, 1);  		notify_aa_path_ctls(codec);  	} -	return change; +	via_hp_automute(codec); +	vt1708_update_hp_work(spec); +	return 1;  }  static const struct snd_kcontrol_new vt1708_jack_detect_ctl = { @@ -2771,6 +2778,7 @@ static int via_init(struct hda_codec *codec)  	via_auto_init_unsol_event(codec);  	via_hp_automute(codec); +	vt1708_update_hp_work(spec);  	return 0;  } @@ -2787,7 +2795,9 @@ static void vt1708_update_hp_jack_state(struct work_struct *work)  		spec->vt1708_hp_present ^= 1;  		via_hp_automute(spec->codec);  	} -	vt1708_start_hp_work(spec); +	if (spec->vt1708_jack_detect) +		schedule_delayed_work(&spec->vt1708_hp_work, +				      msecs_to_jiffies(100));  }  static int get_mux_nids(struct hda_codec *codec)  |