Commit bd911ec4 authored by Jerome Brunet's avatar Jerome Brunet Committed by Ulf Hansson

mmc: meson-gx: rework clock init function

Thanks to devm, carrying the clock structure around after init is not
necessary. Rework the function to remove these from the controller host
data.

Finally, set initial mmc clock rate before enabling it, simplifying the
exit condition.
Reviewed-by: default avatarKevin Hilman <khilman@baylibre.com>
Signed-off-by: default avatarJerome Brunet <jbrunet@baylibre.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
parent f89f55df
...@@ -42,10 +42,7 @@ ...@@ -42,10 +42,7 @@
#define SD_EMMC_CLOCK 0x0 #define SD_EMMC_CLOCK 0x0
#define CLK_DIV_MASK GENMASK(5, 0) #define CLK_DIV_MASK GENMASK(5, 0)
#define CLK_DIV_MAX 63
#define CLK_SRC_MASK GENMASK(7, 6) #define CLK_SRC_MASK GENMASK(7, 6)
#define CLK_SRC_XTAL 0 /* external crystal */
#define CLK_SRC_PLL 1 /* FCLK_DIV2 */
#define CLK_CORE_PHASE_MASK GENMASK(9, 8) #define CLK_CORE_PHASE_MASK GENMASK(9, 8)
#define CLK_TX_PHASE_MASK GENMASK(11, 10) #define CLK_TX_PHASE_MASK GENMASK(11, 10)
#define CLK_RX_PHASE_MASK GENMASK(13, 12) #define CLK_RX_PHASE_MASK GENMASK(13, 12)
...@@ -137,13 +134,9 @@ struct meson_host { ...@@ -137,13 +134,9 @@ struct meson_host {
spinlock_t lock; spinlock_t lock;
void __iomem *regs; void __iomem *regs;
struct clk *core_clk; struct clk *core_clk;
struct clk_mux mux; struct clk *mmc_clk;
struct clk *mux_clk;
unsigned long req_rate; unsigned long req_rate;
struct clk_divider cfg_div;
struct clk *cfg_div_clk;
unsigned int bounce_buf_size; unsigned int bounce_buf_size;
void *bounce_buf; void *bounce_buf;
dma_addr_t bounce_dma_addr; dma_addr_t bounce_dma_addr;
...@@ -291,7 +284,7 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate) ...@@ -291,7 +284,7 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate)
return 0; return 0;
} }
ret = clk_set_rate(host->cfg_div_clk, clk_rate); ret = clk_set_rate(host->mmc_clk, clk_rate);
if (ret) { if (ret) {
dev_err(host->dev, "Unable to set cfg_div_clk to %lu. ret=%d\n", dev_err(host->dev, "Unable to set cfg_div_clk to %lu. ret=%d\n",
clk_rate, ret); clk_rate, ret);
...@@ -299,7 +292,7 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate) ...@@ -299,7 +292,7 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate)
} }
host->req_rate = clk_rate; host->req_rate = clk_rate;
mmc->actual_clock = clk_get_rate(host->cfg_div_clk); mmc->actual_clock = clk_get_rate(host->mmc_clk);
dev_dbg(host->dev, "clk rate: %u Hz\n", mmc->actual_clock); dev_dbg(host->dev, "clk rate: %u Hz\n", mmc->actual_clock);
if (clk_rate != mmc->actual_clock) if (clk_rate != mmc->actual_clock)
...@@ -321,10 +314,13 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate) ...@@ -321,10 +314,13 @@ static int meson_mmc_clk_set(struct meson_host *host, unsigned long clk_rate)
static int meson_mmc_clk_init(struct meson_host *host) static int meson_mmc_clk_init(struct meson_host *host)
{ {
struct clk_init_data init; struct clk_init_data init;
struct clk_mux *mux;
struct clk_divider *div;
struct clk *clk;
char clk_name[32]; char clk_name[32];
int i, ret = 0; int i, ret = 0;
const char *mux_parent_names[MUX_CLK_NUM_PARENTS]; const char *mux_parent_names[MUX_CLK_NUM_PARENTS];
const char *clk_div_parents[1]; const char *clk_parent[1];
u32 clk_reg; u32 clk_reg;
/* init SD_EMMC_CLOCK to sane defaults w/min clock rate */ /* init SD_EMMC_CLOCK to sane defaults w/min clock rate */
...@@ -353,55 +349,57 @@ static int meson_mmc_clk_init(struct meson_host *host) ...@@ -353,55 +349,57 @@ static int meson_mmc_clk_init(struct meson_host *host)
} }
/* create the mux */ /* create the mux */
mux = devm_kzalloc(host->dev, sizeof(*mux), GFP_KERNEL);
if (!mux)
return -ENOMEM;
snprintf(clk_name, sizeof(clk_name), "%s#mux", dev_name(host->dev)); snprintf(clk_name, sizeof(clk_name), "%s#mux", dev_name(host->dev));
init.name = clk_name; init.name = clk_name;
init.ops = &clk_mux_ops; init.ops = &clk_mux_ops;
init.flags = 0; init.flags = 0;
init.parent_names = mux_parent_names; init.parent_names = mux_parent_names;
init.num_parents = MUX_CLK_NUM_PARENTS; init.num_parents = MUX_CLK_NUM_PARENTS;
host->mux.reg = host->regs + SD_EMMC_CLOCK;
host->mux.shift = __bf_shf(CLK_SRC_MASK);
host->mux.mask = CLK_SRC_MASK >> host->mux.shift;
host->mux.flags = 0;
host->mux.table = NULL;
host->mux.hw.init = &init;
host->mux_clk = devm_clk_register(host->dev, &host->mux.hw); mux->reg = host->regs + SD_EMMC_CLOCK;
if (WARN_ON(IS_ERR(host->mux_clk))) mux->shift = __bf_shf(CLK_SRC_MASK);
return PTR_ERR(host->mux_clk); mux->mask = CLK_SRC_MASK >> mux->shift;
mux->hw.init = &init;
clk = devm_clk_register(host->dev, &mux->hw);
if (WARN_ON(IS_ERR(clk)))
return PTR_ERR(clk);
/* create the divider */ /* create the divider */
div = devm_kzalloc(host->dev, sizeof(*div), GFP_KERNEL);
if (!div)
return -ENOMEM;
snprintf(clk_name, sizeof(clk_name), "%s#div", dev_name(host->dev)); snprintf(clk_name, sizeof(clk_name), "%s#div", dev_name(host->dev));
init.name = clk_name; init.name = clk_name;
init.ops = &clk_divider_ops; init.ops = &clk_divider_ops;
init.flags = CLK_SET_RATE_PARENT; init.flags = CLK_SET_RATE_PARENT;
clk_div_parents[0] = __clk_get_name(host->mux_clk); clk_parent[0] = __clk_get_name(clk);
init.parent_names = clk_div_parents; init.parent_names = clk_parent;
init.num_parents = ARRAY_SIZE(clk_div_parents); init.num_parents = 1;
host->cfg_div.reg = host->regs + SD_EMMC_CLOCK; div->reg = host->regs + SD_EMMC_CLOCK;
host->cfg_div.shift = __bf_shf(CLK_DIV_MASK); div->shift = __bf_shf(CLK_DIV_MASK);
host->cfg_div.width = __builtin_popcountl(CLK_DIV_MASK); div->width = __builtin_popcountl(CLK_DIV_MASK);
host->cfg_div.hw.init = &init; div->hw.init = &init;
host->cfg_div.flags = CLK_DIVIDER_ONE_BASED | div->flags = (CLK_DIVIDER_ONE_BASED |
CLK_DIVIDER_ROUND_CLOSEST; CLK_DIVIDER_ROUND_CLOSEST);
host->cfg_div_clk = devm_clk_register(host->dev, &host->cfg_div.hw);
if (WARN_ON(PTR_ERR_OR_ZERO(host->cfg_div_clk)))
return PTR_ERR(host->cfg_div_clk);
ret = clk_prepare_enable(host->cfg_div_clk);
if (ret)
return ret;
/* Get the nearest minimum clock to 400KHz */ host->mmc_clk = devm_clk_register(host->dev, &div->hw);
host->mmc->f_min = clk_round_rate(host->cfg_div_clk, 400000); if (WARN_ON(PTR_ERR_OR_ZERO(host->mmc_clk)))
return PTR_ERR(host->mmc_clk);
ret = meson_mmc_clk_set(host, host->mmc->f_min); /* init SD_EMMC_CLOCK to sane defaults w/min clock rate */
host->mmc->f_min = clk_round_rate(host->mmc_clk, 400000);
ret = clk_set_rate(host->mmc_clk, host->mmc->f_min);
if (ret) if (ret)
clk_disable_unprepare(host->cfg_div_clk); return ret;
return ret; return clk_prepare_enable(host->mmc_clk);
} }
static void meson_mmc_set_tuning_params(struct mmc_host *mmc) static void meson_mmc_set_tuning_params(struct mmc_host *mmc)
...@@ -949,7 +947,7 @@ static int meson_mmc_probe(struct platform_device *pdev) ...@@ -949,7 +947,7 @@ static int meson_mmc_probe(struct platform_device *pdev)
meson_mmc_irq_thread, IRQF_SHARED, meson_mmc_irq_thread, IRQF_SHARED,
NULL, host); NULL, host);
if (ret) if (ret)
goto err_div_clk; goto err_init_clk;
mmc->caps |= MMC_CAP_CMD23; mmc->caps |= MMC_CAP_CMD23;
mmc->max_blk_count = CMD_CFG_LENGTH_MASK; mmc->max_blk_count = CMD_CFG_LENGTH_MASK;
...@@ -965,7 +963,7 @@ static int meson_mmc_probe(struct platform_device *pdev) ...@@ -965,7 +963,7 @@ static int meson_mmc_probe(struct platform_device *pdev)
if (host->bounce_buf == NULL) { if (host->bounce_buf == NULL) {
dev_err(host->dev, "Unable to map allocate DMA bounce buffer.\n"); dev_err(host->dev, "Unable to map allocate DMA bounce buffer.\n");
ret = -ENOMEM; ret = -ENOMEM;
goto err_div_clk; goto err_init_clk;
} }
host->descs = dma_alloc_coherent(host->dev, SD_EMMC_DESC_BUF_LEN, host->descs = dma_alloc_coherent(host->dev, SD_EMMC_DESC_BUF_LEN,
...@@ -984,8 +982,8 @@ static int meson_mmc_probe(struct platform_device *pdev) ...@@ -984,8 +982,8 @@ static int meson_mmc_probe(struct platform_device *pdev)
err_bounce_buf: err_bounce_buf:
dma_free_coherent(host->dev, host->bounce_buf_size, dma_free_coherent(host->dev, host->bounce_buf_size,
host->bounce_buf, host->bounce_dma_addr); host->bounce_buf, host->bounce_dma_addr);
err_div_clk: err_init_clk:
clk_disable_unprepare(host->cfg_div_clk); clk_disable_unprepare(host->mmc_clk);
err_core_clk: err_core_clk:
clk_disable_unprepare(host->core_clk); clk_disable_unprepare(host->core_clk);
free_host: free_host:
...@@ -1007,7 +1005,7 @@ static int meson_mmc_remove(struct platform_device *pdev) ...@@ -1007,7 +1005,7 @@ static int meson_mmc_remove(struct platform_device *pdev)
dma_free_coherent(host->dev, host->bounce_buf_size, dma_free_coherent(host->dev, host->bounce_buf_size,
host->bounce_buf, host->bounce_dma_addr); host->bounce_buf, host->bounce_dma_addr);
clk_disable_unprepare(host->cfg_div_clk); clk_disable_unprepare(host->mmc_clk);
clk_disable_unprepare(host->core_clk); clk_disable_unprepare(host->core_clk);
mmc_free_host(host->mmc); mmc_free_host(host->mmc);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment