On Thu, 5 May 2022 17:09:58 +0200 Alaa Mohamed wrote: > + NL_SET_ERR_MSG(extack, > + "DST, VNI, ifindex and port are mutually exclusive with NH_ID"); This continuation line still does not align with the opening bracket. Look here if your email client makes it hard to see: https://lore.kernel.org/all/ac4b6c650b6519cc56baa32ef20415460a5aa8ee.1651762830.git.eng.alaamohamedsoliman.am@xxxxxxxxx/ Same story in patch 1: > static int ocelot_port_fdb_del(struct ndmsg *ndm, struct nlattr *tb[], > struct net_device *dev, > - const unsigned char *addr, u16 vid) > + const unsigned char *addr, u16 vid, > + struct netlink_ext_ack *extack) and here: > static int vxlan_fdb_delete(struct ndmsg *ndm, struct nlattr *tb[], > struct net_device *dev, > - const unsigned char *addr, u16 vid) > + const unsigned char *addr, u16 vid, > + struct netlink_ext_ack *extack)