Hello.
On 30/07/15 10:55, Alexander Aring wrote:
This patch introduce two static inline functions. The first to get the
frame control field from an sk_buff. The second is for checking on the
acknowledgment request bit on the frame control field. Later we can
introduce more functions to check on the frame control fields.
These will deprecate the current behaviour which requires a
host-byteorder conversion and manually bit handling.
Signed-off-by: Alexander Aring<alex.aring@xxxxxxxxx>
---
Some language suggestions inside.
include/linux/ieee802154.h | 29 +++++++++++++++++++++++++++++
1 file changed, 29 insertions(+)
diff --git a/include/linux/ieee802154.h b/include/linux/ieee802154.h
index 1dc1f4e..4f26c01 100644
--- a/include/linux/ieee802154.h
+++ b/include/linux/ieee802154.h
@@ -25,6 +25,8 @@
#include <linux/types.h>
#include <linux/random.h>
+#include <linux/skbuff.h>
+#include <linux/unaligned/memmove.h>
#include <asm/byteorder.h>
#define IEEE802154_MTU 127
@@ -205,6 +207,33 @@ enum {
IEEE802154_SCAN_IN_PROGRESS = 0xfc,
};
+/* frame control handling */
+#define IEEE802154_FCTL_ACKREQ 0x0020
+
+/**
+ * ieee802154_is_ackreq - check if acknowledgment request bit is set
+ * @fc: frame control bytes in little-endian byteorder
+ */
+static inline bool ieee802154_is_ackreq(__le16 fc)
+{
+ return fc & cpu_to_le16(IEEE802154_FCTL_ACKREQ);
+}
+
+/**
+ * ieee802154_get_fc_from_skb - get the frame control field from an skb
... from a skb
+ * @skb: skb where the frame control field will be get from
Maybe:
skb which contains the frame control field
+ */
+static inline __le16 ieee802154_get_fc_from_skb(const struct sk_buff *skb)
+{
+ /* return some invalid fc on failure */
Maybe:
return on invalid fc
+ if (unlikely(skb->mac_len < 2)) {
+ WARN_ON(1);
+ return cpu_to_le16(0);
+ }
+
+ return (__force __le16)__get_unaligned_memmove16(skb_mac_header(skb));
Just to make sure we don't run into problems like we did with the
6lowpan stack. __get_unaligned_memmove16 is not pulling the fc bytes out
of the skb, right? The skb stays as it is.
+}
+
/**
* ieee802154_is_valid_psdu_len - check if psdu len is valid
* available lengths:
Given the above is true you have my:
Reviewed-by: Stefan Schmidt <stefan@xxxxxxxxxxxxxxx>
regards
Stefan Schmidt
--
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
