diff mbox

power: max77693: fix platform_no_drv_owner.cocci warnings

Message ID 20150121041134.GA205330@athens (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Fengguang Wu Jan. 21, 2015, 4:11 a.m. UTC
drivers/power/max77693_charger.c:747:3-8: No need to set .owner here. The core will do it.

 Remove .owner field if calls are used which set it automatically

Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

CC: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

 max77693_charger.c |    1 -
 1 file changed, 1 deletion(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Krzysztof Kozlowski Jan. 21, 2015, 7:58 a.m. UTC | #1
On ?ro, 2015-01-21 at 12:11 +0800, kbuild test robot wrote:
> drivers/power/max77693_charger.c:747:3-8: No need to set .owner here. The core will do it.
> 
>  Remove .owner field if calls are used which set it automatically
> 
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> 
> CC: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
> 
>  max77693_charger.c |    1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>


> 
> --- a/drivers/power/max77693_charger.c
> +++ b/drivers/power/max77693_charger.c
> @@ -744,7 +744,6 @@ MODULE_DEVICE_TABLE(platform, max77693_c
>  
>  static struct platform_driver max77693_charger_driver = {
>  	.driver = {
> -		.owner	= THIS_MODULE,
>  		.name	= "max77693-charger",
>  	},
>  	.probe		= max77693_charger_probe,

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sebastian Reichel Jan. 21, 2015, 1:05 p.m. UTC | #2
Hi,

On Wed, Jan 21, 2015 at 12:11:34PM +0800, kbuild test robot wrote:
> drivers/power/max77693_charger.c:747:3-8: No need to set .owner here. The core will do it.
> 
>  Remove .owner field if calls are used which set it automatically
> 
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> 
> CC: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Thanks, applied with Reviewed-By from Krzysztof.

-- Sebastian
diff mbox

Patch

--- a/drivers/power/max77693_charger.c
+++ b/drivers/power/max77693_charger.c
@@ -744,7 +744,6 @@  MODULE_DEVICE_TABLE(platform, max77693_c
 
 static struct platform_driver max77693_charger_driver = {
 	.driver = {
-		.owner	= THIS_MODULE,
 		.name	= "max77693-charger",
 	},
 	.probe		= max77693_charger_probe,