Re: [PATCH 000/107] staging: comedi: tidy up ni_stc.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 01, 2015 at 02:58:25PM -0700, H Hartley Sweeten wrote:
> The ni_stc.h header contains the DAQ-STC and board specific register
> information used by ni_mio_common.c to provide the "guts" for the
> National Instruments MIO drivers ni_atmio, ni_mio_cs, and ni_pcimio.
> Those drivers all currently include the ni_mio_common.c file.
> 
> This header is a bit of a mess...
> 
>   1) almost everything is CamelCase
>   2) bits are described as
>      a) #defines with custom _bit defines instead of the BIT() macro
>      b) raw values or bit shifts
>      b) enums with either custom _bit defines, raw values, or bit shifts
>      c) inline functions
>   3) there is quite a bit of unused information that is duplicated in the
>      ni_tio_internal.h header
> 
> This series addresses these issues and gets the ni_stc.h file into a
> consistent, easier maintainable, format.
> 
> It also refactors the STC register remapping for the M-Series boards and
> the ni_tio GPCT (general purpose counter timer) to STC.
> 
> In addition a number of unreachable, or unnecessary, BUG() and BUG_ON() checks
> are removed.

This series adds some build warnings, so can you send a follow-on patch
to fix that?  I'm sure you will get a notice from the 0-day bot about
it soon.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux