Hi Matthew,
On Thu, Sep 16, 2010 at 8:58 PM, Matthew Garrett <mjg@xxxxxxxxxx> wrote:
> The main reason for these urbs to error out on submission is that runtime
> pm has kicked in, which is unnecessary noise. Let's just drop them.
>
> Signed-off-by: Matthew Garrett <mjg@xxxxxxxxxx>
> ---
> drivers/bluetooth/btusb.c | 15 +++------------
> 1 files changed, 3 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index d22ce3c..3ace025 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -229,11 +229,8 @@ static void btusb_intr_complete(struct urb *urb)
> usb_anchor_urb(urb, &data->intr_anchor);
>
> err = usb_submit_urb(urb, GFP_ATOMIC);
> - if (err < 0) {
> - BT_ERR("%s urb %p failed to resubmit (%d)",
> - hdev->name, urb, -err);
> + if (err < 0)
> usb_unanchor_urb(urb);
> - }
> }
>
I do not get the point here. Cannot you just undef BT_ERR if you
enabled debug? I believe this is standard behavior.
IMO NACK
Regards,
Andrei
> static int btusb_submit_intr_urb(struct hci_dev *hdev, gfp_t mem_flags)
> @@ -313,11 +310,8 @@ static void btusb_bulk_complete(struct urb *urb)
> usb_mark_last_busy(data->udev);
>
> err = usb_submit_urb(urb, GFP_ATOMIC);
> - if (err < 0) {
> - BT_ERR("%s urb %p failed to resubmit (%d)",
> - hdev->name, urb, -err);
> + if (err < 0)
> usb_unanchor_urb(urb);
> - }
> }
>
> static int btusb_submit_bulk_urb(struct hci_dev *hdev, gfp_t mem_flags)
> @@ -402,11 +396,8 @@ static void btusb_isoc_complete(struct urb *urb)
> usb_anchor_urb(urb, &data->isoc_anchor);
>
> err = usb_submit_urb(urb, GFP_ATOMIC);
> - if (err < 0) {
> - BT_ERR("%s urb %p failed to resubmit (%d)",
> - hdev->name, urb, -err);
> + if (err < 0)
> usb_unanchor_urb(urb);
> - }
> }
>
> static void inline __fill_isoc_descriptor(struct urb *urb, int len, int mtu)
> --
> 1.7.2.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
