diff mbox series

input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly

Message ID 1569315731-2387-1-git-send-email-Anson.Huang@nxp.com (mailing list archive)
State New, archived
Headers show
Series input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly | expand

Commit Message

Anson Huang Sept. 24, 2019, 9:02 a.m. UTC 
Add helper variable dev = &pdev->dev to simply the code.

Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
 drivers/input/touchscreen/imx6ul_tsc.c | 37 +++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 18 deletions(-)

Comments

Dmitry Torokhov Sept. 30, 2019, 11:08 p.m. UTC | #1 
On Tue, Sep 24, 2019 at 05:02:11PM +0800, Anson Huang wrote:
> Add helper variable dev = &pdev->dev to simply the code.

Do we get any code savings from this?

> 
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
>  drivers/input/touchscreen/imx6ul_tsc.c | 37 +++++++++++++++++-----------------
>  1 file changed, 19 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/input/touchscreen/imx6ul_tsc.c b/drivers/input/touchscreen/imx6ul_tsc.c
> index 9ed2588..4555aa9 100644
> --- a/drivers/input/touchscreen/imx6ul_tsc.c
> +++ b/drivers/input/touchscreen/imx6ul_tsc.c
> @@ -361,7 +361,8 @@ static void imx6ul_tsc_close(struct input_dev *input_dev)
>  
>  static int imx6ul_tsc_probe(struct platform_device *pdev)
>  {
> -	struct device_node *np = pdev->dev.of_node;
> +	struct device *dev = &pdev->dev;
> +	struct device_node *np = dev->of_node;
>  	struct imx6ul_tsc *tsc;
>  	struct input_dev *input_dev;
>  	int err;
> @@ -369,11 +370,11 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  	int adc_irq;
>  	u32 average_samples;
>  
> -	tsc = devm_kzalloc(&pdev->dev, sizeof(*tsc), GFP_KERNEL);
> +	tsc = devm_kzalloc(dev, sizeof(*tsc), GFP_KERNEL);
>  	if (!tsc)
>  		return -ENOMEM;
>  
> -	input_dev = devm_input_allocate_device(&pdev->dev);
> +	input_dev = devm_input_allocate_device(dev);
>  	if (!input_dev)
>  		return -ENOMEM;
>  
> @@ -389,14 +390,14 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  
>  	input_set_drvdata(input_dev, tsc);
>  
> -	tsc->dev = &pdev->dev;
> +	tsc->dev = dev;
>  	tsc->input = input_dev;
>  	init_completion(&tsc->completion);
>  
> -	tsc->xnur_gpio = devm_gpiod_get(&pdev->dev, "xnur", GPIOD_IN);
> +	tsc->xnur_gpio = devm_gpiod_get(dev, "xnur", GPIOD_IN);
>  	if (IS_ERR(tsc->xnur_gpio)) {
>  		err = PTR_ERR(tsc->xnur_gpio);
> -		dev_err(&pdev->dev,
> +		dev_err(dev,
>  			"failed to request GPIO tsc_X- (xnur): %d\n", err);
>  		return err;
>  	}
> @@ -404,28 +405,28 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  	tsc->tsc_regs = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(tsc->tsc_regs)) {
>  		err = PTR_ERR(tsc->tsc_regs);
> -		dev_err(&pdev->dev, "failed to remap tsc memory: %d\n", err);
> +		dev_err(dev, "failed to remap tsc memory: %d\n", err);
>  		return err;
>  	}
>  
>  	tsc->adc_regs = devm_platform_ioremap_resource(pdev, 1);
>  	if (IS_ERR(tsc->adc_regs)) {
>  		err = PTR_ERR(tsc->adc_regs);
> -		dev_err(&pdev->dev, "failed to remap adc memory: %d\n", err);
> +		dev_err(dev, "failed to remap adc memory: %d\n", err);
>  		return err;
>  	}
>  
> -	tsc->tsc_clk = devm_clk_get(&pdev->dev, "tsc");
> +	tsc->tsc_clk = devm_clk_get(dev, "tsc");
>  	if (IS_ERR(tsc->tsc_clk)) {
>  		err = PTR_ERR(tsc->tsc_clk);
> -		dev_err(&pdev->dev, "failed getting tsc clock: %d\n", err);
> +		dev_err(dev, "failed getting tsc clock: %d\n", err);
>  		return err;
>  	}
>  
> -	tsc->adc_clk = devm_clk_get(&pdev->dev, "adc");
> +	tsc->adc_clk = devm_clk_get(dev, "adc");
>  	if (IS_ERR(tsc->adc_clk)) {
>  		err = PTR_ERR(tsc->adc_clk);
> -		dev_err(&pdev->dev, "failed getting adc clock: %d\n", err);
> +		dev_err(dev, "failed getting adc clock: %d\n", err);
>  		return err;
>  	}
>  
> @@ -439,18 +440,18 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  
>  	err = devm_request_threaded_irq(tsc->dev, tsc_irq,
>  					NULL, tsc_irq_fn, IRQF_ONESHOT,
> -					dev_name(&pdev->dev), tsc);
> +					dev_name(dev), tsc);
>  	if (err) {
> -		dev_err(&pdev->dev,
> +		dev_err(dev,
>  			"failed requesting tsc irq %d: %d\n",
>  			tsc_irq, err);
>  		return err;
>  	}
>  
>  	err = devm_request_irq(tsc->dev, adc_irq, adc_irq_fn, 0,
> -				dev_name(&pdev->dev), tsc);
> +				dev_name(dev), tsc);
>  	if (err) {
> -		dev_err(&pdev->dev,
> +		dev_err(dev,
>  			"failed requesting adc irq %d: %d\n",
>  			adc_irq, err);
>  		return err;
> @@ -484,7 +485,7 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  		tsc->average_select = ilog2(average_samples) - 2;
>  		break;
>  	default:
> -		dev_err(&pdev->dev,
> +		dev_err(dev,
>  			"touchscreen-average-samples (%u) must be 1, 4, 8, 16 or 32\n",
>  			average_samples);
>  		return -EINVAL;
> @@ -492,7 +493,7 @@ static int imx6ul_tsc_probe(struct platform_device *pdev)
>  
>  	err = input_register_device(tsc->input);
>  	if (err) {
> -		dev_err(&pdev->dev,
> +		dev_err(dev,
>  			"failed to register input device: %d\n", err);
>  		return err;
>  	}
> -- 
> 2.7.4
>
Michal Vokáč Oct. 2, 2019, 5:32 a.m. UTC | #2 
On 01. 10. 19 1:08, Dmitry Torokhov wrote:
> On Tue, Sep 24, 2019 at 05:02:11PM +0800, Anson Huang wrote:
>> Add helper variable dev = &pdev->dev to simply the code.
> 
> Do we get any code savings from this?

I noticed Anson submitted similar patches into various other subsystems.
Here [1] is a very detailed response from David Laight shedding some light
on this topic.

[1] https://lkml.org/lkml/2019/9/24/151

>>
>> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>> ---
>>   drivers/input/touchscreen/imx6ul_tsc.c | 37 +++++++++++++++++-----------------
>>   1 file changed, 19 insertions(+), 18 deletions(-)
diff mbox series

Patch

diff --git a/drivers/input/touchscreen/imx6ul_tsc.c b/drivers/input/touchscreen/imx6ul_tsc.c
index 9ed2588..4555aa9 100644
--- a/drivers/input/touchscreen/imx6ul_tsc.c
+++ b/drivers/input/touchscreen/imx6ul_tsc.c
@@ -361,7 +361,8 @@  static void imx6ul_tsc_close(struct input_dev *input_dev)
 
 static int imx6ul_tsc_probe(struct platform_device *pdev)
 {
-	struct device_node *np = pdev->dev.of_node;
+	struct device *dev = &pdev->dev;
+	struct device_node *np = dev->of_node;
 	struct imx6ul_tsc *tsc;
 	struct input_dev *input_dev;
 	int err;
@@ -369,11 +370,11 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 	int adc_irq;
 	u32 average_samples;
 
-	tsc = devm_kzalloc(&pdev->dev, sizeof(*tsc), GFP_KERNEL);
+	tsc = devm_kzalloc(dev, sizeof(*tsc), GFP_KERNEL);
 	if (!tsc)
 		return -ENOMEM;
 
-	input_dev = devm_input_allocate_device(&pdev->dev);
+	input_dev = devm_input_allocate_device(dev);
 	if (!input_dev)
 		return -ENOMEM;
 
@@ -389,14 +390,14 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 
 	input_set_drvdata(input_dev, tsc);
 
-	tsc->dev = &pdev->dev;
+	tsc->dev = dev;
 	tsc->input = input_dev;
 	init_completion(&tsc->completion);
 
-	tsc->xnur_gpio = devm_gpiod_get(&pdev->dev, "xnur", GPIOD_IN);
+	tsc->xnur_gpio = devm_gpiod_get(dev, "xnur", GPIOD_IN);
 	if (IS_ERR(tsc->xnur_gpio)) {
 		err = PTR_ERR(tsc->xnur_gpio);
-		dev_err(&pdev->dev,
+		dev_err(dev,
 			"failed to request GPIO tsc_X- (xnur): %d\n", err);
 		return err;
 	}
@@ -404,28 +405,28 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 	tsc->tsc_regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(tsc->tsc_regs)) {
 		err = PTR_ERR(tsc->tsc_regs);
-		dev_err(&pdev->dev, "failed to remap tsc memory: %d\n", err);
+		dev_err(dev, "failed to remap tsc memory: %d\n", err);
 		return err;
 	}
 
 	tsc->adc_regs = devm_platform_ioremap_resource(pdev, 1);
 	if (IS_ERR(tsc->adc_regs)) {
 		err = PTR_ERR(tsc->adc_regs);
-		dev_err(&pdev->dev, "failed to remap adc memory: %d\n", err);
+		dev_err(dev, "failed to remap adc memory: %d\n", err);
 		return err;
 	}
 
-	tsc->tsc_clk = devm_clk_get(&pdev->dev, "tsc");
+	tsc->tsc_clk = devm_clk_get(dev, "tsc");
 	if (IS_ERR(tsc->tsc_clk)) {
 		err = PTR_ERR(tsc->tsc_clk);
-		dev_err(&pdev->dev, "failed getting tsc clock: %d\n", err);
+		dev_err(dev, "failed getting tsc clock: %d\n", err);
 		return err;
 	}
 
-	tsc->adc_clk = devm_clk_get(&pdev->dev, "adc");
+	tsc->adc_clk = devm_clk_get(dev, "adc");
 	if (IS_ERR(tsc->adc_clk)) {
 		err = PTR_ERR(tsc->adc_clk);
-		dev_err(&pdev->dev, "failed getting adc clock: %d\n", err);
+		dev_err(dev, "failed getting adc clock: %d\n", err);
 		return err;
 	}
 
@@ -439,18 +440,18 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 
 	err = devm_request_threaded_irq(tsc->dev, tsc_irq,
 					NULL, tsc_irq_fn, IRQF_ONESHOT,
-					dev_name(&pdev->dev), tsc);
+					dev_name(dev), tsc);
 	if (err) {
-		dev_err(&pdev->dev,
+		dev_err(dev,
 			"failed requesting tsc irq %d: %d\n",
 			tsc_irq, err);
 		return err;
 	}
 
 	err = devm_request_irq(tsc->dev, adc_irq, adc_irq_fn, 0,
-				dev_name(&pdev->dev), tsc);
+				dev_name(dev), tsc);
 	if (err) {
-		dev_err(&pdev->dev,
+		dev_err(dev,
 			"failed requesting adc irq %d: %d\n",
 			adc_irq, err);
 		return err;
@@ -484,7 +485,7 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 		tsc->average_select = ilog2(average_samples) - 2;
 		break;
 	default:
-		dev_err(&pdev->dev,
+		dev_err(dev,
 			"touchscreen-average-samples (%u) must be 1, 4, 8, 16 or 32\n",
 			average_samples);
 		return -EINVAL;
@@ -492,7 +493,7 @@  static int imx6ul_tsc_probe(struct platform_device *pdev)
 
 	err = input_register_device(tsc->input);
 	if (err) {
-		dev_err(&pdev->dev,
+		dev_err(dev,
 			"failed to register input device: %d\n", err);
 		return err;
 	}