All of lore.kernel.org
 help / color / mirror / Atom feed
From: Santosh Shilimkar <santosh.shilimkar-l0cyMroinI0@public.gmane.org>
To: Peter De Schrijver
	<pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: Colin Cross <ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>,
	"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	Joseph Lo <josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH V3 1/5] ARM: tegra: add pending SGI checking API
Date: Thu, 20 Dec 2012 17:36:35 +0530	[thread overview]
Message-ID: <50D2FF4B.5010401@ti.com> (raw)
In-Reply-To: <20121220111409.GH19258-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>

On Thursday 20 December 2012 04:44 PM, Peter De Schrijver wrote:
>>>
>>> So why does OMAP4 have working coupled idle without the SGI check then?
>>>
>> Because whenever the OMAP4 CPUs enter into any power states apart from
>> ON, clock-domain force wakeup method us being used to wakeup
>> secondary CPUs. And secondly CPU RET is not supported on purpose because
>> of some known IP block issues.
>>
>
> Aha. That explains indeed why you don't need the check.
>
Just for the information on OMAP5 SOC, CPU RET is supported and the
couple idle works on it without IPI. As I mentioned, we do use
clock-domain force wakeup method for wakeup. Code is out of the
tree hence i didn't mention that in first place.

Regards
Santosh

WARNING: multiple messages have this Message-ID (diff)
From: santosh.shilimkar@ti.com (Santosh Shilimkar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V3 1/5] ARM: tegra: add pending SGI checking API
Date: Thu, 20 Dec 2012 17:36:35 +0530	[thread overview]
Message-ID: <50D2FF4B.5010401@ti.com> (raw)
In-Reply-To: <20121220111409.GH19258@tbergstrom-lnx.Nvidia.com>

On Thursday 20 December 2012 04:44 PM, Peter De Schrijver wrote:
>>>
>>> So why does OMAP4 have working coupled idle without the SGI check then?
>>>
>> Because whenever the OMAP4 CPUs enter into any power states apart from
>> ON, clock-domain force wakeup method us being used to wakeup
>> secondary CPUs. And secondly CPU RET is not supported on purpose because
>> of some known IP block issues.
>>
>
> Aha. That explains indeed why you don't need the check.
>
Just for the information on OMAP5 SOC, CPU RET is supported and the
couple idle works on it without IPI. As I mentioned, we do use
clock-domain force wakeup method for wakeup. Code is out of the
tree hence i didn't mention that in first place.

