Hi Samuel,
> > @@ -208,11 +489,13 @@ static int nfcwilink_send(struct sk_buff *skb)
> >
> > nfc_dev_dbg(&drv->pdev->dev, "send entry, len %d", skb->len);
> >
> > - if (!test_bit(NFCWILINK_RUNNING, &drv->flags))
> > - return -EBUSY;
> > + if (!test_bit(NFCWILINK_RUNNING, &drv->flags)) {
> > + kfree_skb(skb);
> > + return -EINVAL;
> > + }
> This is not related to this patch.
>
>
> > /* add the ST hdr to the start of the buffer */
> > - hdr.len = skb->len;
> > + hdr.len = cpu_to_le16(skb->len);
> Same here.
>
>
> > memcpy(skb_push(skb, NFCWILINK_HDR_LEN), &hdr,
> NFCWILINK_HDR_LEN);
> >
> > /* Insert skb to shared transport layer's transmit queue.
> > @@ -239,7 +522,7 @@ static int nfcwilink_probe(struct
> platform_device *pdev)
> > {
> > static struct nfcwilink *drv;
> > int rc;
> > - u32 protocols;
> > + __u32 protocols;
> Ditto.
You're correct, I will split the patch.
> Cheers,
> Samuel.
Thanks & BR,
Ilan
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
