Re: [PATCH 00/12] staging: comedi: adv_pci1710: more async command cleanup

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

 



On 2014-04-28 18:41, H Hartley Sweeten wrote:
This continues the cleanup of the async command support code for this driver
and addresses a number of issues pointed out by Ian Abbott.

H Hartley Sweeten (12):
   staging: comedi: adv_pci1710: tidy up pci171x_ai_check_chanlist()
   staging: comedi: adv_pci1710: don't check the chanlist twice
   staging: comedi: adv_pci1710: remove 'neverending_ai' from private data
   staging: comedi: adv_pci1710: remove 'ai_buf_ptr' from private data
   staging: comedi: adv_pci1710: remove 'ai_eos' from private data
   staging: comedi: adv_pci1710: only calc the pacer divisors once
   staging: comedi: adv_pci1710: remove local var in pci171x_ai_cmd()
   staging: comedi: adv_pci1710: tidy up start_pacer()
   staging: comedi: adv_pci1710: rename interrupt helper functions
   staging: comedi: adv_pci1710: always enable PCI171x_PARANOIDCHECK code
   staging: comedi: adv_pci1710: remove 'i8254_osc_base' from private data
   staging: comedi: adv_pci1710: remove 'ai_act_chan' from private data

  drivers/staging/comedi/drivers/adv_pci1710.c | 431 +++++++++++----------------
  1 file changed, 182 insertions(+), 249 deletions(-)


Looks good apart from a typo bug in patch 08/12 that I missed the first time these patches were posted.

Reviewed-by: Ian Abbott <abbotti@xxxxxxxxx>
for patches 01-07,09-12

Feel free to add me as a Reviewed-by for patch 08 once the typo bug is corrected.

--
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@xxxxxxxxx>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
_______________________________________________
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