* Andreas Kemnade [160808 22:36]:
> Calls to musb_platform_enable() occur at only 1 place.
> musb_platform_disable() is called at 4 places.
>
> about balancing:
> There is musb_start() and musb_stop(). They are called from
> musb_gadget_start/stop()
> These call musb_platform_enable() and musb_pl
On Fri, 5 Aug 2016 23:21:35 -0700
Tony Lindgren wrote:
> * Andreas Kemnade [160805 08:35]:
> > I repeat the subject line of the patch:
> > [PATCH v2] musb: omap2430: do not assume balanced enable()/disable()
> > It is *not* fix charging.
> >
> > So you mean t
* Andreas Kemnade [160805 08:35]:
> I repeat the subject line of the patch:
> [PATCH v2] musb: omap2430: do not assume balanced enable()/disable()
> It is *not* fix charging.
>
> So you mean that the phy should magically know at which refcounter
> it should power on and off
d separately increment the PM runtime count and then
> enable the component specific things. And then we have the
> errata to deal with when VBUS is enabled.
I repeat the subject line of the patch:
[PATCH v2] musb: omap2430: do not assume balanced enable()/disable()
It is *not* fix charging.
S
* Andreas Kemnade [160804 09:44]:
> Nothing happens here, so the previous state of the phy remains.
> It would be disabled by the generic phy layer in drivers/phy/phy-core.c
>
> > gadget driver is loaded.
> > musb_start() is called
> > omap2430_musb_enable() is called
> > calls phy_power_o
On Thu, 4 Aug 2016 16:49:30 +0200
"H. Nikolaus Schaller" wrote:
> > Rationale:
> >
> > The charger driver calls pm_runtime_get_sync(bci->transceiver->dev);
> > which should indirectly call twl4030_usb_set_mode to set the
> > POWER_CTRL_OTG_ENAB bit. This enables the prescaler hardware
> > for AD
On Thu, 4 Aug 2016 18:31:29 +0200
Andreas Kemnade wrote:
> Hi,
>
> On Thu, 4 Aug 2016 07:29:19 -0700
> Tony Lindgren wrote:
>
> > Hi,
> >
> > * H. Nikolaus Schaller [160803 10:07]:
> > > All this prevents detection of cable plugin-events and VBUS
> > > measurement and setting OTG_EN before c
Hi,
On Thu, 4 Aug 2016 07:29:19 -0700
Tony Lindgren wrote:
> Hi,
>
> * H. Nikolaus Schaller [160803 10:07]:
> > All this prevents detection of cable plugin-events and VBUS
> > measurement and setting OTG_EN before charging is attempted.
>
> So I gave this patch a try but it now blocks all dee
* H. Nikolaus Schaller [160804 07:50]:
> > Am 04.08.2016 um 16:29 schrieb Tony Lindgren :
> >
> > So I gave this patch a try but it now blocks all deeper SoC idle
> > states as the PHY stays active. I think the real fix is to make
> > sure the charger behaves independent of the USB PHY state.
>
Hi Tony,
> Am 04.08.2016 um 16:29 schrieb Tony Lindgren :
>
> Hi,
>
> * H. Nikolaus Schaller [160803 10:07]:
>> All this prevents detection of cable plugin-events and VBUS measurement
>> and setting OTG_EN before charging is attempted.
>
> So I gave this patch a try but it now blocks all deepe
Hi,
* H. Nikolaus Schaller [160803 10:07]:
> All this prevents detection of cable plugin-events and VBUS measurement
> and setting OTG_EN before charging is attempted.
So I gave this patch a try but it now blocks all deeper SoC idle
states as the PHY stays active. I think the real fix is to make
> Am 03.08.2016 um 17:38 schrieb Andreas Kemnade :
>
> The code assumes that omap2430_musb_enable() and
> omap2430_musb_disable() are called in a balanced way.
> That fact is broken by the fact that musb_init_controller() calls
> musb_platform_disable() to switch from unknown state to off state
>
The code assumes that omap2430_musb_enable() and
omap2430_musb_disable() are called in a balanced way.
That fact is broken by the fact that musb_init_controller() calls
musb_platform_disable() to switch from unknown state to off state
on initialisation.
That means that phy_power_off() is called fi
13 matches
Mail list logo
