soundwire: cadence: simplifiy cdns_init()
authorRander Wang <rander.wang@intel.com>
Tue, 17 Mar 2020 16:33:14 +0000 (11:33 -0500)
committerVinod Koul <vkoul@kernel.org>
Thu, 19 Mar 2020 07:13:03 +0000 (12:43 +0530)
There is no need for the clock_stop_exit argument with the latest
implementation

Signed-off-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20200317163329.25501-3-pierre-louis.bossart@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/soundwire/cadence_master.c
drivers/soundwire/cadence_master.h
drivers/soundwire/intel.c

index a1a889d..941809e 100644 (file)
@@ -1018,22 +1018,12 @@ static u32 cdns_set_initial_frame_shape(int n_rows, int n_cols)
  * sdw_cdns_init() - Cadence initialization
  * @cdns: Cadence instance
  */
-int sdw_cdns_init(struct sdw_cdns *cdns, bool clock_stop_exit)
+int sdw_cdns_init(struct sdw_cdns *cdns)
 {
        struct sdw_bus *bus = &cdns->bus;
        struct sdw_master_prop *prop = &bus->prop;
        u32 val;
        int divider;
-       int ret;
-
-       if (clock_stop_exit) {
-               ret = cdns_clear_bit(cdns, CDNS_MCP_CONTROL,
-                                    CDNS_MCP_CONTROL_CLK_STOP_CLR);
-               if (ret < 0) {
-                       dev_err(cdns->dev, "Couldn't exit from clock stop\n");
-                       return ret;
-               }
-       }
 
        /* Set clock divider */
        divider = (prop->mclk_freq / prop->max_clk_freq) - 1;
index 2de1b24..44e802b 100644 (file)
@@ -138,7 +138,7 @@ extern struct sdw_master_ops sdw_cdns_master_ops;
 irqreturn_t sdw_cdns_irq(int irq, void *dev_id);
 irqreturn_t sdw_cdns_thread(int irq, void *dev_id);
 
-int sdw_cdns_init(struct sdw_cdns *cdns, bool clock_stop_exit);
+int sdw_cdns_init(struct sdw_cdns *cdns);
 int sdw_cdns_pdi_init(struct sdw_cdns *cdns,
                      struct sdw_cdns_stream_config config);
 int sdw_cdns_exit_reset(struct sdw_cdns *cdns);
index a327669..3c83e76 100644 (file)
@@ -1077,7 +1077,7 @@ static int intel_init(struct sdw_intel *sdw)
        intel_link_power_up(sdw);
        intel_shim_init(sdw);
 
-       return sdw_cdns_init(&sdw->cdns, false);
+       return sdw_cdns_init(&sdw->cdns);
 }
 
 /*