Subject: Re: [PATCH] generic check_legacy_ioport

From: Segher Boessenkool <[email-masked]>
Date: Wed, 25 Apr 2007 15:33:05 +0200

>> SLOF/JS21 (at least some versions of it) have an "ipmi"
>> node on the "isa" bus. =A0And the kernel ipmi driver actually
>> works on it, too (no idea about the currently proposed
>> scanning though -- but the principle is correct at least).
>
> It won'd be found by the of_platform_driver part of ipmi,
> since devices on the ISA bus do not get added to the linux
> device tree.

Drat.  Why not?

> If the ipmi node on the ISA bus is located at the standardized
> legacy I/O port range, it should get found by the later probing,
> if check_legacy_ioport allows it.

"Standardised" hahaha.  No, it is at some other
address.

> We might still want to have the driver use a proper of_device,
> which would require the maple platform code to add this device
> during probing.

That sounds like the only option then.  Does
this have to be done per platform though?  It
sounds perfectly safe to do it in more generic
code.

Segher