Messages in this thread | | | From | Neeraj sanjay kale <> | Subject | Re: [PATCH v10 3/3] Bluetooth: NXP: Add protocol support for NXP Bluetooth chipsets | Date | Tue, 14 Mar 2023 15:40:34 +0000 |
| |
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
| |