Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 03 Feb 2005 23:22:50 -0800
From:      Nate Lawson <nate@root.org>
To:        Marcel Moolenaar <marcel@xcllnt.net>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/ia64/ia64 machdep.c
Message-ID:  <420322CA.3090201@root.org>
In-Reply-To: <a14b03e8aa5974ebd7e24436d989317b@xcllnt.net>
References:  <20050204063905.8AE4716A4E9@hub.freebsd.org> <42031DC4.3000109@root.org> <a14b03e8aa5974ebd7e24436d989317b@xcllnt.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Marcel Moolenaar wrote:
> 
> On Feb 3, 2005, at 11:01 PM, Nate Lawson wrote:
> 
>> Marcel Moolenaar wrote:
>>
>>> marcel      2005-02-04 06:38:58 UTC
>>>   FreeBSD src repository
>>>   Modified files:
>>>     sys/ia64/ia64        machdep.c   Log:
>>>   Include sys/bus.h before sys/cpu.h. The latter needs device_t.
>>>     Revision  Changes    Path
>>>   1.195     +1 -0      src/sys/ia64/ia64/machdep.c
>>
>>
>> Thanks, I've made similar changes to the other arches too.
> 
> 
> No worries. Sorry I didn't had the time to test before committing.
> 
> BTW, I still have a link error:
> 
> pluto2# make
> linking kernel.debug
> acpi_cpu.o(.text+0x2302): In function `acpi_cpu_startup':
> ../../../dev/acpica/acpi_cpu.c:757: undefined reference to `cpu_devclass'
> acpi_cpu.o(.data+0x1a8): In function `acpi_cpu_probe':
> ./bus_if.h:107: undefined reference to `cpu_devclass'
> *** Error code 1
> 
> Stop in /q/6.x/src/sys/ia64/compile/PLUTO2.
> 
> Can it be that you missed a file?

No, it's something I need to fix one way or the other and had on my todo 
list.  I'm not sure if my approach is correct -- should different 
drivers for the same hardware share the same devclass?  I have 
legacy::cpu and acpi::cpu sharing the same devclass value since neither 
can be attached at the same time.  I thought in the past there might be 
some advantage to this but can't remember the rationale.

In any case, until I decide whether to go all the way in doing this for 
all archs or falling back to static devclasses, I'll fix it by making 
acpi's dc static.

-- 
Nate



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?420322CA.3090201>