Re: [linux-usb-devel] ti_usb_3410_5052 serial driver (again)

2007-05-30 Thread Oleg Verych
On Wed, May 30, 2007 at 11:43:16PM -0500, Al Borchers wrote: > Oleg -- > > Quoting Oleg Verych <[EMAIL PROTECTED]>: > > 1) sysfsless usb-reconfiguration > > I will test you patch again with the current kernel. We can > make it work, patching usb-serial if needed. It would be great to have it in

Re: [linux-usb-devel] ti_usb_3410_5052 serial driver (again)

2007-05-30 Thread Al Borchers
Oleg -- Quoting Oleg Verych <[EMAIL PROTECTED]>: > 1) sysfsless usb-reconfiguration I will test you patch again with the current kernel. We can make it work, patching usb-serial if needed. > 2) fw loader. I have a patch from Multitech with support for new devices and the firmware loader as an

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Roland Dreier
> > If I just see > > > >for (pos = list_entry((head)->next, typeof(*pos), member), > >n = list_entry(pos->member.next, typeof(*pos), member); > > &pos->member != (head); > > pos = n, n = list_entry(n->member.next, typeof(*n), member)) > > > > then what am

[linux-usb-devel] register

2007-05-30 Thread 张红(163)
-- 张红(163) 2007-05-31 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Cl

Re: [linux-usb-devel] [PATCH 4/6] USB: add reset_resume method

