Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-30 Thread Greg KH
On Fri, Jan 25, 2013 at 04:32:16PM -0800, Sarah Sharp wrote:
> On Fri, Jan 25, 2013 at 03:19:22PM +0100, Gerd Hoffmann wrote:
> >   Hi,
> > 
> > >> diff --git a/MAINTAINERS b/MAINTAINERS
> > >> index 8ae709e..c5b37de 100644
> > >> --- a/MAINTAINERS
> > >> +++ b/MAINTAINERS
> > >> @@ -7911,9 +7911,10 @@ F:drivers/net/wireless/ath/ar5523/
> > >>  USB ATTACHED SCSI
> > >>  M:  Matthew Wilcox 
> > >>  M:  Sarah Sharp 
> > >> +M:  Gerd Hoffmann 
> > > 
> > > Should Matthew be removed from this?
> > 
> > Dunno, Sarah?
> 
> Matthew has said he doesn't have time to work on UAS, so I would suggest
> removing him.

Can someone send me a patch doing this please?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-28 Thread Gerd Hoffmann
  Hi,

>> Problem is that uas is pretty much the only device using streams,
>> so uas will be the one who triggers any stream bugs in xhci.
>> I have no idea how solid xhci streams support is at the moment.
> 
> The xHCI streams support isn't well tested, because the UAS devices I
> had were so buggy that I couldn't fully test them, and the UAS driver
> didn't (doesn't still?) properly support cancellation or device reset.

There have been a number of improvements fixes there, it should do alot
better now.

> Also, I do know there are a couple of streams work-arounds that need to
> be created for the Intel Panther Point xHCI host controller, so I would
> suggest testing with an NEC/Renesas xHCI host instead.  I will get
> around to implementing them, but other bugs have taken a higher
> priority.

Hmm.  I have a nec xhci for that, but streams not working on Intel
pretty much blocks removing the BROKEN tag from uas :(

cheers,
  Gerd

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Sarah Sharp
On Fri, Jan 25, 2013 at 03:19:22PM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index 8ae709e..c5b37de 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -7911,9 +7911,10 @@ F:  drivers/net/wireless/ath/ar5523/
> >>  USB ATTACHED SCSI
> >>  M:Matthew Wilcox 
> >>  M:Sarah Sharp 
> >> +M:Gerd Hoffmann 
> > 
> > Should Matthew be removed from this?
> 
> Dunno, Sarah?

Matthew has said he doesn't have time to work on UAS, so I would suggest
removing him.

> > Also, any word on when I can remove the CONFIG_BROKEN marking on this
> > driver?
> 
> With the patches in -next uas itself should be reasonable solid.
> 
> Problem is that uas is pretty much the only device using streams,
> so uas will be the one who triggers any stream bugs in xhci.
> I have no idea how solid xhci streams support is at the moment.

The xHCI streams support isn't well tested, because the UAS devices I
had were so buggy that I couldn't fully test them, and the UAS driver
didn't (doesn't still?) properly support cancellation or device reset.
So the first SCSI command that the device didn't like basically hung the
driver.

Also, I do know there are a couple of streams work-arounds that need to
be created for the Intel Panther Point xHCI host controller, so I would
suggest testing with an NEC/Renesas xHCI host instead.  I will get
around to implementing them, but other bugs have taken a higher
priority.

> Sarah, is there some way to avoid using streams?  The UAS specs seems to
> imply using streams is mandatory when connected to a USB-3 port, is that
> correct?  Is there some way to force usb3 devices into usb2 mode even
> when plugged into a usb3 port?  I'd like to have a no_streams module
> option if possible ...

USB 3.0 UAS devices are really supposed to be using streams.  You might
as well just use Bulk-only-Transport (BoT) if you're not using streams.

As for forcing a USB 3.0 port to USB 2.0 speeds, I think you could write
a userspace program to use usbfs to disable the USB 3.0 port.  I've
never tried it though.

Sarah Sharp
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Greg KH
On Fri, Jan 25, 2013 at 04:04:33PM +0100, Gerd Hoffmann wrote:
> > Oh, and any hints on what device on the market today actually follows
> > the UAS spec so I can buy one for testing?
> 
> /me asked the same a while ago, here is the reply
> 
> 
> 
> I would suggest getting a TI UAS evaluation board.  They seem to be the
> most stable UAS device out there:
> 
> http://www.ti.com/tool/tusb9261demo
> 
> I have one of their boards from a year or so ago, but I suspect there's
> a new revision by now.  I got the sample from Kevin Main ,
> and I suspect he might give you one for free as well.

Ah, I already have one of these somewhere, I was wondering if there were
devices you could buy on the market just yet.  I guess not, which is
fine.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Gerd Hoffmann
  Hi,

>> Sarah, is there some way to avoid using streams?  The UAS specs seems to
>> imply using streams is mandatory when connected to a USB-3 port, is that
>> correct?  Is there some way to force usb3 devices into usb2 mode even
>> when plugged into a usb3 port?  I'd like to have a no_streams module
>> option if possible ...
> 
> Well, I think we want to use streams, that's the whole advantage of UAS
> over the old spec. 

