diff mbox series

[linux-next] mlxsw: core: remove the unneeded result variable

Message ID 20220912072933.16994-1-xu.panda@zte.com.cn (mailing list archive)
State Changes Requested
Delegated to: Netdev Maintainers
Headers show
Series [linux-next] mlxsw: core: remove the unneeded result variable | expand

Checks

Context Check Description
netdev/tree_selection success Guessing tree name failed - patch did not apply

Commit Message

CGEL Sept. 12, 2022, 7:29 a.m. UTC
From: Xu Panda <xu.panda@zte.com.cn>

Return the value mlxsw_core_bus_device_register() directly instead of
storing it in another redundant variable.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Xu Panda <xu.panda@zte.com.cn>
---
 drivers/net/ethernet/mellanox/mlxsw/core.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

Comments

Ido Schimmel Sept. 12, 2022, 8:21 a.m. UTC | #1
On Mon, Sep 12, 2022 at 07:29:34AM +0000, cgel.zte@gmail.com wrote:
> From: Xu Panda <xu.panda@zte.com.cn>
> 
> Return the value mlxsw_core_bus_device_register() directly instead of
> storing it in another redundant variable.
> 
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Xu Panda <xu.panda@zte.com.cn>

For net-next:

Reviewed-by: Ido Schimmel <idosch@nvidia.com>

Please use "PATCH net-next" prefix for such patches:
https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html#how-do-i-indicate-which-tree-net-vs-net-next-my-patch-should-be-in
Jakub Kicinski Sept. 19, 2022, 9:50 p.m. UTC | #2
On Mon, 12 Sep 2022 07:29:34 +0000 cgel.zte@gmail.com wrote:
> From: Xu Panda <xu.panda@zte.com.cn>
> 
> Return the value mlxsw_core_bus_device_register() directly instead of
> storing it in another redundant variable.

This patch does not apply cleanly to net-next, please rebase.
diff mbox series

Patch

diff --git a/drivers/net/ethernet/mellanox/mlxsw/core.c b/drivers/net/ethernet/mellanox/mlxsw/core.c
index e2a985ec2c76..8daedf6c4496 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/core.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/core.c
@@ -1513,15 +1513,13 @@  mlxsw_devlink_core_bus_device_reload_up(struct devlink *devlink, enum devlink_re
                                        struct netlink_ext_ack *extack)
 {
        struct mlxsw_core *mlxsw_core = devlink_priv(devlink);
-       int err;

        *actions_performed = BIT(DEVLINK_RELOAD_ACTION_DRIVER_REINIT) |
                             BIT(DEVLINK_RELOAD_ACTION_FW_ACTIVATE);
-       err = mlxsw_core_bus_device_register(mlxsw_core->bus_info,
-                                            mlxsw_core->bus,
-                                            mlxsw_core->bus_priv, true,
-                                            devlink, extack);
-       return err;
+       return mlxsw_core_bus_device_register(mlxsw_core->bus_info,
+                                             mlxsw_core->bus,
+                                             mlxsw_core->bus_priv, true,
+                                             devlink, extack);
 }

 static int mlxsw_devlink_flash_update(struct devlink *devlink,