Commit aa29d840 authored by Sascha Hauer's avatar Sascha Hauer

spi i.MX: do not depend on grouped clocks

the current i.MX clock support groups together unrelated clocks
to a single clock which is then used by the driver. This can't
be accomplished with the generic clock framework so we instead
request the individual clocks in the driver. For i.MX there are
generally three different clocks:

ipg: bus clock (needed to access registers)
ahb: dma relevant clock, sometimes referred to as hclk in the datasheet
per: bit clock, pixel clock

This patch changes the driver to request the individual clocks.
Currently all clk_get will get the same clock until the SoCs
are converted to the generic clock framework
Signed-off-by: default avatarSascha Hauer <s.hauer@pengutronix.de>
parent 7560e3f3
...@@ -85,7 +85,8 @@ struct spi_imx_data { ...@@ -85,7 +85,8 @@ struct spi_imx_data {
struct completion xfer_done; struct completion xfer_done;
void __iomem *base; void __iomem *base;
int irq; int irq;
struct clk *clk; struct clk *clk_per;
struct clk *clk_ipg;
unsigned long spi_clk; unsigned long spi_clk;
unsigned int count; unsigned int count;
...@@ -845,15 +846,22 @@ static int __devinit spi_imx_probe(struct platform_device *pdev) ...@@ -845,15 +846,22 @@ static int __devinit spi_imx_probe(struct platform_device *pdev)
goto out_iounmap; goto out_iounmap;
} }
spi_imx->clk = clk_get(&pdev->dev, NULL); spi_imx->clk_ipg = devm_clk_get(&pdev->dev, "ipg");
if (IS_ERR(spi_imx->clk)) { if (IS_ERR(spi_imx->clk_ipg)) {
dev_err(&pdev->dev, "unable to get clock\n"); ret = PTR_ERR(spi_imx->clk_ipg);
ret = PTR_ERR(spi_imx->clk);
goto out_free_irq; goto out_free_irq;
} }
clk_enable(spi_imx->clk); spi_imx->clk_per = devm_clk_get(&pdev->dev, "per");
spi_imx->spi_clk = clk_get_rate(spi_imx->clk); if (IS_ERR(spi_imx->clk_per)) {
ret = PTR_ERR(spi_imx->clk_per);
goto out_free_irq;
}
clk_prepare_enable(spi_imx->clk_per);
clk_prepare_enable(spi_imx->clk_ipg);
spi_imx->spi_clk = clk_get_rate(spi_imx->clk_per);
spi_imx->devtype_data->reset(spi_imx); spi_imx->devtype_data->reset(spi_imx);
...@@ -871,8 +879,8 @@ static int __devinit spi_imx_probe(struct platform_device *pdev) ...@@ -871,8 +879,8 @@ static int __devinit spi_imx_probe(struct platform_device *pdev)
return ret; return ret;
out_clk_put: out_clk_put:
clk_disable(spi_imx->clk); clk_disable_unprepare(spi_imx->clk_per);
clk_put(spi_imx->clk); clk_disable_unprepare(spi_imx->clk_ipg);
out_free_irq: out_free_irq:
free_irq(spi_imx->irq, spi_imx); free_irq(spi_imx->irq, spi_imx);
out_iounmap: out_iounmap:
...@@ -900,8 +908,8 @@ static int __devexit spi_imx_remove(struct platform_device *pdev) ...@@ -900,8 +908,8 @@ static int __devexit spi_imx_remove(struct platform_device *pdev)
spi_bitbang_stop(&spi_imx->bitbang); spi_bitbang_stop(&spi_imx->bitbang);
writel(0, spi_imx->base + MXC_CSPICTRL); writel(0, spi_imx->base + MXC_CSPICTRL);
clk_disable(spi_imx->clk); clk_disable_unprepare(spi_imx->clk_per);
clk_put(spi_imx->clk); clk_disable_unprepare(spi_imx->clk_ipg);
free_irq(spi_imx->irq, spi_imx); free_irq(spi_imx->irq, spi_imx);
iounmap(spi_imx->base); iounmap(spi_imx->base);
......
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