The 'devpriv' macro usage in this driver is holding up other cleanup of the comedi drivers. This patch series removes all the macros used to read/write the hardware registers. All of them are simple wrappers around standard {read,write}[rwl] calls with some of them caching the write value in a shadow variable in the private data. Each of the macros is removed in a separate patch to ease reviewing. They can all be squashed into one patch if desired. After all the macros are gone, the devpriv macro itself can be removed along with the boardinfo macro. The "find pci device" code is also refactored to follow the style of the other comedi pci drivers. The range tables are also cleaned up for aesthetic reasons. The large whitespace causes some pretty nasty line breaks in order to keep the lines < 80 characters. H Hartley Sweeten (83): staging: comedi: rtd520: remove RtdResetBoard macro staging: comedi: rtd520: remove RtdResetCGT macro staging: comedi: rtd520: remove RtdClearCGT macro staging: comedi: rtd520: remove RtdEnableCGT macro staging: comedi: rtd520: remove RtdWriteCGTable macro staging: comedi: rtd520: remove RtdWriteCGLatch macro staging: comedi: rtd520: remove RtdAdcClearFifo macro staging: comedi: rtd520: remove RtdAdcConversionSource macro staging: comedi: rtd520: remove RtdBurstStartSource macro staging: comedi: rtd520: remove RtdPacerStartSource macro staging: comedi: rtd520: remove RtdPacerStopSource macro staging: comedi: rtd520: remove RtdPacerClockSource macro staging: comedi: rtd520: remove RtdAdcSampleCounterSource macro staging: comedi: rtd520: remove RtdPacerTriggerMode macro staging: comedi: rtd520: remove RtdAboutStopEnable macro staging: comedi: rtd520: remove RtdTriggerPolarity macro staging: comedi: rtd520: remove RtdAdcStart macro staging: comedi: rtd520: remove RtdAdcFifoGet macro staging: comedi: rtd520: remove RtdAdcFifoGet2 macro staging: comedi: rtd520: remove RtdFifoStatus macro staging: comedi: rtd520: remove RtdPacerStart macro staging: comedi: rtd520: remove RtdPacerStop macro staging: comedi: rtd520: remove RtdInterruptStatus macro staging: comedi: rtd520: remove RtdInterruptMask macro staging: comedi: rtd520: remove RtdInterruptClear macro staging: comedi: rtd520: remove RtdInterruptClearMask macro staging: comedi: rtd520: remove RtdInterruptOverrunStatus macro staging: comedi: rtd520: remove RtdInterruptOverrunClear macro staging: comedi: rtd520: remove RtdPacerCount macro staging: comedi: rtd520: remove RtdPacerCounter macro staging: comedi: rtd520: remove RtdBurstCount macro staging: comedi: rtd520: remove RtdBurstCounter macro staging: comedi: rtd520: remove RtdDelayCount macro staging: comedi: rtd520: remove RtdDelayCounter macro staging: comedi: rtd520: remove RtdAboutCount macro staging: comedi: rtd520: remove RtdAboutCounter macro staging: comedi: rtd520: remove RtdAdcSampleCount macro staging: comedi: rtd520: remove RtdAdcSampleCounter macro staging: comedi: rtd520: remove RtdUtcCounterGet macro staging: comedi: rtd520: remove RtdUtcCounterPut macro staging: comedi: rtd520: remove RtdUtcCtrlPut macro staging: comedi: rtd520: remove RtdUtcClockSource macro staging: comedi: rtd520: remove RtdUtcGateSource macro staging: comedi: rtd520: remove RtdUsrOutSource macro staging: comedi: rtd520: remove RtdDio0Read macro staging: comedi: rtd520: remove RtdDio0Write macro staging: comedi: rtd520: remove RtdDio1Read macro staging: comedi: rtd520: remove RtdDio1Write macro staging: comedi: rtd520: remove RtdDioStatusRead macro staging: comedi: rtd520: remove RtdDioStatusWrite macro staging: comedi: rtd520: remove RtdDio0CtrlRead macro staging: comedi: rtd520: remove RtdDio0CtrlWrite macro staging: comedi: rtd520: remove RtdDacFifoPut macro staging: comedi: rtd520: remove RtdDacUpdate macro staging: comedi: rtd520: remove RtdDacBothUpdate macro staging: comedi: rtd520: remove RtdDacRange macro staging: comedi: rtd520: remove RtdDacClearFifo macro staging: comedi: rtd520: remove RtdDma0Source macro staging: comedi: rtd520: remove RtdDma1Source macro staging: comedi: rtd520: remove RtdDma0Reset macro staging: comedi: rtd520: remove RtdDma1Reset macro staging: comedi: rtd520: remove RtdPlxInterruptRead macro staging: comedi: rtd520: remove RtdPlxInterruptWrite macro staging: comedi: rtd520: remove RtdDma0Mode macro staging: comedi: rtd520: remove RtdDma0PciAddr macro staging: comedi: rtd520: remove RtdDma0LocalAddr macro staging: comedi: rtd520: remove RtdDma0Count macro staging: comedi: rtd520: remove RtdDma0Count macro staging: comedi: rtd520: remove RtdDma1Mode macro staging: comedi: rtd520: remove RtdDma1PciAddr macro staging: comedi: rtd520: remove RtdDma1LocalAddr macro staging: comedi: rtd520: remove RtdDma1Count macro staging: comedi: rtd520: remove RtdDma1Next macro staging: comedi: rtd520: remove RtdDma0Control macro staging: comedi: rtd520: remove RtdDma0Status macro staging: comedi: rtd520: remove RtdDma1Control macro staging: comedi: rtd520: remove RtdDma1Status macro staging: comedi: rtd520: remove devpriv macro staging: comedi: rtd520: remove thisboard macro staging: comedi: rtd520: factor out the "find pci device" code staging: comedi: rtd520: cleanup the "find pci device" code staging: comedi: rtd520: cleanup the range tables staging: comedi: rtd520: cleanup the boardinfo drivers/staging/comedi/drivers/rtd520.c | 915 +++++++++++--------------------- 1 file changed, 318 insertions(+), 597 deletions(-) -- 1.7.11 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel