RE: Linux USB gadget mass storage

2012-11-01 Thread Alan Stern
On Thu, 1 Nov 2012, victor wrote: > Alan, > > The partial patch is here. In fact I am using file_storage.c in u-boot, the > context and buf pointer are mismatch when the value is returned back from > irq routine. The file-storage.c is not in u-boot release, so I post my > question on linux usb. I

RE: Linux USB gadget mass storage

2012-10-31 Thread victor
>> In file_storage.c, send_status(), if (transport_is_bbb()), I add the >> line >> bh->inreq->buf = bh -> buf, >> >> and in get_next_command(), if (transport_is_bbb()), I add the line >> bh->outreq->context = bh, to make the bh address to be the same. >> >> Is this patch ok? > What patch? You

RE: Linux USB gadget mass storage

2012-10-31 Thread Alan Stern
On Wed, 31 Oct 2012, victor wrote: > Hi, > > In usb gadget file_storage.c, check_command(), the part "Check that only > command bytes listed in the mask are non-zero" cannot handle the > SCSI_INQUIRY command of 12 01 80 00 FF 00 > > The code will return EINVAL. > > Is this a bug? No. Alan S

RE: Linux USB gadget mass storage

2012-10-31 Thread Alan Stern
On Wed, 31 Oct 2012, victor wrote: > Hi, > > In file_storage.c, send_status(), if (transport_is_bbb()), I add the line > bh->inreq->buf = bh -> buf, > > and in get_next_command(), if (transport_is_bbb()), I add the line > bh->outreq->context = bh, to make the bh address to be the same. > > Is t

RE: Linux USB gadget mass storage

2012-10-31 Thread victor
om: victor [mailto:vic...@keyasic.com] Sent: Wednesday, 31 October, 2012 1:20 PM To: 'linux-usb@vger.kernel.org' Subject: RE: Linux USB gadget mass storage Hi, In file_storage.c, send_status(), if (transport_is_bbb()), I add the line bh->inreq->buf = bh -> buf, and in get_next_command(

RE: Linux USB gadget mass storage

2012-10-30 Thread victor
Hi, In file_storage.c, send_status(), if (transport_is_bbb()), I add the line bh->inreq->buf = bh -> buf, and in get_next_command(), if (transport_is_bbb()), I add the line bh->outreq->context = bh, to make the bh address to be the same. Is this patch ok? victor -Original Message- From