diff options
| author | Serhey Popovych <serhe.popovych@gmail.com> | 2018-03-29 16:51:36 +0200 |
|---|---|---|
| committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2019-03-19 22:18:49 +0100 |
| commit | b0ddfe2bb2bd80b1090d5bf42bb65243b76d3b97 (patch) | |
| tree | 6e490599d6b3e29b1f85331441b98a0be734e8ea /drivers/net/ethernet/intel/e100.c | |
| parent | virtio_net: remove hcpu from virtnet_clean_affinity (diff) | |
| download | linux-b0ddfe2bb2bd80b1090d5bf42bb65243b76d3b97.tar.xz linux-b0ddfe2bb2bd80b1090d5bf42bb65243b76d3b97.zip | |
intel: correct return from set features callback
According to comments in <linux/netdevice.h> we should return either >0
or -errno from ->ndo_set_features() if changing dev->features by itself.
Return 1 in such places to notify netdev_update_features() about applied
changes in dev->features.
Signed-off-by: Serhey Popovych <serhe.popovych@gmail.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/e100.c')
| -rw-r--r-- | drivers/net/ethernet/intel/e100.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/e100.c b/drivers/net/ethernet/intel/e100.c index 0fd268070fb4..a65d5a9ba7db 100644 --- a/drivers/net/ethernet/intel/e100.c +++ b/drivers/net/ethernet/intel/e100.c @@ -2797,7 +2797,7 @@ static int e100_set_features(struct net_device *netdev, netdev->features = features; e100_exec_cb(nic, NULL, e100_configure); - return 0; + return 1; } static const struct net_device_ops e100_netdev_ops = { |