Messages in this thread | | | Date | Wed, 2 Nov 2022 09:07:34 -0700 | Subject | Re: [PATCH linux-next] ionic: remove redundant ret variable | From | Shannon Nelson <> |
| |
On 11/2/22 6:48 AM, zhang.songyi@zte.com.cn wrote: > From 06579895d9e3f6441fa30d52e3b585b2015c7e2e Mon Sep 17 00:00:00 2001 > From: zhang songyi <zhang.songyi@zte.com.cn> > Date: Wed, 2 Nov 2022 20:57:40 +0800 > Subject: [PATCH linux-next] ionic: remove redundant ret variable > > Return value from ionic_set_features() directly instead of taking this in > another redundant variable. > > Reported-by: Zeal Robot <zealci@zte.com.cn> > Signed-off-by: zhang songyi <zhang.songyi@zte.com.cn>
Acked-by: Shannon Nelson <snelson@pensando.io>
... although these changes usually go through the net or net-next trees. sln
> --- > drivers/net/ethernet/pensando/ionic/ionic_lif.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/net/ethernet/pensando/ionic/ionic_lif.c b/drivers/net/ethernet/pensando/ionic/ionic_lif.c > index 4dd16c487f2b..a68d748502fd 100644 > --- a/drivers/net/ethernet/pensando/ionic/ionic_lif.c > +++ b/drivers/net/ethernet/pensando/ionic/ionic_lif.c > @@ -1557,14 +1557,11 @@ static int ionic_set_features(struct net_device *netdev, > netdev_features_t features) > { > struct ionic_lif *lif = netdev_priv(netdev); > - int err; > > netdev_dbg(netdev, "%s: lif->features=0x%08llx new_features=0x%08llx\n", > __func__, (u64)lif->netdev->features, (u64)features); > > - err = ionic_set_nic_features(lif, features); > - > - return err; > + return ionic_set_nic_features(lif, features); > } > > static int ionic_set_attr_mac(struct ionic_lif *lif, u8 *mac) > -- > 2.15.2
| |