diff options
author | Jiri Pirko <jpirko@redhat.com> | 2011-12-08 19:52:37 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-12-08 19:52:37 -0500 |
commit | 8e586137e6b63af1e881b328466ab5ffbe562510 (patch) | |
tree | da0767e1b1361aa24bd32f485453079e31854c0c /drivers/net/ethernet/tehuti | |
parent | 7da82c06ded105bf601bfa0eafc92e84eb0ceeed (diff) |
net: make vlan ndo_vlan_rx_[add/kill]_vid return error value
Let caller know the result of adding/removing vlan id to/from vlan
filter.
In some drivers I make those functions to just return 0. But in those
where there is able to see if hw setup went correctly, return value is
set appropriately.
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/tehuti')
-rw-r--r-- | drivers/net/ethernet/tehuti/tehuti.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/ethernet/tehuti/tehuti.c b/drivers/net/ethernet/tehuti/tehuti.c index 3a90af6d111c..4b19e9b0606b 100644 --- a/drivers/net/ethernet/tehuti/tehuti.c +++ b/drivers/net/ethernet/tehuti/tehuti.c | |||
@@ -727,9 +727,10 @@ static void __bdx_vlan_rx_vid(struct net_device *ndev, uint16_t vid, int enable) | |||
727 | * @ndev network device | 727 | * @ndev network device |
728 | * @vid VLAN vid to add | 728 | * @vid VLAN vid to add |
729 | */ | 729 | */ |
730 | static void bdx_vlan_rx_add_vid(struct net_device *ndev, uint16_t vid) | 730 | static int bdx_vlan_rx_add_vid(struct net_device *ndev, uint16_t vid) |
731 | { | 731 | { |
732 | __bdx_vlan_rx_vid(ndev, vid, 1); | 732 | __bdx_vlan_rx_vid(ndev, vid, 1); |
733 | return 0; | ||
733 | } | 734 | } |
734 | 735 | ||
735 | /* | 736 | /* |
@@ -737,9 +738,10 @@ static void bdx_vlan_rx_add_vid(struct net_device *ndev, uint16_t vid) | |||
737 | * @ndev network device | 738 | * @ndev network device |
738 | * @vid VLAN vid to kill | 739 | * @vid VLAN vid to kill |
739 | */ | 740 | */ |
740 | static void bdx_vlan_rx_kill_vid(struct net_device *ndev, unsigned short vid) | 741 | static int bdx_vlan_rx_kill_vid(struct net_device *ndev, unsigned short vid) |
741 | { | 742 | { |
742 | __bdx_vlan_rx_vid(ndev, vid, 0); | 743 | __bdx_vlan_rx_vid(ndev, vid, 0); |
744 | return 0; | ||
743 | } | 745 | } |
744 | 746 | ||
745 | /** | 747 | /** |