On 04/09/2015 05:23 PM, Alexander Aring wrote: > On Thu, Apr 09, 2015 at 03:02:01PM +0530, Varka Bhadram wrote: >> Signed-off-by: Varka Bhadram <varkab@xxxxxxx> >> --- >> include/net/mac802154.h | 95 ++++++++++++++++++++++++++++++++++++++++++++++- >> 1 file changed, 93 insertions(+), 2 deletions(-) >> >> diff --git a/include/net/mac802154.h b/include/net/mac802154.h >> index e18e7fd..474dafb 100644 >> --- a/include/net/mac802154.h >> +++ b/include/net/mac802154.h >> @@ -247,19 +247,110 @@ static inline void ieee802154_le64_to_be64(void *be64_dst, const void *le64_src) >> __put_unaligned_memmove64(swab64p(le64_src), be64_dst); >> } >> >> -/* Basic interface to register ieee802154 device */ >> +/** >> + * ieee802154_alloc_hw - Allocate a new hardware device >> + * >> + * This must be called once for each hardware device. The returned pointer >> + * must be used to refer to this device when calling other functions. >> + * mac802154 allocates a private data area for the driver pointed to by >> + * @priv in &struct ieee802154_hw, the size of this area is given as >> + * @priv_data_len. >> + * >> + * @priv_data_len: length of private data >> + * @ops: callbacks for this device >> + * >> + * Return: A pointer to the new hardware device, or %NULL on error. >> + */ >> struct ieee802154_hw * >> ieee802154_alloc_hw(size_t priv_data_len, const struct ieee802154_ops *ops); >> + >> +/** >> + * ieee802154_free_hw - free hardware descriptor >> + * >> + * This function frees everything that was allocated, including the >> + * private data for the driver. You must call ieee802154_unregister_hw() >> + * before calling this function. >> + * >> + * @hw: the hardware to free >> + */ >> void ieee802154_free_hw(struct ieee802154_hw *hw); >> + >> +/** >> + * ieee802154_register_hw - Register hardware device >> + * >> + * You must call this function before any other functions in >> + * mac802154. Note that before a hardware can be registered, you >> + * need to fill the contained wpan_phy's information. >> + * >> + * @hw: the device to register as returned by ieee802154_alloc_hw() >> + * >> + * Return: 0 on success. An error code otherwise. >> + */ >> int ieee802154_register_hw(struct ieee802154_hw *hw); >> + >> +/** >> + * ieee802154_unregister_hw - Unregister a hardware device >> + * >> + * This function instructs mac802154 to free allocated resources >> + * and unregister netdevices from the networking subsystem. >> + * >> + * @hw: the hardware to unregister >> + */ >> void ieee802154_unregister_hw(struct ieee802154_hw *hw); >> >> +/** >> + * ieee802154_rx - receive frame >> + * >> + * Use this function to hand received frames to mac802154. The receive >> + * buffer in @skb must start with an IEEE 802.15.4 header. In case of a >> + * paged @skb is used, the driver is recommended to put the ieee802154 >> + * header of the frame on the linear part of the @skb to avoid memory >> + * allocation and/or memcpy by the stack. >> + * >> + * This function may not be called in IRQ context. Calls to this function >> + * for a single hardware must be synchronized against each other. >> + * >> + * @hw: the hardware this frame came in on >> + * @skb: the buffer to receive, owned by mac802154 after this call >> + */ >> void ieee802154_rx(struct ieee802154_hw *hw, struct sk_buff *skb); >> + >> +/** >> + * ieee802154_rx_irqsafe - receive frame >> + * >> + * Like ieee802154_rx() but can be called in IRQ context >> + * (internally defers to a tasklet.) >> + * >> + * @hw: the hardware this frame came in on >> + * @skb: the buffer to receive, owned by mac802154 after this call > missing lqi param description. will do. >> + */ >> void ieee802154_rx_irqsafe(struct ieee802154_hw *hw, struct sk_buff *skb, >> u8 lqi); >> - >> +/** >> + * ieee802154_wake_queue - wake specific queue >> + * @hw: pointer as obtained from ieee802154_alloc_hw(). >> + * @queue: queue number (counted from zero). >> + * > which specific queue? This have wireless only. wake wireless queue.... Is OK? What do you suggest? >> + * Drivers should use this function instead of netif_wake_queue. >> + */ >> void ieee802154_wake_queue(struct ieee802154_hw *hw); >> + >> +/** >> + * ieee802154_stop_queue - stop specific queue >> + * @hw: pointer as obtained from ieee802154_alloc_hw(). >> + * @queue: queue number (counted from zero). >> + * > same here, we have no fancy multiple queues. What do you suggest? >> + * Drivers should use this function instead of netif_stop_queue. >> + */ >> void ieee802154_stop_queue(struct ieee802154_hw *hw); >> + >> +/** >> + * ieee802154_xmit_complete - packet transmission complete >> + * >> + * @hw: pointer as obtained from ieee802154_alloc_hw(). >> + * @skb: buffer for transmission >> + * @ifs_handling: indicate interframe space handling >> + */ >> void ieee802154_xmit_complete(struct ieee802154_hw *hw, struct sk_buff *skb, >> bool ifs_handling); > - Alex -- Varka Bhadram -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html