Messages in this thread | | | From | Bjorn Helgaas <> | Subject | Re: [PATCH 2.6.15 1/2] ia64: use i386 dmi_scan.c | Date | Thu, 5 Jan 2006 09:41:15 -0700 |
| |
On Wednesday 04 January 2006 16:29, Matt Domsch wrote: > On Wed, Jan 04, 2006 at 03:36:03PM -0700, Alex Williamson wrote: > > On Wed, 2006-01-04 at 16:16 -0600, Matt Domsch wrote: > > > Andi Kleen has a patch in his x86_64 tree which enables the use > > > of i386 dmi_scan.c on x86_64. dmi_scan.c functions are being > > > used by the drivers/char/ipmi/ipmi_si_intf.c driver for > > > autodetecting the ports or memory spaces where the IPMI > > > controllers may be found. > > > > Can't this be done via ACPI/EFI? I'm really opposed to adding > > anything to ia64 that blindly picks memory ranges and starts > > scanning for magic legacy tables. If nothing else, this can be > > found via efi.smbios. Thanks, > > I'll redo this to use efi.smbios. Thanks for the tip.
The DMI scan looks like it's done in try_init_smbios(). But try_init_acpi() is done first. Since every ia64 machine has ACPI, I would think try_init_acpi() should be sufficient.
Or do you have a machine that doesn't supply the SPMI table used by try_init_acpi()?
Personally, I think try_init_acpi() should be re-done so it uses the normal acpi_bus_register_driver() mechanism, which would locate the IPMI device in the ACPI namespace. I don't think there's any need to rely on the SPMI, which is primarily there to support OS's that want to do IPMI stuff early in boot, before the ACPI machinery is ready.
Bjorn - 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/
| |