Re: [linux-usb-devel] hid powersave by autosuspend #2

2007-02-07 Thread Oliver Neukum
Am Dienstag, 6. Februar 2007 18:39 schrieb Alan Stern: On Tue, 6 Feb 2007, Oliver Neukum wrote: With this infrastructure in place, all you have to do is set_bit(USB_DF_DEVICE_BUSY, hid_to_usb_dev(hid)-dflags); whenever an event occurs. No messing around with extra timers.

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Oliver Neukum
Am Dienstag, 6. Februar 2007 21:39 schrieb Frédéric RISS: Hi, Is there any way to debug the remote wakeup functionality of usb devices? I'm trying to wake my MacMini up using the built-in IR Receiver. This device is seen as an raw HID device. static int hub_port_suspend: if

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Frederic Riss
2007/2/7, Oliver Neukum [EMAIL PROTECTED]: Am Dienstag, 6. Februar 2007 21:39 schrieb Frédéric RISS: Hi, Is there any way to debug the remote wakeup functionality of usb devices? I'm trying to wake my MacMini up using the built-in IR Receiver. This device is seen as an raw HID device.

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 10:54 schrieb Frederic Riss: 2007/2/7, Oliver Neukum [EMAIL PROTECTED]: Am Dienstag, 6. Februar 2007 21:39 schrieb Frédéric RISS: Hi, Is there any way to debug the remote wakeup functionality of usb devices? I'm trying to wake my MacMini up using the

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Frederic Riss
2007/2/7, Oliver Neukum [EMAIL PROTECTED]: As I wrote it's seen by Linux as a raw HID device, thus it's bound to the usbhid driver. IIRC there're 2 sorts of HID devices, input ones and 'raw' ones. This one isn't recognized as an input device, its data is simply sent to /dev/hiddev0.

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

2007-02-07 Thread Phil Endecott
I won't pretend to understand all the subtleties of __attribute__((packed)), but I did learn something about it when I hacked USB/IP to run on ARM, for the NSLU2, last year. The issue boiled down to this: struct S { int intfield; char charfield; } /* maybe add: __attribute__((packed))

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 11:31 schrieb Frederic Riss: 2007/2/7, Oliver Neukum [EMAIL PROTECTED]: As I wrote it's seen by Linux as a raw HID device, thus it's bound to the usbhid driver. IIRC there're 2 sorts of HID devices, input ones and 'raw' ones. This one isn't recognized as an

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

2007-02-07 Thread Oliver Neukum
Am Dienstag, 6. Februar 2007 23:48 schrieb Pete Zaitcev: On Tue, 6 Feb 2007 13:08:19 -0800, Inaky Perez-Gonzalez [EMAIL PROTECTED] wrote: [btw, I truly have little idea about which are those specific costs, out of professional curiosity, got any pointers?]

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 08:00 schrieb David Brownell: In scope of (1) i can't understand: structs are padded implicitly, member access is coded explicitly. But the struct declarations in question are not memory layouts. They're on-the-wire protocol structures, where the compiler

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

2007-02-07 Thread Sergei Organov
Inaky Perez-Gonzalez [EMAIL PROTECTED] writes: [...] Here is my point: it's way easier and more maintenable to do struct some_descriptor { __le16 foo; __u8 bar; } __attribute__((packed)); ... struct some_descriptor descr; ...

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov:   struct some_descriptor descr;   ...   read_from_wire(descr, sizeof(descr));   do_something_with_values(usb_get_uint16(descr-foo), bar);   ...   usb_set_uint16(descr-foo, 2385);   where:     static inline unsigned

[linux-usb-devel] [PATCH 1/3]cdc-acm: acm_set_control optional

2007-02-07 Thread Oliver Neukum
According to the USB CDC class specification, line state control is an optional feature for ACM devices. However, the cdc-acm driver bails out when acm_set_control() fails while opening the device. The proposed patch changes that behaviour: Failure of acm_set_control is now only considered an

[linux-usb-devel] [PATCH 2/3]cdc-acm: clear throttle on open

