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