Re: [linux-usb-devel] [PATCH 0/3] ehci-hcd: complete iso urbs sooner, take 5

2008-01-06 Thread David Brownell
On Thursday 03 January 2008, Karsten Wiese wrote: > Am Donnerstag, 3. Januar 2008 schrieb David Brownell: > > On Wednesday 02 January 2008, Karsten Wiese wrote: > > > > > > In scan_periodic()'s case Q_TYPE_ITD: > > > those lines: > > > *q_p = q.itd->itd_next; > > > *hw_p = q.it

[patch 2.6.24-rc7] usb: ehci minor sparse fix

2008-01-06 Thread David Brownell
This gets rid of an "sparse" warning in EHCI. We don't really need two variables named "status". Signed-off-by: David Brownell <[EMAIL PROTECTED]> --- I think this didn't get sent to you yet. Meanwhile, I may just hold off on forwarding new patches till you catch up. :) --- g26

Re: [patch/rft 2.6.24-rc6] usb: ehci saves even more memory in ISO transfer descriptors

2008-01-06 Thread David Brownell
On Sunday 06 January 2008, Karsten Wiese wrote: > >   for (uframe = 0; uframe < 8; uframe++) { > > - if (likely (itd->index[uframe] == -1)) > > + t = hc32_to_cpup(ehci, &itd->hw_bufp[uframe]); > > NACK, struct ehci_itd has hw_bufp[7], not 8. Hmm, that would be called a

Re: clutter in /var/log/messages (drivers/usb/serial/usb-serial.c)

2008-01-06 Thread Greg KH
On Sat, Jan 05, 2008 at 12:41:16PM +0100, [EMAIL PROTECTED] wrote: > > what`s the advantage of having this and not just "usb-serial:" instead ? > can this probable be optimized (as most drivers only print "drivername:" > anyway) ? Yes it should be cleaned up, patches gladly accepted to do so.

Re: [patch/rft 2.6.24-rc6] usb: ehci saves even more memory in ISO transfer descriptors

2008-01-06 Thread Karsten Wiese
Am Montag, 31. Dezember 2007 schrieb David Brownell: > @@ -1574,16 +1569,20 @@ itd_complete ( > struct usb_iso_packet_descriptor*desc; > u32 t; > unsigneduframe; > - int

Re: WD My Book 500G external drive: error -110

2008-01-06 Thread Matthew Dharm
On Sun, Jan 06, 2008 at 05:00:08PM -0500, sean darcy wrote: > It's a dual boot desktop. XP sees the usb hard drive, and I can run the > WD extended diagnostics on the drive. > googling did show some other reports, e.g. > http://forum.freespire.org/archive/index.php/t-6479.html, but no solutions.

Re: WD My Book 500G external drive: error -110

2008-01-06 Thread Matthew Dharm
On Sun, Jan 06, 2008 at 05:00:08PM -0500, sean darcy wrote: > Alan Stern wrote: > >On Sat, 5 Jan 2008, seandarcy wrote: > > > >>I'm running Fedora 8: > >> > >>uname -r > >>2.6.23.8-63.fc8 > >> > >>I connected my new Western Digital My Book 500G external drive to the > >>usb port. > >> > >>syslog:

Re: WD My Book 500G external drive: error -110

2008-01-06 Thread sean darcy
Alan Stern wrote: On Sat, 5 Jan 2008, seandarcy wrote: I'm running Fedora 8: uname -r 2.6.23.8-63.fc8 I connected my new Western Digital My Book 500G external drive to the usb port. syslog: Jan 5 21:53:35 testbox kernel: usb 1-4: new high speed USB device using ehci_hcd and address 2 Ja

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Sam Ravnborg
On Sun, Jan 06, 2008 at 06:50:43PM +0100, Stefan Richter wrote: > Al Boldi wrote: > > Stefan Richter wrote: > >> Al Boldi wrote: > >>> menuconfig USB_STORAGE > >>> tristate "USB Mass Storage support" > >>> - depends on USB && SCSI > >>> + depends on USB && BLOCK > >>> + select S

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Alan Stern wrote: > Another disadvantage appears when somebody tries by hand to remove a > component (like SCSI) and finds that it magically reappears. Evidently > something is selecting it, but there's no way to find out what. xconfig's "Show Debug Info" and menuconfig's help screen show you w

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Stefan Richter wrote: > Al Boldi wrote: >> Stefan Richter wrote: >>> Still wrong. SCSI also needs HAS_DMA and SCSI_DMA. >> I don't think so. SCSI selects SCSI_DMA, it doesn't depend on it. > > "A selects B" == "A depends on B, but please don't hide A when B is off > and silently switch B on when

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Al Boldi wrote: > Stefan Richter wrote: >> Al Boldi wrote: >>> menuconfig USB_STORAGE >>> tristate "USB Mass Storage support" >>> - depends on USB && SCSI >>> + depends on USB && BLOCK >>> + select SCSI >> Still wrong. SCSI also needs HAS_DMA and SCSI_DMA. > > I don't think so. SCSI

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Alan Stern
On Sun, 6 Jan 2008 [EMAIL PROTECTED] wrote: > what sysadmins like me would really like is a set of scripts that could > generate a .config from an existing system. After we have one that covers > the hardware for the system we will then have a much better starting point > to work from. We may d

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Al Boldi
Stefan Richter wrote: > Al Boldi wrote: > > Sam Ravnborg wrote: > >> And that requires you to use select only to select symbols with > >> no dependencies. > >> > >> In this case we do not know if BLOCK is enabled or not. > > > > Good point! How about we solve it like this: > > > > menuconfig U

Missing keys on Gyration Universal Remote

2008-01-06 Thread Adolfo R. Brandes
Hey folks, I recently got an USB Gyration Media Center Universal Remote, and it works fine except for 5 buttons. In short, 4 of the keys are not recognized by HID, and one of them repeats endlessly, even under 2.6.24-rc6. I gathered this info running evbug and CONFIG_HID_DEBUG=yes. This is

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Al Boldi wrote: > Sam Ravnborg wrote: >> And that requires you to use select only to select symbols with >> no dependencies. >> >> In this case we do not know if BLOCK is enabled or not. > > Good point! How about we solve it like this: > > menuconfig USB_STORAGE > tristate "USB Mass

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
[EMAIL PROTECTED] wrote: > On Sun, 6 Jan 2008, Stefan Richter wrote: >> build-time configuration has to deal with different, more >> complex questions than run-time configuration. ... > what config options must be selected by the user at build time? (i.e. no > sane default can possibly be deduced f

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 02:13:09PM +0100, Stefan Richter wrote: >> Module autoloading is quite different. > > Both are "hardware -> required kernel support" mappings. > > I know that people don't like this idea since the CML2 discussions, but > there even don't seem to be an

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread david
On Sun, 6 Jan 2008, Stefan Richter wrote: David Lang wrote: On Sun, 6 Jan 2008, Stefan Richter wrote: Module autoloading is quite different. but if boot scripts can figure out what modules to autoload, why can't scripts be written to figure out what drives to build into a system? Because b

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Sam Ravnborg
> > You miss the fundamental point: Adrian - it is you that miss the important piont in this discussion. We have two simple options: 1) Allow the user to define non-valid configurations 2) And the opposite And we want to define only valid configurations which is why we with the present select b

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
David Lang wrote: > On Sun, 6 Jan 2008, Stefan Richter wrote: >> Module autoloading is quite different. > > but if boot scripts can figure out what modules to autoload, why can't > scripts be written to figure out what drives to build into a system? Because build-time configuration has to deal wi

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Adrian Bunk
On Sun, Jan 06, 2008 at 02:13:09PM +0100, Stefan Richter wrote: > Adrian Bunk wrote: > > On Sun, Jan 06, 2008 at 01:18:48PM +0100, Stefan Richter wrote: > >> I'm afraid this can't be put into practice. (User says which hardware > >> and protocols he needs to be supported, scripts magically assembl

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread david
On Sun, 6 Jan 2008, Stefan Richter wrote: Adrian Bunk wrote: On Sun, Jan 06, 2008 at 01:18:48PM +0100, Stefan Richter wrote: I'm afraid this can't be put into practice. (User says which hardware and protocols he needs to be supported, scripts magically assemble a suitable configuration.) Di

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 01:32:35PM +0100, Stefan Richter wrote: >> No, the dependency relationships alone do not carry enough information. >> I am aware of that. >> >> For example, we certainly want [...] > > I don't think your implementation would result in a better UI. I'

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 01:18:48PM +0100, Stefan Richter wrote: >> I'm afraid this can't be put into practice. (User says which hardware >> and protocols he needs to be supported, scripts magically assemble a >> suitable configuration.) > > Distribution installers and live CD

Re: [patch 2.6.24-rc6-git] usb: at91_udc uses generic GPIO calls; minor cleanup

2008-01-06 Thread steve birtles
David, Just like to report that the patch appears works , including the inversion on the D+pullup. if the inversion is set to ".pullup_active_low=0" then the gadgets don't show up on the remote host . By setting it ".pullup_active_low=1" they show up on the host. (which is correct f

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Adrian Bunk
On Sun, Jan 06, 2008 at 01:32:35PM +0100, Stefan Richter wrote: > Adrian Bunk wrote: > > On Sun, Jan 06, 2008 at 12:29:46PM +0100, Stefan Richter wrote: > >> Adrian Bunk wrote: > >>> Duplicating the structure in each UI should be an improvement? > >>> > >>> Hardly. > >> What do you mean? > ... > >

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 12:54:04PM +0100, Stefan Richter wrote: >> The user who wants to enable usb-storage /has/ to go into the SCSI menu >> anyway to answer whether he needs sd, sr, st, sg, command logging... > > That's a different UI problem that has to be fixed. > > The "

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Adrian Bunk
On Sun, Jan 06, 2008 at 01:18:48PM +0100, Stefan Richter wrote: > Adrian Bunk wrote: > > On Sun, Jan 06, 2008 at 01:35:21AM +0100, Stefan Richter wrote: > >> Adrian Bunk wrote: > >>> Whether or not an option requires an additional subsystem like e.g. SCSI > >>> or SSB are hardware and implementati

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 12:29:46PM +0100, Stefan Richter wrote: >> Adrian Bunk wrote: >>> Duplicating the structure in each UI should be an improvement? >>> >>> Hardly. >> What do you mean? ... > You said: > "The graphic UIs including menuconfig currently work best for tree-lik

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 01:35:21AM +0100, Stefan Richter wrote: >> Adrian Bunk wrote: >>> Whether or not an option requires an additional subsystem like e.g. SCSI >>> or SSB are hardware and implementation details we shouldn't bother >>> kconfig users with. >> What is an impl

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Adrian Bunk
On Sun, Jan 06, 2008 at 12:54:04PM +0100, Stefan Richter wrote: > Randy Dunlap wrote: > > Sam Ravnborg wrote: > >> On Sat, Jan 05, 2008 at 11:03:30PM +0200, Adrian Bunk wrote: > >>> For kconfig users, "select" is _much_ better than sending them > >>> through different menus. > >> Only if used withi

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Adrian Bunk
On Sun, Jan 06, 2008 at 12:29:46PM +0100, Stefan Richter wrote: > Adrian Bunk wrote: > > On Sun, Jan 06, 2008 at 01:35:21AM +0100, Stefan Richter wrote: > >> instead work on better UIs if you have got > >> trouble with the complexities of the dependencies graph. The graphic > >> UIs including menu

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Randy Dunlap wrote: > Sam Ravnborg wrote: >> On Sat, Jan 05, 2008 at 11:03:30PM +0200, Adrian Bunk wrote: >>> For kconfig users, "select" is _much_ better than sending them >>> through different menus. >> Only if used within the current limitations of Kconfig. >> And that requires you to use select

Re: [PATCH 2/5] USB Kconfig: Select SCSI for USB Mass Storage support

2008-01-06 Thread Stefan Richter
Adrian Bunk wrote: > On Sun, Jan 06, 2008 at 01:35:21AM +0100, Stefan Richter wrote: >> instead work on better UIs if you have got >> trouble with the complexities of the dependencies graph. The graphic >> UIs including menuconfig currently work best for tree-like dependencies, >> but the graph is

Re: Which hub chip?

2008-01-06 Thread Diego Zuccato
Karl Hiramoto ha scritto: But your report is about problems with Win drivers. Does it happen in Linux too? If not, then it's not an issue, since the host is running Linux (kernel 2.6.8 at least). I don't think so, but i haven't tried that hub chip in quite a while. Anyone else have had exper

Re: clutter in /var/log/messages (drivers/usb/serial/usb-serial.c)

2008-01-06 Thread Tilman Schmidt
On Sat, 05 Jan 2008 12:41:16 +0100, [EMAIL PROTECTED] said: [...] > > modprobe -r keyspan: > usbcore: deregistering interface driver keyspan > drivers/usb/serial/usb-serial.c: USB Serial deregistering driver Keyspan > - (without firmware) > drivers/usb/serial/usb-serial.c: USB Serial deregiste