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