Re: [PATCH 10/15] usb: musb: Don't set d+ high before enable for 2430 glue layer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, May 16, 2016 at 07:57:57AM -0700, Tony Lindgren wrote:
> * Bin Liu <b-liu@xxxxxx> [160516 07:17]:
> > Hi,
> > 
> > On Sat, May 14, 2016 at 04:30:32PM +0300, Sergei Shtylyov wrote:
> > > On 5/14/2016 12:03 AM, Bin Liu wrote:
> > > 
> > > >>At least 2430 glue layer pulls d+ high on start up even if there are
> > > >>no gadgets configured. This is bad at least for anything using a separate
> > > >>battery charger chip as it can confuse the charger detection.
> > > >>
> > > >>Let's fix the issue by getting rid of omap2430_musb_set_mode() and only
> > > >
> > > >By doing so, you lost the feature of switching mode from sysfs, I am not
> > > >sure if there is anyone using it though, still, it is a regression.
> > > 
> > >    BTW, set_mode() implemented in the OMAP glue always seemed bogus to me.
> > > Instead of forcing host/gadget/OTG modes, it sets the Session bit...
> > 
> > Setting the session bit here seems to be okay. Other glues rely
> > otg_timer() instead.
> 
> Looks like host mode starts just fine without writing to session bit
> for 2430 glue. Let's just remove it, I'll send an updated patch shortly.

So setting host mode in sysfs also works without writting to session
bit? If so, yeah, we can remove it.

Regards,
-Bin.

> 
> For other musb core register tinkering, let's attempt to remove
> those from the glue layers eventually. I think Felipe wanted to do this
> for a long time as this allows us to switch to a shared interrupt model
> for the glue layer and musb core.
> 
> Regards,
> 
> Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux