From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751930AbcDPVAX (ORCPT ); Sat, 16 Apr 2016 17:00:23 -0400 Received: from mail-wm0-f46.google.com ([74.125.82.46]:37683 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751614AbcDPVAW (ORCPT ); Sat, 16 Apr 2016 17:00:22 -0400 Date: Sat, 16 Apr 2016 23:00:19 +0200 From: Jiri Pirko To: Arnd Bergmann Cc: Jiri Pirko , "David S. Miller" , Ido Schimmel , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] devlink: fix devlink_sb_register prototype Message-ID: <20160416210018.GF1954@nanopsycho.orion> References: <1460838446-1246570-1-git-send-email-arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1460838446-1246570-1-git-send-email-arnd@arndb.de> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Sat, Apr 16, 2016 at 10:27:06PM CEST, arnd@arndb.de wrote: >The devlink shared buffer interface contains two different >prototypes for devlink_sb_register, and the one that is >used when NET_DEVLINK is disabled does not work: > >drivers/net/ethernet/mellanox/mlxsw/spectrum_buffers.c: In function 'mlxsw_sp_buffers_init': >drivers/net/ethernet/mellanox/mlxsw/spectrum_buffers.c:547:9: error: too many arguments to function 'devlink_sb_register' > return devlink_sb_register(priv_to_devlink(mlxsw_sp->core), 0, > >This makes the two prototypes have the same argument list. > >Signed-off-by: Arnd Bergmann >Fixes: bf7974710a40 ("devlink: add shared buffer configuration") Already fixed by: commit de33efd0fb7f923cd41671b1f743c3a0d44dd953 Author: Jiri Pirko Date: Fri Apr 15 09:17:08 2016 +0200 devlink: fix sb register stub in case devlink is disabled