* [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() [not found] <1654227.8mz0SueHsU@kreacher> @ 2020-02-11 23:28 ` Rafael J. Wysocki 2020-02-12 18:38 ` Greg KH 2020-02-19 1:09 ` Peter Chen 0 siblings, 2 replies; 6+ messages in thread From: Rafael J. Wysocki @ 2020-02-11 23:28 UTC (permalink / raw) To: Linux PM; +Cc: LKML, Amit Kucheria, Peter Chen, linux-usb From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> Call cpu_latency_qos_add/remove_request() instead of pm_qos_add/remove_request(), respectively, because the latter are going to be dropped. No intentional functional impact. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> --- drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c b/drivers/usb/chipidea/ci_hdrc_imx.c index d8e7eb2f97b9..a479af3ae31d 100644 --- a/drivers/usb/chipidea/ci_hdrc_imx.c +++ b/drivers/usb/chipidea/ci_hdrc_imx.c @@ -393,8 +393,7 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) } if (pdata.flags & CI_HDRC_PMQOS) - pm_qos_add_request(&data->pm_qos_req, - PM_QOS_CPU_DMA_LATENCY, 0); + cpu_latency_qos_add_request(&data->pm_qos_req, 0); ret = imx_get_clks(dev); if (ret) @@ -478,7 +477,7 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) /* don't overwrite original ret (cf. EPROBE_DEFER) */ regulator_disable(data->hsic_pad_regulator); if (pdata.flags & CI_HDRC_PMQOS) - pm_qos_remove_request(&data->pm_qos_req); + cpu_latency_qos_remove_request(&data->pm_qos_req); data->ci_pdev = NULL; return ret; } @@ -499,7 +498,7 @@ static int ci_hdrc_imx_remove(struct platform_device *pdev) if (data->ci_pdev) { imx_disable_unprepare_clks(&pdev->dev); if (data->plat_data->flags & CI_HDRC_PMQOS) - pm_qos_remove_request(&data->pm_qos_req); + cpu_latency_qos_remove_request(&data->pm_qos_req); if (data->hsic_pad_regulator) regulator_disable(data->hsic_pad_regulator); } @@ -527,7 +526,7 @@ static int __maybe_unused imx_controller_suspend(struct device *dev) imx_disable_unprepare_clks(dev); if (data->plat_data->flags & CI_HDRC_PMQOS) - pm_qos_remove_request(&data->pm_qos_req); + cpu_latency_qos_remove_request(&data->pm_qos_req); data->in_lpm = true; @@ -547,8 +546,7 @@ static int __maybe_unused imx_controller_resume(struct device *dev) } if (data->plat_data->flags & CI_HDRC_PMQOS) - pm_qos_add_request(&data->pm_qos_req, - PM_QOS_CPU_DMA_LATENCY, 0); + cpu_latency_qos_add_request(&data->pm_qos_req, 0); ret = imx_prepare_enable_clks(dev); if (ret) -- 2.16.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() 2020-02-11 23:28 ` [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() Rafael J. Wysocki @ 2020-02-12 18:38 ` Greg KH 2020-02-18 8:03 ` Peter Chen 2020-02-19 1:09 ` Peter Chen 1 sibling, 1 reply; 6+ messages in thread From: Greg KH @ 2020-02-12 18:38 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: Linux PM, LKML, Amit Kucheria, Peter Chen, linux-usb On Wed, Feb 12, 2020 at 12:28:44AM +0100, Rafael J. Wysocki wrote: > From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > > Call cpu_latency_qos_add/remove_request() instead of > pm_qos_add/remove_request(), respectively, because the > latter are going to be dropped. > > No intentional functional impact. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > --- > drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() 2020-02-12 18:38 ` Greg KH @ 2020-02-18 8:03 ` Peter Chen 2020-02-18 8:08 ` Greg KH 0 siblings, 1 reply; 6+ messages in thread From: Peter Chen @ 2020-02-18 8:03 UTC (permalink / raw) To: Greg KH Cc: Rafael J. Wysocki, Linux PM, LKML, Amit Kucheria, linux-usb@vger.kernel.org On 20-02-12 10:38:27, Greg KH wrote: > On Wed, Feb 12, 2020 at 12:28:44AM +0100, Rafael J. Wysocki wrote: > > From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > > > > Call cpu_latency_qos_add/remove_request() instead of > > pm_qos_add/remove_request(), respectively, because the > > latter are going to be dropped. > > > > No intentional functional impact. > > > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > --- > > drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- > > 1 file changed, 5 insertions(+), 7 deletions(-) > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Hi Greg, With this patch applied, the usb-next can't compile pass. -- Thanks, Peter Chen ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() 2020-02-18 8:03 ` Peter Chen @ 2020-02-18 8:08 ` Greg KH 2020-02-18 8:11 ` Peter Chen 0 siblings, 1 reply; 6+ messages in thread From: Greg KH @ 2020-02-18 8:08 UTC (permalink / raw) To: Peter Chen Cc: Rafael J. Wysocki, Linux PM, LKML, Amit Kucheria, linux-usb@vger.kernel.org On Tue, Feb 18, 2020 at 08:03:13AM +0000, Peter Chen wrote: > On 20-02-12 10:38:27, Greg KH wrote: > > On Wed, Feb 12, 2020 at 12:28:44AM +0100, Rafael J. Wysocki wrote: > > > From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > > > > > > Call cpu_latency_qos_add/remove_request() instead of > > > pm_qos_add/remove_request(), respectively, because the > > > latter are going to be dropped. > > > > > > No intentional functional impact. > > > > > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > > --- > > > drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- > > > 1 file changed, 5 insertions(+), 7 deletions(-) > > > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Hi Greg, > > With this patch applied, the usb-next can't compile pass. Did I apply this? It looks to need to go through Rafael's tree which introduces the new api, right? thanks, greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() 2020-02-18 8:08 ` Greg KH @ 2020-02-18 8:11 ` Peter Chen 0 siblings, 0 replies; 6+ messages in thread From: Peter Chen @ 2020-02-18 8:11 UTC (permalink / raw) To: Greg KH Cc: Rafael J. Wysocki, Linux PM, LKML, Amit Kucheria, linux-usb@vger.kernel.org > > On Tue, Feb 18, 2020 at 08:03:13AM +0000, Peter Chen wrote: > > On 20-02-12 10:38:27, Greg KH wrote: > > > On Wed, Feb 12, 2020 at 12:28:44AM +0100, Rafael J. Wysocki wrote: > > > > From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > > > > > > > > Call cpu_latency_qos_add/remove_request() instead of > > > > pm_qos_add/remove_request(), respectively, because the latter are > > > > going to be dropped. > > > > > > > > No intentional functional impact. > > > > > > > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > > > --- > > > > drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- > > > > 1 file changed, 5 insertions(+), 7 deletions(-) > > > > > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > > Hi Greg, > > > > With this patch applied, the usb-next can't compile pass. > > Did I apply this? It looks to need to go through Rafael's tree which introduces the > new api, right? > Not yet, I just try it at my local for my chipidea tree. Peter ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() 2020-02-11 23:28 ` [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() Rafael J. Wysocki 2020-02-12 18:38 ` Greg KH @ 2020-02-19 1:09 ` Peter Chen 1 sibling, 0 replies; 6+ messages in thread From: Peter Chen @ 2020-02-19 1:09 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Linux PM, LKML, Amit Kucheria, linux-usb@vger.kernel.org On 20-02-12 00:28:44, Rafael J. Wysocki wrote: > From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > > Call cpu_latency_qos_add/remove_request() instead of > pm_qos_add/remove_request(), respectively, because the > latter are going to be dropped. > > No intentional functional impact. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > --- > drivers/usb/chipidea/ci_hdrc_imx.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c b/drivers/usb/chipidea/ci_hdrc_imx.c > index d8e7eb2f97b9..a479af3ae31d 100644 > --- a/drivers/usb/chipidea/ci_hdrc_imx.c > +++ b/drivers/usb/chipidea/ci_hdrc_imx.c > @@ -393,8 +393,7 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) > } > > if (pdata.flags & CI_HDRC_PMQOS) > - pm_qos_add_request(&data->pm_qos_req, > - PM_QOS_CPU_DMA_LATENCY, 0); > + cpu_latency_qos_add_request(&data->pm_qos_req, 0); > > ret = imx_get_clks(dev); > if (ret) > @@ -478,7 +477,7 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) > /* don't overwrite original ret (cf. EPROBE_DEFER) */ > regulator_disable(data->hsic_pad_regulator); > if (pdata.flags & CI_HDRC_PMQOS) > - pm_qos_remove_request(&data->pm_qos_req); > + cpu_latency_qos_remove_request(&data->pm_qos_req); > data->ci_pdev = NULL; > return ret; > } > @@ -499,7 +498,7 @@ static int ci_hdrc_imx_remove(struct platform_device *pdev) > if (data->ci_pdev) { > imx_disable_unprepare_clks(&pdev->dev); > if (data->plat_data->flags & CI_HDRC_PMQOS) > - pm_qos_remove_request(&data->pm_qos_req); > + cpu_latency_qos_remove_request(&data->pm_qos_req); > if (data->hsic_pad_regulator) > regulator_disable(data->hsic_pad_regulator); > } > @@ -527,7 +526,7 @@ static int __maybe_unused imx_controller_suspend(struct device *dev) > > imx_disable_unprepare_clks(dev); > if (data->plat_data->flags & CI_HDRC_PMQOS) > - pm_qos_remove_request(&data->pm_qos_req); > + cpu_latency_qos_remove_request(&data->pm_qos_req); > > data->in_lpm = true; > > @@ -547,8 +546,7 @@ static int __maybe_unused imx_controller_resume(struct device *dev) > } > > if (data->plat_data->flags & CI_HDRC_PMQOS) > - pm_qos_add_request(&data->pm_qos_req, > - PM_QOS_CPU_DMA_LATENCY, 0); > + cpu_latency_qos_add_request(&data->pm_qos_req, 0); > > ret = imx_prepare_enable_clks(dev); > if (ret) > -- > 2.16.4 > Acked-by: Peter Chen <peter.chen@nxp.com> -- Thanks, Peter Chen ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-02-19 1:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1654227.8mz0SueHsU@kreacher>
2020-02-11 23:28 ` [PATCH 23/28] drivers: usb: Call cpu_latency_qos_*() instead of pm_qos_*() Rafael J. Wysocki
2020-02-12 18:38 ` Greg KH
2020-02-18 8:03 ` Peter Chen
2020-02-18 8:08 ` Greg KH
2020-02-18 8:11 ` Peter Chen
2020-02-19 1:09 ` Peter Chen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).