* [PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly
@ 2019-09-24 9:02 Anson Huang
2019-09-30 23:08 ` Dmitry Torokhov
0 siblings, 1 reply; 3+ messages in thread
From: Anson Huang @ 2019-09-24 9:02 UTC (permalink / raw)
To: dmitry.torokhov, shawnguo, s.hauer, kernel, festevam, swboyd,
mojha, linux-input, linux-arm-kernel, linux-kernel
Cc: Linux-imx
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(-)
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly
2019-09-24 9:02 [PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly Anson Huang
@ 2019-09-30 23:08 ` Dmitry Torokhov
2019-10-02 5:32 ` Michal Vokáč
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Torokhov @ 2019-09-30 23:08 UTC (permalink / raw)
To: Anson Huang
Cc: shawnguo, s.hauer, linux-kernel, swboyd, Linux-imx, kernel,
linux-input, festevam, mojha, linux-arm-kernel
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
>
--
Dmitry
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly
2019-09-30 23:08 ` Dmitry Torokhov
@ 2019-10-02 5:32 ` Michal Vokáč
0 siblings, 0 replies; 3+ messages in thread
From: Michal Vokáč @ 2019-10-02 5:32 UTC (permalink / raw)
To: Dmitry Torokhov, Anson Huang
Cc: shawnguo, s.hauer, linux-kernel, swboyd, Linux-imx, kernel,
linux-input, festevam, mojha, linux-arm-kernel
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(-)
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-02 5:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-24 9:02 [PATCH] input: touchscreen: imx6ul_tsc: Use 'dev' instead of dereferencing it repeatedly Anson Huang
2019-09-30 23:08 ` Dmitry Torokhov
2019-10-02 5:32 ` Michal Vokáč
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).