[PATCH 00/28] staging: comedi: ni_mio_common: more cleanup
Ian Abbott
abbotti at mev.co.uk
Thu Jul 17 11:54:41 UTC 2014
On 2014-07-16 17:36, Hartley Sweeten wrote:
> On Wednesday, July 16, 2014 3:54 AM, Ian Abbott wrote:
>> On 2014-07-14 20:23, H Hartley Sweeten wrote:
>>> The ni_mio_common.c file is included by a number of National Instruments
>>> drivers to provide the "common" driver support. This series does some
>>> more cleanup to get the file closer to being changed into a module instead
>>> of including it in the drivers.
>>
>> Jolly good!
>>
>> One thing you missed is simplifying ni_ao_config_chanlist(),
>> ni_old_ao_config_chanlist() and ni_m_series_ao_config_chanlist() a
>> little more by changing them to return 'void' and removing the 'invert'
>> variable from ni_old_ao_config_chanlist().
>
> The ni_mio_common stuff still needs additional cleanup. I'll get
> around to tidying up the config_chanlist stuff eventually. ;-)
Okay. I just thought I'd mention it because your patches were changing
those functions anyway!
--
-=( Ian Abbott @ MEV Ltd. E-mail: <abbotti at mev.co.uk> )=-
-=( Tel: +44 (0)161 477 1898 FAX: +44 (0)161 718 3587 )=-
More information about the devel
mailing list