[PATCH v2] net: aquantia: add an error handling in aq_nic_set_multicast_list

From: Chenwandun <redacted>

add an error handling in aq_nic_set_multicast_list, it may not
work when hw_multicast_list_set error; and at the same time
it will remove gcc Wunused-but-set-variable warning.

Signed-off-by: Chenwandun <redacted>
---
 drivers/net/ethernet/aquantia/atlantic/aq_nic.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_nic.c b/drivers/net/ethernet/aquantia/atlantic/aq_nic.c
index 2a18439..137c1de 100644
--- a/drivers/net/ethernet/aquantia/atlantic/aq_nic.c
+++ b/drivers/net/ethernet/aquantia/atlantic/aq_nic.c

@@ -664,6 +664,8 @@ int aq_nic_set_multicast_list(struct aq_nic_s *self, struct net_device *ndev)
 		err = hw_ops->hw_multicast_list_set(self->aq_hw,
 						    self->mc_list.ar,
 						    self->mc_list.count);
+		if (err < 0)
+			return err;
 	}
 	return aq_nic_set_packet_filter(self, packet_filter);
 }

-- 
2.7.4

Thanks, Chen.

On 18.10.2019 13:20, Chen Wandun wrote:

From: Chenwandun <redacted>

add an error handling in aq_nic_set_multicast_list, it may not
work when hw_multicast_list_set error; and at the same time
it will remove gcc Wunused-but-set-variable warning.

Signed-off-by: Chenwandun <redacted>

Reviewed-by: Igor Russkikh <redacted>

Regards,
  Igor

On Fri, Oct 18, 2019 at 06:20:37PM +0800, Chen Wandun wrote:

From: Chenwandun <redacted>

add an error handling in aq_nic_set_multicast_list, it may not
work when hw_multicast_list_set error; and at the same time
it will remove gcc Wunused-but-set-variable warning.

Signed-off-by: Chenwandun <redacted>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

From: Chen Wandun <redacted>
Date: Fri, 18 Oct 2019 18:20:37 +0800

From: Chenwandun <redacted>

add an error handling in aq_nic_set_multicast_list, it may not
work when hw_multicast_list_set error; and at the same time
it will remove gcc Wunused-but-set-variable warning.

Signed-off-by: Chenwandun <redacted>

Applied, thanks.