RE: Kernel interrupts disable at user level - RIGHT/ WRONG - Help

RE: Kernel interrupts disable at user level - RIGHT/ WRONG - Help

[Mukund JB.]

> -----Original Message-----
> From: Dave Jones [mailto:davej@redhat.com]
> Sent: Tuesday, December 20, 2005 12:39 PM
> To: Mukund JB.
> Cc: Alan Cox; linux-kernel@vger.kernel.org
> Subject: Re: Kernel interrupts disable at user level - RIGHT/ WRONG -
> Help
> 
> 
> On Tue, Dec 20, 2005 at 12:27:13PM +0530, Mukund JB. wrote:
>  > 
>  > Dear Alan,
>  > 
>  > I want the contents of A, B, C, D registers of CMOS mapped 
> registers.
>  > But instead the driver gives the details about the bit 
> masks of few of register A, B only.
>  > The others are NOT available. 
>  > 
>  > I would also require to retrieve the day of the week info 
> in RTC information. 
>  > I tried the /dev/rtc but I don't get it there.
> 
> Use /dev/nvram instead.
> 
> 		Dave

/dev/nvram does not give the cpomplete CMOS details. A part of RTC & date, tine and other info will be missing.

Regards,
Mukund Jampala

PATCH: day of week (RE: Kernel interrupts disable at user level - RIGHT/ WRONG - Help)

[Alan Cox]

On Maw, 2005-12-20 at 14:32 +0530, Mukund JB. wrote:
> >  > I tried the /dev/rtc but I don't get it there.
> > 
> > Use /dev/nvram instead.
> > 
> > 		Dave
> 
> /dev/nvram does not give the cpomplete CMOS details. A part of RTC & date, tine and other info will be missing.


What Dave should have said is use /dev/nvram as well.

>From /dev/rtc you get registers 0-9 except as you said 6
>From /dev/nvram you get A+

The lack of day of week is odd but fixable. The struct tm already has a
field for tm_wday (day of week) so we can fill it in. Whether it is
valid is another question.

What do people thing about this ?

Signed-off-by: Alan Cox <alan@redhat.com>

--- drivers/char/rtc.c~	2005-12-20 23:40:34.912559808 +0000
+++ drivers/char/rtc.c	2005-12-20 23:51:16.428034632 +0000
@@ -46,10 +46,10 @@
  *      1.11a   Daniele Bellucci: Audit create_proc_read_entry in rtc_init
  *	1.12	Venkatesh Pallipadi: Hooks for emulating rtc on HPET base-timer
  *		CONFIG_HPET_EMULATE_RTC
- *
+ *	1.12ac	Alan Cox: Allow read access to the day of week register
  */
 
-#define RTC_VERSION		"1.12"
+#define RTC_VERSION		"1.12ac"
 
 #define RTC_IO_EXTENT	0x8
 
@@ -1250,9 +1250,9 @@
 
 	/*
 	 * Only the values that we read from the RTC are set. We leave
-	 * tm_wday, tm_yday and tm_isdst untouched. Even though the
-	 * RTC has RTC_DAY_OF_WEEK, we ignore it, as it is only updated
-	 * by the RTC when initially set to a non-zero value.
+	 * tm_wday, tm_yday and tm_isdst untouched. Note that while the
+	 * RTC has RTC_DAY_OF_WEEK, we should usually ignore it, as it is
+	 * only updated by the RTC when initially set to a non-zero value.
 	 */
 	spin_lock_irq(&rtc_lock);
 	rtc_tm->tm_sec = CMOS_READ(RTC_SECONDS);
@@ -1261,6 +1261,9 @@
 	rtc_tm->tm_mday = CMOS_READ(RTC_DAY_OF_MONTH);
 	rtc_tm->tm_mon = CMOS_READ(RTC_MONTH);
 	rtc_tm->tm_year = CMOS_READ(RTC_YEAR);
+	/* Only set from 2.6.16 onwards */
+	rtc_tm->tm_wday = CMOS_READ(RTC_DAY_OF_WEEK);
+	
 #ifdef CONFIG_MACH_DECSTATION
 	real_year = CMOS_READ(RTC_DEC_YEAR);
 #endif
@@ -1275,6 +1278,7 @@
 		BCD_TO_BIN(rtc_tm->tm_mday);
 		BCD_TO_BIN(rtc_tm->tm_mon);
 		BCD_TO_BIN(rtc_tm->tm_year);
+		BCD_TO_BIN(rtc_tm->tm_wday);
 	}
 
 #ifdef CONFIG_MACH_DECSTATION

Re: PATCH: day of week (RE: Kernel interrupts disable at user level - RIGHT/ WRONG - Help)

[Jan Engelhardt]

>What do people thing about this ?

Well why not. I doubt any program relied on the - anyway broken - tm_wday 
field so far.


Jan Engelhardt
--