On Thu, Feb 26, 2015 at 01:30:21PM -0600, Bin Liu wrote: > Felipe, > > On Thu, Feb 26, 2015 at 12:27 PM, Felipe Balbi <balbi@xxxxxx> wrote: > > On Thu, Feb 26, 2015 at 12:25:16PM -0600, Felipe Balbi wrote: > >> FSDEV is set for both HIGH and FULL speeds, > >> the correct HIGHSPEED check is done through > >> power register's HSMODE bit. > >> > >> Signed-off-by: Felipe Balbi <balbi@xxxxxx> > > > > I'm still unsure if we should really ignore babble on FS/LS. It seems to > > me we should never ignore it, but I really don't have a way to prove > > this statement. For the sake of reducing impact, we will just fix HS > > check here. > > > > I believe we should drop speed check in here and not ignore babble > regardless. We have seen many cases that full-speed babble causes MUSB > stop working. I'll make that as a separate patch then, just to make sure we can revert it later if something goes wrong ;-) -- balbi
Attachment:
signature.asc
Description: Digital signature