Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-02 Thread Bjorn Andersson
On Wed 02 Nov 12:24 PDT 2016, Sarangdhar Joshi wrote: > On 11/01/2016 04:11 PM, Stephen Boyd wrote: > >On 10/28, Sarangdhar Joshi wrote: > >>@@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); > >> static int qcom_scm_probe(struct platform_device *pdev) > >> { > >>struct qcom_scm

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-02 Thread Bjorn Andersson
On Wed 02 Nov 12:24 PDT 2016, Sarangdhar Joshi wrote: > On 11/01/2016 04:11 PM, Stephen Boyd wrote: > >On 10/28, Sarangdhar Joshi wrote: > >>@@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); > >> static int qcom_scm_probe(struct platform_device *pdev) > >> { > >>struct qcom_scm

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-02 Thread Sarangdhar Joshi
On 11/01/2016 04:11 PM, Stephen Boyd wrote: On 10/28, Sarangdhar Joshi wrote: @@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); static int qcom_scm_probe(struct platform_device *pdev) { struct qcom_scm *scm; + uint64_t clks; int ret; scm =

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-02 Thread Sarangdhar Joshi
On 11/01/2016 04:11 PM, Stephen Boyd wrote: On 10/28, Sarangdhar Joshi wrote: @@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); static int qcom_scm_probe(struct platform_device *pdev) { struct qcom_scm *scm; + uint64_t clks; int ret; scm =

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-01 Thread Stephen Boyd
On 10/28, Sarangdhar Joshi wrote: > @@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); > static int qcom_scm_probe(struct platform_device *pdev) > { > struct qcom_scm *scm; > + uint64_t clks; > int ret; > > scm = devm_kzalloc(>dev, sizeof(*scm), GFP_KERNEL); >

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-11-01 Thread Stephen Boyd
On 10/28, Sarangdhar Joshi wrote: > @@ -380,33 +384,43 @@ EXPORT_SYMBOL(qcom_scm_is_available); > static int qcom_scm_probe(struct platform_device *pdev) > { > struct qcom_scm *scm; > + uint64_t clks; > int ret; > > scm = devm_kzalloc(>dev, sizeof(*scm), GFP_KERNEL); >

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-10-28 Thread kbuild test robot
Hi Sarangdhar, [auto build test WARNING on linus/master] [also build test WARNING on v4.9-rc2 next-20161028] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to

Re: [PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-10-28 Thread kbuild test robot
Hi Sarangdhar, [auto build test WARNING on linus/master] [also build test WARNING on v4.9-rc2 next-20161028] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to

[PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-10-28 Thread Sarangdhar Joshi
Core, iface and bus clocks are not required to be voted from SCM driver for some of the Qualcomm chipsets. Remove dependency on these clocks from driver. Suggested-by: Bjorn Andersson Signed-off-by: Sarangdhar Joshi ---

[PATCH 2/3] firmware: qcom: scm: Remove core, iface and bus clocks dependency

2016-10-28 Thread Sarangdhar Joshi
Core, iface and bus clocks are not required to be voted from SCM driver for some of the Qualcomm chipsets. Remove dependency on these clocks from driver. Suggested-by: Bjorn Andersson Signed-off-by: Sarangdhar Joshi --- drivers/firmware/qcom_scm.c | 65