Sure, but being able to turn them off for trouble-shooting purposes
would still be useful IMO.

> I wasn't aware that the bugs were in the xhci
> driver, I thought they were in the uas driver, but I could be totally
> wrong.

Oh, uas had bugs too, pretty serious ones included, no question.

> Oh, and any hints on what device on the market today actually follows
> the UAS spec so I can buy one for testing?

/me asked the same a while ago, here is the reply



I would suggest getting a TI UAS evaluation board.  They seem to be the
most stable UAS device out there:

http://www.ti.com/tool/tusb9261demo

I have one of their boards from a year or so ago, but I suspect there's
a new revision by now.  I got the sample from Kevin Main ,
and I suspect he might give you one for free as well.

Another option might be to use the Linux UAS gadget stack with a OMAP5
board with the Synopsis Designware 3 USB 3.0 device controller.  You
could talk with Sebastian Andrzej Siewior , since
he has been doing a lot of work on the UAS gadget driver lately.



cheers,
  Gerd

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Greg KH
On Fri, Jan 25, 2013 at 03:19:22PM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index 8ae709e..c5b37de 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -7911,9 +7911,10 @@ F:  drivers/net/wireless/ath/ar5523/
> >>  USB ATTACHED SCSI
> >>  M:Matthew Wilcox 
> >>  M:Sarah Sharp 
> >> +M:Gerd Hoffmann 
> > 
> > Should Matthew be removed from this?
> 
> Dunno, Sarah?
> 
> > Also, any word on when I can remove the CONFIG_BROKEN marking on this
> > driver?
> 
> With the patches in -next uas itself should be reasonable solid.
> 
> Problem is that uas is pretty much the only device using streams,
> so uas will be the one who triggers any stream bugs in xhci.
> I have no idea how solid xhci streams support is at the moment.
> 
> Sarah, is there some way to avoid using streams?  The UAS specs seems to
> imply using streams is mandatory when connected to a USB-3 port, is that
> correct?  Is there some way to force usb3 devices into usb2 mode even
> when plugged into a usb3 port?  I'd like to have a no_streams module
> option if possible ...

Well, I think we want to use streams, that's the whole advantage of UAS
over the old spec.  I wasn't aware that the bugs were in the xhci
driver, I thought they were in the uas driver, but I could be totally
wrong.

Oh, and any hints on what device on the market today actually follows
the UAS spec so I can buy one for testing?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Gerd Hoffmann
  Hi,

>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 8ae709e..c5b37de 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -7911,9 +7911,10 @@ F:drivers/net/wireless/ath/ar5523/
>>  USB ATTACHED SCSI
>>  M:  Matthew Wilcox 
>>  M:  Sarah Sharp 
>> +M:  Gerd Hoffmann 
> 
> Should Matthew be removed from this?

Dunno, Sarah?

> Also, any word on when I can remove the CONFIG_BROKEN marking on this
> driver?

With the patches in -next uas itself should be reasonable solid.

Problem is that uas is pretty much the only device using streams,
so uas will be the one who triggers any stream bugs in xhci.
I have no idea how solid xhci streams support is at the moment.

Sarah, is there some way to avoid using streams?  The UAS specs seems to
imply using streams is mandatory when connected to a USB-3 port, is that
correct?  Is there some way to force usb3 devices into usb2 mode even
when plugged into a usb3 port?  I'd like to have a no_streams module
option if possible ...

cheers,
  Gerd
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Greg KH
On Fri, Jan 25, 2013 at 02:52:09PM +0100, Gerd Hoffmann wrote:
> Cc: Sarah Sharp 
> Signed-off-by: Gerd Hoffmann 
> ---
>  MAINTAINERS |3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8ae709e..c5b37de 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7911,9 +7911,10 @@ F: drivers/net/wireless/ath/ar5523/
>  USB ATTACHED SCSI
>  M:   Matthew Wilcox 
>  M:   Sarah Sharp 
> +M:   Gerd Hoffmann 

Should Matthew be removed from this?

Also, any word on when I can remove the CONFIG_BROKEN marking on this
driver?

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/2] usb-uas: update MAINTAINERS entry

2013-01-25 Thread Gerd Hoffmann
Cc: Sarah Sharp 
Signed-off-by: Gerd Hoffmann 
---
 MAINTAINERS |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8ae709e..c5b37de 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7911,9 +7911,10 @@ F:   drivers/net/wireless/ath/ar5523/
 USB ATTACHED SCSI
 M: Matthew Wilcox 
 M: Sarah Sharp 
+M: Gerd Hoffmann 
 L: linux-usb@vger.kernel.org
 L: linux-s...@vger.kernel.org
-S: Supported
+S: Maintained
 F: drivers/usb/storage/uas.c
 
 USB CDC ETHERNET DRIVER
-- 
1.7.9.7

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html