Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-22 Thread Jorge Ramirez
On 1/22/19 19:47, Stephen Boyd wrote: > Quoting Jorge Ramirez (2019-01-17 02:46:21) >> On 1/17/19 11:08, Viresh Kumar wrote: >>> On 17-01-19, 09:38, Jorge Ramirez wrote: COMMON_CLK_DISABLED_UNUSED relies on the enable_count reference counter to disable the clocks that were enabled by the

Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-22 Thread Stephen Boyd
Quoting Jorge Ramirez (2019-01-17 02:46:21) > On 1/17/19 11:08, Viresh Kumar wrote: > > On 17-01-19, 09:38, Jorge Ramirez wrote: > >> COMMON_CLK_DISABLED_UNUSED relies on the enable_count reference counter > >> to disable the clocks that were enabled by the firwmare and not by the > >> drivers. > >

Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-17 Thread Jorge Ramirez
On 1/17/19 11:08, Viresh Kumar wrote: > On 17-01-19, 09:38, Jorge Ramirez wrote: >> COMMON_CLK_DISABLED_UNUSED relies on the enable_count reference counter >> to disable the clocks that were enabled by the firwmare and not by the >> drivers. >> >> the cpufreq driver does not enable the cpu clock. >

Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-17 Thread Viresh Kumar
On 17-01-19, 09:38, Jorge Ramirez wrote: > COMMON_CLK_DISABLED_UNUSED relies on the enable_count reference counter > to disable the clocks that were enabled by the firwmare and not by the > drivers. > > the cpufreq driver does not enable the cpu clock. > > so when clk_change_rate is called, the e

Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-17 Thread Jorge Ramirez
On 1/17/19 07:33, Bjorn Andersson wrote: > On Mon 17 Dec 01:46 PST 2018, Jorge Ramirez-Ortiz wrote: > >> When COMMON_CLK_DISABLED_UNUSED is set, in an effort to save power and >> to keep the software model of the clock in line with reality, the >> framework transverses the clock tree and disables

Re: [PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2019-01-16 Thread Bjorn Andersson
On Mon 17 Dec 01:46 PST 2018, Jorge Ramirez-Ortiz wrote: > When COMMON_CLK_DISABLED_UNUSED is set, in an effort to save power and > to keep the software model of the clock in line with reality, the > framework transverses the clock tree and disables those clocks that > were enabled by the firmware

[PATCH 08/13] clk: qcom: hfpll: CLK_IGNORE_UNUSED

2018-12-17 Thread Jorge Ramirez-Ortiz
When COMMON_CLK_DISABLED_UNUSED is set, in an effort to save power and to keep the software model of the clock in line with reality, the framework transverses the clock tree and disables those clocks that were enabled by the firmware but have not been enabled by any device driver. If CPUFREQ is en