From: Alexander Smirnov <alex.bluesman.smir...@gmail.com>
Date: Mon, 19 Dec 2011 19:33:43 +0300

> +void ieee802154_rx(struct ieee802154_dev *dev, struct sk_buff *skb, u8 lqi);
> +void ieee802154_rx_irqsafe(struct ieee802154_dev *dev, struct sk_buff *skb,
> +             u8 lqi);

Line up the "u8 lqi" argument with the openning parenthesis on the previous 
line.

> +static void
> +mac802154_subif_rx(struct ieee802154_dev *hw, struct sk_buff *skb, u8 lqi)
> +{
> +     struct mac802154_priv *priv = mac802154_to_priv(hw);
> +
> +     BUG_ON(!skb);

This assertion is excessive.

You are adding all of the call sites, therefore you can see if the SKB
argument is NULL or not.


------------------------------------------------------------------------------
Learn Windows Azure Live!  Tuesday, Dec 13, 2011
Microsoft is holding a special Learn Windows Azure training event for 
developers. It will provide a great way to learn Windows Azure and what it 
provides. You can attend the event by watching it streamed LIVE online.  
Learn more at http://p.sf.net/sfu/ms-windowsazure
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to