Re: [PATCH v10 3/3] Bluetooth: NXP: Add protocol support for NXP Bluetooth chipsets

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Simon

Thank you for reviewing the patch. I have a comment below:

> 
> > +send_skb:
> > +     /* Prepend skb with frame type */
> > +     memcpy(skb_push(skb, 1), &hci_skb_pkt_type(skb), 1);
> > +     skb_queue_tail(&nxpdev->txq, skb);
> > +
> > +     btnxpuart_tx_wakeup(nxpdev);
> > +ret:
> > +     return 0;
> > +
> > +free_skb:
> > +     kfree_skb(skb);
> > +     goto ret;
> 
> nit: I think it would be nicer to simply return 0 here.
>      And remove the ret label entirely.
> 
> > +}
> 
We need to return from this function without clearing the skbs, unless "goto free_skb" is called.
If I remove the ret label and return after kfree_skb() it causes a kernel crash.
Keeping this change as it is.

Please let me know if you have any further review comments on the v11 patch.

Thanks,
Neeraj




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux