On Mon, 5 Aug 2013, Ming Lei wrote:
> Some host controllers(such as xHCI) can support building
> packet from discontinuous buffers, so introduce one flag
> and helper for this kind of host controllers, then the
> feature can help some applications(such as usbnet) by
> supporting arbitrary length of sg buffers.
> +static inline bool usb_device_no_sg_constraint(struct usb_device *udev)
> +{
> + if (udev && udev->bus && udev->bus->no_sg_constraint)
> + return true;
> + else
> + return false;
> +}
A more elegant implementation would be:
return udev && udev->bus && udev->bus->no_sg_constraint;
I doubt it will make any difference to the object code, though. Apart
from that, this looks fine.
Acked-by: Alan Stern <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html