On Tue, 17 May 2011, Alexey Orishko wrote:
> Changes:
> - claim slave/data interface during bind() and release in
> unbind() unconditionally
> - in case of error during bind(), release claimed data
> interface in the same function
> - remove obsolited "*_claimed" entries from driver context
...
> @@ -572,42 +559,32 @@ advance:
> goto error;
>
> /* claim interfaces, if any */
> - if (ctx->data != intf) {
> - temp = usb_driver_claim_interface(driver, ctx->data, dev);
> - if (temp)
> - goto error;
> - ctx->data_claimed = 1;
> - }
> -
> - if (ctx->control != intf) {
> - temp = usb_driver_claim_interface(driver, ctx->control, dev);
> - if (temp)
> - goto error;
> - ctx->control_claimed = 1;
> - }
> + temp = usb_driver_claim_interface(driver, ctx->data, dev);
> + if (temp)
> + goto error;
Here and later on, the patch seems to have forgotten about the control
interface. Is this deliberate or an oversight?
Alan Stern
--
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
