Re: [linux-usb-devel] usb nasties

2007-02-09 Thread Tony Olech
Hi David, unfortunately I had been sick as well being under pressure on a different project. Now my boss has promised me time from Monday next week to resolve the issues. Tony Olech On 9 Feb 2007, at 02:52, David Brownell wrote: > On Thursday 08 February 2007 5:13 pm, Andrew Morton wrote: >

Re: [linux-usb-devel] PATCH: usb: descriptor structures have to be packed

2007-02-09 Thread Sergei Organov
Alan Stern <[EMAIL PROTECTED]> writes: > On Thu, 8 Feb 2007, Sergei Organov wrote: > >> IMHO aligned(N) should be used in portable code only to *increase* >> alignment (this effectively excludes usage of 'packed'), and then the >> alignment won't affect the low-level operations, that automatically

[linux-usb-devel] code review requested for tranzport driver

2007-02-09 Thread Mike Taht
I have been working on a usb kernel driver for the frontier designs "tranzport" control surface. It's a pretty unique device - 20 character LCD, 7 lights, 21 buttons, and a shuttle wheel. Anyway, as I have something that more or less works better than the current ardour2 libusb driver... and as

Re: [linux-usb-devel] PATCH: usb: descriptor structures have to be packed

2007-02-09 Thread Sergei Organov
Alan Stern <[EMAIL PROTECTED]> writes: > On Thu, 8 Feb 2007, Sergei Organov wrote: > >> IMHO aligned(N) should be used in portable code only to *increase* >> alignment (this effectively excludes usage of 'packed'), and then the >> alignment won't affect the low-level operations, that automatically

Re: [linux-usb-devel] code review request: simple usb stream interface (susi)

2007-02-09 Thread Alan Stern
On Thu, 8 Feb 2007, Edwin Olson wrote: > > So I see. Do you happen to have the URB_NO_FSBR flag turned on in your > > URBs? > Not in my code, at least... What does it do? It does pretty much what you said you saw -- it hints to the controller driver that this URB doesn't need more than one pack

Re: [linux-usb-devel] Gadget Zero patch for ZLP

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Pandita, Vikram wrote: > You are right, however I will correct my analysis about the issue. > For IN transfers, the Gadget Zero driver is setting the ZLP flag > (req->zero) even if the packet size is a short packet. Is that right? Yes. > Example: 66 byte packet size sent out

[linux-usb-devel] autosuspend for storage devices

2007-02-09 Thread Oliver Neukum
Hi, here's autosuspend for storage devices. It was surprisingly easy. I've tested it and I can play an mp3 and see how the stick wakes up every few seconds. Comments? To test this you'll likely have to turn off haldaemon. This still needs a solution. Alan, do I recall correctly that you have a mou

Re: [linux-usb-devel] Blackberry driver and pm

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Oliver Neukum wrote: > Am Donnerstag, 8. Februar 2007 17:09 schrieb Alan Stern: > > You shouldn't call usb_autopm_get_interface().  The changes it makes are > > lost when the driver fails to bind to the interface.  You should call > > usb_autopm_resume_device() instead -- unf

Re: [linux-usb-devel] PATCH: usb: descriptor structures have to be packed

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Sergei Organov wrote: > Alan Stern <[EMAIL PROTECTED]> writes: > > > On Thu, 8 Feb 2007, Sergei Organov wrote: > > > >> IMHO aligned(N) should be used in portable code only to *increase* > >> alignment (this effectively excludes usage of 'packed'), and then the > >> alignment

Re: [linux-usb-devel] code review request: simple usb stream interface (susi)

2007-02-09 Thread Pete Zaitcev
On Thu, 08 Feb 2007 18:02:12 -0500, Edwin Olson <[EMAIL PROTECTED]> wrote: > [...] staring at usbmon, it > appears that sometimes I do in fact get multiple callbacks from the same > frame. But it's quite rare... This is quite alarming. Maybe you want to check the 1s file and see how many events

Re: [linux-usb-devel] code review request: simple usb stream interface (susi)

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Pete Zaitcev wrote: > If we get multiply callbacks in reality, many things my break, especially > when files are closed, devices disconnected, modules removed, etc. Pete, note that Edwin is talking about callbacks for multiple URBs in the same frame. Not multiple callbacks fo

Re: [linux-usb-devel] code review request: simple usb stream interface (susi)

