From: Jiri Pirko <j...@mellanox.com>

__devlink_port_type_set() returns void, it makes no sense to pass it on,
so don't do that.

Signed-off-by: Jiri Pirko <j...@mellanox.com>
---
 net/core/devlink.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/net/core/devlink.c b/net/core/devlink.c
index b40fd1addf14..35ae9980ce86 100644
--- a/net/core/devlink.c
+++ b/net/core/devlink.c
@@ -5384,8 +5384,7 @@ static void __devlink_port_type_set(struct devlink_port 
*devlink_port,
 void devlink_port_type_eth_set(struct devlink_port *devlink_port,
                               struct net_device *netdev)
 {
-       return __devlink_port_type_set(devlink_port,
-                                      DEVLINK_PORT_TYPE_ETH, netdev);
+       __devlink_port_type_set(devlink_port, DEVLINK_PORT_TYPE_ETH, netdev);
 }
 EXPORT_SYMBOL_GPL(devlink_port_type_eth_set);
 
@@ -5398,8 +5397,7 @@ EXPORT_SYMBOL_GPL(devlink_port_type_eth_set);
 void devlink_port_type_ib_set(struct devlink_port *devlink_port,
                              struct ib_device *ibdev)
 {
-       return __devlink_port_type_set(devlink_port,
-                                      DEVLINK_PORT_TYPE_IB, ibdev);
+       __devlink_port_type_set(devlink_port, DEVLINK_PORT_TYPE_IB, ibdev);
 }
 EXPORT_SYMBOL_GPL(devlink_port_type_ib_set);
 
@@ -5410,8 +5408,7 @@ EXPORT_SYMBOL_GPL(devlink_port_type_ib_set);
  */
 void devlink_port_type_clear(struct devlink_port *devlink_port)
 {
-       return __devlink_port_type_set(devlink_port,
-                                      DEVLINK_PORT_TYPE_NOTSET, NULL);
+       __devlink_port_type_set(devlink_port, DEVLINK_PORT_TYPE_NOTSET, NULL);
 }
 EXPORT_SYMBOL_GPL(devlink_port_type_clear);
 
-- 
2.17.2

Reply via email to