2007-05-30 Thread Greg KH
On Thu, May 31, 2007 at 01:55:43AM +0200, Jiri Kosina wrote: > On Wed, 30 May 2007, Alan Stern wrote: > > > In addition, the pre_reset and post_reset method return types are > > changed; they now must return an error code. The return value is unused > > at present, but at some later time we may

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Pete Zaitcev
On Wed, 30 May 2007 16:42:41 -0700, Roland Dreier <[EMAIL PROTECTED]> wrote: > If I just see > > for (pos = list_entry((head)->next, typeof(*pos), member), > n = list_entry(pos->member.next, typeof(*pos), member); >&pos->member != (head); >pos = n, n =

Re: [linux-usb-devel] [PATCH 4/6] USB: add reset_resume method

2007-05-30 Thread Jiri Kosina
On Wed, 30 May 2007, Alan Stern wrote: > In addition, the pre_reset and post_reset method return types are > changed; they now must return an error code. The return value is unused > at present, but at some later time we may unbind drivers and re-probe if > they encounter an error during reset

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Roland Dreier
> > > The negative is the sheer number of helper functions in list.h. > > > Personally, > > > I find it difficult to retain a working knowledge of them. Iterators are > > > particularly nasty that way. I'm thinking about dropping all of these > > > list_for_each_with_murky_argument_requiremen

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Pete Zaitcev
On Wed, 30 May 2007 16:14:01 -0700, Roland Dreier <[EMAIL PROTECTED]> wrote: > > The negative is the sheer number of helper functions in list.h. Personally, > > I find it difficult to retain a working knowledge of them. Iterators are > > particularly nasty that way. I'm thinking about dropping

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Roland Dreier
> The negative is the sheer number of helper functions in list.h. Personally, > I find it difficult to retain a working knowledge of them. Iterators are > particularly nasty that way. I'm thinking about dropping all of these > list_for_each_with_murky_argument_requirements_and_odd_side_effects(

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Pete Zaitcev
On Thu, 31 May 2007 02:44:17 +0530, "Satyam Sharma" <[EMAIL PROTECTED]> wrote: > > > - list_for_each(p, &sc->luns) { > > > - lun = list_entry(p, struct ub_lun, link); > > > + list_for_each_entry(lun, &sc->luns, link) { > > This patch straddles the border of acceptable. The poi

Re: [linux-usb-devel] [PATCH 6/6] UNUSUAL_DEV: Sync up some reported devices from Ubuntu

2007-05-30 Thread Phil Dibowitz
On Wed, May 30, 2007 at 05:50:28PM -0400, Dave Jones wrote: > On Wed, May 23, 2007 at 06:39:24PM -0400, Ben Collins wrote: > > Various unusual dev entries accumulated from Ubuntu bug reports. > > > > CC: Phil Dibowitz <[EMAIL PROTECTED]> > > CC: Alan Stern <[EMAIL PROTECTED]> > > CC: linux-us

Re: [linux-usb-devel] [PATCH 6/6] UNUSUAL_DEV: Sync up some reported devices from Ubuntu

2007-05-30 Thread Dave Jones
On Wed, May 23, 2007 at 06:39:24PM -0400, Ben Collins wrote: > Various unusual dev entries accumulated from Ubuntu bug reports. > > CC: Phil Dibowitz <[EMAIL PROTECTED]> > CC: Alan Stern <[EMAIL PROTECTED]> > CC: linux-usb-devel@lists.sourceforge.net > Signed-off-by: Ben Collins <[EMAIL PROT

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Satyam Sharma
Hi Pete, On 5/31/07, Pete Zaitcev <[EMAIL PROTECTED]> wrote: > On Wed, 30 May 2007 10:47:52 +0200, Matthias Kaehlcke <[EMAIL PROTECTED]> > wrote: > > > @@ -1608,8 +1605,7 @@ static void ub_reset_task(struct work_struct *work) > > spin_lock_irqsave(sc->lock, flags); > > sc->reset = 0;

[linux-usb-devel] [PATCH 1b/6] USB: remove __usb_port_suspend

2007-05-30 Thread Alan Stern
This patch (as915b) combines the public routine usb_port_suspend() and the private routine __usb_port_suspend() into a single function. By removing the explicit mention of otg_port in the call to __usb_port_suspend(), we prevent a possible error in which the system tries to perform HNP on the wron

Re: [linux-usb-devel] [PATCH 1/6] USB: remove __usb_port_suspend

2007-05-30 Thread Alan Stern
On Wed, 30 May 2007, David Brownell wrote: > > According to the original code, plugging a non-targeted device into > > a non-OTG port would cause the host to initiate HNP on the OTG port! > > At least, that's what it looks like to me. Have I got it wrong? > > Yeah, I thought I noticed that to

Re: [linux-usb-devel] [PATCH 1/6] USB: remove __usb_port_suspend

2007-05-30 Thread David Brownell
On Wednesday 30 May 2007, Alan Stern wrote: > On Wed, 30 May 2007, David Brownell wrote: > > > On Wednesday 30 May 2007, Alan Stern wrote: > > > > > > Propagating the change into the OTG HNP handler requires a slight > > > addition to the OTG whitelist checking code: A device not attached to > >

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Matthias Kaehlcke
El Wed, May 30, 2007 at 12:38:40PM -0700 Pete Zaitcev ha dit: > On Wed, 30 May 2007 10:47:52 +0200, Matthias Kaehlcke <[EMAIL PROTECTED]> > wrote: > > > @@ -1608,8 +1605,7 @@ static void ub_reset_task(struct work_struct *work) > > spin_lock_irqsave(sc->lock, flags); > > sc->reset = 0; >

Re: [linux-usb-devel] [PATCH 1/6] USB: remove __usb_port_suspend

2007-05-30 Thread Alan Stern
On Wed, 30 May 2007, David Brownell wrote: > On Wednesday 30 May 2007, Alan Stern wrote: > > > > Propagating the change into the OTG HNP handler requires a slight > > addition to the OTG whitelist checking code: A device not attached to > > the OTG port should always count as "targeted". > > No,

Re: [linux-usb-devel] [PATCH 1/6] USB: remove __usb_port_suspend

2007-05-30 Thread David Brownell
On Wednesday 30 May 2007, Alan Stern wrote: > > Propagating the change into the OTG HNP handler requires a slight > addition to the OTG whitelist checking code: A device not attached to > the OTG port should always count as "targeted". No, that's not what the OTG spec says. If there's a list of

[linux-usb-devel] [PATCH 6/6] USB: add power/persist device attribute

2007-05-30 Thread Alan Stern
This patch (as920) adds an extra level of protection to the USB-Persist facility. Now it will apply by default only to hubs; for all other devices the user must enable it explicitly by setting the power/persist device attribute. The disconnect_all_children() routine in hub.c has been removed and

[linux-usb-devel] [PATCH 5/6] USB: unify reset_resume and normal resume

2007-05-30 Thread Alan Stern
This patch (as919) unifies the code paths used for normal resume and for reset-resume. Earlier I had failed to note a section in the USB spec which requires the host to resume a suspended port before resetting it if the attached device is enabled for remote wakeup. Since the port has to be resumed

Re: [linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Pete Zaitcev
On Wed, 30 May 2007 10:47:52 +0200, Matthias Kaehlcke <[EMAIL PROTECTED]> wrote: > @@ -1608,8 +1605,7 @@ static void ub_reset_task(struct work_struct *work) > spin_lock_irqsave(sc->lock, flags); > sc->reset = 0; > tasklet_schedule(&sc->tasklet); > - list_for_each(p, &sc->luns

[linux-usb-devel] [PATCH 4/6] USB: add reset_resume method

2007-05-30 Thread Alan Stern
This patch (as918) introduces a new USB driver method: reset_resume. It is called when a device needs to be reset as part of a resume procedure (whether because of a device quirk or because of the USB-Persist facility), thereby taking over a role formerly assigned to the post_reset method. As a co

[linux-usb-devel] [PATCH 3/6] USB: remove excess code from hub.c

2007-05-30 Thread Alan Stern
This patch (as917) removes a now-unnecessary level of subroutine nesting from hub.c. Since usb_port_suspend() does nothing but call hub_port_suspend(), and usb_port_resume() does nothing but call hub_port_resume(), there's no reason to keep the routines separate. Also included in the patch are a

[linux-usb-devel] [PATCH 2/6] USB: separate root and non-root suspend/resume

2007-05-30 Thread Alan Stern
This patch (as916) completes the separation of code paths for suspend and resume of root hubs as opposed to non-root devices. Root hubs will be power-managed through their bus_suspend and bus_resume methods, whereas normal devices will use usb_port_suspend() and usb_port_resume(). Changes to the

[linux-usb-devel] [PATCH 1/6] USB: remove __usb_port_suspend

2007-05-30 Thread Alan Stern
This patch (as915) combines the public routine usb_port_suspend() and the private routine __usb_port_suspend() into a single function. Propagating the change into the OTG HNP handler requires a slight addition to the OTG whitelist checking code: A device not attached to the OTG port should always c

[linux-usb-devel] [PATCH 0/6] Update reset-resume and USB-Persist handling

2007-05-30 Thread Alan Stern
The following series of patches makes several improvements to the reset-resume and USB-persist implementations, as discussed online during the past few weeks. Among the changes: A new reset_resume method is added, in place of overloading the post_reset method. Drivers can

Re: [linux-usb-devel] autosuspend for HID devices, take #5

2007-05-30 Thread Oliver Neukum
Am Mittwoch, 30. Mai 2007 17:25 schrieb Jiri Kosina: > On Wed, 23 May 2007, Oliver Neukum wrote: > > > del_timer_sync(&usbhid->io_retry); > > + cancel_work_sync(&usbhid->restart_work); > > flush_scheduled_work(); > > Hi Oliver, > > isn't the call to flush_scheduled_work() after cancel_

Re: [linux-usb-devel] autosuspend for HID devices, take #5

2007-05-30 Thread Oliver Neukum
Am Mittwoch, 30. Mai 2007 17:25 schrieb Jiri Kosina: > On Wed, 23 May 2007, Oliver Neukum wrote: > > > del_timer_sync(&usbhid->io_retry); > > + cancel_work_sync(&usbhid->restart_work); > > flush_scheduled_work(); > > Hi Oliver, > > isn't the call to flush_scheduled_work() after cancel_

Re: [linux-usb-devel] Dealing with flaky USB storage devices and rootfs

2007-05-30 Thread Alan Stern
On Wed, 30 May 2007, Dan Aloni wrote: > On Tue, May 29, 2007 at 05:50:49PM -0400, Alan Stern wrote: > > On Tue, 29 May 2007, Dan Aloni wrote: > > > > > Hello, > > > > > > We have a system where the rootfs is a partition on a USB device, > > > and I've noticed upon a few rare cases where the USB

Re: [linux-usb-devel] autosuspend for HID devices, take #5

2007-05-30 Thread Jiri Kosina
On Wed, 23 May 2007, Oliver Neukum wrote: > del_timer_sync(&usbhid->io_retry); > + cancel_work_sync(&usbhid->restart_work); > flush_scheduled_work(); Hi Oliver, isn't the call to flush_scheduled_work() after cancel_work_sync() redundant? Also, we could very probably use the hot

[linux-usb-devel] [PATCH] USBHID: avoid flush_scheduled_work()

2007-05-30 Thread Alan Stern
This patch (as914) replaces a call to flush_scheduled_work() with cancel_work_sync(), in order to help avoid potential deadlocks. Signed-off-by: Alan Stern <[EMAIL PROTECTED]> --- Index: usb-2.6/drivers/hid/usbhid/hid-core.c === ---

[linux-usb-devel] [PATCH] g_file_storage: call allow_signal()

2007-05-30 Thread Alan Stern
From: Oleg Nesterov <[EMAIL PROTECTED]> New changes in the signal-handling code require compensating changes in g_file_storage. This patch (as913) by Oleg Nesterov makes the code use allow_signal() instead of sigprocmask(). Signed-off-by: Alan Stern <[EMAIL PROTECTED]> --- Index: usb-2.6/drive

Re: [linux-usb-devel] Removing Gadget File Storage module

2007-05-30 Thread Alan Stern
On Tue, 29 May 2007, Ragner N Magalhães wrote: > On 5/28/07, Alan Stern <[EMAIL PROTECTED]> wrote: > > On Mon, 28 May 2007, Ragner N Magalhães wrote: > > > > > Hi all, > > > I am working with OMAP H2 and when I run "rmmod g_file_storage", it > > > stay waiting some thing and not terminate ... > >

[linux-usb-devel] [PATCH] ftdi_sio: set default latency at device probe time

2007-05-30 Thread Edwin Olson
The FTDI devices default to a very long timeout period (16ms) which is detrimental to performance. Previous patches have exposed the latency timer via sysfs attributes, but increasingly, it seems that *everyone* wants lower timeout latencies. Given that there's basically no downside (a tiny inc

[linux-usb-devel] [patch 2.6.22-rc2-mm1]m66592-udc: fix use old interrupt flags

2007-05-30 Thread Yoshihiro Shimoda
This patch fixes the problem that used SA_* flags. Signed-off-by: Yoshihiro Shimoda <[EMAIL PROTECTED]> --- drivers/usb/gadget/m66592-udc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/usb/gadget/m66592-udc.c.orig 2007-05-30 21:34:10.0 +0900 +++ a/driv

Re: [linux-usb-devel] Grabbing characters from a Symbol barcode scanner

2007-05-30 Thread Jiri Kosina
On Wed, 30 May 2007, Andy Stewart wrote: > I have a Symbol USB barcode scanner that appears to work with Linux. > If I bring up a Konsole window and give it focus, characters are typed > into that window as I scan barcodes. However, what I'd like to do is > grab those characters in a C program

[linux-usb-devel] [patch 2.6.22-rc2-mm1] r8a66597-hcd: fix NULL access

2007-05-30 Thread Yoshihiro Shimoda
This patch fixes the problem that accesses NULL pointer when disconnected a cable while play music with usb-speaker. Signed-off-by: Yoshihiro Shimoda <[EMAIL PROTECTED]> --- drivers/usb/host/r8a66597-hcd.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) --- a/drivers/usb/host/r8a665

[linux-usb-devel] Grabbing characters from a Symbol barcode scanner

2007-05-30 Thread Andy Stewart
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 HI everybody, I have a Symbol USB barcode scanner that appears to work with Linux. If I bring up a Konsole window and give it focus, characters are typed into that window as I scan barcodes. However, what I'd like to do is grab those characters in

[linux-usb-devel] 业务/代理。

2007-05-30 Thread pengyou
负责人:你好! 深圳市众鑫集团有剩余的发.票优惠代开欢迎有需要的朋友欢迎来电联系。 范围包含较广有:商品销售,服务代理,广告运输安装装饰等票.据。税率比较低, 可电脑查询绝对正规。双方彼此合作一次必成永久朋友! 联系人:陈福生/经理 联系手机:136 918 41676 地址:深圳市八卦岭科技大厦 顺祝:商祺! - This SF.net email is sponsored by DB2 Express Download DB2 Express

Re: [linux-usb-devel] [patch 3/5] introduction of usb_anchor and overhaul of usb-skeleton

2007-05-30 Thread Oliver Neukum
Am Mittwoch, 30. Mai 2007 01:07 schrieb Greg KH: > On Fri, May 25, 2007 at 01:44:19PM +0200, Oliver Neukum wrote: > > Hi, > > > > this patch set introduces usb_anchor and uses it to implement all modern > > APIs in the skeleton driver. > > > > - support for suspend/resume > > This doesn't apply

[linux-usb-devel] [PATCH] drivers/block/ub.c: use list_for_each_entry()

2007-05-30 Thread Matthias Kaehlcke
Low performance USB storage driver: Use list_for_each_entry() instead of list_for_each() Signed-off-by: Matthias Kaehlcke <[EMAIL PROTECTED]> -- diff --git a/drivers/block/ub.c b/drivers/block/ub.c index 746a118..18c8b6c 100644 --- a/drivers/block/ub.c +++ b/drivers/block/ub.c @@ -1547,10 +1547,

Re: [linux-usb-devel] [patch 3/5] introduction of usb_anchor and overhaul of usb-skeleton

2007-05-30 Thread Greg KH
On Fri, May 25, 2007 at 01:44:19PM +0200, Oliver Neukum wrote: > Hi, > > this patch set introduces usb_anchor and uses it to implement all modern > APIs in the skeleton driver. > > - support for suspend/resume This doesn't apply for some reason, are you working on a different version than I have