2007-02-09 Thread Pete Zaitcev
On Fri, 9 Feb 2007 11:01:08 -0500 (EST), Alan Stern <[EMAIL PROTECTED]> wrote: > On Fri, 9 Feb 2007, Pete Zaitcev wrote: > > > If we get multiply callbacks in reality, many things my break, especially > > when files are closed, devices disconnected, modules removed, etc. > > Pete, note that Edwin

Re: [linux-usb-devel] [PATCH 59/70] USB Input: Added kernel module to support all GTCO CalComp USB InterWrite School products

2007-02-09 Thread Greg KH
On Thu, Feb 08, 2007 at 09:37:47PM -0700, Roberson, Jeremy wrote: > The HID driver doesn't parse Digitizer data properly which, is why > Wacom, etc requires custom drivers for their digitizers. Same for our > digitizers. As far as I know, Digitizers have never been parsed > properly by the HID Dr

Re: [linux-usb-devel] patch to autosuspend mice

2007-02-09 Thread Greg KH
On Thu, Feb 01, 2007 at 11:59:43AM -0500, Alan Stern wrote: > > > We should write sysfs_add_to_group(), so that these two > > > attributes can be added to the existing device/power > > > subdirectory. (sysfs_remove_from_group() doesn't seem to > > > be needed, but it could be written as we

Re: [linux-usb-devel] code review requested for tranzport driver

2007-02-09 Thread Laurent Pinchart
Hi Mike, I'll answer to the best of my knowledge, which means only your question nr 1 will get an answer :-) > I have been working on a usb kernel driver for the frontier designs > "tranzport" control surface. It's a pretty unique device - 20 character > LCD, 7 lights, 21 buttons, and a shuttle

Re: [linux-usb-devel] autosuspend for storage devices

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Oliver Neukum wrote: > Hi, > > here's autosuspend for storage devices. It was surprisingly easy. > I've tested it and I can play an mp3 and see how the stick wakes > up every few seconds. Comments? > To test this you'll likely have to turn off haldaemon. This still > needs a s

Re: [linux-usb-devel] USB host controller testing in linux

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Deepak Katagade wrote: > Hi All, >Thanks for the immediate reply.We are now test testing the > massstorage device with our hostcontroller driver by writing a script > that will mount the device copy the some large file in and out of the > device and run md5sum.This will run

Re: [linux-usb-devel] code review requested for tranzport driver

2007-02-09 Thread Clemens Ladisch
Mike Taht wrote: > 2) Will I need to request specific usb minor numbers for this (raw?, > Alsa?) ALSA's device nodes are handled by the framework. > 3) Is there a midi layer that is a model of it's kind? There is no MIDI driver that converts MIDI from/to something entirely different. However, s

Re: [linux-usb-devel] [PATCH]cdc-acm:fix incorrect throtteling, make set_control optional

