Alexander Aring wrote:
> ok, I remove this patch from current series. It's not necessary to add
> this now.
It's a minor detail, you don't have to pull the patch for it.
Just though I'd mention it.
- Werner
--
October We
Hi Werner,
On Thu, Oct 24, 2013 at 09:50:44AM -0300, Werner Almesberger wrote:
> Alexander Aring wrote:
> > - dev->needed_tailroom= 2; /* FCS */
> > + dev->needed_tailroom= IEEE802154_MFR_SIZE;
>
> Looking at include/net/ieee802154.h
>
> /* MAC footer size */
> #define IEEE802154
Alexander Aring wrote:
> - dev->needed_tailroom= 2; /* FCS */
> + dev->needed_tailroom= IEEE802154_MFR_SIZE;
Looking at include/net/ieee802154.h
/* MAC footer size */
#define IEEE802154_MFR_SIZE 2 /* 2 octets */
we "lose" the information that MFR is the FCS. You may want
This patch replaces a redundant open-coded calculation
with the already defined MAC802154_FRAME_HARD_HEADER_LEN.
Also replace a magic number with the already defined
IEEE802154_MFR_SIZE.
Signed-off-by: Alexander Aring
---
net/ieee802154/6lowpan.c | 6 +++---
1 file changed, 3 insertions(+), 3 d
