All of lore.kernel.org
 help / color / mirror / Atom feed
From: Colin Cross <ccross@android.com>
To: Stephen Boyd <sboyd@codeaurora.org>
Cc: linux-tegra@vger.kernel.org,
	Russell King <linux@arm.linux.org.uk>,
	konkers@android.com, Gary King <gking@nvidia.com>,
	linux-kernel@vger.kernel.org, olof@lixom.net,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 18/28] ARM: tegra: add CPU_IDLE driver
Date: Thu, 27 Jan 2011 14:07:01 -0800	[thread overview]
Message-ID: <AANLkTimZcg7Q-Tc=P7zf7=RQsyKJg_amWogP2Jafgxu0@mail.gmail.com> (raw)
In-Reply-To: <AANLkTi=o5o2_4Ur2GTS=0XSoYqFMYqSF99aU9N_wxWuu@mail.gmail.com>

On Thu, Jan 27, 2011 at 1:51 PM, Colin Cross <ccross@android.com> wrote:
> On Thu, Jan 27, 2011 at 12:38 PM, Stephen Boyd <sboyd@codeaurora.org> wrote:
>> On 01/23/2011 06:01 PM, Colin Cross wrote:
>>>
>>> +static int __init tegra_cpuidle_debug_init(void)
>>> +{
>>> +     struct dentry *dir;
>>> +     struct dentry *d;
>>> +
>>> +     dir = debugfs_create_dir("cpuidle", NULL);
>>> +     if (!dir)
>>> +             return -ENOMEM;
>>> +
>>> +     d = debugfs_create_file("lp2", S_IRUGO, dir, NULL,
>>> +             &tegra_lp2_debug_ops);
>>> +     if (!d)
>>> +             return -ENOMEM;
>>> +
>>> +     return 0;
>>> +}
>>> +#endif
>>> +
>>> +late_initcall(tegra_cpuidle_debug_init);
>>
>> Will this compile with CONFIG_DEBUG_FS=n?
>
> Yes.  All debugfs ops return ERR_PTR(-ENODEV) if CONFIG_DEBUG_FS=n, so
> tegra_cpuidle_debug_init will do nothing and return 0.
>

Oops no, it will not compile.  Will fix.

WARNING: multiple messages have this Message-ID (diff)
From: ccross@android.com (Colin Cross)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 18/28] ARM: tegra: add CPU_IDLE driver
Date: Thu, 27 Jan 2011 14:07:01 -0800	[thread overview]
Message-ID: <AANLkTimZcg7Q-Tc=P7zf7=RQsyKJg_amWogP2Jafgxu0@mail.gmail.com> (raw)
In-Reply-To: <AANLkTi=o5o2_4Ur2GTS=0XSoYqFMYqSF99aU9N_wxWuu@mail.gmail.com>

On Thu, Jan 27, 2011 at 1:51 PM, Colin Cross <ccross@android.com> wrote:
> On Thu, Jan 27, 2011 at 12:38 PM, Stephen Boyd <sboyd@codeaurora.org> wrote:
>> On 01/23/2011 06:01 PM, Colin Cross wrote:
>>>
>>> +static int __init tegra_cpuidle_debug_init(void)
>>> +{
>>> + ? ? struct dentry *dir;
>>> + ? ? struct dentry *d;
>>> +
>>> + ? ? dir = debugfs_create_dir("cpuidle", NULL);
>>> + ? ? if (!dir)
>>> + ? ? ? ? ? ? return -ENOMEM;
>>> +
>>> + ? ? d = debugfs_create_file("lp2", S_IRUGO, dir, NULL,
>>> + ? ? ? ? ? ? &tegra_lp2_debug_ops);
>>> + ? ? if (!d)
>>> + ? ? ? ? ? ? return -ENOMEM;
>>> +
>>> + ? ? return 0;
>>> +}
>>> +#endif
>>> +
>>> +late_initcall(tegra_cpuidle_debug_init);
>>
>> Will this compile with CONFIG_DEBUG_FS=n?
>
> Yes. ?All debugfs ops return ERR_PTR(-ENODEV) if CONFIG_DEBUG_FS=n, so
> tegra_cpuidle_debug_init will do nothing and return 0.
>

