Re: [PATCH v6 0/6] DVFS for IO devices on sdm845 and sc7180

[Rajendra Nayak]

Hey Matthias, thanks for summarizing this.

On 6/18/2020 3:45 AM, Matthias Kaehlcke wrote:
> What is the plan for landing these, it seems not all must/should
> go through the QCOM tree.
>
> My guesses:
>
> tty: serial: qcom_geni_serial: Use OPP API to set clk/perf state
> spi: spi-geni-qcom: Use OPP API to set clk/perf state
>    QCOM tree due to shared dependency on change in include/linux/qcom-geni-se.h

That's correct, Bjorn/Andy, can these be pulled in now for 5.9?
They have acks from Greg for serial and Mark for the spi patch.
 
> drm/msm/dpu: Use OPP API to set clk/perf state
> drm/msm: dsi: Use OPP API to set clk/perf state
>    drm/msm tree

Correct, the dsi patch is still not reviewed by Rob, so once that's done,
I am guessing Rob would pull both of these.

> media: venus: core: Add support for opp tables/perf voting
>    venus tree

correct, this is pending review/ack from Stan.

> spi: spi-qcom-qspi: Use OPP API to set clk/perf state
>    SPI tree

Right, Mark has this acked, I am guessing he will pull this in at
some point.

thanks,
Rajendra

> Does this make sense or are there any dependencies I'm missing?
>
> Thanks
>
> Matthias
>
> On Mon, Jun 15, 2020 at 05:32:38PM +0530, Rajendra Nayak wrote:
> > Changes in v6:
> > 1. rebased on 5.8-rc1, no functional change.
> >
> > Changes in v5:
> > 1. Opp cleanup path fixed up across drivers
> >
> > Changes in v4:
> > 1. Fixed all review feedback on v3
> > 2. Dropped the dts patches, will post as a seperate series once
> > driver changes are reviewed and merged.
> > The driver changes without DT updates to include OPP tables will
> > have zero functional change.
> > 3. Dropped the mmc/sdhc patch, which is a standalone patch. will
> > repost if needed seperately.
> >
> > Changes in v3:
> > 1. Added better error handling for dev_pm_opp_of_add_table()
> > 2. Some minor changes and fixes in 'PATCH 12/17' as compared to v2
> > 3. Dropped the mmc patch picked up by Ulf [2]
> >
> > Changes in v2:
> > 1. Added error handling for dev_pm_opp_set_clkname()
> > and dev_pm_opp_of_add_table()
> > 2. Used dev_pm_opp_put_clkname() in the cleanup path
> > 3. Dropped the OPP patch pulled in by Viresh [1]
> > 4. Dropped the UFS patches since they had some major rework
> > needed because of changes that were merged in the merge window
> > and I don't have a UFS device currently to validate the changes.
> >
> > We have had support added in the OPP core for a while now to support
> > DVFS for IO devices, and this series uses that infrastructure to
> > add DVFS support for various IO devices in sdm845 and sc7180 SoCs.
> >
> > [1] https://lkml.org/lkml/2020/4/14/98
> > [2] https://lore.kernel.org/patchwork/patch/1226381/
> >
> > Rajendra Nayak (6):
> >    tty: serial: qcom_geni_serial: Use OPP API to set clk/perf state
> >    spi: spi-geni-qcom: Use OPP API to set clk/perf state
> >    drm/msm/dpu: Use OPP API to set clk/perf state
> >    drm/msm: dsi: Use OPP API to set clk/perf state
> >    media: venus: core: Add support for opp tables/perf voting
> >    spi: spi-qcom-qspi: Use OPP API to set clk/perf state
> >
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c  |  3 +-
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c        | 26 +++++++++++-
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h        |  4 ++
> >   drivers/gpu/drm/msm/dsi/dsi.h                  |  2 +
> >   drivers/gpu/drm/msm/dsi/dsi_cfg.c              |  4 +-
> >   drivers/gpu/drm/msm/dsi/dsi_host.c             | 58 ++++++++++++++++++++++++++
> >   drivers/media/platform/qcom/venus/core.c       | 43 ++++++++++++++++---
> >   drivers/media/platform/qcom/venus/core.h       |  5 +++
> >   drivers/media/platform/qcom/venus/pm_helpers.c | 54 ++++++++++++++++++++++--
> >   drivers/spi/spi-geni-qcom.c                    | 26 ++++++++++--
> >   drivers/spi/spi-qcom-qspi.c                    | 28 ++++++++++++-
> >   drivers/tty/serial/qcom_geni_serial.c          | 34 ++++++++++++---
> >   include/linux/qcom-geni-se.h                   |  4 ++
> >   13 files changed, 268 insertions(+), 23 deletions(-)
> >
> > --
> > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> > of Code Aurora Forum, hosted by The Linux Foundation

--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation