linux-mediatek.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings
@ 2019-03-21  1:45 YueHaibing
  2019-03-21  6:07 ` Stanley Chu
  2019-03-26  2:07 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: YueHaibing @ 2019-03-21  1:45 UTC (permalink / raw)
  To: Alim Akhtar, Avri Altman, Pedro Sousa, James E.J. Bottomley,
	Martin K. Petersen, Matthias Brugger, Stanley Chu
  Cc: linux-mediatek, kernel-janitors, YueHaibing, linux-arm-kernel,
	linux-scsi

Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/scsi/ufs/ufs-mediatek.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/scsi/ufs/ufs-mediatek.c b/drivers/scsi/ufs/ufs-mediatek.c
index c3b78fc83d3f..e9590bff5b5c 100644
--- a/drivers/scsi/ufs/ufs-mediatek.c
+++ b/drivers/scsi/ufs/ufs-mediatek.c
@@ -358,7 +358,6 @@ static struct platform_driver ufs_mtk_pltform = {
 	.shutdown   = ufshcd_pltfrm_shutdown,
 	.driver = {
 		.name   = "ufshcd-mtk",
-		.owner  = THIS_MODULE,
 		.pm     = &ufs_mtk_pm_ops,
 		.of_match_table = ufs_mtk_of_match,
 	},

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings
  2019-03-21  1:45 [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings YueHaibing
@ 2019-03-21  6:07 ` Stanley Chu
  2019-03-26  2:07 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Stanley Chu @ 2019-03-21  6:07 UTC (permalink / raw)
  To: YueHaibing
  Cc: linux-scsi, Martin K. Petersen, James E.J. Bottomley,
	kernel-janitors, Avri Altman, linux-mediatek, Alim Akhtar,
	Matthias Brugger, Pedro Sousa, linux-arm-kernel

Hi,

On Thu, 2019-03-21 at 01:45 +0000, YueHaibing wrote:
> Remove .owner field if calls are used which set it automatically
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> 
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---
>  drivers/scsi/ufs/ufs-mediatek.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/scsi/ufs/ufs-mediatek.c b/drivers/scsi/ufs/ufs-mediatek.c
> index c3b78fc83d3f..e9590bff5b5c 100644
> --- a/drivers/scsi/ufs/ufs-mediatek.c
> +++ b/drivers/scsi/ufs/ufs-mediatek.c
> @@ -358,7 +358,6 @@ static struct platform_driver ufs_mtk_pltform = {
>  	.shutdown   = ufshcd_pltfrm_shutdown,
>  	.driver = {
>  		.name   = "ufshcd-mtk",
> -		.owner  = THIS_MODULE,
>  		.pm     = &ufs_mtk_pm_ops,
>  		.of_match_table = ufs_mtk_of_match,
>  	},
> 
> 
This is redundant currently. Thanks for helping removing it.

Reviewed-by: Stanley Chu <stanley.chu@mediatek.com>


Thanks,
Stanley

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings
  2019-03-21  1:45 [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings YueHaibing
  2019-03-21  6:07 ` Stanley Chu
@ 2019-03-26  2:07 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2019-03-26  2:07 UTC (permalink / raw)
  To: YueHaibing
  Cc: linux-scsi, Martin K. Petersen, James E.J. Bottomley,
	kernel-janitors, Avri Altman, linux-mediatek, Alim Akhtar,
	Matthias Brugger, Stanley Chu, Pedro Sousa, linux-arm-kernel


YueHaibing,

> Remove .owner field if calls are used which set it automatically
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

Applied to 5.2/scsi-queue, thanks.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-03-26  2:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-21  1:45 [PATCH -next] scsi: ufs-mediatek: Fix platform_no_drv_owner.cocci warnings YueHaibing
2019-03-21  6:07 ` Stanley Chu
2019-03-26  2:07 ` Martin K. Petersen

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).