lkml.org 
[lkml]   [2020]   [May]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 2/6] spi: spi-geni-qcom: Use OPP API to set clk/perf state
On Wed 13 May 22:03 PDT 2020, Rajendra Nayak wrote:

>
> []..
>
> > > spi->bus_num = -1;
> > > spi->dev.of_node = dev->of_node;
> > > @@ -596,6 +607,9 @@ static int spi_geni_probe(struct platform_device *pdev)
> > > spi_geni_probe_runtime_disable:
> > > pm_runtime_disable(dev);
> > > spi_master_put(spi);
> > > + if (mas->se.has_opp_table)
> >
> > Why do you need has_opp_table?
> >
> > Afaict if dev_pm_opp_of_add_table() returns -ENODEV there's no attached
> > opp-table and dev_pm_opp_of_remove_table() is a nop.
>
> Apparently its not. Calling dev_pm_opp_of_remove_table() when dev_pm_opp_of_add_table()
> failed causes use-count mismatch.
> You can see https://lkml.org/lkml/2020/4/15/18 for more details.
>

Must have missed that when I glanced at the code, thanks for clarifying.

Regards,
Bjorn

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

\
 
 \ /
  Last update: 2020-05-14 07:39    [W:0.058 / U:2.772 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site