On 10/15/21 7:20 AM, MichelleJin wrote:
> 'netdev->priv_flags & IFF_BONDING && netdev->flags & IFF_MASTER'
> is defined as netif_is_bond_master() in netdevice.h.
> So, replace it with netif_is_bond_master() for cleaning code.
>
> Signed-off-by: MichelleJin <shjy180909@xxxxxxxxx>
> ---
> drivers/scsi/fcoe/fcoe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c
> index 5ae6c207d3ac..6415f88738ad 100644
> --- a/drivers/scsi/fcoe/fcoe.c
> +++ b/drivers/scsi/fcoe/fcoe.c
> @@ -307,7 +307,7 @@ static int fcoe_interface_setup(struct fcoe_interface *fcoe,
> }
>
> /* Do not support for bonding device */
> - if (netdev->priv_flags & IFF_BONDING && netdev->flags & IFF_MASTER) {
> + if (netif_is_bond_master(netdev)) {
> FCOE_NETDEV_DBG(netdev, "Bonded interfaces not supported\n");
> return -EOPNOTSUPP;
> }
>
Reviewed-by: Lee Duncan <lduncan@xxxxxxxx>
