On Fri, Aug 08, 2014 at 12:07:02PM +0530, Varka Bhadram wrote:
> On 08/08/2014 11:58 AM, Alexander Aring wrote:
> >On Fri, Aug 08, 2014 at 08:24:39AM +0200, Alexander Aring wrote:
> >>Signed-off-by: Alexander Aring <alex.aring@xxxxxxxxx>
> >>---
> >> net/ieee802154/6lowpan_rtnl.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >>diff --git a/net/ieee802154/6lowpan_rtnl.c b/net/ieee802154/6lowpan_rtnl.c
> >>index 0acd273..b6ec3b1 100644
> >>--- a/net/ieee802154/6lowpan_rtnl.c
> >>+++ b/net/ieee802154/6lowpan_rtnl.c
> >>@@ -238,7 +238,7 @@ lowpan_alloc_frag(struct sk_buff *skb, int size,
> >> return ERR_PTR(-rc);
> >> }
> >> } else {
> >>- frag = ERR_PTR(ENOMEM);
> >>+ frag = ERR_PTR(-ENOMEM);
> >> }
> >> return frag;
> >>--
> >>2.0.3
> >>
> >Sorry, I will also add a "ieee802154: " to beginning of this commit msg.
> >
> Also some description about patch..?
>
No, that's enough it's only one line. What should I say about that? I
could say this was introduce by commit blabla... but it doesn't matter.
I talked to you about you should write something in your commit msg, but
this was a whole driver and there stands "Add support for cc2520..." and
nothing more, nothing about hw aack support (address filter), I mean
what's the driver supports for these guys which wants to use this driver.
Okay, maybe they could check the hardware flags for this.
I detected it by doing some experimental things to check how we can do
things better.
- Alex
--
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
