[2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock
diff mbox series

Message ID 1543562950-23997-3-git-send-email-rohitkr@codeaurora.org
State New
Headers show
Series
  • qcom_adsp_pil: Remove voting for lpass_aon clock
Related show

Commit Message

Rohit Kumar Nov. 30, 2018, 7:29 a.m. UTC
Lpass_aon clock is on by default. Remove it from lpass
clock list to avoid voting for it.

Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
---
 drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rohit Kumar Jan. 4, 2019, 4:36 a.m. UTC | #1
Hello Bjorn,

Can you please review this patch series too.

LPASS_AON clock support is already removed from lpass clock driver.


Thanks,

Rohit

On 11/30/2018 12:59 PM, Rohit kumar wrote:
> Lpass_aon clock is on by default. Remove it from lpass
> clock list to avoid voting for it.
>
> Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
> ---
>   drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc/qcom_q6v5_adsp.c
> index 79374d1..4829173 100644
> --- a/drivers/remoteproc/qcom_q6v5_adsp.c
> +++ b/drivers/remoteproc/qcom_q6v5_adsp.c
> @@ -48,7 +48,7 @@
>   
>   /* list of clocks required by ADSP PIL */
>   static const char * const adsp_clk_id[] = {
> -	"sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
> +	"sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
>   	"qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
>   };
>
Bjorn Andersson Jan. 7, 2019, 7:13 a.m. UTC | #2
On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote:

> Hello Bjorn,
> 
> Can you please review this patch series too.
> 
> LPASS_AON clock support is already removed from lpass clock driver.
> 

Applied the two patches.

Thanks,
Bjorn

> 
> Thanks,
> 
> Rohit
> 
> On 11/30/2018 12:59 PM, Rohit kumar wrote:
> > Lpass_aon clock is on by default. Remove it from lpass
> > clock list to avoid voting for it.
> > 
> > Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
> > ---
> >   drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc/qcom_q6v5_adsp.c
> > index 79374d1..4829173 100644
> > --- a/drivers/remoteproc/qcom_q6v5_adsp.c
> > +++ b/drivers/remoteproc/qcom_q6v5_adsp.c
> > @@ -48,7 +48,7 @@
> >   /* list of clocks required by ADSP PIL */
> >   static const char * const adsp_clk_id[] = {
> > -	"sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
> > +	"sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
> >   	"qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
> >   };
> 
> -- 
> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member
> of the Code Aurora Forum, hosted by the Linux Foundation.
>
Rohit Kumar Jan. 14, 2019, 10:44 a.m. UTC | #3
On 1/7/2019 12:43 PM, Bjorn Andersson wrote:
> On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote:
>
>> Hello Bjorn,
>>
>> Can you please review this patch series too.
>>
>> LPASS_AON clock support is already removed from lpass clock driver.
>>
> Applied the two patches.

Hello Bjorn,


Can you please point me to the tree where this is applied.


Thanks,

Rohit

> Thanks,
> Bjorn
>
>> Thanks,
>>
>> Rohit
>>
>> On 11/30/2018 12:59 PM, Rohit kumar wrote:
>>> Lpass_aon clock is on by default. Remove it from lpass
>>> clock list to avoid voting for it.
>>>
>>> Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
>>> ---
>>>    drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc/qcom_q6v5_adsp.c
>>> index 79374d1..4829173 100644
>>> --- a/drivers/remoteproc/qcom_q6v5_adsp.c
>>> +++ b/drivers/remoteproc/qcom_q6v5_adsp.c
>>> @@ -48,7 +48,7 @@
>>>    /* list of clocks required by ADSP PIL */
>>>    static const char * const adsp_clk_id[] = {
>>> -	"sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
>>> +	"sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
>>>    	"qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
>>>    };
>> -- 
>> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member
>> of the Code Aurora Forum, hosted by the Linux Foundation.
>>
Rohit Kumar Jan. 30, 2019, 5:08 a.m. UTC | #4
Hello Bjorn,

On 1/14/2019 4:14 PM, Rohit Kumar wrote:
>
> On 1/7/2019 12:43 PM, Bjorn Andersson wrote:
>> On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote:
>>
>>> Hello Bjorn,
>>>
>>> Can you please review this patch series too.
>>>
>>> LPASS_AON clock support is already removed from lpass clock driver.
>>>
>> Applied the two patches.
>
I did not see this in next yet. Can you please pick this up.

Thanks,

Rohit

> Hello Bjorn,
>
>
> Can you please point me to the tree where this is applied.
>
>
> Thanks,
>
> Rohit
>
>> Thanks,
>> Bjorn
>>
>>> Thanks,
>>>
>>> Rohit
>>>
>>> On 11/30/2018 12:59 PM, Rohit kumar wrote:
>>>> Lpass_aon clock is on by default. Remove it from lpass
>>>> clock list to avoid voting for it.
>>>>
>>>> Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
>>>> ---
>>>>    drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c 
>>>> b/drivers/remoteproc/qcom_q6v5_adsp.c
>>>> index 79374d1..4829173 100644
>>>> --- a/drivers/remoteproc/qcom_q6v5_adsp.c
>>>> +++ b/drivers/remoteproc/qcom_q6v5_adsp.c
>>>> @@ -48,7 +48,7 @@
>>>>    /* list of clocks required by ADSP PIL */
>>>>    static const char * const adsp_clk_id[] = {
>>>> -    "sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", 
>>>> "lpass_ahbm_aon_cbcr",
>>>> +    "sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
>>>>        "qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
>>>>    };
>>> -- 
>>> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a 
>>> member
>>> of the Code Aurora Forum, hosted by the Linux Foundation.
>>>
Rohit Kumar Feb. 18, 2019, 4:54 a.m. UTC | #5
Hello Bjorn,


On 1/30/2019 10:38 AM, Rohit Kumar wrote:
> Hello Bjorn,
>
> On 1/14/2019 4:14 PM, Rohit Kumar wrote:
>>
>> On 1/7/2019 12:43 PM, Bjorn Andersson wrote:
>>> On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote:
>>>
>>>> Hello Bjorn,
>>>>
>>>> Can you please review this patch series too.
>>>>
>>>> LPASS_AON clock support is already removed from lpass clock driver.
>>>>
>>> Applied the two patches.
>>
Can you please apply these patches. I don't see them in next.
>>
> I did not see this in next yet. Can you please pick this up.
>
> Thanks,
>
> Rohit
>
>> Hello Bjorn,
>>
>>
>> Can you please point me to the tree where this is applied.
>>
>>
>> Thanks,
>>
>> Rohit
>>
>>> Thanks,
>>> Bjorn
>>>
>>>> Thanks,
>>>>
>>>> Rohit
>>>>
>>>> On 11/30/2018 12:59 PM, Rohit kumar wrote:
>>>>> Lpass_aon clock is on by default. Remove it from lpass
>>>>> clock list to avoid voting for it.
>>>>>
>>>>> Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
>>>>> ---
>>>>>    drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
>>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c 
>>>>> b/drivers/remoteproc/qcom_q6v5_adsp.c
>>>>> index 79374d1..4829173 100644
>>>>> --- a/drivers/remoteproc/qcom_q6v5_adsp.c
>>>>> +++ b/drivers/remoteproc/qcom_q6v5_adsp.c
>>>>> @@ -48,7 +48,7 @@
>>>>>    /* list of clocks required by ADSP PIL */
>>>>>    static const char * const adsp_clk_id[] = {
>>>>> -    "sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", 
>>>>> "lpass_ahbm_aon_cbcr",
>>>>> +    "sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
>>>>>        "qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
>>>>>    };
>>>> -- 
>>>> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a 
>>>> member
>>>> of the Code Aurora Forum, hosted by the Linux Foundation.
>>>>
Thanks,

Rohit

Patch
diff mbox series

diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc/qcom_q6v5_adsp.c
index 79374d1..4829173 100644
--- a/drivers/remoteproc/qcom_q6v5_adsp.c
+++ b/drivers/remoteproc/qcom_q6v5_adsp.c
@@ -48,7 +48,7 @@ 
 
 /* list of clocks required by ADSP PIL */
 static const char * const adsp_clk_id[] = {
-	"sway_cbcr", "lpass_aon", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
+	"sway_cbcr", "lpass_ahbs_aon_cbcr", "lpass_ahbm_aon_cbcr",
 	"qdsp6ss_xo", "qdsp6ss_sleep", "qdsp6ss_core",
 };