On 18.07.2018 20:54, David Miller wrote:
> From: Claudiu Beznea <claudiu.bez...@microchip.com>
> Date: Wed, 18 Jul 2018 15:58:09 +0300
>
>>
>> +static int macb_pad_and_fcs(struct sk_buff **skb, struct net_device *ndev)
>> +{
>> + struct sk_buff *nskb;
>> + int padlen = ETH_ZLEN - (*skb)->len;
>> + int headroom = skb_headroom(*skb);
>> + int tailroom = skb_tailroom(*skb);
>> + bool cloned = skb_cloned(*skb) || skb_header_cloned(*skb);
>> + u32 fcs;
>
> Please keep local variable ordered from longest to shortest line
> (ie. reverse christmas tree format).
OK! Thank you!
>
> Thank you.
>
