lkml.org 
[lkml]   [2011]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] rtc-cmos: fix broken NVRAM bank 2 writing
Date
On Wednesday 23 November 2011, Andrew Morton wrote:
> On Sun, 13 Nov 2011 23:20:31 +0100
>
> Ondrej Zary <linux@rainbow-software.org> wrote:
> > Fix writing to NVRAM bank 2 in rtc-cmos driver. It never worked since its
> > introduction in 2.6.28 because of a typo.
> >
> > Signed-off-by: Ondrej Zary <linux@rainbow-software.org>
> >
> > --- linux-3.1-orig/drivers/rtc/rtc-cmos.c 2011-10-24 09:10:05.000000000
> > +0200 +++ linux-3.1/drivers/rtc/rtc-cmos.c 2011-11-08 17:43:29.000000000
> > +0100 @@ -164,7 +164,7 @@ static inline unsigned char cmos_read_ba
> > static inline void cmos_write_bank2(unsigned char val, unsigned char
> > addr) {
> > outb(addr, RTC_PORT(2));
> > - outb(val, RTC_PORT(2));
> > + outb(val, RTC_PORT(3));
> > }
> >
> > #else
>
> Strange. Can you suggest why this bug has lived for so long without anyone
> noticing it?

Probably because nobody uses it. It's hidden in sysfs
(/sys/class/rtc/rtc0/device/nvram) and there's another (old) module "nvram"
that does (mostly) the same thing (except there's no bank 2 support).

--
Ondrej Zary


\
 
 \ /
  Last update: 2011-11-23 08:33    [W:0.044 / U:0.592 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site