Hi,

> +        } else if (descriptors[i + 5] != s->configuration) {
> +            fprintf(stderr, "not requested configuration %d\n",
> +                    s->configuration);
> +            i += (descriptors[i + 3] << 8) + descriptors[i + 2];
> +            continue;
> +        }

That message doesn't indicate an error and should be a DPRINTF instead
of a fprintf.  Otherwise the patch looks fine.

cheers,
  Gerd


Reply via email to