Re: [PATCH 13/21] mmc: sdhci-msm: Use OPP API to set clk/perf state

From: Ulf Hansson
Date: Thu Apr 16 2020 - 04:39:57 EST


On Wed, 15 Apr 2020 at 18:43, Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> wrote:
>
>
>
> On 4/15/2020 7:22 PM, Ulf Hansson wrote:
> > On Wed, 8 Apr 2020 at 15:48, Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> wrote:
> >>
> >> On some qualcomm SoCs we need to vote on a performance state of a power
> >> domain depending on the clock rates. Hence move to using OPP api to set
> >> the clock rate and performance state specified in the OPP table.
> >> On platforms without an OPP table, dev_pm_opp_set_rate() is eqvivalent to
> >> clk_set_rate()
> >>
> >> Signed-off-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx>
> >> Cc: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> >> Cc: Pradeep P V K <ppvk@xxxxxxxxxxxxxx>
> >> Cc: Veerabhadrarao Badiganti <vbadigan@xxxxxxxxxxxxxx>
> >> Cc: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx>
> >> Cc: linux-mmc@xxxxxxxxxxxxxxx
> >
> > This looks good to me!
> >
> > However, are there any of the other patches in the series that
> > $subject patch depends on - or can I apply this as a standalone mmc
> > patch?
>
> Hey Ulf, thanks for the review. I'll just need to respin these to make
> sure I do not do a dev_pm_opp_of_remove_table() if dev_pm_opp_of_add_table()
> isn;t successful as discussed with Viresh on another thread [1]
>
> As for the dependencies, its only PATCH 01/21 in this series and that's
> already been queued by Viresh [2]

I see, thanks!

Looks like Viresh is sending it to be included for v5.7-rc2, so I can
pick your new version of $subject patch next week.

[...]

Kind regards
Uffe