From: kbuild test robot <fengguang...@intel.com>
Date: Fri, 25 Aug 2017 08:03:28 +0800
> All errors (new ones prefixed by >>):
>
> drivers/net//ethernet/mellanox/mlxsw/spectrum_dpipe.c: In function
> 'mlxsw_sp_dpipe_erif_table_init':
>>> drivers/net//ethernet/mellanox/mlxsw/spectrum_dpipe.c:323:9: error: too few
>>> arguments to function 'devlink_dpipe_table_register'
> return devlink_dpipe_table_register(devlink,
> ^
> In file included from
> drivers/net//ethernet/mellanox/mlxsw/spectrum_dpipe.c:36:0:
> include/net/devlink.h:401:1: note: declared here
> devlink_dpipe_table_register(struct devlink *devlink,
> ^
> drivers/net//ethernet/mellanox/mlxsw/spectrum_dpipe.c:327:1: warning:
> control reaches end of non-void function [-Wreturn-type]
> }
> ^
I'll push the following fix into net-next for this:
====================
>From 790c6056686cc4dd5b149b330bbd5ae208d4d721 Mon Sep 17 00:00:00 2001
From: "David S. Miller" <da...@davemloft.net>
Date: Thu, 24 Aug 2017 18:10:46 -0700
Subject: [PATCH] devlink: Fix devlink_dpipe_table_register() stub signature.
One too many arguments compared to the non-stub version.
Reported-by: kbuild test robot <fengguang...@intel.com>
Fixes: ffd3cdccf214 ("devlink: Add support for dynamic table size")
Signed-off-by: David S. Miller <da...@davemloft.net>
---
include/net/devlink.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/net/devlink.h b/include/net/devlink.h
index 047a0c54f652..aaf7178127a2 100644
--- a/include/net/devlink.h
+++ b/include/net/devlink.h
@@ -402,8 +402,7 @@ static inline int
devlink_dpipe_table_register(struct devlink *devlink,
const char *table_name,
struct devlink_dpipe_table_ops *table_ops,
- void *priv, u64 size,
- bool counter_control_extern)
+ void *priv, bool counter_control_extern)
{
return 0;
}
--
2.13.5
