On Fri, Mar 23, 2018 at 06:00:38PM +0100, Sergio Paracuellos wrote: > +static inline void hostif_pmklist_init(struct ks_wlan_private *priv) > +{ > + int i; > memset(&priv->pmklist, 0, sizeof(priv->pmklist)); Put a blank line after the declaration block. Otherwise it looks good. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel