lkml.org 
[lkml]   [2006]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH, RFC] Driver for reading HP laptop LCD brightness
On Sun, Feb 05, 2006 at 03:27:26PM +0100, Arjan van de Ven wrote:

> disable_irq() and enable_irq() are really really evil. Are you sure you
> need these? To me on first sight it looks like a bug (think of shared
> interrupts for example), can you explain what you are trying to achieve
> with these?

We're talking to the hardware directly. There's a potential race where
the BIOS will try to access the cmos at the same time, with potentially
interesting results (We set the address we want to read with the outb.
The BIOS runs, outbs its own address, and then reads. We then read from
the address the BIOS was looking at, rather than what we were looking
at).

On all the hardware this will run on, IRQ 8 is the RTC. I don't believe
it can share interrupts.
--
Matthew Garrett | mjg59@srcf.ucam.org
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-05 15:37    [W:0.030 / U:0.832 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site