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

Groups > linux.kernel > #1529017

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Show all headers | View raw

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

On Thu, Nov 24, 2016 at 11:18 AM, hl <hl@rock-chips.com> wrote:
> Hi MyungJoo Ham,
[]
>>
>> We still need to sync the all status even i call target() in
>> devfreq_suspend/resume_device
>> directly, so still need update_devfreq() other setp except
>> devfreq->governor->get_target_freq(devfreq, &freq);
>
> And i think it better to be governor behaviors, for userspace they may not
> want to change
> the suspend frequency like other governor, the frequency should decide by
> the user, if they
> want this function, they should like other governor to rigister a
> devfreq_monitor_suspend().


> What do you think about my rev6 patch?

If I understand the intention correctly, this is for the stability of
the device due to the behavior or bootloader/SoC-initializer, which
has nothing to do with governors.

Even if users are using userspace, as long as they set the custom
frequencies lower than the default, they have the possibility of
being unstable as ondemand is going to have.


To reuse the update_devfreq() code, you may do something like:

static int _update_freq(struct devfreq *devfreq, bool is_suspending)
{
    /* original contents of update_freq with if statement with is_suspending wrapping get_target_freq */
}
int update_freq(struct devfreq *devfreq)
{
	return _update_freq(devfreq, false);
}


There should be other good non-invasive methods that are not governoe-specific as well.



Cheers,
MyungJoo

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

Thread

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support MyungJoo Ham <myungjoo.ham@samsung.com> - 2016-11-24 07:20 +0100
  Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support Chanwoo Choi <cw00.choi@samsung.com> - 2016-11-24 08:20 +0100
    Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support hl <hl@rock-chips.com> - 2016-11-24 08:40 +0100
      Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support Chanwoo Choi <cw00.choi@samsung.com> - 2016-11-24 09:20 +0100
        Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support hl <hl@rock-chips.com> - 2016-11-24 09:40 +0100
          Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support Chanwoo Choi <cw00.choi@samsung.com> - 2016-11-24 10:30 +0100
            Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support Chanwoo Choi <cw00.choi@samsung.com> - 2016-11-24 11:00 +0100

csiph-web