[input:next 1341/1344] drivers/input/touchscreen/ft6236.c:316:3-8: No need to set .owner here. The core will do it.

[input:next 1341/1344] drivers/input/touchscreen/ft6236.c:316:3-8: No need to set .owner here. The core will do it.

[kbuild test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
head:   c65cf815653ec64e73d40819806c2d1aed91681e
commit: 92deea1350f9fa97a841d45cd1f8228d5e8aa667 [1341/1344] Input: add support for FocalTech FT6236 touchscreen controller


coccinelle warnings: (new ones prefixed by >>)

>> drivers/input/touchscreen/ft6236.c:316:3-8: No need to set .owner here. The core will do it.

Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/ft6236.c:316: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: Noralf Trønnes <noralf@tronnes.org>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/ft6236.c
+++ b/drivers/input/touchscreen/ft6236.c
@@ -313,7 +313,6 @@ MODULE_DEVICE_TABLE(i2c, ft6236_id);
 static struct i2c_driver ft6236_driver = {
 	.driver = {
 		.name = "ft6236",
-		.owner = THIS_MODULE,
 		.of_match_table = of_match_ptr(ft6236_of_match),
 	},
 	.probe = ft6236_probe,

Re: [PATCH] Input: fix platform_no_drv_owner.cocci warnings

[Dmitry Torokhov]

On Tue, Oct 06, 2015 at 09:35:31AM +0800, kbuild test robot wrote:
> drivers/input/touchscreen/ft6236.c:316: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: Noralf Trønnes <noralf@tronnes.org>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Applied, thank you.

> ---
> 
>  ft6236.c |    1 -
>  1 file changed, 1 deletion(-)
> 
> --- a/drivers/input/touchscreen/ft6236.c
> +++ b/drivers/input/touchscreen/ft6236.c
> @@ -313,7 +313,6 @@ MODULE_DEVICE_TABLE(i2c, ft6236_id);
>  static struct i2c_driver ft6236_driver = {
>  	.driver = {
>  		.name = "ft6236",
> -		.owner = THIS_MODULE,
>  		.of_match_table = of_match_ptr(ft6236_of_match),
>  	},
>  	.probe = ft6236_probe,

-- 
Dmitry

Re: [PATCH] Input: fix platform_no_drv_owner.cocci warnings

[Dmitry Torokhov]

On Tue, Oct 06, 2015 at 09:35:31AM +0800, kbuild test robot wrote:
> drivers/input/touchscreen/ft6236.c:316: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: Noralf Trønnes <noralf@tronnes.org>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Applied, thank you.

> ---
> 
>  ft6236.c |    1 -
>  1 file changed, 1 deletion(-)
> 
> --- a/drivers/input/touchscreen/ft6236.c
> +++ b/drivers/input/touchscreen/ft6236.c
> @@ -313,7 +313,6 @@ MODULE_DEVICE_TABLE(i2c, ft6236_id);
>  static struct i2c_driver ft6236_driver = {
>  	.driver = {
>  		.name = "ft6236",
> -		.owner = THIS_MODULE,
>  		.of_match_table = of_match_ptr(ft6236_of_match),
>  	},
>  	.probe = ft6236_probe,

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

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/cyttsp5.c:1098: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

Fixes: 68d0bd1e4815 ("Input: Add driver for Cypress Generation 5 touchscreen")
CC: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/cyttsp5.c
+++ b/drivers/input/touchscreen/cyttsp5.c
@@ -1095,7 +1095,6 @@ static int cyttsp5_i2c_remove(struct i2c
 static struct i2c_driver cyttsp5_i2c_driver = {
 	.driver = {
 		.name = CYTTSP5_NAME,
-		.owner = THIS_MODULE,
 		.of_match_table = of_match_ptr(cyttsp5_of_match),
 	},
 	.probe = cyttsp5_i2c_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/cyttsp5.c:1098: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

Fixes: 68d0bd1e4815 ("Input: Add driver for Cypress Generation 5 touchscreen")
CC: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/cyttsp5.c
+++ b/drivers/input/touchscreen/cyttsp5.c
@@ -1095,7 +1095,6 @@ static int cyttsp5_i2c_remove(struct i2c
 static struct i2c_driver cyttsp5_i2c_driver = {
 	.driver = {
 		.name = CYTTSP5_NAME,
-		.owner = THIS_MODULE,
 		.of_match_table = of_match_ptr(cyttsp5_of_match),
 	},
 	.probe = cyttsp5_i2c_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/goodix-ts-sunrise/goodix_gtx5_i2c.c:872: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: Wang Yafei <wangyafei@goodix.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/goodix-ts-sunrise/goodix_gtx5_i2c.c
+++ b/drivers/input/touchscreen/goodix-ts-sunrise/goodix_gtx5_i2c.c
@@ -869,7 +869,6 @@ MODULE_DEVICE_TABLE(i2c, gtx5_id_table);
 static struct i2c_driver goodix_i2c_driver = {
 	.driver = {
 		.name = TS_DRIVER_NAME,
-		.owner = THIS_MODULE,
 		.of_match_table = of_match_ptr(gtx5_of_matchs),
 	},
 	.probe = goodix_i2c_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/goodix-ts-sunrise/goodix_ts_core.c:1335: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: Wang Yafei <wangyafei@goodix.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/goodix-ts-sunrise/goodix_ts_core.c
+++ b/drivers/input/touchscreen/goodix-ts-sunrise/goodix_ts_core.c
@@ -1332,7 +1332,6 @@ MODULE_DEVICE_TABLE(platform, ts_core_id
 static struct platform_driver goodix_ts_driver = {
 	.driver = {
 		.name = GOODIX_CORE_DRIVER_NAME,
-		.owner = THIS_MODULE,
 		.pm = &dev_pm_ops,
 	},
 	.probe = goodix_ts_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/cyttsp5.c:1162: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: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/cyttsp5.c
+++ b/drivers/input/touchscreen/cyttsp5.c
@@ -1159,7 +1159,6 @@ static int cyttsp5_i2c_remove(struct i2c
 static struct i2c_driver cyttsp5_i2c_driver = {
 	.driver = {
 		.name = CYTTSP5_NAME,
-		.owner = THIS_MODULE,
 		.of_match_table = cyttsp5_of_match,
 	},
 	.probe = cyttsp5_i2c_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/touchscreen/cyttsp5.c:1162: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: Mylène Josserand <mylene.josserand@free-electrons.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/touchscreen/cyttsp5.c
+++ b/drivers/input/touchscreen/cyttsp5.c
@@ -1159,7 +1159,6 @@ static int cyttsp5_i2c_remove(struct i2c
 static struct i2c_driver cyttsp5_i2c_driver = {
 	.driver = {
 		.name = CYTTSP5_NAME,
-		.owner = THIS_MODULE,
 		.of_match_table = cyttsp5_of_match,
 	},
 	.probe = cyttsp5_i2c_probe,

[PATCH] Input: fix platform_no_drv_owner.cocci warnings

[kbuild test robot]

drivers/input/rmi4/rmi_smbus.c:448: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: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--- a/drivers/input/rmi4/rmi_smbus.c
+++ b/drivers/input/rmi4/rmi_smbus.c
@@ -445,7 +445,6 @@ MODULE_DEVICE_TABLE(i2c, rmi_id);
 
 static struct i2c_driver rmi_smb_driver = {
 	.driver = {
-		.owner	= THIS_MODULE,
 		.name	= "rmi4_smbus",
 		.pm	= &rmi_smb_pm,
 		.resume	= rmi_smb_resume,