Re: [PATCH 5/5] Input: ts4800-ts: Use devm_platform_ioremap_resource()

From: Dmitry Torokhov
Date: Tue Jul 16 2019 - 11:52:56 EST


On Thu, Apr 04, 2019 at 12:47:23PM +0530, Mukesh Ojha wrote:
> devm_platform_ioremap_resource() internally have platform_get_resource()
> and devm_ioremap_resource() in it. So instead of calling them separately
> use devm_platform_ioremap_resource() directly.
>
> Signed-off-by: Mukesh Ojha <mojha@xxxxxxxxxxxxxx>

Applied, thank you.

> ---
> drivers/input/touchscreen/ts4800-ts.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/input/touchscreen/ts4800-ts.c b/drivers/input/touchscreen/ts4800-ts.c
> index fed73ee..5b4f536 100644
> --- a/drivers/input/touchscreen/ts4800-ts.c
> +++ b/drivers/input/touchscreen/ts4800-ts.c
> @@ -148,7 +148,6 @@ static int ts4800_ts_probe(struct platform_device *pdev)
> {
> struct input_polled_dev *poll_dev;
> struct ts4800_ts *ts;
> - struct resource *res;
> int error;
>
> ts = devm_kzalloc(&pdev->dev, sizeof(*ts), GFP_KERNEL);
> @@ -159,8 +158,7 @@ static int ts4800_ts_probe(struct platform_device *pdev)
> if (error)
> return error;
>
> - res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> - ts->base = devm_ioremap_resource(&pdev->dev, res);
> + ts->base = devm_platform_ioremap_resource(pdev, 0);
> if (IS_ERR(ts->base))
> return PTR_ERR(ts->base);
>
> --
> Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center,
> Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
>

--
Dmitry