Re: [PATCH V2 0/2] clk: qcom: gdsc: Add support for gdscs with HW control

[Stanimir Varbanov <svarbanov@mm-sol.com>]

Hi,

On 11/18/2016 02:28 PM, Sricharan R wrote:
> This series adds support for gdscs(powerdomains) that can be configured
> in hw controlled mode. So they are turned 'ON' based on needs dynamically,
> helping to save power. Also updated the venus video ip's gdsc/clock
> data to put them in hw control.
> 
> V2:
>  Dropped patch#3 [1] as it was concluded that the patch was effectively
>  masking the fact the clocks were not getting turned on when the gdsc
>  is put in hwctrl. With some change in sequence from venus core, masking
>  is not needed and so patch needs to handled in venus driver.

Which sequence should be changed in venus driver?

> 
>  Fixed a comment in patch#1 to check the return value for gdsc_hwctrl
> 
> [1] https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1270922.html
> 
> Rajendra Nayak (1):
>   clk: qcom: gdsc: Add support for gdscs with HW control
> 
> Sricharan R (1):
>   clk: qcom: Put venus core0/1 gdscs to hw control mode
> 
>  drivers/clk/qcom/gdsc.c         | 19 +++++++++++++++++++
>  drivers/clk/qcom/gdsc.h         |  1 +
>  drivers/clk/qcom/mmcc-msm8996.c |  2 ++
>  3 files changed, 22 insertions(+)
>