Re: [PATCH v2 2/3] usb: gadget: file_storage: Make CD-ROM emulationwork with Mac OS-X

From: Michal Nazarewicz
Date: Wed Mar 23 2011 - 12:15:26 EST

On the other hand, I don't think any implementations would get upset if we simply ended the transfer with a short packet instead of
adhering strictly to the spec.

On Wed, 23 Mar 2011, Roger Quadros wrote:
I tried your patch with the CD-ROM implementation and it works perfectly. I do not see the unnecessary zero padded transfers
any more.

Do you think we should have this patch in? with the risk of not strictly
adhering to spec for cases where controller cannot stall?

On Wed, 23 Mar 2011 16:17:32 +0100, Alan Stern wrote:
There already is another place where not stalling forces the driver to
violate the spec. I don't think this makes things much worse... but it
is a significant change in behavior.

This affects Michal's driver too; we should ask his opinion. Michal,
in case you didn't see it, the proposed patch is here:

Thanks. I don't have original message so I'll reply here:

+ /* Don't know what to do if common->fsg is NULL */
+ } else if (!common->fsg) {
+ rc = -EIO;

I'd change that to "if (!fsg_is_set(common))". fsg_is_set() prints out a warning
if common->fsg is not set and I think that's something we want.

(Come to think of it, this check should be done at the beginning of the function
simplifying the rest of the function, but that's unrelated, I'll do it myself I

Other then that, looks good to me.

As for violating the spec, I don't think I have anything clever to add...

Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michal "mina86" Nazarewicz (o o)
ooo +-----<email/xmpp: mnazarewicz@xxxxxxxxxx>-----ooO--(_)--Ooo--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at