Re: [PATCH] x86: restrict keyboard io ports reservation to make ipmidriver work

From: Thomas Gleixner
Date: Tue Apr 29 2008 - 18:57:43 EST




On Tue, 29 Apr 2008, Yinghai Lu wrote:

> On Tue, Apr 29, 2008 at 2:46 PM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> > From: Helge Wagner <Helge.Wagner@xxxxxxxxxxx>
> >
> > On some of our (single board computer) boards (x86) we are using an
> > IPMI controller that uses I/O ports 0x62 and 0x66 for a KCS (keyboard
> > controller style) IPMI system interface.
> >
> > Trying to load the openipmi driver fails, because the ports
> > (0x62/0x66) are reserved for keyboard. keyboard reserves the full
> > range 0x60-0x6F while it doesn't need to.
> >
> > Reserve only ports 0x60 and 0x64 for the legacy PS/2 i8042 keyboad
> > controller instead of 0x60-0x6F to allow the openipmi driver to work.
> >
> > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > ---
> > arch/x86/kernel/setup_32.c | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > Index: linux-2.6/arch/x86/kernel/setup_32.c
> > ===================================================================
> > --- linux-2.6.orig/arch/x86/kernel/setup_32.c
> > +++ linux-2.6/arch/x86/kernel/setup_32.c
> > @@ -127,7 +127,12 @@ static struct resource standard_io_resou
> > }, {
> > .name = "keyboard",
> > .start = 0x0060,
> > - .end = 0x006f,
> > + .end = 0x0060,
> > + .flags = IORESOURCE_BUSY | IORESOURCE_IO
> > +}, {
> > + .name = "keyboard",
> > + .start = 0x0064,
> > + .end = 0x0064,
> > .flags = IORESOURCE_BUSY | IORESOURCE_IO
> > }, {
> > .name = "dma page reg",
> > --
>
> 64 bit need to be fixed too...
> { .name = "keyboard", .start = 0x60, .end = 0x6f,
> .flags = IORESOURCE_BUSY | IORESOURCE_IO },

Doh, yes.

Thanks,
tglx

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/