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 <stern@xxxxxxxxxxxxxxxxxxx> -- 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