> From: Felipe Balbi [mailto:me@xxxxxxxxxxxxxxx] > Sent: Saturday, February 21, 2009 1:19 AM > To: David Brownell > Cc: me@xxxxxxxxxxxxxxx; giuseppe.gorgoglione@xxxxxx; > greg@xxxxxxxxx; Sergei Shtylyov; Felipe Balbi; > linux-usb@xxxxxxxxxxxxxxx > Subject: Re: [patch 00/12] musb mess > > On Fri, Feb 20, 2009 at 04:16:53PM -0800, David Brownell wrote: > > On Friday 20 February 2009, Felipe Balbi wrote: > > > > > > > ISTR that needed to be reissued against mainline; the > patch I saw > > > > was against some older code. (And didn't he have some other > > > > bugfix too?) > > > > > > he did, here it is (updated as he made a patch -p0 version): > > > > > > >From 96da2c37a19e3a003ba0484af129e66c13d9b81f Mon Sep 17 > 00:00:00 > > > >2001 > > > From: Giuseppe GORGOGLIONE <giuseppe.gorgoglione@xxxxxx> > > > Date: Sat, 21 Feb 2009 00:35:43 +0200 > > > > Saturday is *tomorrow* ... you and your silly timezone! ;) > > hehe, working late :-p > > > I'll bundle this with the patches I send for 2.6.30, > assuming this is > > the patch I'm thinking of: comes up only with static FIFO > config, not > > the dynamic FIFO support. > > The problem he found was only with static FIFO (ep_config_from hw()). Yes, I can confirm this. Our SoC (STM STA2062) has two embedded Inventra devices, one with static FIFO config and the other with dynamic config. The current driver (as found in 2.6.28) is working fine for us with dynamic FIFO config, and needed a couple of patches for static config. Regards, Giuseppe. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html