ALSA: x86: Call event callback directly
authorTakashi Iwai <tiwai@suse.de>
Mon, 30 Jan 2017 16:38:00 +0000 (17:38 +0100)
committerTakashi Iwai <tiwai@suse.de>
Fri, 3 Feb 2017 16:23:15 +0000 (17:23 +0100)
Currently the driver calls the event callback stored in its ctx
pointer, but it's obviously inefficient.  Replace it with the direct
calls.

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

index 7165f14d522998bed0365eb71dae3abcdda14ba2..571ec07a36113c3a0103e567b544b4fa8e794fd4 100644 (file)
@@ -1591,7 +1591,6 @@ int hdmi_audio_probe(struct platform_device *devptr,
        int retval;
        struct snd_pcm *pcm;
        struct snd_card *card;
-       struct had_callback_ops ops_cb;
        struct snd_intelhad *intelhaddata;
        struct had_pvt_data *had_stream;
 
@@ -1610,16 +1609,6 @@ int hdmi_audio_probe(struct platform_device *devptr,
                goto free_haddata;
        }
 
-       ops_cb.intel_had_event_call_back = had_event_handler;
-
-       /* registering with display driver to get access to display APIs */
-
-       retval = mid_hdmi_audio_setup(ops_cb.intel_had_event_call_back);
-       if (retval) {
-               pr_err("querying display driver APIs failed %#x\n", retval);
-               goto free_hadstream;
-       }
-
        spin_lock_init(&intelhaddata->had_spinlock);
        intelhaddata->drv_status = HAD_DRV_DISCONNECTED;
        pr_debug("%s @ %d:DEBUG PLUG/UNPLUG : HAD_DRV_DISCONNECTED\n",
index 5a82a3f429d7e3a7ed11a39dcbe07e5d46c36c93..110d1d083000c9124c48171676e5d80d0e7d1dda 100644 (file)
@@ -99,10 +99,6 @@ struct had_pvt_data {
        enum had_status_stream          stream_type;
 };
 
-struct had_callback_ops {
-       had_event_call_back intel_had_event_call_back;
-};
-
 /**
  * struct snd_intelhad - intelhad driver structure
  *
index 0c11d82eb99bfc19a1b8db9cd4e1eda387e52e37..54cc30f034f3e8f1c5e9654a7732c6d163f40ea9 100644 (file)
@@ -44,7 +44,6 @@ static union otm_hdmi_eld_t hlpe_eld;
 struct hdmi_lpe_audio_ctx {
        int irq;
        void __iomem *mmio_start;
-       had_event_call_back had_event_callbacks;
        struct snd_intelhad *had;
        int tmds_clock_speed;
        bool dp_output;
@@ -118,8 +117,7 @@ void mid_hdmi_audio_signal_event(enum had_event_type event)
        /* The handler is protected in the respective
         * event handlers to avoid races
         */
-       if (ctx->had_event_callbacks)
-               (*ctx->had_event_callbacks)(event, ctx->had);
+       had_event_handler(event, ctx->had);
 }
 
 /*
@@ -257,19 +255,6 @@ int mid_hdmi_audio_set_caps(enum had_caps_list set_element,
        return 0;
 }
 
-int mid_hdmi_audio_setup(had_event_call_back audio_callbacks)
-{
-       struct hdmi_lpe_audio_ctx *ctx;
-
-       ctx = platform_get_drvdata(hlpe_pdev);
-
-       dev_dbg(&hlpe_pdev->dev, "%s: called\n",  __func__);
-
-       ctx->had_event_callbacks = audio_callbacks;
-
-       return 0;
-}
-
 static void _had_wq(struct work_struct *work)
 {
        mid_hdmi_audio_signal_event(HAD_EVENT_HOT_PLUG);
index 0d285ce8d4e60f48a45c0320229002dd1ce3a1c8..511bdc30dca15e02b5ee67a6994e43a72c98fc37 100644 (file)
@@ -638,16 +638,12 @@ enum had_event_type {
  * HDMI Display Controller Audio Interface
  *
  */
-typedef int (*had_event_call_back) (enum had_event_type event_type,
-               void *ctxt_info);
-
 struct hdmi_audio_event {
        int type;
 };
 
 bool mid_hdmi_audio_is_busy(void *dev);
 void mid_hdmi_audio_signal_event(enum had_event_type event);
-int mid_hdmi_audio_setup(had_event_call_back audio_callbacks);
 
 int mid_hdmi_audio_read(u32 reg, u32 *val);
 int mid_hdmi_audio_write(u32 reg, u32 val);