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.