Regards
Santosh

  parent reply	other threads:[~2012-12-20 12:06 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-18  2:30 [PATCH V3 0/5] ARM: tegra20: cpuidle: add power-down state Joseph Lo
2012-12-18  2:30 ` Joseph Lo
     [not found] ` <1355797861-12759-1-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2012-12-18  2:30   ` [PATCH V3 1/5] ARM: tegra: add pending SGI checking API Joseph Lo
2012-12-18  2:30     ` Joseph Lo
2012-12-18  2:42     ` Colin Cross
2012-12-18  2:42       ` Colin Cross
     [not found]       ` <CAMbhsRSorDf-BebFn+t39zOK4VX4N0Fu0iPrM7XpKZcssTgX9A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-12-18  2:57         ` Joseph Lo
2012-12-18  2:57           ` Joseph Lo
2012-12-18 10:15         ` Peter De Schrijver
2012-12-18 10:15           ` Peter De Schrijver
     [not found]           ` <20121218101548.GO19258-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-12-18 19:36             ` Colin Cross
2012-12-18 19:36               ` Colin Cross
     [not found]               ` <CAMbhsRSDwSoHoEs93NZhDchCS28-vQCuW2p+kqfNKeqLygL2sQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-12-19  1:06                 ` Joseph Lo
2012-12-19  1:06                   ` Joseph Lo
     [not found]                   ` <1355879172.4449.10.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2012-12-19  3:47                     ` Joseph Lo
2012-12-19  3:47                       ` Joseph Lo
2012-12-20  7:16                 ` Santosh Shilimkar
2012-12-20  7:16                   ` Santosh Shilimkar
     [not found]                   ` <50D2BB51.9000109-l0cyMroinI0@public.gmane.org>
2012-12-20  9:34                     ` Peter De Schrijver
2012-12-20  9:34                       ` Peter De Schrijver
     [not found]                       ` <20121220093412.GE19258-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-12-20  9:49                         ` Santosh Shilimkar
2012-12-20  9:49                           ` Santosh Shilimkar
     [not found]                           ` <50D2DF45.1000305-l0cyMroinI0@public.gmane.org>
2012-12-20  9:59                             ` Peter De Schrijver
2012-12-20  9:59                               ` Peter De Schrijver
     [not found]                               ` <20121220095951.GG19258-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-12-20 10:24                                 ` Santosh Shilimkar
2012-12-20 10:24                                   ` Santosh Shilimkar
     [not found]                                   ` <50D2E76C.7050309-l0cyMroinI0@public.gmane.org>
2012-12-20 11:14                                     ` Peter De Schrijver
2012-12-20 11:14                                       ` Peter De Schrijver
     [not found]                                       ` <20121220111409.GH19258-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-12-20 12:06                                         ` Santosh Shilimkar [this message]
2012-12-20 12:06                                           ` Santosh Shilimkar
2012-12-18  2:30   ` [PATCH V3 2/5] ARM: tegra20: cpuidle: add powered-down state for secondary CPU Joseph Lo
2012-12-18  2:30     ` Joseph Lo
2012-12-18  2:46     ` Colin Cross
2012-12-18  2:46       ` Colin Cross
     [not found]       ` <CAMbhsRTLo6jS_7hqVVtgaa8_ZweAwicrgghaaFWfnzHpc6f1Pg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-12-18  3:06         ` Joseph Lo
2012-12-18  3:06           ` Joseph Lo
     [not found]     ` <1355797861-12759-3-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2012-12-20 17:43       ` Stephen Warren
2012-12-20 17:43         ` Stephen Warren
     [not found]         ` <50D34E28.5000208-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-12-21  6:36           ` Joseph Lo
2012-12-21  6:36             ` Joseph Lo
     [not found]             ` <1356071804.26029.34.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2012-12-21 21:04               ` Stephen Warren
2012-12-21 21:04                 ` Stephen Warren
2012-12-18  2:30   ` [PATCH V3 3/5] ARM: tegra20: clocks: add CPU low-power function into tegra_cpu_car_ops Joseph Lo
2012-12-18  2:30     ` Joseph Lo
     [not found]     ` <1355797861-12759-4-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2012-12-20 17:46       ` Stephen Warren
2012-12-20 17:46         ` Stephen Warren
     [not found]         ` <50D34F12.9080801-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-12-21  5:02           ` Joseph Lo
2012-12-21  5:02             ` Joseph Lo
     [not found]             ` <1356066131.26029.26.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2012-12-21 21:10               ` Stephen Warren
2012-12-21 21:10                 ` Stephen Warren
2012-12-18  2:31   ` [PATCH V3 4/5] ARM: tegra20: flowctrl: add support for cpu_suspend_enter/exit Joseph Lo
2012-12-18  2:31     ` Joseph Lo
2012-12-18  2:31   ` [PATCH V3 5/5] ARM: tegra20: cpuidle: apply coupled cpuidle for powered-down mode Joseph Lo
2012-12-18  2:31     ` Joseph Lo
     [not found]     ` <1355797861-12759-6-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2012-12-18 10:18       ` Peter De Schrijver
2012-12-18 10:18         ` Peter De Schrijver
2012-12-20 17:54       ` Stephen Warren
2012-12-20 17:54         ` Stephen Warren
     [not found]         ` <50D350C1.6070400-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-12-21  7:10           ` Joseph Lo
2012-12-21  7:10             ` Joseph Lo
     [not found]             ` <1356073828.26029.49.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2012-12-21 21:06               ` Stephen Warren
2012-12-21 21:06                 ` Stephen Warren
2013-01-02 19:05   ` [PATCH V3 0/5] ARM: tegra20: cpuidle: add power-down state Stephen Warren
2013-01-02 19:05     ` Stephen Warren
     [not found]     ` <50E484ED.90108-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-01-03  8:39       ` Joseph Lo
2013-01-03  8:39         ` Joseph Lo

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=50D2FF4B.5010401@ti.com \
    --to=santosh.shilimkar-l0cymroini0@public.gmane.org \
    --cc=ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
    --cc=josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@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.