Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1400774

Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

Baolin Wang <baolin.wang@linaro.org> writes:
>>>> why does it need restart? Why is dwc3 powered off? Who powers it off?
>>>
>>> Because when the dwc3 Vbus is off (no cable pluging in now),
>>> especially for some mobile device, the system need to power off the
>>> dwc3 to save power in this situation.
>>
>> but dwc3 doesn't do this by itself, so who's doing it?
>
> Yes, the dwc3 clock is controlled by the Soc system, so the Soc system
> can disable the dwc3 clock when there is no cable plugging in.

understood.

>>>> This looks like a *really* bad power management implementation. Do you
>>>> have hibernation enabled? Do you have Clock gating enabled? Which dwc3
>>>> version are you using? How was it configured?
>>>
>>> This is not hibernation, we want to power off the dwc3 to save power
>>> when no cable plugging in. Yes, we have clock gating, at this
>>> situation we will disable the clock and shutdown the phy to save
>>> power. For mobile device, most time no cable plugging in, so we need
>>> to think about the power consuming. How do you think this requirement?
>>
>> Well, seems like you're missing *proper* runtime PM. I've been meaning
>> to work on it for weeks, but I still have a few other things to do
>> before I get to that. In any case, we don't need to do what you did
>> here. There are better ways.
>
> Make sense.

cool, if you wanna work on it, let me know and I can give some details
of what I have in mind.

-- 
balbi

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-13 12:30 +0200
  Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-13 12:50 +0200
    Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-13 13:40 +0200
      Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-13 14:20 +0200
        Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-13 14:40 +0200
          Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-13 14:50 +0200
            Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-16 04:10 +0200
            Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-17 10:00 +0200
              Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-17 10:10 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-17 10:50 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-17 11:30 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-17 12:50 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-18 12:00 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-18 12:20 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-18 12:20 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-18 12:30 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-18 13:10 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until  gadget is power on Baolin Wang <baolin.wang@linaro.org> - 2016-05-18 13:30 +0200
                Re: [PATCH] dwc3: gadget: Defer starting the gadget device until gadget is power on Felipe Balbi <balbi@kernel.org> - 2016-05-18 13:30 +0200

csiph-web