From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Fastabend Subject: [net-next PATCH v2 6/7] dcb: fix return type on dcb_setapp() Date: Tue, 21 Jun 2011 10:34:58 -0700 Message-ID: <20110621173458.2777.38457.stgit@jf-dev1-dcblab> References: <20110621173431.2777.39482.stgit@jf-dev1-dcblab> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, shmulikr@broadcom.com To: davem@davemloft.net Return-path: Received: from mga14.intel.com ([143.182.124.37]:2969 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757082Ab1FURlh (ORCPT ); Tue, 21 Jun 2011 13:41:37 -0400 In-Reply-To: <20110621173431.2777.39482.stgit@jf-dev1-dcblab> Sender: netdev-owner@vger.kernel.org List-ID: Incorrect return type on dcb_setapp() this routine returns negative error codes. All call sites of dcb_setapp() assign the return value to an int already so no need to update drivers. Signed-off-by: John Fastabend --- include/net/dcbnl.h | 2 +- net/dcb/dcbnl.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/net/dcbnl.h b/include/net/dcbnl.h index c14fd94..d5bbb79 100644 --- a/include/net/dcbnl.h +++ b/include/net/dcbnl.h @@ -28,7 +28,7 @@ struct dcb_app_type { struct list_head list; }; -u8 dcb_setapp(struct net_device *, struct dcb_app *); +int dcb_setapp(struct net_device *, struct dcb_app *); u8 dcb_getapp(struct net_device *, struct dcb_app *); int dcb_ieee_setapp(struct net_device *, struct dcb_app *); int dcb_ieee_delapp(struct net_device *, struct dcb_app *); diff --git a/net/dcb/dcbnl.c b/net/dcb/dcbnl.c index b7d0be0..f54c784 100644 --- a/net/dcb/dcbnl.c +++ b/net/dcb/dcbnl.c @@ -1885,7 +1885,7 @@ EXPORT_SYMBOL(dcb_getapp); * removes applications from the app list if the priority is * set to zero. */ -u8 dcb_setapp(struct net_device *dev, struct dcb_app *new) +int dcb_setapp(struct net_device *dev, struct dcb_app *new) { struct dcb_app_type *itr; struct dcb_app_type event;