All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Ingo Molnar <mingo@elte.hu>
Cc: Pavel Machek <pavel@ucw.cz>,
	linux-pm@lists.linux-foundation.org,
	kernel list <linux-kernel@vger.kernel.org>
Subject: Re: [linux-pm] sleepy linux self-test
Date: Sat, 2 Feb 2008 23:18:02 -0800	[thread overview]
Message-ID: <200802022318.03597.david-b@pacbell.net> (raw)
In-Reply-To: <20080203051425.GA20740@elte.hu>

On Saturday 02 February 2008, Ingo Molnar wrote:
> > [   23.509562] Calling initcall 0xc0c49e00: be_sleepy+0x0/0x170()
> > [   23.515837] PM: no wakelarm-capable RTC
> > [   23.517562] initcall 0xc0c49e00: be_sleepy+0x0/0x170() returned 0.

Because CONFIG_RTC_DRV_CMOS was not configured, though
you had several other RTC drivers ready.

> # CONFIG_RTC_DRV_CMOS is not set

Then later you had that enabled, but you also had the
legacy driver enabled:

> CONFIG_RTC=y

So that was the driver which succesfully bound to the RTC
(since it's probed first) and prevented the more generic
code from kicking in.


Nobody has yet submitted a patch to help arbitrate between the
legacy and "newfangled" RTC drivers.  The general "how to Kconfig"
policy has been to expect a lot (too much?) from users, but this
type of annoyance is too common.

Maybe we need something to more actively avoid the "two drivers"
on PC hardware ... like the appended.  (The "comment" is there as
a reminder to folk who still look to that menu for RTC stuff.)

- Dave


--- g26.orig/drivers/char/Kconfig
+++ g26/drivers/char/Kconfig
@@ -715,9 +715,12 @@ config NVRAM
 	  To compile this driver as a module, choose M here: the
 	  module will be called nvram.
 
+comment "You are using the RTC framework, not the legacy CMOS RTC driver"
+	depends on RTC_DRV_CMOS
+
 config RTC
 	tristate "Enhanced Real Time Clock Support"
-	depends on !PPC && !PARISC && !IA64 && !M68K && !SPARC && !FRV && !ARM && !SUPERH && !S390
+	depends on !PPC && !PARISC && !IA64 && !M68K && !SPARC && !FRV && !ARM && !SUPERH && !S390 && !RTC_DRV_CMOS
 	---help---
 	  If you say Y here and create a character special file /dev/rtc with
 	  major number 10 and minor number 135 using mknod ("man mknod"), you
--- g26.orig/drivers/rtc/Kconfig
+++ g26/drivers/rtc/Kconfig
@@ -281,6 +281,7 @@ comment "Platform RTC drivers"
 config RTC_DRV_CMOS
 	tristate "PC-style 'CMOS'"
 	depends on X86 || ALPHA || ARM || M32R || ATARI || PPC || MIPS
+	default y if X86
 	help
 	  Say "yes" here to get direct support for the real time clock
 	  found in every PC or ACPI-based system, and some other boards.
 

  parent reply	other threads:[~2008-02-03  7:18 UTC|newest]

Thread overview: 102+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-30 13:17 sleepy linux self-test Pavel Machek
2008-01-30 16:35 ` Ingo Molnar
2008-01-30 16:35   ` Ingo Molnar
2008-01-30 16:39   ` Pavel Machek
2008-01-30 19:36 ` Ingo Molnar
2008-01-30 23:26   ` Pavel Machek
2008-02-01 14:22     ` Ingo Molnar
2008-02-02 12:45       ` Pavel Machek
2008-02-02 13:49         ` Ingo Molnar
2008-02-02 13:51         ` Ingo Molnar
2008-02-01  1:55 ` David Brownell
2008-02-01  1:55 ` [linux-pm] " David Brownell
2008-02-02 12:47   ` Pavel Machek
2008-02-02 12:47   ` [linux-pm] " Pavel Machek
2008-02-02 13:50     ` Ingo Molnar
2008-02-02 13:50     ` [linux-pm] " Ingo Molnar
2008-02-02 17:49       ` David Brownell
2008-02-02 18:06         ` Ingo Molnar
2008-02-02 19:47           ` David Brownell
2008-02-02 19:47           ` [linux-pm] " David Brownell
2008-02-02 18:06         ` Ingo Molnar
2008-02-02 17:49       ` David Brownell
2008-02-02 17:31     ` [linux-pm] " David Brownell
2008-02-02 17:51       ` David Brownell
2008-02-02 17:51       ` David Brownell
2008-02-02 18:00       ` [linux-pm] " Ingo Molnar
2008-02-02 19:13         ` David Brownell
2008-02-02 19:13         ` [linux-pm] " David Brownell
2008-02-02 19:32           ` Pavel Machek
2008-02-02 19:32           ` [linux-pm] " Pavel Machek
2008-02-02 19:38             ` Ingo Molnar
2008-02-02 19:38             ` [linux-pm] " Ingo Molnar
2008-02-02 19:59               ` Pavel Machek
2008-02-02 19:59               ` [linux-pm] " Pavel Machek
2008-02-03  2:37               ` David Brownell
2008-02-03  2:37               ` [linux-pm] " David Brownell
2008-02-03  5:05                 ` Ingo Molnar
2008-02-03  5:14                   ` Ingo Molnar
2008-02-03  5:19                     ` Ingo Molnar
2008-02-03  5:19                     ` [linux-pm] " Ingo Molnar
2008-02-03  5:35                       ` Ingo Molnar
2008-02-03  5:35                       ` [linux-pm] " Ingo Molnar
2008-02-03  5:54                         ` Ingo Molnar
2008-02-03  5:54                         ` [linux-pm] " Ingo Molnar
2008-02-03  7:05                           ` Ingo Molnar
2008-02-03  7:05                           ` [linux-pm] " Ingo Molnar
2008-02-03  7:32                             ` David Brownell
2008-02-03  7:32                             ` [linux-pm] " David Brownell
2008-02-03 12:21                               ` Rafael J. Wysocki
2008-02-03 13:16                                 ` David Brownell
2008-02-03 21:29                                   ` Rafael J. Wysocki
2008-02-03 21:29                                   ` [linux-pm] " Rafael J. Wysocki
2008-02-03 22:42                                     ` David Brownell
2008-02-03 22:43                                       ` Rafael J. Wysocki
2008-02-03 22:43                                       ` [linux-pm] " Rafael J. Wysocki
2008-02-03 22:48                                       ` Pavel Machek
2008-02-03 22:48                                       ` [linux-pm] " Pavel Machek
2008-02-03 23:08                                         ` David Brownell
2008-02-03 23:08                                         ` [linux-pm] " David Brownell
2008-02-10 21:03                                           ` Pavel Machek
2008-02-10 21:03                                           ` [linux-pm] " Pavel Machek
2008-02-03 22:42                                     ` David Brownell
2008-02-18  8:56                                   ` Pavel Machek
2008-02-18  8:56                                   ` [linux-pm] " Pavel Machek
2008-02-18  9:46                                     ` [patch] suspend/resume self-test Ingo Molnar
2008-02-18  9:53                                       ` Pavel Machek
2008-02-18  9:53                                       ` Pavel Machek
2008-02-18 10:40                                         ` David Brownell
2008-02-18 11:04                                           ` Rafael J. Wysocki
2008-02-18 11:04                                           ` Rafael J. Wysocki
2008-02-18 13:09                                           ` Ingo Molnar
2008-02-18 20:16                                             ` David Brownell
2008-02-18 20:16                                             ` David Brownell
2008-02-19 10:11                                               ` Pavel Machek
2008-02-19 10:11                                                 ` Pavel Machek
2008-02-19 14:43                                                 ` Ingo Molnar
2008-02-19 14:43                                                 ` Ingo Molnar
2008-02-19 19:12                                                   ` David Brownell
2008-02-19 19:12                                                   ` David Brownell
2008-02-20 10:15                                                     ` Ingo Molnar
2008-02-20 10:15                                                     ` Ingo Molnar
2008-02-19 14:40                                               ` Ingo Molnar
2008-02-19 14:40                                               ` Ingo Molnar
2008-02-18 13:09                                           ` Ingo Molnar
2008-02-18 10:40                                         ` David Brownell
2008-02-18 11:06                                       ` Rafael J. Wysocki
2008-02-18 11:06                                       ` Rafael J. Wysocki
2008-02-18  9:46                                     ` Ingo Molnar
2008-02-03 13:16                                 ` sleepy linux self-test David Brownell
2008-02-03 12:21                               ` Rafael J. Wysocki
2008-02-10 21:02                           ` Pavel Machek
2008-02-10 21:02                           ` [linux-pm] " Pavel Machek
2008-02-03  7:18                     ` David Brownell
2008-02-03  7:18                     ` David Brownell [this message]
2008-02-03  7:51                       ` [linux-pm] " Sam Ravnborg
2008-02-03  8:26                         ` David Brownell
2008-02-03  8:26                         ` [linux-pm] " David Brownell
2008-02-03  7:51                       ` Sam Ravnborg
2008-02-03  5:14                   ` Ingo Molnar
2008-02-03  5:05                 ` Ingo Molnar
2008-02-02 18:00       ` Ingo Molnar
2008-02-02 17:31     ` David Brownell

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=200802022318.03597.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=mingo@elte.hu \
    --cc=pavel@ucw.cz \
    /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.