Skip to contentSkip to conversation

i8253 != rtc

DISCUSSION Pavel Machek 3 participants 3 messages Jun 16, 2003

lore
Share & tools
lore
#0 lore raw

Hi!

/* XXX this driverfs stuff should probably go elsewhere later -john
*/
static struct sys_device device_i8253 = {
- .name = "rtc",
.id = 0,
- .dev = {
- .name = "i8253 Real Time Clock",
- },
+ .cls = &rtc_sysclass,
};

...but i8253 is *not* real time clock. Its clock since bootup. Realtime clock is near battery-backed CMOS RAM, its driver is linux/drivers/char/rtc.c...

Pavel

Signature · 2 lines
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]
#1 lore raw
Quoted 11 lines · Pavel Machek

/* XXX this driverfs stuff should probably go elsewhere later -john
*/
static struct sys_device device_i8253 = {

(quoted diff) +1 -4
-       .name           = "rtc",
        .id             = 0,
-       .dev    = {
-               .name   = "i8253 Real Time Clock",
-       },
+       .cls    = &rtc_sysclass,
 };
Quoted 3 lines

...but i8253 is *not* real time clock. Its clock since bootup. Realtime clock is near battery-backed CMOS RAM, its driver is linux/drivers/char/rtc.c...

FWIW, the legacy Intel databooks call it a "Programmable Interval Timer" (PIT). Likewise the 8254, a superset of the 8253. I don't know that Intel ever used the 'i' prefix on these parts.

Signature · 1 line
/Jonathan Lundell.
#2 lore raw
In reply to Pavel Machek
Quoted 13 lines · Pavel Machek

Hi!

/* XXX this driverfs stuff should probably go elsewhere later -john
*/
static struct sys_device device_i8253 = {

(quoted diff) +1 -4
-       .name           = "rtc",
        .id             = 0,
-       .dev    = {
-               .name   = "i8253 Real Time Clock",
-       },
+       .cls    = &rtc_sysclass,
 };
Quoted 1 line

...but i8253 is *not* real time clock.

D'oh. Noted and changed to 'pit'.

-pat