Oops no, it will not compile.  Will fix.

  reply	other threads:[~2011-01-27 22:07 UTC|newest]

Thread overview: 137+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-24  2:01 [PATCH v2 00/28] Updates for Tegra support in 2.6.39 Colin Cross
2011-01-24  2:01 ` [PATCH v2 01/28] ARM: tegra: irq: Rename gic pointers to avoid conflicts Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-26  7:10   ` Colin Cross
2011-01-26  7:10     ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 02/28] ARM: gic: Add functions to save and restore gic state Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-02-01 13:03   ` Russell King - ARM Linux
2011-02-01 13:03     ` Russell King - ARM Linux
2011-01-24  2:01 ` [PATCH v2 03/28] ARM: gic: Export irq chip functions Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  6:39   ` Santosh Shilimkar
2011-01-24  6:39     ` Santosh Shilimkar
2011-01-24  2:01 ` [PATCH v2 04/28] ARM: mm: cache-l2x0: Add support for re-enabling l2x0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-25 15:14   ` Catalin Marinas
2011-01-25 15:14     ` Catalin Marinas
2011-01-25 15:41     ` Russell King - ARM Linux
2011-01-25 15:41       ` Russell King - ARM Linux
2011-01-25 18:14       ` Catalin Marinas
2011-01-25 18:14         ` Catalin Marinas
2011-01-25 18:32         ` Santosh Shilimkar
2011-01-25 18:32           ` Santosh Shilimkar
2011-01-25 18:39           ` Will Deacon
     [not found]           ` <-8932138696981683633@unknownmsgid>
