Re: mmap

From: Bernd Porr <bp1_at_cn.stir.ac.uk> · Date: Fri, 23 May 2003 21:02:19 +0100

and shame on me. I haven't used the CVS tree. Only comedi-0.7.66. How 
stable is the CVS just now?

/Bernd

Bernd Porr wrote:

> David,
>
> so it is option[COMEDI_DEVCONF_AUX_DATA] and 
> option[COMEDI_DEVCONF_AUX_DATA_LENGTH]?
>
> If so the driver daqboard2000.c should be modified? Cause I used it 
> for my driver:
>
>    if(copy_from_user(ptr,(void *)it->options[0],size)){
>      result=-EFAULT;
>      goto out;
>    }
>
> /Bernd
> David Schleef wrote:
>
>> On Fri, May 23, 2003 at 01:35:15PM +0100, Bernd Porr wrote:
>>  
>>
>>> Hi Anne,
>>>
>>> looks good. The function is not very well documented and that it is 
>>> supplied as option[0] and option[1] prevents passing other 
>>> parameters.   
>>
>>
>> Actually, it is COMEDI_DEVCONF_AUX_DATA and
>> COMEDI_DEVCONF_AUX_DATA_LENGTH.
>>
>>  
>>
>>> There was a problem with the type of the option array. Maybe it's a 
>>> good idea to pass the firmware with the help of a "real" pointer and
>>> a size field. Just now there's only one driver in the comedi 
>>> framwork which needs firmware (except the one I'm just writing) and 
>>> therefore it might be the last moment to fix this. What does the 
>>> community think about this?
>>>   
>>
>>
>> It's only a problem with 64-bit machines, and can be worked around
>> when the need arises.
>>
>>
>>
>> dave...
>>
>>
>> _______________________________________________
>> comedi mailing list
>> comedi_at_comedi.org
>> https://cvs.comedi.org/cgi-bin/mailman/listinfo/comedi
>>  
>>
>

-- 
http://www.cn.stir.ac.uk/~bp1/
mailto:bp1_at_cn.stir.ac.uk