2007-02-09 Thread David Brownell
On Thursday 08 February 2007 1:06 am, Oliver Neukum wrote: > --- a/include/linux/usb/cdc.h 2007-02-06 14:11:50.0 +0100 > +++ b/include/linux/usb/cdc.h 2007-02-08 09:50:28.0 +0100 > @@ -73,6 +73,13 @@ > __u8bmCapabilities; > } __attribute__ ((packed)); > > +/* capabilit

[linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Stephan Esterhuizen
I'm using Cypress bulkloop.hex firmware on an fx2lp (CY7C68013A) with simple host-side user-land code to transmit data to EP2 (out endpoint) and read the 'echo' on EP6 (in endpoint). The very first time the code is run it writes, say, 60 bytes to EP2 and reads the echo (60 bytes) from EP6. When

Re: [linux-usb-devel] usb: descriptor structures have to be packed

2007-02-09 Thread Alan
On Thu, 08 Feb 2007 19:20:39 + "Phil Endecott" <[EMAIL PROTECTED]> wrote: > > From a > > higher-level (source code) point of view, the alignment doesn't affect the > > set of operations which can be applied to the data type. > > It affects whether you can legally use & to take its address.

Re: [linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Stephan Esterhuizen wrote: > I'm using Cypress bulkloop.hex firmware on an fx2lp (CY7C68013A) with simple > host-side user-land code to transmit data to EP2 (out endpoint) and read the > 'echo' on EP6 (in endpoint). The very first time the code is run it writes, > say, 60 by

Re: [linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Stephan Esterhuizen
Pardon the very long lines. This has been fixed now. My usbmon shows the following: --- (1) Test for 12 bytes TX/RX _without_ writing the extra 1 byte bulk --- packet --- (12 byte tx/rx: run program for first time) eb13c640 4268765364 S Co:028:00 s 00 09 0001 0 eb13c640 4268765473 C Co

Re: [linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Stephan Esterhuizen
This is even more bizarre, the output below is from usbmon when I do two 12-byte writes/reads within the same program (in the previous example I posted I did 2 12-byte writes/reads by calling my program twice). --- (2x 12-byte writes/reads within the same program) eb0e9cc0 1713314624 S Co:030:00 s

Re: [linux-usb-devel] autosuspend for storage devices

2007-02-09 Thread Oliver Neukum
Am Freitag, 9. Februar 2007 19:06 schrieb Alan Stern: > On Fri, 9 Feb 2007, Oliver Neukum wrote: > > > Hi, > > > > here's autosuspend for storage devices. It was surprisingly easy. > > I've tested it and I can play an mp3 and see how the stick wakes > > up every few seconds. Comments? > > To test

Re: [linux-usb-devel] [PATCH]cdc-acm:fix incorrect throtteling, make set_control optional

2007-02-09 Thread Oliver Neukum
> > +#define ACM_CTRL_CAP_COMM_FTR 0x01 > > +#define ACM_CTRL_CAP_LINE 0x02 > > +#define ACM_CTRL_CAP_BRK 0x04 > > +#define ACM_CTRL_CAP_NOTIFY0x08 > > All other #defines in that code use USB_CDC_* prefixes; > could you make this match? No problem. Greg, do you want an updated p

[linux-usb-devel] [PATCH] usbcore: clean up the new blacklist code

2007-02-09 Thread Alan Stern
This patch (as853) cleans up the new USB blacklist code in several ways: Replace the STRING_FETCH quirk with STRING_FETCH_255, meaning not to use a 255-byte query when retrieving string descriptors. This is all that the problem-causing HP scanners require.

[linux-usb-devel] [RFC] Autosuspend update

2007-02-09 Thread Alan Stern
Oliver: This patch includes the various facilities we have been discussing. It seems to work okay. Obviously it isn't in final form. It needs to be cleaned up and broken apart into several simpler patches. But you ought to be able to test it easily enough. It needs to go on top of as853, p

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Alan Stern
On Fri, 19 Jan 2007, Jon Smirl wrote: > On 1/19/07, Oliver Neukum <[EMAIL PROTECTED]> wrote: > > Hi, > > > > I did the dirty deed. Could you please tell me which quirky devices > > you know of? I am afraid this list needs to be populated and there > > are outstanding regressions in Adrian's list.

Re: [linux-usb-devel] [PATCH]cdc-acm:fix incorrect throtteling, make set_control optional

2007-02-09 Thread Greg KH
On Fri, Feb 09, 2007 at 10:58:43PM +0100, Oliver Neukum wrote: > > > > +#define ACM_CTRL_CAP_COMM_FTR0x01 > > > +#define ACM_CTRL_CAP_LINE0x02 > > > +#define ACM_CTRL_CAP_BRK 0x04 > > > +#define ACM_CTRL_CAP_NOTIFY 0x08 > > > > All other #defines in that code use USB_CDC_* prefix

Re: [linux-usb-devel] autosuspend for storage devices

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Oliver Neukum wrote: > > Also, it's debatable whether USB should autosuspend a storage device at > > all. Like with HID devices, it seems more logical to have autosuspend > > requests come up from the lower layer (in this case the SCSI layer). > > After all, suspending a dis

Re: [linux-usb-devel] [RFC] Autosuspend update

2007-02-09 Thread Oliver Neukum
Am Freitag, 9. Februar 2007 23:12 schrieb Alan Stern: Firstly, cool. I'll look at it closer again, but just some first comments. > +static ssize_t > +set_autosuspend(struct device *dev, struct device_attribute *attr, > +   const char *buf, size_t count) > +{ > +   struct usb_devic

Re: [linux-usb-devel] [RFC] Autosuspend update

2007-02-09 Thread Oliver Neukum
Am Freitag, 9. Februar 2007 23:12 schrieb Alan Stern: > +static ssize_t > +set_suspended(struct device *dev, struct device_attribute *attr, > +   const char *buf, size_t count) > +{ > +   struct usb_device *udev = to_usb_device(dev); > +   unsigned value; > +   int rc; > + >

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Jon Smirl
On 2/9/07, Alan Stern <[EMAIL PROTECTED]> wrote: > On Fri, 19 Jan 2007, Jon Smirl wrote: > > > On 1/19/07, Oliver Neukum <[EMAIL PROTECTED]> wrote: > > > Hi, > > > > > > I did the dirty deed. Could you please tell me which quirky devices > > > you know of? I am afraid this list needs to be populate

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Jon Smirl
On 2/9/07, Jon Smirl <[EMAIL PROTECTED]> wrote: > On 2/9/07, Alan Stern <[EMAIL PROTECTED]> wrote: > > On Fri, 19 Jan 2007, Jon Smirl wrote: > > > > > On 1/19/07, Oliver Neukum <[EMAIL PROTECTED]> wrote: > > > > Hi, > > > > > > > > I did the dirty deed. Could you please tell me which quirky devices

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Jon Smirl wrote: > > I received the device in the mail today and tested it. My experience > > echoes yours exactly, except that I didn't actually try to use the device > > after resuming it. I presume you have tried and it doesn't work? > > Device does not work after resume.

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Jon Smirl wrote: > Reading through that doc it appears that the device operates in a USB > standard mode and PSA2 mode ( a MS Windows mode). > > Maybe when it is resuming it is resuming to MS Windows mode instead of > the USB standard mode, or the other way around. > > There

Re: [linux-usb-devel] [RFC] Autosuspend update

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Oliver Neukum wrote: > Am Freitag, 9. Februar 2007 23:12 schrieb Alan Stern: > > Firstly, cool. > I'll look at it closer again, but just some first comments. > > > +static ssize_t > > +set_autosuspend(struct device *dev, struct device_attribute *attr, > > +   cons

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Jon Smirl
On 2/9/07, Alan Stern <[EMAIL PROTECTED]> wrote: > Are you certain? The mute button is on the top front edge, but the light > appears about 2/3 of the way up the front side. (There may be another > light right on the mute button as well; I can't remember.) The light on the front is the volume in

Re: [linux-usb-devel] introduction of blacklist for autosuspend

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Jon Smirl wrote: > On 2/9/07, Alan Stern <[EMAIL PROTECTED]> wrote: > > Are you certain? The mute button is on the top front edge, but the light > > appears about 2/3 of the way up the front side. (There may be another > > light right on the mute button as well; I can't remem

Re: [linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Alan Stern
On Fri, 9 Feb 2007, Stephan Esterhuizen wrote: > My usbmon shows the following: > > --- (1) Test for 12 bytes TX/RX _without_ writing the extra 1 byte bulk > --- packet --- > (12 byte tx/rx: run program for first time) > eb13c640 4268765364 S Co:028:00 s 00 09 0001 0 > eb13c640 4268765

Re: [linux-usb-devel] [PATCH 59/70] USB Input: Added kernel module to support all GTCO CalComp USB InterWrite School products

2007-02-09 Thread Roberson, Jeremy
Here is my concern. If the HID layer is fixed to support Digitizers which means we would no longer need separate drivers, then how are we(Digitizer manufacturers) supposed to get the digitizer input before the X Mouse driver takes it? Please forgive my ignorance, but I'm still learning. -- Jer

[linux-usb-devel] Elektrikli Bisiklet

2007-02-09 Thread baris elektronik
http://hemalhemsat.com/main/auctiondetail/1778386/bisiklet_ve_scooter_aksesuarlari/elektrikli_bisiklet.php Akü kapasitesi 16amper.Bir žarj ile 50km. yol yapar.ešer isterseniz yedek akü temin edip iki katż yol 100km. gidersiniz. Tekerlek ēapż : 26" inc. Hżz : 35-40km. Motor : 240W. 24V. fżrē

Re: [linux-usb-devel] FX2 Bulk In/Out fails with bulkloop.hex

2007-02-09 Thread Stephan Esterhuizen
Ah yes, everything works now. See below: > On Fri, 9 Feb 2007, Stephan Esterhuizen wrote: > > > My usbmon shows the following: > > > > --- (1) Test for 12 bytes TX/RX _without_ writing the extra 1 byte bulk > > --- packet --- > > (12 byte tx/rx: run program for first time) > > eb13c640 42687653

[linux-usb-devel] Patch for expanded text API in usbmon

2007-02-09 Thread Pete Zaitcev
I decided not to cram the ISO descriptors into the old line format, primarily because my parser would still choke on ISO no matter what. I added a new format '0u' in addition to the old one '0t'. So we have a bit of a bloat, all structures grew somewhat, output lines grew too... Numbers do not seem