Re: CVS commit: src/sys/dev/pci

[Christoph Egger <Christoph_Egger%gmx.de@localhost>]

On 24.12.10 14:11, Michael wrote:
> Hello,
> 
> On Dec 24, 2010, at 4:13 AM, Christoph Egger wrote:
> 
>> On 24.12.10 00:56, Michael wrote:
>>> Hello,
>>>
>>> On Dec 23, 2010, at 4:34 PM, Christoph Egger wrote:
>>>
>>>> Module Name:    src
>>>> Committed By:    cegger
>>>> Date:        Thu Dec 23 21:34:02 UTC 2010
>>>
>>>> Modified Files:
>>>>    src/sys/dev/pci: voyagerfb.c
>>>
>>>> Log Message:
>>>> read 32bits instead of 64bits and truncate the upper 32bits
>>>
>>> Please stop making unnecessary changes to drivers you can't test on
>>> actual hardware.
> 
>> Sorry, should have mentioned this makes the i386 ALL kernel linking.
> 
> Are you trying to tell me that i386 doesn't have bus_space_*_8()
> methods? I'd call that a bug.

It is declared but not implemented. I got an 'undefined reference'
linking error.

Christoph