Use dev_name() for requesting an interrupt so we don't get an interrupt requested with same name for multiple device instances. While at it, tidy up the driver data handling. Signed-off-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> --- drivers/media/video/s5p-fimc/fimc-core.c | 9 +++------ drivers/media/video/s5p-fimc/fimc-core.h | 2 ++ 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/media/video/s5p-fimc/fimc-core.c b/drivers/media/video/s5p-fimc/fimc-core.c index 2da6638..f67c3b6 100644 --- a/drivers/media/video/s5p-fimc/fimc-core.c +++ b/drivers/media/video/s5p-fimc/fimc-core.c @@ -792,15 +792,12 @@ static int fimc_m2m_resume(struct fimc_dev *fimc) static int fimc_probe(struct platform_device *pdev) { + struct fimc_drvdata *drv_data = fimc_get_drvdata(pdev); + struct s5p_platform_fimc *pdata; struct fimc_dev *fimc; struct resource *res; - struct fimc_drvdata *drv_data; - struct s5p_platform_fimc *pdata; int ret = 0; - drv_data = (struct fimc_drvdata *) - platform_get_device_id(pdev)->driver_data; - if (pdev->id >= ARRAY_SIZE(drv_data->variant)) { dev_err(&pdev->dev, "Invalid platform device id: %d\n", pdev->id); @@ -844,7 +841,7 @@ static int fimc_probe(struct platform_device *pdev) platform_set_drvdata(pdev, fimc); ret = devm_request_irq(&pdev->dev, res->start, fimc_irq_handler, - 0, pdev->name, fimc); + 0, dev_name(&pdev->dev), fimc); if (ret) { dev_err(&pdev->dev, "failed to install irq (%d)\n", ret); goto err_clk; diff --git a/drivers/media/video/s5p-fimc/fimc-core.h b/drivers/media/video/s5p-fimc/fimc-core.h index e3078d3..cbd1137 100644 --- a/drivers/media/video/s5p-fimc/fimc-core.h +++ b/drivers/media/video/s5p-fimc/fimc-core.h @@ -401,6 +401,8 @@ struct fimc_drvdata { unsigned long lclk_frequency; }; +#define fimc_get_drvdata(_pdev) \ + ((struct fimc_drvdata *) platform_get_device_id(_pdev)->driver_data) struct fimc_ctx; -- 1.7.10 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html