mbox series

[0/6] clk: qcom: cpu-8996: additional cleanup for the driver

Message ID 20220714100351.1834711-1-dmitry.baryshkov@linaro.org
Headers show
Series clk: qcom: cpu-8996: additional cleanup for the driver | expand

Message

Dmitry Baryshkov July 14, 2022, 10:03 a.m. UTC 
This patch series depends on patches 1-5 from [1].

Path 1 is slightly fixed version of patch 6 from the mentioned
patch series (fixed to use parent_hws where applicable). The rest is
minor cleanup of the driver.

[1] https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/


Dmitry Baryshkov (5):
  clk: qcom: cpu-8996: switch to devm_clk_notifier_register
  clk: qcom: cpu-8996: declare ACD clocks
  clk: qcom: cpu-8996: move ACD logic to
    clk_cpu_8996_pmux_determine_rate
  clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
  clk: qcom: cpu-8996: use constant mask for pmux

Yassine Oudjana (1):
  clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks

 drivers/clk/qcom/clk-cpu-8996.c | 191 +++++++++++++++++---------------
 1 file changed, 100 insertions(+), 91 deletions(-)


base-commit: ca48adcc40b09d7f26a7754d4d54cfc4bd611f38
prerequisite-patch-id: ff67ff7bea1aef8e367a2589c46cf2c9ebb48664
prerequisite-patch-id: 1fdf02d8161689f3e571816d73ec94b115f51c34
prerequisite-patch-id: 837945fbb40427dac2e95a58b7660a3cf26d7d53
prerequisite-patch-id: df10945929f6f558c1363a23e2993d748a40236f
prerequisite-patch-id: a657a27256ef4be0cb932cb0ca7b3e4768e466f9

Comments

Dmitry Baryshkov Sept. 9, 2022, 10:22 a.m. UTC | #1 
On 14/07/2022 13:03, Dmitry Baryshkov wrote:
> This patch series depends on patches 1-5 from [1].
> 
> Path 1 is slightly fixed version of patch 6 from the mentioned
> patch series (fixed to use parent_hws where applicable). The rest is
> minor cleanup of the driver.
> 
> [1] https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/
> 

Gracious ping. Yassing, Bjorn, Konrad?

> 
> Dmitry Baryshkov (5):
>    clk: qcom: cpu-8996: switch to devm_clk_notifier_register
>    clk: qcom: cpu-8996: declare ACD clocks
>    clk: qcom: cpu-8996: move ACD logic to
>      clk_cpu_8996_pmux_determine_rate
>    clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
>    clk: qcom: cpu-8996: use constant mask for pmux
> 
> Yassine Oudjana (1):
>    clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks
> 
>   drivers/clk/qcom/clk-cpu-8996.c | 191 +++++++++++++++++---------------
>   1 file changed, 100 insertions(+), 91 deletions(-)
> 
> 
> base-commit: ca48adcc40b09d7f26a7754d4d54cfc4bd611f38
> prerequisite-patch-id: ff67ff7bea1aef8e367a2589c46cf2c9ebb48664
> prerequisite-patch-id: 1fdf02d8161689f3e571816d73ec94b115f51c34
> prerequisite-patch-id: 837945fbb40427dac2e95a58b7660a3cf26d7d53
> prerequisite-patch-id: df10945929f6f558c1363a23e2993d748a40236f
> prerequisite-patch-id: a657a27256ef4be0cb932cb0ca7b3e4768e466f9
Bjorn Andersson Sept. 14, 2022, 3:19 a.m. UTC | #2 
On Fri, Sep 09, 2022 at 01:22:40PM +0300, Dmitry Baryshkov wrote:
> On 14/07/2022 13:03, Dmitry Baryshkov wrote:
> > This patch series depends on patches 1-5 from [1].
> > 
> > Path 1 is slightly fixed version of patch 6 from the mentioned
> > patch series (fixed to use parent_hws where applicable). The rest is
> > minor cleanup of the driver.
> > 
> > [1] https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/
> > 
> 
> Gracious ping. Yassing, Bjorn, Konrad?
> 

