ASoC: Intel: Remove sst_dsp_get_thread_context
authorCezary Rojewski <cezary.rojewski@intel.com>
Tue, 6 Oct 2020 06:49:07 +0000 (08:49 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 6 Oct 2020 14:12:29 +0000 (15:12 +0100)
While sst_dsp_get_thread_context() is declared as solution-agnostic, it
is only used by /skylake/ solution. Majority of thread_context field
usages are direct accesses. Improve code cohesiveness and convert to
single usage model.

Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Liam Girdwood <liam.r.girdwood@intel.com>
Link: https://lore.kernel.org/r/20201006064907.16277-14-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/common/sst-dsp-priv.h
sound/soc/intel/skylake/cnl-sst.c
sound/soc/intel/skylake/skl-sst-ipc.c

index 5a5ce50..de2b445 100644 (file)
@@ -98,9 +98,4 @@ struct sst_dsp {
        struct snd_dma_buffer dmab;
 };
 
-static inline void *sst_dsp_get_thread_context(struct sst_dsp *sst)
-{
-       return sst->thread_context;
-}
-
 #endif
index 355a7b1..fcd8dff 100644 (file)
@@ -311,7 +311,7 @@ static struct sst_ops cnl_ops = {
 static irqreturn_t cnl_dsp_irq_thread_handler(int irq, void *context)
 {
        struct sst_dsp *dsp = context;
-       struct skl_dev *cnl = sst_dsp_get_thread_context(dsp);
+       struct skl_dev *cnl = dsp->thread_context;
        struct sst_generic_ipc *ipc = &cnl->ipc;
        struct skl_ipc_header header = {0};
        u32 hipcida, hipctdr, hipctdd;
index 667cddd..7a42527 100644 (file)
@@ -489,7 +489,7 @@ void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 irqreturn_t skl_dsp_irq_thread_handler(int irq, void *context)
 {
        struct sst_dsp *dsp = context;
-       struct skl_dev *skl = sst_dsp_get_thread_context(dsp);
+       struct skl_dev *skl = dsp->thread_context;
        struct sst_generic_ipc *ipc = &skl->ipc;
        struct skl_ipc_header header = {0};
        u32 hipcie, hipct, hipcte;