On Thu, Apr 24 2014, Andrzej Pietrasiewicz wrote: > @@ -438,8 +440,46 @@ static int config_desc(struct usb_composite_dev *cdev, > unsigned w_value) > > } > > - /* This is a lookup by config *INDEX* */ > w_value &= 0xff; > + /* > + * This is a lookup to make non-compliant USB hosts happy. > + * > + * By non-compliant USB hosts I mean those which only ask for > + * configuration @index 0. > + * If one configuration is marked "special" by storing > + * its address in cdev->os_desc_config, we try returning > + * exactly this configuration. > + * > + * Otherwise, regular list lookup is performed. > + * > + * If we are connected to a compliant USB host _and_ > + * one configuration is "special" we report it @ index 0, > + * but report remaining configurations as well - with adjusted > + * indices. > + */
The purpose of this is confusing to me. Why not have the configurations
in desired order? Or why not rearrange configurations instead of
setting os_desc_config?
> + c = cdev->os_desc_config;
> + if (c) {
> + if (!w_value) {
> + switch (speed) {
> + case USB_SPEED_SUPER:
> + if (!c->superspeed)
> + goto list_lookup;
> + break;
> + case USB_SPEED_HIGH:
> + if (!c->highspeed)
> + goto list_lookup;
> + break;
> + default:
> + if (!c->fullspeed)
> + goto list_lookup;
> + }
> +
> + return config_buf(c, speed, cdev->req->buf, type);
> + }
> + --w_value; /* account for conf returned @ index 0 */
> + }
> +list_lookup:
> + /* This is a lookup by config *INDEX* */
> list_for_each_entry(c, &cdev->configs, list) {
> /* ignore configs that won't work at this speed */
> switch (speed) {
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michał “mina86” Nazarewicz (o o)
ooo +--<m...@google.com>--<xmpp:min...@jabber.org>--ooO--(_)--Ooo--
signature.asc
Description: PGP signature
