Subject: Re: structural bus_space_handles
To: Sam Leffler <sam@errno.com>
From: Garrett D'Amore <garrett_damore@tadpole.com>
List: tech-kern
Date: 06/05/2006 09:54:04
Sam Leffler wrote:
> Garrett D'Amore wrote:
>
>> Sam Leffler wrote:
>>
>>> Garrett D'Amore wrote:
>>>
>>>
>>>
>>>> He passes a void * token (two actually, one for the tag and one for the
>>>> handle) now.
>>>>
>>>> The point was that cleaning this up required all architectures to get an
>>>> extra indirect access, whereas only sparc truly needs it.
>>>>
>>>>
>>> The extra indirect was #ifdef'd by the arch. It's only done for sparc64
>>> where it's required. Everywhere else there is no extra indirection; the
>>> data are passed through as opaque values and cast/converted before use.
>>>
>>>
>> Yes, I know that. I had briefly put it back, but I've since changed
>> it. What I committed now stores the handle/tag in the softc, and pass
>> a pointer to these to the HAL. It works on SPARC, i386, and the
>> AR5312. I have not been able to make the HAL work on Au1550 (MIPS
>> based) for some time, but at this point I don't know whether the problem
>> is in the HAL or in the Au1550 port. I'd love to have someone else with
>> alternate MIPS hardware test this out for me.
>>
>
> Sounds like you've tied the definition of the softc structure to the hal
> which is a step backward.
>
No. I have the softc for the OS-specific busses have the structure.
Take a look at what I committed.
-- Garrett
> Sam
>
--
Garrett D'Amore, Principal Software Engineer
Tadpole Computer / Computing Technologies Division,
General Dynamics C4 Systems
http://www.tadpolecomputer.com/
Phone: 951 325-2134 Fax: 951 325-2191