All of lore.kernel.org
 help / color / mirror / Atom feed
From: vipul kumar samar <vipulkumar.samar-qxv4g6HH51o@public.gmane.org>
To: Ulf HANSSON <ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
Cc: Ulf Hansson <ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Viresh Kumar
	<viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Linus Walleij
	<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org"
	<spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH 2/3] Revert "spi/pl022: enable runtime PM"
Date: Fri, 28 Sep 2012 17:20:41 +0530	[thread overview]
Message-ID: <50658F11.5010405@st.com> (raw)
In-Reply-To: <1348831266-16721-3-git-send-email-ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>

Hello ulf,

On 9/28/2012 4:51 PM, Ulf HANSSON wrote:
> From: Ulf Hansson<ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
>
> This reverts commit 2fb30d1147c599f5657e8c62c862f9a0f58d9d99.
>
> Signed-off-by: Ulf Hansson<ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> ---
>   drivers/spi/spi-pl022.c |    4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c
> index 3f2f36c..05cfb61e 100644
> --- a/drivers/spi/spi-pl022.c
> +++ b/drivers/spi/spi-pl022.c
> @@ -2188,9 +2188,6 @@ pl022_probe(struct amba_device *adev, const struct amba_id *id)
>   	printk(KERN_INFO "pl022: mapped registers from 0x%08x to %p\n",
>   	       adev->res.start, pl022->virtbase);
>
> -	pm_runtime_enable(dev);
> -	pm_runtime_resume(dev);
> -
>   	pl022->clk = clk_get(&adev->dev, NULL);
>   	if (IS_ERR(pl022->clk)) {
>   		status = PTR_ERR(pl022->clk);
> @@ -2303,7 +2300,6 @@ pl022_remove(struct amba_device *adev)
>   	clk_disable(pl022->clk);
>   	clk_unprepare(pl022->clk);
>   	clk_put(pl022->clk);
> -	pm_runtime_disable(&adev->dev);
>   	iounmap(pl022->virtbase);
>   	amba_release_regions(adev);
>   	tasklet_disable(&pl022->pump_transfers);

sorry..Now i got it you revert complete rumtime patch.

Regards
Vipul Samar

------------------------------------------------------------------------------
Got visibility?
Most devs has no idea what their production app looks like.
Find out how fast your code is with AppDynamics Lite.
http://ad.doubleclick.net/clk;262219671;13503038;y?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html

WARNING: multiple messages have this Message-ID (diff)
From: vipulkumar.samar@st.com (vipul kumar samar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/3] Revert "spi/pl022: enable runtime PM"
Date: Fri, 28 Sep 2012 17:20:41 +0530	[thread overview]
Message-ID: <50658F11.5010405@st.com> (raw)
In-Reply-To: <1348831266-16721-3-git-send-email-ulf.hansson@stericsson.com>

Hello ulf,

On 9/28/2012 4:51 PM, Ulf HANSSON wrote:
> From: Ulf Hansson<ulf.hansson@linaro.org>
>
> This reverts commit 2fb30d1147c599f5657e8c62c862f9a0f58d9d99.
>
> Signed-off-by: Ulf Hansson<ulf.hansson@linaro.org>
> ---
>   drivers/spi/spi-pl022.c |    4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c
> index 3f2f36c..05cfb61e 100644
> --- a/drivers/spi/spi-pl022.c
> +++ b/drivers/spi/spi-pl022.c
> @@ -2188,9 +2188,6 @@ pl022_probe(struct amba_device *adev, const struct amba_id *id)
>   	printk(KERN_INFO "pl022: mapped registers from 0x%08x to %p\n",
>   	       adev->res.start, pl022->virtbase);
>
> -	pm_runtime_enable(dev);
> -	pm_runtime_resume(dev);
> -
>   	pl022->clk = clk_get(&adev->dev, NULL);
>   	if (IS_ERR(pl022->clk)) {
>   		status = PTR_ERR(pl022->clk);
> @@ -2303,7 +2300,6 @@ pl022_remove(struct amba_device *adev)
>   	clk_disable(pl022->clk);
>   	clk_unprepare(pl022->clk);
>   	clk_put(pl022->clk);
> -	pm_runtime_disable(&adev->dev);
>   	iounmap(pl022->virtbase);
>   	amba_release_regions(adev);
>   	tasklet_disable(&pl022->pump_transfers);

sorry..Now i got it you revert complete rumtime patch.

Regards
Vipul Samar

  parent reply	other threads:[~2012-09-28 11:50 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-28 11:21 [PATCH 0/3] Fixup use of runtime pm Ulf Hansson
2012-09-28 11:21 ` Ulf Hansson
     [not found] ` <1348831266-16721-1-git-send-email-ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
2012-09-28 11:21   ` [PATCH 1/3] Revert "spi/pl022: fix spi-pl022 pm enable at probe" Ulf Hansson
2012-09-28 11:21     ` Ulf Hansson
     [not found]     ` <1348831266-16721-2-git-send-email-ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
2012-09-28 11:45       ` vipul kumar samar
2012-09-28 11:45         ` vipul kumar samar
2012-09-30  8:44       ` Linus Walleij
2012-09-30  8:44         ` Linus Walleij
     [not found]         ` <CACRpkdb+bnCzYV2Dzqou5kGyc1hrh0VT-pYXZm1auu4uik2GNg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-09-30 10:14           ` Russell King - ARM Linux
2012-09-30 10:14             ` Russell King - ARM Linux
     [not found]             ` <20120930101418.GF6933-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2012-09-30 21:43               ` Linus Walleij
2012-09-30 21:43                 ` Linus Walleij
2012-10-03 13:43       ` Ulf Hansson
2012-10-03 13:43         ` Ulf Hansson
     [not found]         ` <CAPDyKFo5-UeUe7PkxCBMnRziWh+eSC_ZjJ0pKhcBkHUkFdSH+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-10-03 13:55           ` Mark Brown
2012-10-03 13:55             ` Mark Brown
     [not found]             ` <20121003135556.GE4360-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2012-10-03 14:28               ` Ulf Hansson
2012-10-03 14:28                 ` Ulf Hansson
     [not found]                 ` <CAPDyKFrQ9UhEc_jEKdR=J8T-NUxsYQHB-yMh2vTyxJ+y+R38Og-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-10-03 14:29                   ` Mark Brown
2012-10-03 14:29                     ` Mark Brown
2012-09-28 11:21   ` [PATCH 2/3] Revert "spi/pl022: enable runtime PM" Ulf Hansson
2012-09-28 11:21     ` Ulf Hansson
     [not found]     ` <1348831266-16721-3-git-send-email-ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
2012-09-28 11:50       ` vipul kumar samar [this message]
2012-09-28 11:50         ` vipul kumar samar
2012-09-30 21:46       ` Linus Walleij
2012-09-30 21:46         ` Linus Walleij
2012-09-28 11:21   ` [PATCH 3/3] spi: spi-pl022: Minor simplification for runtime pm Ulf Hansson
2012-09-28 11:21     ` Ulf Hansson
     [not found]     ` <1348831266-16721-4-git-send-email-ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
2012-09-30 21:47       ` Linus Walleij
2012-09-30 21:47         ` Linus Walleij
2012-09-28 11:25   ` [PATCH 0/3] Fixup use of " Ulf Hansson
2012-09-28 11:25     ` Ulf Hansson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50658F11.5010405@st.com \
    --to=vipulkumar.samar-qxv4g6hh51o@public.gmane.org \
    --cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=ulf.hansson-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org \
    --cc=ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.