2007-02-07 Thread Oliver Neukum
It seems that the CDC-ACM driver can get stuck in a throttling condition. The proposed patch fixes this by resetting the acm-throttle flag at device open time. This patch was tested on i386, ohci-hcd with a custom USB device and appears to fix the problem. Signed-off-by: Joris van Rantijk [EMAIL

[linux-usb-devel] [PATCH 3/3]cdc-acm: check throttle once under lock

2007-02-07 Thread Oliver Neukum
Ok. The version below is more similar in structure to the current code. The essential change is that acm-throttle is now read only once so that it can not change under our nose. Tested, still works. Signed-off-by: Joris van Rantwijk [EMAIL PROTECTED] Acked-by: Oliver Neukum [EMAIL PROTECTED]

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

2007-02-07 Thread Sergei Organov
Oliver Neukum [EMAIL PROTECTED] writes: Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov:   struct some_descriptor descr;   ...   read_from_wire(descr, sizeof(descr));   do_something_with_values(usb_get_uint16(descr-foo), bar);   ...   usb_set_uint16(descr-foo, 2385);   where:

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 15:04 schrieb Sergei Organov: Unfortunately le16_to_cpu() is not exactly what's needed here, and if I re-implement the above in terms of it: The issues of endianness and alignment are orthogonal. If you face both do: x = le16_to_cpu(get_unaligned(p)); Don't

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

2007-02-07 Thread Greg KH
On Wed, Feb 07, 2007 at 12:23:03PM +0530, Deepak Katagade wrote: Hi All, We have written usb host controller driver for our arm board which does not have any pci interface.We are trying to test the this usb host controller with the usb test driver provided in the linux kernel.We tried to

[linux-usb-devel] Hi

2007-02-07 Thread Gardenia Letcher
Hi, Viuagra - 3.35 Vaulium - 1.25 Ciualis - 3.75 Amabien - 2.90 Souma - 1.15 http://www.zodr!x.com Important: Remove ! in the above link - Using Tomcat but need to do more? Need to support web services, security? Get

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

2007-02-07 Thread Sergei Organov
Oliver Neukum [EMAIL PROTECTED] writes: Am Mittwoch, 7. Februar 2007 15:04 schrieb Sergei Organov: Unfortunately le16_to_cpu() is not exactly what's needed here, and if I re-implement the above in terms of it: The issues of endianness and alignment are orthogonal. If you face both do: x =

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

2007-02-07 Thread Pete Zaitcev
On Wed, 7 Feb 2007 14:39:28 +0100, Oliver Neukum [EMAIL PROTECTED] wrote: Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov:   static inline unsigned usb_get_uint16(__u8 const *p)   {     return p[0] | (p[1] 8);   } It makes little sense to reinvent le16_to_cpu() It would

Re: [linux-usb-devel] hid powersave by autosuspend #2

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Oliver Neukum wrote: Okay, I'll do it. There will also be a per-device delay variable, whose value will be settable via sysfs. The question then becomes, with all those different delay values, what delay should the timer actually use? The simplest answer is to

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Oliver Neukum wrote: Am Mittwoch, 7. Februar 2007 10:54 schrieb Frederic Riss: 2007/2/7, Oliver Neukum [EMAIL PROTECTED]: Am Dienstag, 6. Februar 2007 21:39 schrieb Frédéric RISS: Hi, Is there any way to debug the remote wakeup functionality of usb

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

2007-02-07 Thread Sergei Organov
Pete Zaitcev [EMAIL PROTECTED] writes: On Wed, 7 Feb 2007 14:39:28 +0100, Oliver Neukum [EMAIL PROTECTED] wrote: Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov:   static inline unsigned usb_get_uint16(__u8 const *p)   {     return p[0] | (p[1] 8);   } It makes little

[linux-usb-devel] Bug in usb_match_one_id()?

2007-02-07 Thread Alan Stern
Greg: When you refactored the USB device-matching code, you may have introduced a bug. Does it seem reasonable that an entry might contain both device-specific and interface-specific criteria to match? In which case both sets of matches would have to succeed, not just one. So is the patch

Re: [linux-usb-devel] Bug in usb_match_one_id()?

2007-02-07 Thread Greg KH
On Wed, Feb 07, 2007 at 12:53:31PM -0500, Alan Stern wrote: Greg: When you refactored the USB device-matching code, you may have introduced a bug. Does it seem reasonable that an entry might contain both device-specific and interface-specific criteria to match? I don't know of any such

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

2007-02-07 Thread David Brownell
On Wednesday 07 February 2007 4:25 am, Oliver Neukum wrote: 1. Disallow any padding 2. Assume the structure may be in unaligned memory Issue number #1 is usually handled by the USB specification. Key word usually; they haven't AFAIK promised to do so. #2 is not. Therefore packed data

Re: [linux-usb-devel] [PATCH 1/3]cdc-acm: acm_set_control optional

2007-02-07 Thread David Brownell
On Wednesday 07 February 2007 6:01 am, Oliver Neukum wrote: --- a/drivers/usb/class/cdc-acm.h 2007-01-10 20:10:37.0 +0100 +++ b/drivers/usb/class/cdc-acm.h 2007-02-01 21:06:24.0 +0100 @@ -45,10 +45,13 @@ #define ACM_CTRL_FRAMING 0x10 #define ACM_CTRL_PARITY

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

2007-02-07 Thread David Brownell
On Wednesday 07 February 2007 3:04 am, Phil Endecott wrote: I won't pretend to understand all the subtleties of __attribute__((packed)), but I did learn something about it when I hacked USB/IP to run on ARM, for the NSLU2, last year. The issue boiled down to this: struct S { int

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

2007-02-07 Thread Phil Endecott
David Brownell wrote: On Wednesday 07 February 2007 3:04 am, Phil Endecott wrote: I won't pretend to understand all the subtleties of __attribute__((packed)), but I did learn something about it when I hacked USB/IP to run on ARM, for the NSLU2, last year. The issue boiled down to this:

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

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Sergei Organov wrote: In fact if there were macro/function similar to get_unaligned(), but converting from little-/big-endian to native, say, get_unaligned_le(), I'd use it: ... Unfortunately there is no such macro, or did I miss it? I have often thought exactly the

Re: [linux-usb-devel] Bug in usb_match_one_id()?

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Greg KH wrote: On Wed, Feb 07, 2007 at 12:53:31PM -0500, Alan Stern wrote: Greg: When you refactored the USB device-matching code, you may have introduced a bug. Does it seem reasonable that an entry might contain both device-specific and interface-specific

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

2007-02-07 Thread Sergei Organov
David Brownell [EMAIL PROTECTED] writes: On Tuesday 06 February 2007 2:48 pm, Pete Zaitcev wrote: On Tue, 6 Feb 2007 13:08:19 -0800, Inaky Perez-Gonzalez [EMAIL PROTECTED] wrote: [btw, I truly have little idea about which are those specific costs, out of professional curiosity, got any

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

2007-02-07 Thread David Brownell
On Wednesday 07 February 2007 10:41 am, Phil Endecott wr Err, maybe my example was over-simplified then, sorry. Try another level of indirection: void inc(int* i) { (*i)++; } void f(struct S* s) { inc((s-intfield); One would expect that to generate a warning ... that you're

Re: [linux-usb-devel] hid powersave by autosuspend #2

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 18:40 schrieb Alan Stern: On Wed, 7 Feb 2007, Oliver Neukum wrote: So the answer would not be the largest delay, but the largest delay currently requested to be active. We haven't yet defined how a driver can request an interface delay to be active or

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 18:45 schrieb Alan Stern: On Wed, 7 Feb 2007, Oliver Neukum wrote: If the device is not bound to a driver, that is ignored. Not so. The attribute is checked for every device that gets suspended. Of course, the device itself may choose not to report wakeup

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 19:23 schrieb David Brownell: On Wednesday 07 February 2007 4:25 am, Oliver Neukum wrote: 1. Disallow any padding 2. Assume the structure may be in unaligned memory Issue number #1 is usually handled by the USB specification. Key word usually; they

Re: [linux-usb-devel] [PATCH 1/3]cdc-acm: acm_set_control optional

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 19:25 schrieb David Brownell: On Wednesday 07 February 2007 6:01 am, Oliver Neukum wrote: --- a/drivers/usb/class/cdc-acm.h 2007-01-10 20:10:37.0 +0100 +++ b/drivers/usb/class/cdc-acm.h 2007-02-01 21:06:24.0 +0100 @@ -45,10 +45,13 @@

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 19:47 schrieb Alan Stern: On Wed, 7 Feb 2007, Sergei Organov wrote: In fact if there were macro/function similar to get_unaligned(), but converting from little-/big-endian to native, say, get_unaligned_le(), I'd use it: ... Unfortunately there is no

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 18:31 schrieb Pete Zaitcev: On Wed, 7 Feb 2007 14:39:28 +0100, Oliver Neukum [EMAIL PROTECTED] wrote: Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov: static inline unsigned usb_get_uint16(__u8 const *p) { return p[0] | (p[1] 8); }

Re: [linux-usb-devel] Howto debug failure to use remote wakup?

2007-02-07 Thread Frédéric Riss
Le mercredi 07 février 2007 à 13:10 +0100, Oliver Neukum a écrit : Am Mittwoch, 7. Februar 2007 11:31 schrieb Frederic Riss: 2007/2/7, Oliver Neukum [EMAIL PROTECTED]: As I wrote it's seen by Linux as a raw HID device, thus it's bound to the usbhid driver. IIRC there're 2 sorts of HID

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

2007-02-07 Thread Phil Endecott
Just to throw one other factor into the discussion: if you add something like __attribute__((packed)) to the definition of a struct in an include file (e.g. ch9.h), it will affect third-party user-land code that happens to #include it. People writing such code probably don't get any benefit

[linux-usb-devel] Fw: [Bugme-new] [Bug 7960] New: kernel panic when inserting usb isdn modem

2007-02-07 Thread Andrew Morton
whee.. Begin forwarded message: Date: Wed, 7 Feb 2007 12:15:21 -0800 From: [EMAIL PROTECTED] To: [EMAIL PROTECTED] Subject: [Bugme-new] [Bug 7960] New: kernel panic when inserting usb isdn modem http://bugzilla.kernel.org/show_bug.cgi?id=7960 Summary: kernel panic when inserting

[linux-usb-devel] gcc's packed vs hardware (Re: PATCH: usb: descriptor structures have to be packed)

2007-02-07 Thread Oleg Verych
On Tue, Feb 06, 2007 at 11:00:14PM -0800, David Brownell wrote: On Tuesday 06 February 2007 5:58 pm, Oleg Verych wrote: Would you clarify non aligned access issue on well known platforms (*x86-*). I see on gcc's output, that they are not carry much about it: X86 has hardware logic to

Re: [linux-usb-devel] hid powersave by autosuspend #2

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Oliver Neukum wrote: Am Mittwoch, 7. Februar 2007 18:40 schrieb Alan Stern: On Wed, 7 Feb 2007, Oliver Neukum wrote: So the answer would not be the largest delay, but the largest delay currently requested to be active. We haven't yet defined how a driver can

Re: [linux-usb-devel] gcc's packed vs hardware

2007-02-07 Thread Phil Endecott
Oleg Verych [EMAIL PROTECTED] wrote: And what Phil Endecott told is just a bug in GCC! I don't know if he tried with and without optimization. Oleg, I'm just a messenger. I'm just saying that I've encountered these things before, and I've asked the gcc experts, and I've told you what they

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

2007-02-07 Thread Alan Stern
On Wed, 7 Feb 2007, Oliver Neukum wrote: I have often thought exactly the same thing. It seems so natural to combine unaligned reads and writes with byte-reordering. I even brought up the issue on LKML: Why shouldn't there be architecture-dependent versions of

[linux-usb-devel] [PATCH] EHCI: disable wakeup on all ports during shutdown

2007-02-07 Thread Alan Stern
This patch (as850) disables remote wakeup on EHCI ports when the shutdown() method is called. If it is left active then some systems will reboot instead of powering off. This solves Bugzilla #7828. Signed-off-by: Alan Stern [EMAIL PROTECTED] --- Index: usb-2.6/drivers/usb/host/ehci-hcd.c

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

2007-02-07 Thread Phil Endecott
David Brownell wrote: On Wednesday 07 February 2007 10:41 am, Phil Endecott wr void inc(int* i) { (*i)++; } here's a quote from Paul Brook (http://gcc.gnu.org/ml/gcc-help/2006-12/msg00115.html): the compiler is allowed to assume that the low 2 bits of an int* are zero. Unless

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

2007-02-07 Thread Oleg Verych
From: Phil Endecott [EMAIL PROTECTED] Newsgroups: gmane.linux.usb.devel Subject: Re: usb: descriptor structures have to be packed Date: Wed, 07 Feb 2007 22:14:47 + Just question. void inc(int __attribute__((aligned(1))) * i) void inc(int * __attribute__((aligned(1))) i) void inc(int *

Re: [linux-usb-devel] hid powersave by autosuspend #2

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 22:44 schrieb Alan Stern: On Wed, 7 Feb 2007, Oliver Neukum wrote: Am Mittwoch, 7. Februar 2007 18:40 schrieb Alan Stern: On Wed, 7 Feb 2007, Oliver Neukum wrote: So the answer would not be the largest delay, but the largest delay currently requested

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

2007-02-07 Thread David Brownell
On Wednesday 07 February 2007 2:14 pm, Phil Endecott wrote: David Brownell wrote: On Wednesday 07 February 2007 10:41 am, Phil Endecott wr void inc(int* i) { (*i)++; } here's a quote from Paul Brook (http://gcc.gnu.org/ml/gcc-help/2006-12/msg00115.html): the compiler is

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

2007-02-07 Thread Pete Zaitcev
On Wed, 7 Feb 2007 21:26:36 +0100, Oliver Neukum [EMAIL PROTECTED] wrote: It would not, if le16_to_cpu took a pointer to u8. But as it is, functions are quite different. That's what casts are for. However, why do you have no type information in the first place? Untyped data is risky. You

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

2007-02-07 Thread Oliver Neukum
Am Mittwoch, 7. Februar 2007 23:45 schrieb Pete Zaitcev: On Wed, 7 Feb 2007 21:26:36 +0100, Oliver Neukum [EMAIL PROTECTED] wrote: It would not, if le16_to_cpu took a pointer to u8. But as it is, functions are quite different. That's what casts are for. However, why do you have no

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

2007-02-07 Thread Phil Endecott
Oleg Verych wrote: From: Phil Endecott [EMAIL PROTECTED] Just question. void inc(int __attribute__((aligned(1))) * i) void inc(int * __attribute__((aligned(1))) i) void inc(int * i __attribute__((aligned(1 Why 1, and not 2? 1 is the worst unalignment. If I specify 2 I get the same

Re: [linux-usb-devel] [PATCH 1/2] USB: add Freescale high-speed USB SOC device controller driver

2007-02-07 Thread Kumar Gala
On Mon, 5 Feb 2007, Li Yang wrote: Freescale high-speed USB SOC can be found on some Freescale processors among different architectures. It supports both host and device functions. This driver adds its device support for Linux USB Gadget layer. It is tested for MPC834x DR module, but should

[linux-usb-devel] [GIT PATCH] USB patches for 2.6.20

2007-02-07 Thread Greg KH
Here are a bunch of USB patches against 2.6.20-git They include some new drivers (including PS3 support), some new features (dynamic id support for usb-serial drivers), and loads of other stuff. All of these have been in the -mm releases for quite some time. Please pull from:

[linux-usb-devel] [PATCH 1/70] USB: unusual_devs.h for Sony floppy

2007-02-07 Thread Greg KH
From: Luiz Fernando N. Capitulino [EMAIL PROTECTED] This patch increases the range for 0x054c:0x002c devices to make the following Sony USB floppy to work: T: Bus=02 Lev=01 Prnt=01 Port=02 Cnt=02 Dev#= 6 Spd=12 MxCh= 0 D: Ver= 1.10 Cls=00(ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1 P:

[linux-usb-devel] [PATCH 2/70] USB: add EPIC support to the io_edgeport driver

2007-02-07 Thread Greg KH
From: Greg Kroah-Hartman [EMAIL PROTECTED] This patch adds EPiC support to the io_edgeport driver which adds support for a number of NCR printers: - NCR (Axiohm) 7401-K580 printer - NCR (TEC) 7401-K590 printer, 7402-K592 - NCR (TEC) 7167, 7168 printers - NCR (TEC)

[linux-usb-devel] [PATCH 3/70] USB: move usb_device_class class devices to be real devices

2007-02-07 Thread Greg KH
From: Greg Kroah-Hartman [EMAIL PROTECTED] This moves the usb class devices that control the usbfs nodes to show up in the proper place in the larger device tree. Cc: Kay Sievers [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/core/devio.c | 24

[linux-usb-devel] [PATCH 4/70] USB: convert usb class devices to real devices

2007-02-07 Thread Greg KH
From: Greg Kroah-Hartman [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/core/file.c | 13 ++--- include/linux/usb.h |5 +++-- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/usb/core/file.c b/drivers/usb/core/file.c

[linux-usb-devel] [PATCH 5/70] USB: Rework the OHCI quirk mecanism as suggested by David

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] This patch applies David Brownell's suggestion for reworking the OHCI quirk mechanism via a table of PCI IDs. It adapts the existing quirks to use that mechanism. This also moves the quirks to reset() as suggested by the comment in there. This is

[linux-usb-devel] [PATCH 6/70] USB: Implement support for split endian OHCI

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] This patch separates support for big endian MMIO register access and big endian descriptors in order to support the Toshiba SCC implementation which has big endian registers but little endian in-memory descriptors. It simplifies the access functions

[linux-usb-devel] [PATCH 8/70] USB: Fix OHCI warning

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] This patch fixes a warning introduces by the split endian OHCI support patch on platforms that don't have readl_be/writel_be variants (though mostly harmless as those are called in an if (0) statement, but gcc still warns). Signed-off-by: Benjamin

[linux-usb-devel] [PATCH 9/70] USB: Fix EHCI warning

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] This patch fixes a warning introduced by the big endian MMIO EHCI support patch on platforms that don't have readl_be/writel_be variants (though mostly harmless as those are called in an if (0) statement, but gcc still warns). Signed-off-by:

[linux-usb-devel] [PATCH 10/70] USB: linux/usb_ch9.h becomes linux/usb/ch9.h

2007-02-07 Thread Greg KH
From: David Brownell [EMAIL PROTECTED] This moves linux/usb_ch9.h to linux/usb/ch9.h to reduce some of the clutter of usb header files. Signed-off-by: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/i2c/chips/isp1301_omap.c |2 +-

[linux-usb-devel] [PATCH 7/70] USB: Implement support for EHCI with big endian MMIO

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] This patch implements supports for EHCI controllers whose MMIO registers are big endian and enables that functionality for the Toshiba SCC chip. It does _not_ add support for big endian in-memory data structures as this is not needed for that chip

[linux-usb-devel] [PATCH 11/70] USB: define USB_CLASS_MISC in linux/usb/ch9.h

2007-02-07 Thread Greg KH
From: David Brownell [EMAIL PROTECTED] Add USB_CLASS_MISC to linux/usb/ch9.h Signed-off-by: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- include/linux/usb/ch9.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

[linux-usb-devel] [PATCH 13/70] USB: mutexification of rio500

2007-02-07 Thread Greg KH
From: Oliver Neukum [EMAIL PROTECTED] this makes the rio500 misc usb driver use mutexes and turns uninterruptible sleep into interruptible sleep where the semantics are not affected. Signed-off-by: Oliver Neukum [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] ---

[linux-usb-devel] [PATCH 14/70] USB: devio.c add missing INIT_LIST_HEAD()

2007-02-07 Thread Greg KH
From: Dan Carpenter [EMAIL PROTECTED] It should hopefully fix the list corruption bug on: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214402 Add a missing INIT_LIST_HEAD() Signed-off-by: Dan Carpenter [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] ---

[linux-usb-devel] [PATCH 15/70] USB: indicate active altsetting in proc/bus/usb/devices file

2007-02-07 Thread Greg KH
From: David Brownell [EMAIL PROTECTED] Update /proc/bus/usb/devices output to report active altsettings. Signed-off-by: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- Documentation/usb/proc_usb_info.txt | 21 +++--

[linux-usb-devel] [PATCH 12/70] USB: Remove unneeded void * casts in idmouse.c

2007-02-07 Thread Greg KH
From: Tobias Klauser [EMAIL PROTECTED] The patch removes unneeded void * casts for the following (void *) pointers: - struct file: private_data The patch also contains some whitespace and coding style cleanups in the relevant areas. Signed-off-by: Tobias Klauser [EMAIL PROTECTED] Signed-off-by:

[linux-usb-devel] [PATCH 16/70] usbcore: remove unneeded error check

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as830) removes some unnecessary error checking. According to the kerneldoc, schedule_work() can't fail. Signed-off-by: Alan Stern [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/core/message.c |6 +- 1

[linux-usb-devel] [PATCH 17/70] USB: ethernet gadget interop with MCCI Windows driver

2007-02-07 Thread Greg KH
From: David Brownell [EMAIL PROTECTED] It turns out that minor tweaks to the CDC Subset support in the Ethernet gadget driver, just updating a config descriptor, let it be automagically recognized by a Windows driver supported by MCCI. This patch adds those descriptors, so systems using PXA 255

[linux-usb-devel] [PATCH 18/70] rndis_host learns ActiveSync basics

2007-02-07 Thread Greg KH
From: Ole Andre Vadla Ravnas [EMAIL PROTECTED] Windows Mobile 5 based devices described as supporting ActiveSync: - Speak RNDIS but lack the CDC and union descriptors. This patch updates the cdc ethernet code to fake ACM descriptors we need. - Require RNDIS_MSG_QUERY messages to include a

[linux-usb-devel] [PATCH 21/70] USB serial: add dynamic id support to usb-serial core

2007-02-07 Thread Greg KH
From: Greg Kroah-Hartman [EMAIL PROTECTED] Thanks to Johannes Hölzl [EMAIL PROTECTED] for fixing a few things and getting it all working properly. This adds support for dynamic usb ids to the usb serial core. The file new_id will show up under the usb serial driver, not the usb driver

[linux-usb-devel] [PATCH 20/70] ohci: Add support for OHCI controller on the of_platform bus

2007-02-07 Thread Greg KH
From: Sylvain Munaut [EMAIL PROTECTED] PPC embedded systems can have a ohci controller builtin. In the new model, it will end up as a driver on the of_platform bus, this patches takes care of them. Signed-off-by: Sylvain Munaut [EMAIL PROTECTED] Acked-by: David Brownell [EMAIL PROTECTED]

[linux-usb-devel] [PATCH 19/70] ohci: Rework bus glue integration to allow several at once

2007-02-07 Thread Greg KH
From: Sylvain Munaut [EMAIL PROTECTED] The previous model had the module_init module_exit function in the bus glue .c files themselves. That's a problem if several glues need to be selected at once and the driver is built has module. This case is quite common in embedded system where you want to

[linux-usb-devel] [PATCH 23/70] USB: Bugfix for aircable: Add module and name to usb_serial_driver

2007-02-07 Thread Greg KH
From: =?utf-8?q?Johannes_H=C3=B6lzl?= [EMAIL PROTECTED] While adding the dynamic-id support to usb serial I found a small bug in the air cable driver: Adds module and name information to the usb_serial_driver instance of aircable. So the aircable driver is correctly shown under

[linux-usb-devel] [PATCH 22/70] USB serial: add driver pointer to all usb-serial drivers

2007-02-07 Thread Greg KH
From: =?utf-8?q?Johannes_H=C3=B6lzl?= [EMAIL PROTECTED] Every usb serial driver should have a pointer to the corresponding usb driver. So the usb serial core can add a new id not only to the usb serial driver, but also to the usb driver. Also the usb drivers of ark3116, mos7720 and mos7840

[linux-usb-devel] [PATCH 24/70] USB Gadget file_storage.c: remove unnecessary casts

2007-02-07 Thread Greg KH
From: John Daiker [EMAIL PROTECTED] Went looking through some usb stuff and found some unnecessary casts in file_storage.c This is part of the KernelJanitors TODO list. Signed-off-by: John Daiker [EMAIL PROTECTED] Acked-by: Alan Stern [EMAIL PROTECTED] Signed-off-by: David Brownell [EMAIL

[linux-usb-devel] [PATCH 25/70] USB: Add usb_endpoint_xfer_control to usb.h

2007-02-07 Thread Greg KH
From: Sarah Bailey [EMAIL PROTECTED] Added a function to check if an endpoint is a control endpoint. There were similar functions for bulk, interrupt, and isoc, but not for control endpoints. Signed-off-by: Sarah Bailey [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] ---

[linux-usb-devel] [PATCH 26/70] USB: add binary API to usbmon

2007-02-07 Thread Greg KH
From: Pete Zaitcev [EMAIL PROTECTED] This patch adds a new, binary API in addition to the old, text API usbmon had before. The new API allows for less CPU use, and it allows to capture all data from a packet where old API only captured 32 bytes at most. There are some limitations and conditions

[linux-usb-devel] [PATCH 27/70] USB: race on disconnect in mdc800

2007-02-07 Thread Greg KH
From: Oliver Neukum [EMAIL PROTECTED] I overlooked one. Setting the flag and killing the URBs must be under the lock so that no URB is submitted after usb_kill_urb() Signed-off-by: Oliver Neukum [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/image/mdc800.c

[linux-usb-devel] [PATCH 28/70] UHCI: improved debugging checks for the frame list

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as768) improves the debugging checks for the uhci-hcd frame list. The number of entries displayed is limited to 10, and the driver now checks for the correct Skeleton QH link value at the end of each chain of Isochronous TDs. The code to compute

[linux-usb-devel] [PATCH 29/70] UHCI: no dummy TDs for Iso QHs

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] Isochronous queues don't need a dummy TD because the Queue Header isn't managed by the hardware. This patch (as836) removes the unnecessary dummy TDs. The patch also fixes a long-standing typo in a comment (a don't was missing -- potentially very confusing!).

[linux-usb-devel] [PATCH 30/70] usb-storage: SCSI level fixes

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as835) removes from usb-storage the code which sets all devices to a SCSI level of at least SCSI-2. The original reasons for doing this no longer apply, and in fact it prevents certain kinds of ATA pass-thru commands from being used. The patch also

[linux-usb-devel] [PATCH 32/70] usb: gadgetfs whitespace cleanup

2007-02-07 Thread Greg KH
From: David Brownell [EMAIL PROTECTED] Remove some whitespace bugs in gadgetfs (mostly from someone's patch updating the AIO support). Signed-off-by: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/gadget/inode.c | 56

[linux-usb-devel] [PATCH 34/70] USB: power management for kaweth

2007-02-07 Thread Greg KH
From: Oliver Neukum [EMAIL PROTECTED] - implements suspend when the network interface is down - fixes a typo in comments - adds debugging output for power management - fixes a compiler warning Signed-off-by: Oliver Neukum [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] ---

[linux-usb-devel] [PATCH 35/70] USB: better ethtool support for kaweth

2007-02-07 Thread Greg KH
From: Oliver Neukum [EMAIL PROTECTED] this implements enough ethtool support to make NetworkManager happy. Signed-off-by: Oliver Neukum [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/net/kaweth.c | 12 +++- 1 files changed, 11 insertions(+), 1

[linux-usb-devel] [PATCH 33/70] usb: gadgetfs remove delayed init mode

2007-02-07 Thread Greg KH
From: Phil Endecott [EMAIL PROTECTED] Gadgetfs had a mode in which endpoint descriptors were written by the user program before connection. This mode had some bugs, and hasn't seen much (if any) use. This patch removes that mode, leaving the mode of operation where the user program waits for

[linux-usb-devel] [PATCH 31/70] USB: ohci-at91 refcount fix for irq wake enables

2007-02-07 Thread Greg KH
From: Marc Pignat [EMAIL PROTECTED] The attached patch fixes the unbalanced calls to enable_irq_wake() and disable_irq_wake() in the AT91 USB Host driver. It should resolve these kernel messages: Unbalanced IRQ x wake disable BUG: warning at kernel/irq/manage.c:167/set_irq_wake() (The

[linux-usb-devel] [PATCH 38/70] USB: ohci error handling cleanup

2007-02-07 Thread Greg KH
From: Benjamin Herrenschmidt [EMAIL PROTECTED] Restructure the ohci_hcd_mod_init error handling code in to better support the multiple platform drivers. This does not change the functionality. Signed-off-by: Benjamin Herrenschmidt [EMAIL PROTECTED] Cc: David Brownell [EMAIL PROTECTED]

[linux-usb-devel] [PATCH 37/70] USB: ps3 controller hid quirk

2007-02-07 Thread Greg KH
From: Geoff Levand [EMAIL PROTECTED] Add the USB HID quirk HID_QUIRK_SONY_PS3_CONTROLLER. This sends an HID_REQ_GET_REPORT to the the PS3 controller to put the device into 'operational mode'. Signed-off-by: Geoff Levand [EMAIL PROTECTED] Cc: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg

[linux-usb-devel] [PATCH 36/70] USB: ps3 ehci bus glue

2007-02-07 Thread Greg KH
From: Geoff Levand [EMAIL PROTECTED] USB EHCI driver bus glue for the PS3 game console. Signed-off-by: Geoff Levand [EMAIL PROTECTED] Cc: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- arch/powerpc/Kconfig|2 + drivers/usb/host/ehci-hcd.c |

[linux-usb-devel] [PATCH 39/70] USB: ps3 ohci bus glue

2007-02-07 Thread Greg KH
From: Geoff Levand [EMAIL PROTECTED] USB OHCI driver bus glue for the PS3 game console. Signed-off-by: Geoff Levand [EMAIL PROTECTED] Cc: David Brownell [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- arch/powerpc/Kconfig|3 + drivers/usb/host/ohci-hcd.c |

[linux-usb-devel] [PATCH 40/70] UHCI: fix bandwidth allocation

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as840) fixes the bandwidth allocation mechanism in uhci-hcd. It has never worked correctly. Signed-off-by: Alan Stern [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/usb/host/uhci-debug.c | 21 +++-

[linux-usb-devel] [PATCH 41/70] usbcore: remove unused bandwith-related code

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as841) removes from usbcore a couple of support routines meant to help with bandwidth allocation. With the changes to uhci-hcd in the previous patch, these routines are no longer used anywhere. Also removed is the CONFIG_USB_BANDWIDTH option; it no

[linux-usb-devel] [PATCH 42/70] EHCI: local variable for port status register

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as708) introduces a local variable to hold the port status-register address in ehci-hub.c. There's not much improvement in the object code, but it sure is a lot easier to read. Signed-off-by: Alan Stern [EMAIL PROTECTED] Cc: David Brownell [EMAIL

[linux-usb-devel] [PATCH 43/70] EHCI: don't hide ports owned by the companion

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as709) changes the way ehci-hcd presents port status values for ports owned by the companion controller. It no longer hides the information; in particular, it allows the core to see the disconnect event that occurs when a full- or low-speed device

[linux-usb-devel] [PATCH 44/70] EHCI: force high-speed devices to run at full speed

2007-02-07 Thread Greg KH
From: Alan Stern [EMAIL PROTECTED] This patch (as710) adds a sysfs class-device attribute file named companion for EHCI controllers. The file contains a list of port numbers that are dedicated to the companion controller; by writing a port number to the file the user can force a high-speed

  1   2   >