ASoC: Intel: move the lock and wq initialization to routine

This will be used by ACPI code as well, so moving to common routine helps

Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/intel/sst/sst.c b/sound/soc/intel/sst/sst.c
index f9a6d6d117d6d3cded252a7d0e45ed2d9ce10c37..0863471a2c86f6da4a34d2a3976c4ffde613a1d3 100644 (file)
--- a/sound/soc/intel/sst/sst.c
+++ b/sound/soc/intel/sst/sst.c
@@ -222,6 +222,14 @@ static int sst_workqueue_init(struct intel_sst_drv *ctx)
        return 0;
 }
 
+static void sst_init_locks(struct intel_sst_drv *ctx)
+{
+       mutex_init(&ctx->sst_lock);
+       spin_lock_init(&ctx->rx_msg_lock);
+       spin_lock_init(&ctx->ipc_spin_lock);
+       spin_lock_init(&ctx->block_lock);
+}
+
 /*
 * intel_sst_probe - PCI probe function
 *
@@ -259,7 +267,7 @@ static int intel_sst_probe(struct pci_dev *pci,
                return -EINVAL;
 
        ops = sst_drv_ctx->ops;
-       mutex_init(&sst_drv_ctx->sst_lock);
+       sst_init_locks(sst_drv_ctx);
 
        /* pvt_id 0 reserved for async messages */
        sst_drv_ctx->pvt_id = 1;
@@ -270,10 +278,6 @@ static int intel_sst_probe(struct pci_dev *pci,
        sst_drv_ctx->use_dma = 0;
        sst_drv_ctx->use_lli = 0;
 
-       spin_lock_init(&sst_drv_ctx->ipc_spin_lock);
-       spin_lock_init(&sst_drv_ctx->block_lock);
-       spin_lock_init(&sst_drv_ctx->rx_msg_lock);
-
        if (sst_workqueue_init(sst_drv_ctx))
                return -EINVAL;