[U-Boot] [PATCH 2/7] usb: ehci: Move interrupt packet length check to create_int_queue

[Marek Vasut]

On Sunday, September 21, 2014 at 07:53:35 PM, Hans de Goede wrote:
> Hi,
[...]
> >> -       if (length > usb_maxpacket(dev, pipe)) {
> >> -               printf("%s: Interrupt transfers requiring several "
> >> -                       "transactions are not supported.\n", __func__);
> >> -               return -1;
> >> -       }
> >> -
> >> 
> >>         queue = create_int_queue(dev, pipe, 1, length, buffer);
> >> 
> >> +       if (!queue)
> >> +               return -1;
> > 
> > Can you return a more consistent error code?
> 
> I'm just moving code around, and returning the same error code as before.
> Surely changing the error code belongs in another patch ?

Yes, full ACK. This is exactly a prime examply where squashing two fixes into 
one patch would break bisectability absolutely perfectly.

Best regards,
Marek Vasut