From: David Ahern <d...@cumulusnetworks.com>
Date: Wed,  4 May 2016 21:54:09 -0700

> +             if (dev && netif_is_l3_master(dev) &&
> +                 dev->l3mdev_ops->l3mdev_get_saddr) {
> +                     rc = dev->l3mdev_ops->l3mdev_get_saddr(dev, fl4);
> +             }

Please do not use braces for single statement basic blocks.

The same issue exists in patch #2 so please fix it there as well.

Thanks.

Reply via email to