Hi Sifram,
This version applies fine to my for-usb-next branch, so I'll send it off
to Greg today. Thanks for taking the time to work through our hairy
patch submission process. :) I hope it will be easier for you to send
your next patch!
Sarah Sharp
On Wed, Aug 31, 2011 at 04:44:35PM +0530, Sifram Rajas wrote:
> From: Sifram Rajas <Sifram Rajas sifram.rajas@xxxxxxxxx>
>
> The xhci_hcd->devs is an array of pointers rather than pointer to pointer.
> Hence this check is not required.
>
> Signed-off-by: Sifram Rajas <Sifram Rajas sifram.rajas@xxxxxxxxx>
> ---
> drivers/usb/host/xhci.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
> index 3a0f695..18db1a7 100644
> --- a/drivers/usb/host/xhci.c
> +++ b/drivers/usb/host/xhci.c
> @@ -945,8 +945,7 @@ static int xhci_check_args(struct usb_hcd *hcd, struct usb_device *udev,
> return -ENODEV;
>
> if (check_virt_dev) {
> - if (!udev->slot_id || !xhci->devs
> - || !xhci->devs[udev->slot_id]) {
> + if (!udev->slot_id || !xhci->devs[udev->slot_id]) {
> printk(KERN_DEBUG "xHCI %s called with unaddressed "
> "device\n", func);
> return -EINVAL;
> --
> 1.7.4.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
