netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next 0/2] net: Remove redundant initialization owner
@ 2023-08-04  9:59 Li Zetao
  2023-08-04  9:59 ` [PATCH net-next 1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver Li Zetao
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Li Zetao @ 2023-08-04  9:59 UTC (permalink / raw)
  To: ioana.ciornei, davem, edumazet, kuba, pabeni; +Cc: lizetao1, netdev

This patch set removes redundant initialization owner when register a
fsl_mc_driver driver

Li Zetao (2):
  net: dpaa2-eth: Remove redundant initialization owner in
    dpaa2_eth_driver
  net: dpaa2-switch: Remove redundant initialization owner in
    dpaa2_switch_drv

 drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c    | 1 -
 drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c | 1 -
 2 files changed, 2 deletions(-)

-- 
2.34.1


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

* [PATCH net-next 1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver
  2023-08-04  9:59 [PATCH net-next 0/2] net: Remove redundant initialization owner Li Zetao
@ 2023-08-04  9:59 ` Li Zetao
  2023-08-04  9:59 ` [PATCH net-next 2/2] net: dpaa2-switch: Remove redundant initialization owner in dpaa2_switch_drv Li Zetao
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Li Zetao @ 2023-08-04  9:59 UTC (permalink / raw)
  To: ioana.ciornei, davem, edumazet, kuba, pabeni; +Cc: lizetao1, netdev

The fsl_mc_driver_register() will set "THIS_MODULE" to driver.owner when
register a fsl_mc_driver driver, so it is redundant initialization to set
driver.owner in dpaa2_eth_driver statement. Remove it for clean code.

Signed-off-by: Li Zetao <lizetao1@huawei.com>
---
 drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
index a9676d0dece8..15bab41cee48 100644
--- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
+++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
@@ -5087,7 +5087,6 @@ MODULE_DEVICE_TABLE(fslmc, dpaa2_eth_match_id_table);
 static struct fsl_mc_driver dpaa2_eth_driver = {
 	.driver = {
 		.name = KBUILD_MODNAME,
-		.owner = THIS_MODULE,
 	},
 	.probe = dpaa2_eth_probe,
 	.remove = dpaa2_eth_remove,
-- 
2.34.1


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

* [PATCH net-next 2/2] net: dpaa2-switch: Remove redundant initialization owner in dpaa2_switch_drv
  2023-08-04  9:59 [PATCH net-next 0/2] net: Remove redundant initialization owner Li Zetao
  2023-08-04  9:59 ` [PATCH net-next 1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver Li Zetao
@ 2023-08-04  9:59 ` Li Zetao
  2023-08-05 11:38 ` [PATCH net-next 0/2] net: Remove redundant initialization owner Simon Horman
  2023-08-08  2:30 ` patchwork-bot+netdevbpf
  3 siblings, 0 replies; 5+ messages in thread
From: Li Zetao @ 2023-08-04  9:59 UTC (permalink / raw)
  To: ioana.ciornei, davem, edumazet, kuba, pabeni; +Cc: lizetao1, netdev

The fsl_mc_driver_register() will set "THIS_MODULE" to driver.owner when
register a fsl_mc_driver driver, so it is redundant initialization to set
driver.owner in dpaa2_switch_drv statement. Remove it for clean code.

Signed-off-by: Li Zetao <lizetao1@huawei.com>
---
 drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c
index 21cc4e52425a..97d3151076d5 100644
--- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c
+++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c
@@ -3457,7 +3457,6 @@ MODULE_DEVICE_TABLE(fslmc, dpaa2_switch_match_id_table);
 static struct fsl_mc_driver dpaa2_switch_drv = {
 	.driver = {
 		.name = KBUILD_MODNAME,
-		.owner = THIS_MODULE,
 	},
 	.probe = dpaa2_switch_probe,
 	.remove = dpaa2_switch_remove,
-- 
2.34.1


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

* Re: [PATCH net-next 0/2] net: Remove redundant initialization owner
  2023-08-04  9:59 [PATCH net-next 0/2] net: Remove redundant initialization owner Li Zetao
  2023-08-04  9:59 ` [PATCH net-next 1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver Li Zetao
  2023-08-04  9:59 ` [PATCH net-next 2/2] net: dpaa2-switch: Remove redundant initialization owner in dpaa2_switch_drv Li Zetao
@ 2023-08-05 11:38 ` Simon Horman
  2023-08-08  2:30 ` patchwork-bot+netdevbpf
  3 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2023-08-05 11:38 UTC (permalink / raw)
  To: Li Zetao; +Cc: ioana.ciornei, davem, edumazet, kuba, pabeni, netdev

On Fri, Aug 04, 2023 at 05:59:44PM +0800, Li Zetao wrote:
> This patch set removes redundant initialization owner when register a
> fsl_mc_driver driver

Reviewed-by: Simon Horman <horms@kernel.org>


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

* Re: [PATCH net-next 0/2] net: Remove redundant initialization owner
  2023-08-04  9:59 [PATCH net-next 0/2] net: Remove redundant initialization owner Li Zetao
                   ` (2 preceding siblings ...)
  2023-08-05 11:38 ` [PATCH net-next 0/2] net: Remove redundant initialization owner Simon Horman
@ 2023-08-08  2:30 ` patchwork-bot+netdevbpf
  3 siblings, 0 replies; 5+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-08-08  2:30 UTC (permalink / raw)
  To: Li Zetao; +Cc: ioana.ciornei, davem, edumazet, kuba, pabeni, netdev

Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 4 Aug 2023 17:59:44 +0800 you wrote:
> This patch set removes redundant initialization owner when register a
> fsl_mc_driver driver
> 
> Li Zetao (2):
>   net: dpaa2-eth: Remove redundant initialization owner in
>     dpaa2_eth_driver
>   net: dpaa2-switch: Remove redundant initialization owner in
>     dpaa2_switch_drv
> 
> [...]

Here is the summary with links:
  - [net-next,1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver
    https://git.kernel.org/netdev/net-next/c/43265d3fceeb
  - [net-next,2/2] net: dpaa2-switch: Remove redundant initialization owner in dpaa2_switch_drv
    https://git.kernel.org/netdev/net-next/c/ca46d207c972

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



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

end of thread, other threads:[~2023-08-08  2:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-04  9:59 [PATCH net-next 0/2] net: Remove redundant initialization owner Li Zetao
2023-08-04  9:59 ` [PATCH net-next 1/2] net: dpaa2-eth: Remove redundant initialization owner in dpaa2_eth_driver Li Zetao
2023-08-04  9:59 ` [PATCH net-next 2/2] net: dpaa2-switch: Remove redundant initialization owner in dpaa2_switch_drv Li Zetao
2023-08-05 11:38 ` [PATCH net-next 0/2] net: Remove redundant initialization owner Simon Horman
2023-08-08  2:30 ` patchwork-bot+netdevbpf

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