cvs commit: src/sys/amd64/conf NOTES src/sys/conf files.amd64 files.i386 src/sys/dev/ipmi ipmi.c ipmi_acpi.c ipmi_isa.c ipmi_kcs.c ipmi_pci.c ipmi_smbios.c ipmi_smbus.c ipmi_smic.c ipmi_ssif.c ipmivars.h src/sys/i386/conf NOTES

[John Baldwin]

On Monday 25 September 2006 15:50, Doug Ambrisko wrote:
> John Baldwin writes:
> | jhb         2006-09-22 22:11:29 UTC
> | 
> |   FreeBSD src repository
> | 
> |   Modified files:
> |     sys/amd64/conf       NOTES 
> |     sys/conf             files.amd64 files.i386 
> |     sys/dev/ipmi         ipmi.c ipmi_pci.c ipmi_smbios.c 
> |                          ipmivars.h 
> |     sys/i386/conf        NOTES 
> |     sys/modules/ipmi     Makefile 
> |     sys/sys              ipmi.h 
> |   Added files:
> |     sys/dev/ipmi         ipmi_acpi.c ipmi_isa.c ipmi_kcs.c 
> |                          ipmi_smbus.c ipmi_smic.c ipmi_ssif.c 
> |   Log:
> |   Update the ipmi(4) driver:
> 	[snip]
> |   - Add an ACPI attachment.
> 
> This should work, however, my one test machine doesn't attach with it.
> Since it doesn't break anything having it, it is safe to add.  That
> machine attached fine with the SMBIOS info. so it's not a big deal.
> Just don't expect it to work out of the box.  There are some issues 
> that need to get investigated.
> 
> Thanks to John for helping to finish some loose ends and add a lot more
> polish.

Did you get a chance to test the last snap of the patch with the ACPI 
attachment?  It now does a loop to alloc multiple resources if they exist 
which should fix the machine you have I think.

-- 
John Baldwin