Commit 017d84bd authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Chris Ball

mmc: jz4740: Use managed resources

Use managed resources for the mmio memory region and the clock.
Makes the code a bit shorter.
Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent 58e300af
...@@ -121,7 +121,6 @@ struct jz4740_mmc_host { ...@@ -121,7 +121,6 @@ struct jz4740_mmc_host {
int irq; int irq;
int card_detect_irq; int card_detect_irq;
struct resource *mem;
void __iomem *base; void __iomem *base;
struct mmc_request *req; struct mmc_request *req;
struct mmc_command *cmd; struct mmc_command *cmd;
...@@ -755,6 +754,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -755,6 +754,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
struct mmc_host *mmc; struct mmc_host *mmc;
struct jz4740_mmc_host *host; struct jz4740_mmc_host *host;
struct jz4740_mmc_platform_data *pdata; struct jz4740_mmc_platform_data *pdata;
struct resource *res;
pdata = pdev->dev.platform_data; pdata = pdev->dev.platform_data;
...@@ -774,39 +774,25 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -774,39 +774,25 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
goto err_free_host; goto err_free_host;
} }
host->clk = clk_get(&pdev->dev, "mmc"); host->clk = devm_clk_get(&pdev->dev, "mmc");
if (IS_ERR(host->clk)) { if (IS_ERR(host->clk)) {
ret = PTR_ERR(host->clk); ret = PTR_ERR(host->clk);
dev_err(&pdev->dev, "Failed to get mmc clock\n"); dev_err(&pdev->dev, "Failed to get mmc clock\n");
goto err_free_host; goto err_free_host;
} }
host->mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!host->mem) { host->base = devm_ioremap_resource(&pdev->dev, res);
ret = -ENOENT;
dev_err(&pdev->dev, "Failed to get base platform memory\n");
goto err_clk_put;
}
host->mem = request_mem_region(host->mem->start,
resource_size(host->mem), pdev->name);
if (!host->mem) {
ret = -EBUSY;
dev_err(&pdev->dev, "Failed to request base memory region\n");
goto err_clk_put;
}
host->base = ioremap_nocache(host->mem->start, resource_size(host->mem));
if (!host->base) { if (!host->base) {
ret = -EBUSY; ret = -EBUSY;
dev_err(&pdev->dev, "Failed to ioremap base memory\n"); dev_err(&pdev->dev, "Failed to ioremap base memory\n");
goto err_release_mem_region; goto err_free_host;
} }
ret = jz_gpio_bulk_request(jz4740_mmc_pins, jz4740_mmc_num_pins(host)); ret = jz_gpio_bulk_request(jz4740_mmc_pins, jz4740_mmc_num_pins(host));
if (ret) { if (ret) {
dev_err(&pdev->dev, "Failed to request mmc pins: %d\n", ret); dev_err(&pdev->dev, "Failed to request mmc pins: %d\n", ret);
goto err_iounmap; goto err_free_host;
} }
ret = jz4740_mmc_request_gpios(mmc, pdev); ret = jz4740_mmc_request_gpios(mmc, pdev);
...@@ -863,12 +849,6 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -863,12 +849,6 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
jz4740_mmc_free_gpios(pdev); jz4740_mmc_free_gpios(pdev);
err_gpio_bulk_free: err_gpio_bulk_free:
jz_gpio_bulk_free(jz4740_mmc_pins, jz4740_mmc_num_pins(host)); jz_gpio_bulk_free(jz4740_mmc_pins, jz4740_mmc_num_pins(host));
err_iounmap:
iounmap(host->base);
err_release_mem_region:
release_mem_region(host->mem->start, resource_size(host->mem));
err_clk_put:
clk_put(host->clk);
err_free_host: err_free_host:
mmc_free_host(mmc); mmc_free_host(mmc);
...@@ -890,11 +870,6 @@ static int jz4740_mmc_remove(struct platform_device *pdev) ...@@ -890,11 +870,6 @@ static int jz4740_mmc_remove(struct platform_device *pdev)
jz4740_mmc_free_gpios(pdev); jz4740_mmc_free_gpios(pdev);
jz_gpio_bulk_free(jz4740_mmc_pins, jz4740_mmc_num_pins(host)); jz_gpio_bulk_free(jz4740_mmc_pins, jz4740_mmc_num_pins(host));
iounmap(host->base);
release_mem_region(host->mem->start, resource_size(host->mem));
clk_put(host->clk);
mmc_free_host(host->mmc); mmc_free_host(host->mmc);
return 0; return 0;
......
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