On Thu, Feb 26, 2015 at 1:48 PM, Felipe Balbi <balbi@xxxxxx> wrote: > 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 ;-) Agreed. > > -- > balbi -- 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