[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1429618691-7157-1-git-send-email-abbotti@mev.co.uk>
Date: Tue, 21 Apr 2015 13:18:09 +0100
From: Ian Abbott <abbotti@....co.uk>
To: <driverdev-devel@...uxdriverproject.org>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Ian Abbott <abbotti@....co.uk>,
H Hartley Sweeten <hartleys@...ionengravers.com>,
<linux-kernel@...r.kernel.org>
Subject: [PATCH 0/2] staging: comedi: hide subdevice runflags stuff
Keep the details of the comedi subdevice `runflags` member local to
"comedi_fops.c". In particular, the usage of the
`COMEDI_SRF_FREE_SPRIV` run-flag doesn't really fit in all that well
with the others. It's used as a marker to indicate the subdevice's
`private` pointer can be automatically freed by the subdevice
clean-up code, whereas the others are associated with the operation of
asynchronous comedi commands. Abstract it's usage away in a couple of
new wrapper functions.
1) staging: comedi: wrap COMEDI_SRF_FREE_SPRIV usage
2) staging: comedi: move COMEDI_SRF_... macros to "comedi_fops.c"
drivers/staging/comedi/comedi_fops.c | 41 ++++++++++++++++++++--
drivers/staging/comedi/comedi_internal.h | 1 +
drivers/staging/comedi/comedidev.h | 18 +---------
drivers/staging/comedi/drivers.c | 2 +-
.../staging/comedi/drivers/amplc_dio200_common.c | 6 ++--
5 files changed, 45 insertions(+), 23 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists