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

From: Stanimir Varbanov
Date: Fri Nov 18 2016 - 09:49:32 EST


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@xxxxxxxxxxxxxxx/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(+)
>