Subject: Re: [PATCH V2 0/2] clk: qcom: gdsc: Add support for gdscs with HW
 control
To: Sricharan R <sricharan@codeaurora.org>, sboyd@codeaurora.org,
        mturquette@baylibre.com, linux-clk@vger.kernel.org,
        linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
        rnayak@codeaurora.org, stanimir.varbanov@linaro.org
References: <1479472107-18472-1-git-send-email-sricharan@codeaurora.org>
From: Stanimir Varbanov <svarbanov@mm-sol.com>
Message-ID: <8257df0e-0cae-edbf-e845-e31fb50b7ba7@mm-sol.com>
Date: Fri, 18 Nov 2016 16:49:22 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.3.0
MIME-Version: 1.0
In-Reply-To: <1479472107-18472-1-git-send-email-sricharan@codeaurora.org>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1159
Lines: 32

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(+)
>