Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-09-14 Thread Sibi Sankar
Hi Saravana, On 2018-08-02 06:27, Saravana Kannan wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. Example

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-09-14 Thread Sibi Sankar
Hi Saravana, On 2018-08-02 06:27, Saravana Kannan wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. Example

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-09-10 Thread Sibi Sankar
Hi Georgi, This driver uses of_icc_get which is very likely to fail if it probes before the interconnect provider. Would it be possible for icc_get to return/differentiate both -EPROBE_DEFER and other errors to prevent the driver to continually probe defer if the path doesn't actually exist or

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-09-10 Thread Sibi Sankar
Hi Georgi, This driver uses of_icc_get which is very likely to fail if it probes before the interconnect provider. Would it be possible for icc_get to return/differentiate both -EPROBE_DEFER and other errors to prevent the driver to continually probe defer if the path doesn't actually exist or

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-23 Thread Georgi Djakov
Hi Saravana, On 08/02/2018 03:57 AM, Saravana Kannan wrote: > This driver registers itself as a devfreq device that allows devfreq > governors to make bandwidth votes for an interconnect path. This allows > applying various policies for different interconnect paths using devfreq > governors. > >

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-23 Thread Georgi Djakov
Hi Saravana, On 08/02/2018 03:57 AM, Saravana Kannan wrote: > This driver registers itself as a devfreq device that allows devfreq > governors to make bandwidth votes for an interconnect path. This allows > applying various policies for different interconnect paths using devfreq > governors. > >

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-07 Thread skannan
On 2018-08-07 09:51, Rob Herring wrote: On Wed, Aug 01, 2018 at 05:57:42PM -0700, Saravana Kannan wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-07 Thread skannan
On 2018-08-07 09:51, Rob Herring wrote: On Wed, Aug 01, 2018 at 05:57:42PM -0700, Saravana Kannan wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-07 Thread Rob Herring
On Wed, Aug 01, 2018 at 05:57:42PM -0700, Saravana Kannan wrote: > This driver registers itself as a devfreq device that allows devfreq > governors to make bandwidth votes for an interconnect path. This allows > applying various policies for different interconnect paths using devfreq > governors.

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-07 Thread Rob Herring
On Wed, Aug 01, 2018 at 05:57:42PM -0700, Saravana Kannan wrote: > This driver registers itself as a devfreq device that allows devfreq > governors to make bandwidth votes for an interconnect path. This allows > applying various policies for different interconnect paths using devfreq > governors.

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread skannan
On 2018-08-02 03:13, MyungJoo Ham wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. First of all, the name,

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread skannan
On 2018-08-02 03:13, MyungJoo Ham wrote: This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. First of all, the name,

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread Georgi Djakov
Hi MyungJoo, On 08/02/2018 01:13 PM, MyungJoo Ham wrote: >> This driver registers itself as a devfreq device that allows devfreq >> governors to make bandwidth votes for an interconnect path. This allows >> applying various policies for different interconnect paths using devfreq >> governors. >>

Re: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread Georgi Djakov
Hi MyungJoo, On 08/02/2018 01:13 PM, MyungJoo Ham wrote: >> This driver registers itself as a devfreq device that allows devfreq >> governors to make bandwidth votes for an interconnect path. This allows >> applying various policies for different interconnect paths using devfreq >> governors. >>

RE: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread MyungJoo Ham
>This driver registers itself as a devfreq device that allows devfreq >governors to make bandwidth votes for an interconnect path. This allows >applying various policies for different interconnect paths using devfreq >governors. > First of all, the name, "devfreq_icbw", is not appropriate for a

RE: [PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-02 Thread MyungJoo Ham
>This driver registers itself as a devfreq device that allows devfreq >governors to make bandwidth votes for an interconnect path. This allows >applying various policies for different interconnect paths using devfreq >governors. > First of all, the name, "devfreq_icbw", is not appropriate for a

[PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-01 Thread Saravana Kannan
This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. Example uses: * Use the devfreq performance governor to set the CPU to

[PATCH v3 2/2] PM / devfreq: Add devfreq driver for interconnect bandwidth voting

2018-08-01 Thread Saravana Kannan
This driver registers itself as a devfreq device that allows devfreq governors to make bandwidth votes for an interconnect path. This allows applying various policies for different interconnect paths using devfreq governors. Example uses: * Use the devfreq performance governor to set the CPU to