lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 25 Apr 2018 10:46:24 +0900 From: Kunihiko Hayashi <hayashi.kunihiko@...ionext.com> To: Luc Van Oostenryck <luc.vanoostenryck@...il.com> Cc: linux-kernel@...r.kernel.org, "David S. Miller" <davem@...emloft.net>, Jassi Brar <jaswinder.singh@...aro.org>, Andrew Lunn <andrew@...n.ch>, netdev@...r.kernel.org Subject: Re: [PATCH] net: ethernet: ave: fix ave_start_xmit()'s return type Hi, On Tue, 24 Apr 2018 15:17:25 +0200 Luc Van Oostenryck <luc.vanoostenryck@...il.com> wrote: > The method ndo_start_xmit() is defined as returning an 'netdev_tx_t', > which is a typedef for an enum type, but the implementation in this > driver returns an 'int'. > > Fix this by returning 'netdev_tx_t' in this driver too. > > Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@...il.com> Thanks for fixing. I think it's preferable to add 'Fixes'. Fixes: 4c270b55a5af ("net: ethernet: socionext: add AVE ethernet driver") Acked-by: Kunihiko Hayashi <hayashi.kunihiko@...ionext.com> Thank you, > --- > drivers/net/ethernet/socionext/sni_ave.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/socionext/sni_ave.c b/drivers/net/ethernet/socionext/sni_ave.c > index 0b3b7a460..95625bca1 100644 > --- a/drivers/net/ethernet/socionext/sni_ave.c > +++ b/drivers/net/ethernet/socionext/sni_ave.c > @@ -1360,7 +1360,7 @@ static int ave_stop(struct net_device *ndev) > return 0; > } > > -static int ave_start_xmit(struct sk_buff *skb, struct net_device *ndev) > +static netdev_tx_t ave_start_xmit(struct sk_buff *skb, struct net_device *ndev) > { > struct ave_private *priv = netdev_priv(ndev); > u32 proc_idx, done_idx, ndesc, cmdsts; > -- > 2.17.0 --- Best Regards, Kunihiko Hayashi
Powered by blists - more mailing lists