]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ALSA: x86: Rename had_enable_audio_int() to had_ack_irqs()
authorTakashi Iwai <tiwai@suse.de>
Tue, 7 Feb 2017 06:55:27 +0000 (07:55 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 7 Feb 2017 08:43:01 +0000 (09:43 +0100)
commit075a1d46bed386138eb51f92a6a3130c82fdefec
tree52008e09afda38f3f8c7d5cf6aede137b9173d9d
parent3e21a76ca32f535a7640b0657f8d56660527ddab
ALSA: x86: Rename had_enable_audio_int() to had_ack_irqs()

had_enable_audio_int() came from the LPE audio shell set_caps
callback with ENABLE_INT and DISABLE_INT caps.  I interpreted as these
correspond to enabling / disabling the audio interface, but the actual
implementation is only to clear (send ACK) to both BUFFER_DONE and
BUFFER_UNDERRUN interrupts unconditionally.  And, there is no
counterpart, DISABLE_INT, code at all.

For avoiding the further misunderstanding, rename the function to the
more fitting one, had_ack_irqs(), and drop the calls with enable=false
in allover places.  There is no functional changes at all.

After this patch, there is only one caller at the PCM trigger start.
Then it's doubtful whether this call is still really needed or not; I
bet it not, but let's stay in the safer side for now and keep it as
was.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/x86/intel_hdmi_audio.c