ALSA: x86: Drop unused fields from snd_intelhad struct
authorTakashi Iwai <tiwai@suse.de>
Tue, 31 Jan 2017 15:11:27 +0000 (16:11 +0100)
committerTakashi Iwai <tiwai@suse.de>
Fri, 3 Feb 2017 16:33:28 +0000 (17:33 +0100)
Also change the flag_underrun to bool to be clearer.

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

index 8b25687..0a14f5d 100644 (file)
@@ -593,9 +593,8 @@ static int had_register_chmap_ctls(struct snd_intelhad *intelhaddata,
                return err;
 
        intelhaddata->chmap->private_data = intelhaddata;
-       intelhaddata->kctl = intelhaddata->chmap->kctl;
-       intelhaddata->kctl->info = had_chmap_ctl_info;
-       intelhaddata->kctl->get = had_chmap_ctl_get;
+       intelhaddata->chmap->kctl->info = had_chmap_ctl_info;
+       intelhaddata->chmap->kctl->get = had_chmap_ctl_get;
        intelhaddata->chmap->chmap = NULL;
        return 0;
 }
@@ -1331,7 +1330,7 @@ static snd_pcm_uframes_t snd_intelhad_pcm_pointer(
        intelhaddata = snd_pcm_substream_chip(substream);
 
        if (intelhaddata->flag_underrun) {
-               intelhaddata->flag_underrun = 0;
+               intelhaddata->flag_underrun = false;
                return SNDRV_PCM_POS_XRUN;
        }
 
@@ -1690,7 +1689,7 @@ static int had_process_buffer_underrun(struct snd_intelhad *intelhaddata)
 
        if (stream_type == HAD_RUNNING_STREAM) {
                /* Report UNDERRUN error to above layers */
-               intelhaddata->flag_underrun = 1;
+               intelhaddata->flag_underrun = true;
                had_period_elapsed(stream->had_substream);
        }
 
@@ -1776,7 +1775,6 @@ static int had_process_hot_unplug(struct snd_intelhad *intelhaddata)
        spin_unlock_irqrestore(&intelhaddata->had_spinlock, flag_irqs);
        kfree(intelhaddata->chmap->chmap);
        intelhaddata->chmap->chmap = NULL;
-       intelhaddata->audio_reg_base = NULL;
 
        return 0;
 }
@@ -2019,9 +2017,7 @@ static int hdmi_lpe_audio_probe(struct platform_device *pdev)
        ctx->drv_status = HAD_DRV_DISCONNECTED;
        ctx->dev = &pdev->dev;
        ctx->card = card;
-       ctx->card_id = hdmi_card_id;
-       ctx->card_index = card->number;
-       ctx->flag_underrun = 0;
+       ctx->flag_underrun = false;
        ctx->aes_bits = SNDRV_PCM_DEFAULT_CON_SPDIF;
        strcpy(card->driver, INTEL_HAD);
        strcpy(card->shortname, INTEL_HAD);
index bcbb4b2..4549c4d 100644 (file)
@@ -39,8 +39,6 @@
 #include <sound/pcm.h>
 #include "intel_hdmi_lpe_audio.h"
 
-struct platform_device;
-
 #define PCM_INDEX              0
 #define MAX_PB_STREAMS         1
 #define MAX_CAP_STREAMS                0
@@ -102,8 +100,6 @@ struct had_stream_data {
  * struct snd_intelhad - intelhad driver structure
  *
  * @card: ptr to hold card details
- * @card_index: sound card index
- * @card_id: detected sound card id
  * @drv_status: driver status
  * @buf_info: ring buffer info
  * @stream_info: stream information
@@ -114,15 +110,11 @@ struct had_stream_data {
  * @aes_bits: IEC958 status bits
  * @buff_done: id of current buffer done intr
  * @dev: platoform device handle
- * @kctl: holds kctl ptrs used for channel map
  * @chmap: holds channel map info
- * @audio_reg_base: hdmi audio register base offset
  * @underrun_count: PCM stream underrun counter
  */
 struct snd_intelhad {
        struct snd_card *card;
-       int             card_index;
-       char            *card_id;
        enum had_drv_status     drv_status;
        struct          ring_buf_info buf_info[HAD_NUM_OF_RING_BUFS];
        struct          pcm_stream_info stream_info;
@@ -131,15 +123,12 @@ struct snd_intelhad {
        enum            intel_had_aud_buf_type curr_buf;
        int             valid_buf_cnt;
        unsigned int    aes_bits;
-       int flag_underrun;
+       bool flag_underrun;
        struct had_stream_data stream_data;
        spinlock_t had_spinlock;
        enum            intel_had_aud_buf_type buff_done;
        struct device *dev;
-       struct snd_kcontrol *kctl;
        struct snd_pcm_chmap *chmap;
-       unsigned int    *audio_reg_base;
-       unsigned int    audio_cfg_offset;
        int underrun_count;
        enum hdmi_connector_status state;
        int tmds_clock_speed;
@@ -149,7 +138,6 @@ struct snd_intelhad {
        int irq;
        void __iomem *mmio_start;
        unsigned int had_config_offset;
-       int hdmi_audio_interrupt_mask;
        struct work_struct hdmi_audio_wq;
 };