On 08/25/2017 04:11 AM, David Miller wrote:
> 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;
> }
>
Thank you, sorry for the mistake.
