On Wed, Apr 09, 2014 at 08:24:08PM +0400, Sergei Shtylyov wrote: > >diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c b/drivers/staging/rtl8188eu/core/rtw_recv.c > >index 01fcabc..61084d6 100644 > >--- a/drivers/staging/rtl8188eu/core/rtw_recv.c > >+++ b/drivers/staging/rtl8188eu/core/rtw_recv.c > >@@ -551,7 +551,7 @@ static struct recv_frame *portctrl(struct adapter *adapter, > > struct sta_info *psta; > > struct sta_priv *pstapriv; > > struct recv_frame *prtnframe; > >- u16 ether_type = 0; > >+ u16 ether_type; > > I suggest: > > u16 ethertype; > I don't understand this suggestion. Why is that name prefered? regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html