Re: [PATCH] cpufreq-SCPI: Delete unnecessary assignment for the field "owner"

From: Rafael J. Wysocki
Date: Tue Aug 16 2016 - 16:42:49 EST


On Tuesday, August 16, 2016 10:38:02 PM SF Markus Elfring wrote:
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Tue, 16 Aug 2016 22:30:17 +0200
>
> The field "owner" is set by the core.
> Thus delete an unneeded initialisation.
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> ---
> drivers/cpufreq/scpi-cpufreq.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/cpufreq/scpi-cpufreq.c b/drivers/cpufreq/scpi-cpufreq.c
> index e8a7bf5..4a70d241 100644
> --- a/drivers/cpufreq/scpi-cpufreq.c
> +++ b/drivers/cpufreq/scpi-cpufreq.c
> @@ -104,8 +104,7 @@ static int scpi_cpufreq_remove(struct platform_device *pdev)
>
> static struct platform_driver scpi_cpufreq_platdrv = {
> .driver = {
> - .name = "scpi-cpufreq",
> - .owner = THIS_MODULE,
> + .name = "scpi-cpufreq"

Comma, please.

> },
> .probe = scpi_cpufreq_probe,
> .remove = scpi_cpufreq_remove,
>

Thanks.
Rafael