[PATCH 00/26] staging: comedi: comedi_fops: cleanup module

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

 



Remove all the forward declarations.

Move the low-level struct device drvdata (struct comedi_device_file_info)
internal to this module and don't expose it globally.

Don't allow direct access to the spin_lock protected subdevice runflags.
Provide an exported helper function for the one driver module (ni_mio_common)
that checks to see if the subdevice is running a command. Provide a couple
other internal helper functions to help clarify the code.

Refactor the code a bit so the MODULE_* information and module_{init,exit}
code is at the end of the file.

Do a bit of housekeeping of the rest of the code.

H Hartley Sweeten (26):
  staging: comedi: comedi_fops: remove forward declarations
  staging: comedi: don't expose comedi_get_{read,write}_subdevice
  staging: comedi: comedi_fops: introduce comedi_dev_from_minor()
  staging: comedi: use comedi_dev_from_minor()
  staging: comedi: comedi_fops: don't export comedi_get_device_file_info()
  staging: comedi: don't expose struct comedi_device_file_info
  staging: comedi: comedi_fops: rename struct comedi_device_file_info
  staging: comedi: comedi_fops: rename comedi_get_device_file_info()
  staging: comedi: comedi_fops: rename comedi_{read,write}_subdevice()
  staging: comedi: comedi_fops: cleanup do_devinfo_ioctl()
  staging: comedi: comedi_fops: cleanup comedi_poll()
  staging: comedi: comedi_fops: remove the goto's in comedi_read()
  staging: comedi: comedi_fops: remove the goto's in comedi_write()
  staging: comedi: comedi_fops: move the MODULE_* information to EOF
  staging: comedi: comedi_fops: move the module_{init,exit} functions to EOF
  staging: comedi: comedi_fops: move comedi_cleanup_legacy_minors()
  staging: comedi: comedi_fops: fix a number of sizeof(struct foo) forms
  staging: comedi: comedi_fops: move comedi_get_subdevice_runflags()
  staging: comedi: comedi_fops: introduce comedi_is_subdevice_running()
  staging: comedi: use comedi_is_subdevice_running()
  staging: comedi: comedi_fops: don't export comedi_get_subdevice_runflags()
  staging: comedi: comedi_fops: introduce comedi_is_subdevice_in_error()
  staging: comedi: comedi_fops: introduce comedi_is_subdevice_idle()
  staging: comedi: comedi_fops: cleanup comedi_{read,write}_subdevice()
  staging: comedi: comedi_fops: cleanup comedi_mmap()
  staging: comedi: comedi_fops: use kzalloc in comedi_alloc_subdevice_minor()

 drivers/staging/comedi/comedi_fops.c               | 774 ++++++++++-----------
 drivers/staging/comedi/comedidev.h                 |  32 +-
 drivers/staging/comedi/drivers.c                   |  11 +-
 drivers/staging/comedi/drivers/ni_mio_common.c     |   2 +-
 .../staging/comedi/kcomedilib/kcomedilib_main.c    |   8 +-
 drivers/staging/comedi/proc.c                      |   7 +-
 6 files changed, 387 insertions(+), 447 deletions(-)

-- 
1.8.0

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/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