On Fri, Sep 12, 2014 at 3:40 PM, David Miller <davem@xxxxxxxxxxxxx> wrote: > From: Frank Li <Frank.Li@xxxxxxxxxxxxx> > Date: Thu, 11 Sep 2014 02:30:42 +0800 > >> + >> + unsigned tx_align; >> + unsigned rx_align; >> }; > > Please always fully write out "unsigned int" rather than just "unsigned". > >> @@ -2399,6 +2393,7 @@ fec_enet_alloc_rxq_buffers(struct net_device *ndev, unsigned int queue) >> struct sk_buff *skb; >> struct bufdesc *bdp; >> struct fec_enet_priv_rx_q *rxq; >> + unsigned off; > > Likewise. Okay, I will fix it next version. best regards Frank Li -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html