[PATCH 03/10] video: da8xx-fb: use devres
Replace existing resource handling in the driver with managed device resource. Signed-off-by: Afzal Mohammed --- drivers/video/da8xx-fb.c | 35 ++- 1 file changed, 6 insertions(+), 29 deletions(-) diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c index 4f92028..e119ec8 100644 --- a/drivers/video/da8xx-fb.c +++ b/drivers/video/da8xx-fb.c @@ -1040,12 +1040,9 @@ static int __devexit fb_remove(struct platform_device *dev) par->p_palette_base); dma_free_coherent(NULL, par->vram_size, par->vram_virt, par->vram_phys); - free_irq(par->irq, par); pm_runtime_put_sync(>dev); pm_runtime_disable(>dev); framebuffer_release(info); - iounmap(da8xx_fb_reg_base); - release_mem_region(lcdc_regs->start, resource_size(lcdc_regs)); } return 0; @@ -1269,7 +1266,6 @@ static int __devinit fb_probe(struct platform_device *device) struct fb_info *da8xx_fb_info; struct clk *fb_clk = NULL; struct da8xx_fb_par *par; - resource_size_t len; int ret, i; unsigned long ulcm; @@ -1279,29 +1275,16 @@ static int __devinit fb_probe(struct platform_device *device) } lcdc_regs = platform_get_resource(device, IORESOURCE_MEM, 0); - if (!lcdc_regs) { - dev_err(>dev, - "Can not get memory resource for LCD controller\n"); - return -ENOENT; - } - - len = resource_size(lcdc_regs); - - lcdc_regs = request_mem_region(lcdc_regs->start, len, lcdc_regs->name); - if (!lcdc_regs) - return -EBUSY; - - da8xx_fb_reg_base = ioremap(lcdc_regs->start, len); + da8xx_fb_reg_base = devm_request_and_ioremap(>dev, lcdc_regs); if (!da8xx_fb_reg_base) { - ret = -EBUSY; - goto err_request_mem; + dev_err(>dev, "memory resource setup failed\n"); + return -EADDRNOTAVAIL; } - fb_clk = clk_get(>dev, "fck"); + fb_clk = devm_clk_get(>dev, "fck"); if (IS_ERR(fb_clk)) { dev_err(>dev, "Can not get device clock\n"); - ret = -ENODEV; - goto err_ioremap; + return -ENODEV; } pm_runtime_enable(>dev); @@ -1462,7 +1445,7 @@ static int __devinit fb_probe(struct platform_device *device) lcdc_irq_handler = lcdc_irq_handler_rev02; } - ret = request_irq(par->irq, lcdc_irq_handler, 0, + ret = devm_request_irq(>dev, par->irq, lcdc_irq_handler, 0, DRIVER_NAME, par); if (ret) goto irq_freq; @@ -1492,12 +1475,6 @@ err_pm_runtime_disable: pm_runtime_put_sync(>dev); pm_runtime_disable(>dev); -err_ioremap: - iounmap(da8xx_fb_reg_base); - -err_request_mem: - release_mem_region(lcdc_regs->start, len); - return ret; } -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 03/10] video: da8xx-fb: use devres
Replace existing resource handling in the driver with managed device resource. Signed-off-by: Afzal Mohammed af...@ti.com --- drivers/video/da8xx-fb.c | 35 ++- 1 file changed, 6 insertions(+), 29 deletions(-) diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c index 4f92028..e119ec8 100644 --- a/drivers/video/da8xx-fb.c +++ b/drivers/video/da8xx-fb.c @@ -1040,12 +1040,9 @@ static int __devexit fb_remove(struct platform_device *dev) par-p_palette_base); dma_free_coherent(NULL, par-vram_size, par-vram_virt, par-vram_phys); - free_irq(par-irq, par); pm_runtime_put_sync(dev-dev); pm_runtime_disable(dev-dev); framebuffer_release(info); - iounmap(da8xx_fb_reg_base); - release_mem_region(lcdc_regs-start, resource_size(lcdc_regs)); } return 0; @@ -1269,7 +1266,6 @@ static int __devinit fb_probe(struct platform_device *device) struct fb_info *da8xx_fb_info; struct clk *fb_clk = NULL; struct da8xx_fb_par *par; - resource_size_t len; int ret, i; unsigned long ulcm; @@ -1279,29 +1275,16 @@ static int __devinit fb_probe(struct platform_device *device) } lcdc_regs = platform_get_resource(device, IORESOURCE_MEM, 0); - if (!lcdc_regs) { - dev_err(device-dev, - Can not get memory resource for LCD controller\n); - return -ENOENT; - } - - len = resource_size(lcdc_regs); - - lcdc_regs = request_mem_region(lcdc_regs-start, len, lcdc_regs-name); - if (!lcdc_regs) - return -EBUSY; - - da8xx_fb_reg_base = ioremap(lcdc_regs-start, len); + da8xx_fb_reg_base = devm_request_and_ioremap(device-dev, lcdc_regs); if (!da8xx_fb_reg_base) { - ret = -EBUSY; - goto err_request_mem; + dev_err(device-dev, memory resource setup failed\n); + return -EADDRNOTAVAIL; } - fb_clk = clk_get(device-dev, fck); + fb_clk = devm_clk_get(device-dev, fck); if (IS_ERR(fb_clk)) { dev_err(device-dev, Can not get device clock\n); - ret = -ENODEV; - goto err_ioremap; + return -ENODEV; } pm_runtime_enable(device-dev); @@ -1462,7 +1445,7 @@ static int __devinit fb_probe(struct platform_device *device) lcdc_irq_handler = lcdc_irq_handler_rev02; } - ret = request_irq(par-irq, lcdc_irq_handler, 0, + ret = devm_request_irq(device-dev, par-irq, lcdc_irq_handler, 0, DRIVER_NAME, par); if (ret) goto irq_freq; @@ -1492,12 +1475,6 @@ err_pm_runtime_disable: pm_runtime_put_sync(device-dev); pm_runtime_disable(device-dev); -err_ioremap: - iounmap(da8xx_fb_reg_base); - -err_request_mem: - release_mem_region(lcdc_regs-start, len); - return ret; } -- 1.7.9.5 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/