2011-02-04 23:32             ` Colin Cross
2011-02-04 23:32               ` Colin Cross
2011-02-04 23:32               ` Colin Cross
2011-02-04 23:43               ` Russell King - ARM Linux
2011-02-04 23:43                 ` Russell King - ARM Linux
     [not found]                 ` <20110204234331.GF8732-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-02-05  1:44                   ` Colin Cross
2011-02-05  1:44                     ` Colin Cross
2011-02-05  1:44                     ` Colin Cross
     [not found]                     ` <AANLkTi=fHnivHXHnYrQvdP6JWbEA3t1X3DuBxj5gN3H0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-05  7:51                       ` Santosh Shilimkar
2011-02-05  7:51                         ` Santosh Shilimkar
2011-02-05  7:51                         ` Santosh Shilimkar
     [not found]                         ` <1bebe4b5c8590059b70a146d5486fa6a-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-05  9:47                           ` Russell King - ARM Linux
2011-02-05  9:47                             ` Russell King - ARM Linux
2011-02-05  9:47                             ` Russell King - ARM Linux
2011-02-05 10:41                             ` [PATCH v2 04/28] ARM: mm: cache-l2x0: Add support forre-enabling l2x0 Santosh Shilimkar
2011-02-05 10:41                               ` Santosh Shilimkar
2011-02-05 16:36                               ` Colin Cross
2011-02-05 16:36                                 ` Colin Cross
     [not found]                                 ` <AANLkTik_r4k_5o+F47vRbGPcWLwfgHgWqhym49XfhBZ9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-07  6:13                                   ` Santosh Shilimkar
2011-02-07  6:13                                     ` Santosh Shilimkar
2011-02-07  6:13                                     ` Santosh Shilimkar
2011-01-24  2:01 ` [PATCH v2 05/28] ARM: tegra: Centralize macros to define debug uart base Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 06/28] ARM: tegra: Add api to control internal powergating Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 07/28] ARM: tegra: irqs: Update irq list Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 08/28] ARM: tegra: Add prototypes for subsystem suspend functions Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 09/28] ARM: tegra: clock: Suspend fixes, and add new clocks Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 10/28] ARM: tegra: pinmux: Add missing drive pingroups and fix suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 11/28] ARM: tegra: timer: Add idle and suspend support to timers Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 12/28] ARM: tegra: irq: Add support for suspend wake sources Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 13/28] ARM: tegra: Add suspend and hotplug support Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  7:31   ` Colin Cross
2011-01-24  7:31     ` Colin Cross
2011-01-24  9:07   ` Russell King - ARM Linux
2011-01-24  9:07     ` Russell King - ARM Linux
2011-01-24  9:26     ` Colin Cross
2011-01-24  9:26       ` Colin Cross
2011-01-24 10:15       ` Russell King - ARM Linux
2011-01-24 10:15         ` Russell King - ARM Linux
2011-01-24  2:01 ` [PATCH v2 14/28] ARM: tegra: irq: Add set_wake and set_type support for suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 15/28] ARM: tegra: irq: Add debugfs file to show wake irqs Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-27 19:45   ` Stephen Boyd
2011-01-27 19:45     ` Stephen Boyd
2011-01-24  2:01 ` [PATCH v2 16/28] ARM: tegra: irq: Implement retrigger Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 17/28] ARM: tegra: gpio: Add support for waking from suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 18/28] ARM: tegra: add CPU_IDLE driver Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-27 20:38   ` Stephen Boyd
2011-01-27 20:38     ` Stephen Boyd
2011-01-27 21:51     ` Colin Cross
2011-01-27 21:51       ` Colin Cross
2011-01-27 22:07       ` Colin Cross [this message]
2011-01-27 22:07         ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 19/28] ARM: tegra: iomap: Add missing devices, fix use of SZ_8, SZ_64 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 20/28] ARM: tegra: cpufreq: Disable cpufreq during suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24 14:41   ` Mark Brown
2011-01-24 14:41     ` Mark Brown
2011-01-24 18:50     ` Colin Cross
2011-01-24 18:50       ` Colin Cross
2011-01-24 19:35       ` Mark Brown
2011-01-24 19:35         ` Mark Brown
2011-01-24 19:52         ` Colin Cross
2011-01-24 19:52           ` Colin Cross
2011-01-24 20:26           ` Mark Brown
2011-01-24 20:26             ` Mark Brown
2011-01-24 20:52             ` Colin Cross
2011-01-24 20:52               ` Colin Cross
2011-01-24 21:08               ` Mark Brown
2011-01-24 21:08                 ` Mark Brown
2011-01-24 21:24                 ` Colin Cross
2011-01-24 21:24                   ` Colin Cross
2011-01-25  4:26         ` Kyungmin Park
2011-01-25  4:26           ` Kyungmin Park
2011-01-24  2:01 ` [PATCH v2 21/28] ARM: tegra: Allow overriding arch_reset Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 22/28] ARM: tegra: dma: Fix critical data corruption bugs Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 23/28] ARM: tegra: add tegra_defconfig Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 24/28] ARM: tegra: Use writel_relaxed in tegra_init_cache Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 25/28] ARM: tegra: suspend: Save protected aperture across LP0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 26/28] ARM: tegra: suspend: protect suspend functions with CONFIG_PM Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 27/28] ARM: tegra: enable emc clock updates after lp0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 28/28] ARM: tegra: clock: Add forward reference to struct clk Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24 10:26   ` Russell King - ARM Linux
2011-01-24 10:26     ` Russell King - ARM Linux
2011-01-25  1:23     ` Colin Cross
2011-01-25  1:23       ` Colin Cross
2011-01-26  3:19   ` Grant Likely
2011-01-26  3:19     ` Grant Likely
2011-01-26  3:21     ` Colin Cross
2011-01-26  3:21       ` Colin Cross
2011-01-26  4:16       ` Grant Likely
2011-01-26  4:16         ` Grant Likely

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='AANLkTimZcg7Q-Tc=P7zf7=RQsyKJg_amWogP2Jafgxu0@mail.gmail.com' \
    --to=ccross@android.com \
    --cc=gking@nvidia.com \
    --cc=konkers@android.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=olof@lixom.net \
    --cc=sboyd@codeaurora.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.