As I haven't heard anything from Yassine or Konrad I went ahead and
tried to pick this, unfortunately [1] doesn't apply.

Could you please rebase the patches from [1] that you want applied and
resubmit that?

Thanks,
Bjorn

> > 
> > Dmitry Baryshkov (5):
> >    clk: qcom: cpu-8996: switch to devm_clk_notifier_register
> >    clk: qcom: cpu-8996: declare ACD clocks
> >    clk: qcom: cpu-8996: move ACD logic to
> >      clk_cpu_8996_pmux_determine_rate
> >    clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
> >    clk: qcom: cpu-8996: use constant mask for pmux
> > 
> > Yassine Oudjana (1):
> >    clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks
> > 
> >   drivers/clk/qcom/clk-cpu-8996.c | 191 +++++++++++++++++---------------
> >   1 file changed, 100 insertions(+), 91 deletions(-)
> > 
> > 
> > base-commit: ca48adcc40b09d7f26a7754d4d54cfc4bd611f38
> > prerequisite-patch-id: ff67ff7bea1aef8e367a2589c46cf2c9ebb48664
> > prerequisite-patch-id: 1fdf02d8161689f3e571816d73ec94b115f51c34
> > prerequisite-patch-id: 837945fbb40427dac2e95a58b7660a3cf26d7d53
> > prerequisite-patch-id: df10945929f6f558c1363a23e2993d748a40236f
> > prerequisite-patch-id: a657a27256ef4be0cb932cb0ca7b3e4768e466f9
> 
> -- 
> With best wishes
> Dmitry
>
Yassine Oudjana Sept. 14, 2022, 12:18 p.m. UTC | #3 
On Tue, Sep 13 2022 at 22:19:28 -05:00:00, Bjorn Andersson 
<andersson@kernel.org> wrote:
> On Fri, Sep 09, 2022 at 01:22:40PM +0300, Dmitry Baryshkov wrote:
>>  On 14/07/2022 13:03, Dmitry Baryshkov wrote:
>>  > This patch series depends on patches 1-5 from [1].
>>  >
>>  > Path 1 is slightly fixed version of patch 6 from the mentioned
>>  > patch series (fixed to use parent_hws where applicable). The rest 
>> is
>>  > minor cleanup of the driver.
>>  >
>>  > [1] 
>> https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/
>>  >
>> 
>>  Gracious ping. Yassing, Bjorn, Konrad?
>> 
> 
> As I haven't heard anything from Yassine or Konrad I went ahead and
> tried to pick this, unfortunately [1] doesn't apply.

Seems to apply on next-20220914. Do I need to base on a different 
branch?

> 
> Could you please rebase the patches from [1] that you want applied and
> resubmit that?
> 
> Thanks,
> Bjorn
> 
>>  >
>>  > Dmitry Baryshkov (5):
>>  >    clk: qcom: cpu-8996: switch to devm_clk_notifier_register
>>  >    clk: qcom: cpu-8996: declare ACD clocks
>>  >    clk: qcom: cpu-8996: move ACD logic to
>>  >      clk_cpu_8996_pmux_determine_rate
>>  >    clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
>>  >    clk: qcom: cpu-8996: use constant mask for pmux
>>  >
>>  > Yassine Oudjana (1):
>>  >    clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks
>>  >
>>  >   drivers/clk/qcom/clk-cpu-8996.c | 191 
>> +++++++++++++++++---------------
>>  >   1 file changed, 100 insertions(+), 91 deletions(-)
>>  >
>>  >
>>  > base-commit: ca48adcc40b09d7f26a7754d4d54cfc4bd611f38
>>  > prerequisite-patch-id: ff67ff7bea1aef8e367a2589c46cf2c9ebb48664
>>  > prerequisite-patch-id: 1fdf02d8161689f3e571816d73ec94b115f51c34
>>  > prerequisite-patch-id: 837945fbb40427dac2e95a58b7660a3cf26d7d53
>>  > prerequisite-patch-id: df10945929f6f558c1363a23e2993d748a40236f
>>  > prerequisite-patch-id: a657a27256ef4be0cb932cb0ca7b3e4768e466f9
>> 
>>  --
>>  With best wishes
>>  Dmitry
>>
Dmitry Baryshkov Sept. 15, 2022, 5:01 p.m. UTC | #4 
On 14/09/2022 06:19, Bjorn Andersson wrote:
> On Fri, Sep 09, 2022 at 01:22:40PM +0300, Dmitry Baryshkov wrote:
>> On 14/07/2022 13:03, Dmitry Baryshkov wrote:
>>> This patch series depends on patches 1-5 from [1].
>>>
>>> Path 1 is slightly fixed version of patch 6 from the mentioned
>>> patch series (fixed to use parent_hws where applicable). The rest is
>>> minor cleanup of the driver.
>>>
>>> [1] https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/
>>>
>>
>> Gracious ping. Yassing, Bjorn, Konrad?
>>
> 
> As I haven't heard anything from Yassine or Konrad I went ahead and
> tried to pick this, unfortunately [1] doesn't apply.
> 
> Could you please rebase the patches from [1] that you want applied and
> resubmit that?

