[PATCH] staging: remove unneeded static set .owner field in platform_driver
From: vaibhav.sr@gmail.com (Vaibhav Agarwal)
Date: 2018-09-12 05:59:47
Also in:
lkml
On Wed, Sep 12, 2018 at 9:22 AM zhong jiang [off-list ref] wrote:
quoted hunk ↗ jump to hunk
platform_driver_register will set the .owner field. So it is safe to remove the redundant assignment. The issue is detected with the help of Coccinelle. Signed-off-by: zhong jiang <redacted> --- drivers/staging/greybus/audio_codec.c | 1 - drivers/staging/mt7621-eth/gsw_mt7621.c | 1 - drivers/staging/mt7621-eth/mtk_eth_soc.c | 1 - 3 files changed, 3 deletions(-)diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index 35acd55..08746c8 100644 --- a/drivers/staging/greybus/audio_codec.c +++ b/drivers/staging/greybus/audio_codec.c@@ -1087,7 +1087,6 @@ static int gbaudio_codec_remove(struct platform_device *pdev) static struct platform_driver gbaudio_codec_driver = { .driver = { .name = "apb-dummy-codec", - .owner = THIS_MODULE, #ifdef CONFIG_PM .pm = &gbaudio_codec_pm_ops, #endifdiff --git a/drivers/staging/mt7621-eth/gsw_mt7621.c b/drivers/staging/mt7621-eth/gsw_mt7621.c index 2c07b55..53767b1 100644 --- a/drivers/staging/mt7621-eth/gsw_mt7621.c +++ b/drivers/staging/mt7621-eth/gsw_mt7621.c@@ -286,7 +286,6 @@ static int mt7621_gsw_remove(struct platform_device *pdev) .remove = mt7621_gsw_remove, .driver = { .name = "mt7621-gsw", - .owner = THIS_MODULE, .of_match_table = mediatek_gsw_match, }, };diff --git a/drivers/staging/mt7621-eth/mtk_eth_soc.c b/drivers/staging/mt7621-eth/mtk_eth_soc.c index 7135075..363d3c9 100644 --- a/drivers/staging/mt7621-eth/mtk_eth_soc.c +++ b/drivers/staging/mt7621-eth/mtk_eth_soc.c@@ -2167,7 +2167,6 @@ static int mtk_remove(struct platform_device *pdev) .remove = mtk_remove, .driver = { .name = "mtk_soc_eth", - .owner = THIS_MODULE, .of_match_table = of_mtk_match, }, }; --1.7.12.4
Acked-by: Vaibhav Agarwal <vaibhav.sr@gmail.com>