I just checked, patches from [1] apply cleanly on your for-next tree. 
Could you please doublecheck?



> 
> Thanks,
> Bjorn
> 
>>>
>>> Dmitry Baryshkov (5):
>>>     clk: qcom: cpu-8996: switch to devm_clk_notifier_register
>>>     clk: qcom: cpu-8996: declare ACD clocks
>>>     clk: qcom: cpu-8996: move ACD logic to
>>>       clk_cpu_8996_pmux_determine_rate
>>>     clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
>>>     clk: qcom: cpu-8996: use constant mask for pmux
>>>
>>> Yassine Oudjana (1):
>>>     clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks
>>>
>>>    drivers/clk/qcom/clk-cpu-8996.c | 191 +++++++++++++++++---------------
>>>    1 file changed, 100 insertions(+), 91 deletions(-)
>>>
>>>
>>> base-commit: ca48adcc40b09d7f26a7754d4d54cfc4bd611f38
>>> prerequisite-patch-id: ff67ff7bea1aef8e367a2589c46cf2c9ebb48664
>>> prerequisite-patch-id: 1fdf02d8161689f3e571816d73ec94b115f51c34
>>> prerequisite-patch-id: 837945fbb40427dac2e95a58b7660a3cf26d7d53
>>> prerequisite-patch-id: df10945929f6f558c1363a23e2993d748a40236f
>>> prerequisite-patch-id: a657a27256ef4be0cb932cb0ca7b3e4768e466f9
>>
>> -- 
>> With best wishes
>> Dmitry
>>
Bjorn Andersson Sept. 27, 2022, 3:22 a.m. UTC | #5 
On Thu, 14 Jul 2022 13:03:45 +0300, Dmitry Baryshkov wrote:
> This patch series depends on patches 1-5 from [1].
> 
> Path 1 is slightly fixed version of patch 6 from the mentioned
> patch series (fixed to use parent_hws where applicable). The rest is
> minor cleanup of the driver.
> 
> [1] https://lore.kernel.org/linux-arm-msm/20220621160621.24415-1-y.oudjana@protonmail.com/
> 
> [...]

Applied, thanks!

[1/6] clk: qcom: msm8996-cpu: Use parent_data/_hws for all clocks
      commit: da5daae8b412c922e08f86979e84ea80b60092a1
[2/6] clk: qcom: cpu-8996: switch to devm_clk_notifier_register
      commit: a808c7848a52523cb758c844f642b3d9e059c0c3
[3/6] clk: qcom: cpu-8996: declare ACD clocks
      commit: f1e3fcc4fc81e2aa78f4af754c460468e3f19782
[4/6] clk: qcom: cpu-8996: move ACD logic to clk_cpu_8996_pmux_determine_rate
      commit: 81165aca05dc003ea41c9bc725238dc249d477fd
[5/6] clk: qcom: cpu-8996: don't store parents in clk_cpu_8996_pmux
      commit: f387d1c46f53457d0d9687295629f3db2f44d29b
[6/6] clk: qcom: cpu-8996: use constant mask for pmux
      commit: f9ea0f59f7eefe44d82bbd4e86d2fac353fcfcbe

Best regards,