* [PATCH 0/9] staging: comedi: introduce comedi_usb.h header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
"comedidev.h" includes USB-specific stuff that gets included by all
comedi drivers including non-USB ones. Separate it out into its own
header "comedi_usb.h". Make the new header include <linux/usb.h> and
"comedidev.h" so that comedi USB drivers do not need to include them
explicitly.
1) staging: comedi: add comedi_usb.h
2) staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
3) staging: comedi: dt9812: include new "comedi_usb.h" header
4) staging: comedi: ni_usb6501: include new "comedi_usb.h" header
5) staging: comedi: usbdux: include new "comedi_usb.h" header
6) staging: comedi: usbduxfast: include new "comedi_usb.h" header
7) staging: comedi: usbduxsigma: include new "comedi_usb.h" header
8) staging: comedi: vmk80xx: include new "comedi_usb.h" header
9) staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
drivers/staging/comedi/comedi_usb.c | 3 +-
drivers/staging/comedi/comedi_usb.h | 50 ++++++++++++++++++++++++++++
drivers/staging/comedi/comedidev.h | 29 ----------------
drivers/staging/comedi/drivers/dt9812.c | 3 +-
drivers/staging/comedi/drivers/ni_usb6501.c | 3 +-
drivers/staging/comedi/drivers/usbdux.c | 3 +-
drivers/staging/comedi/drivers/usbduxfast.c | 3 +-
drivers/staging/comedi/drivers/usbduxsigma.c | 3 +-
drivers/staging/comedi/drivers/vmk80xx.c | 3 +-
9 files changed, 57 insertions(+), 43 deletions(-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 0/9] staging: comedi: introduce comedi_usb.h header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
"comedidev.h" includes USB-specific stuff that gets included by all
comedi drivers including non-USB ones. Separate it out into its own
header "comedi_usb.h". Make the new header include <linux/usb.h> and
"comedidev.h" so that comedi USB drivers do not need to include them
explicitly.
1) staging: comedi: add comedi_usb.h
2) staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
3) staging: comedi: dt9812: include new "comedi_usb.h" header
4) staging: comedi: ni_usb6501: include new "comedi_usb.h" header
5) staging: comedi: usbdux: include new "comedi_usb.h" header
6) staging: comedi: usbduxfast: include new "comedi_usb.h" header
7) staging: comedi: usbduxsigma: include new "comedi_usb.h" header
8) staging: comedi: vmk80xx: include new "comedi_usb.h" header
9) staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
drivers/staging/comedi/comedi_usb.c | 3 +-
drivers/staging/comedi/comedi_usb.h | 50 ++++++++++++++++++++++++++++
drivers/staging/comedi/comedidev.h | 29 ----------------
drivers/staging/comedi/drivers/dt9812.c | 3 +-
drivers/staging/comedi/drivers/ni_usb6501.c | 3 +-
drivers/staging/comedi/drivers/usbdux.c | 3 +-
drivers/staging/comedi/drivers/usbduxfast.c | 3 +-
drivers/staging/comedi/drivers/usbduxsigma.c | 3 +-
drivers/staging/comedi/drivers/vmk80xx.c | 3 +-
9 files changed, 57 insertions(+), 43 deletions(-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 1/9] staging: comedi: add comedi_usb.h
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Add a new header that Comedi USB drivers can include instead of
"comedidev.h". Currently, it just pulls in <linux/usb.h> and
"comedidev.h", but the plan is to migrate the USB-specific stuff from
"comedidev.h" here.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/comedi_usb.h | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
create mode 100644 drivers/staging/comedi/comedi_usb.h
diff --git a/drivers/staging/comedi/comedi_usb.h b/drivers/staging/comedi/comedi_usb.h
new file mode 100644
index 0000000..a57025b
--- /dev/null
+++ b/drivers/staging/comedi/comedi_usb.h
@@ -0,0 +1,30 @@
+/*
+ * comedi_usb.h
+ * header file for USB Comedi drivers
+ *
+ * COMEDI - Linux Control and Measurement Device Interface
+ * Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _COMEDI_USB_H
+#define _COMEDI_USB_H
+
+#include <linux/usb.h>
+
+#include "comedidev.h"
+
+/*
+ * TODO: Move USB-specific stuff into here from "comedidev.h".
+ */
+
+#endif /* _COMEDI_USB_H */
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 1/9] staging: comedi: add comedi_usb.h
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Add a new header that Comedi USB drivers can include instead of
"comedidev.h". Currently, it just pulls in <linux/usb.h> and
"comedidev.h", but the plan is to migrate the USB-specific stuff from
"comedidev.h" here.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/comedi_usb.h | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
create mode 100644 drivers/staging/comedi/comedi_usb.h
diff --git a/drivers/staging/comedi/comedi_usb.h b/drivers/staging/comedi/comedi_usb.h
new file mode 100644
index 0000000..a57025b
--- /dev/null
+++ b/drivers/staging/comedi/comedi_usb.h
@@ -0,0 +1,30 @@
+/*
+ * comedi_usb.h
+ * header file for USB Comedi drivers
+ *
+ * COMEDI - Linux Control and Measurement Device Interface
+ * Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _COMEDI_USB_H
+#define _COMEDI_USB_H
+
+#include <linux/usb.h>
+
+#include "comedidev.h"
+
+/*
+ * TODO: Move USB-specific stuff into here from "comedidev.h".
+ */
+
+#endif /* _COMEDI_USB_H */
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 2/9] staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "comedi_usb.h" header instead of <linux/usb.h> and
"comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/comedi_usb.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/comedi_usb.c b/drivers/staging/comedi/comedi_usb.c
index 0b862a6..68b75e8 100644
--- a/drivers/staging/comedi/comedi_usb.c
+++ b/drivers/staging/comedi/comedi_usb.c
@@ -17,9 +17,8 @@
*/
#include <linux/module.h>
-#include <linux/usb.h>
-#include "comedidev.h"
+#include "comedi_usb.h"
/**
* comedi_to_usb_interface() - comedi_device pointer to usb_interface pointer.
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 2/9] staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "comedi_usb.h" header instead of <linux/usb.h> and
"comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/comedi_usb.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/comedi_usb.c b/drivers/staging/comedi/comedi_usb.c
index 0b862a6..68b75e8 100644
--- a/drivers/staging/comedi/comedi_usb.c
+++ b/drivers/staging/comedi/comedi_usb.c
@@ -17,9 +17,8 @@
*/
#include <linux/module.h>
-#include <linux/usb.h>
-#include "comedidev.h"
+#include "comedi_usb.h"
/**
* comedi_to_usb_interface() - comedi_device pointer to usb_interface pointer.
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 3/9] staging: comedi: dt9812: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/dt9812.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/dt9812.c b/drivers/staging/comedi/drivers/dt9812.c
index 06c601d..e11c216 100644
--- a/drivers/staging/comedi/drivers/dt9812.c
+++ b/drivers/staging/comedi/drivers/dt9812.c
@@ -42,9 +42,8 @@ for my needs.
#include <linux/module.h>
#include <linux/errno.h>
#include <linux/uaccess.h>
-#include <linux/usb.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#define DT9812_DIAGS_BOARD_INFO_ADDR 0xFBFF
#define DT9812_MAX_WRITE_CMD_PIPE_SIZE 32
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 3/9] staging: comedi: dt9812: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/dt9812.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/dt9812.c b/drivers/staging/comedi/drivers/dt9812.c
index 06c601d..e11c216 100644
--- a/drivers/staging/comedi/drivers/dt9812.c
+++ b/drivers/staging/comedi/drivers/dt9812.c
@@ -42,9 +42,8 @@ for my needs.
#include <linux/module.h>
#include <linux/errno.h>
#include <linux/uaccess.h>
-#include <linux/usb.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#define DT9812_DIAGS_BOARD_INFO_ADDR 0xFBFF
#define DT9812_MAX_WRITE_CMD_PIPE_SIZE 32
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 4/9] staging: comedi: ni_usb6501: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/ni_usb6501.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/ni_usb6501.c b/drivers/staging/comedi/drivers/ni_usb6501.c
index 3b5a1b9..5f649f8 100644
--- a/drivers/staging/comedi/drivers/ni_usb6501.c
+++ b/drivers/staging/comedi/drivers/ni_usb6501.c
@@ -96,9 +96,8 @@
#include <linux/kernel.h>
#include <linux/module.h>
#include <linux/slab.h>
-#include <linux/usb.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#define NI6501_TIMEOUT 1000
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 4/9] staging: comedi: ni_usb6501: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/ni_usb6501.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/ni_usb6501.c b/drivers/staging/comedi/drivers/ni_usb6501.c
index 3b5a1b9..5f649f8 100644
--- a/drivers/staging/comedi/drivers/ni_usb6501.c
+++ b/drivers/staging/comedi/drivers/ni_usb6501.c
@@ -96,9 +96,8 @@
#include <linux/kernel.h>
#include <linux/module.h>
#include <linux/slab.h>
-#include <linux/usb.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#define NI6501_TIMEOUT 1000
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 5/9] staging: comedi: usbdux: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/usbdux.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbdux.c b/drivers/staging/comedi/drivers/usbdux.c
index 0683d6a..1cd7403 100644
--- a/drivers/staging/comedi/drivers/usbdux.c
+++ b/drivers/staging/comedi/drivers/usbdux.c
@@ -80,11 +80,10 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#include "comedi_fc.h"
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 5/9] staging: comedi: usbdux: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/usbdux.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbdux.c b/drivers/staging/comedi/drivers/usbdux.c
index 0683d6a..1cd7403 100644
--- a/drivers/staging/comedi/drivers/usbdux.c
+++ b/drivers/staging/comedi/drivers/usbdux.c
@@ -80,11 +80,10 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
#include "comedi_fc.h"
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 6/9] staging: comedi: usbduxfast: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/usbduxfast.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbduxfast.c b/drivers/staging/comedi/drivers/usbduxfast.c
index 85071da..7ce27c1 100644
--- a/drivers/staging/comedi/drivers/usbduxfast.c
+++ b/drivers/staging/comedi/drivers/usbduxfast.c
@@ -46,11 +46,10 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
#include "comedi_fc.h"
-#include "../comedidev.h"
+#include "../comedi_usb.h"
/*
* timeout for the USB-transfer
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 6/9] staging: comedi: usbduxfast: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/usbduxfast.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbduxfast.c b/drivers/staging/comedi/drivers/usbduxfast.c
index 85071da..7ce27c1 100644
--- a/drivers/staging/comedi/drivers/usbduxfast.c
+++ b/drivers/staging/comedi/drivers/usbduxfast.c
@@ -46,11 +46,10 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
#include "comedi_fc.h"
-#include "../comedidev.h"
+#include "../comedi_usb.h"
/*
* timeout for the USB-transfer
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 7/9] staging: comedi: usbduxsigma: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/usbduxsigma.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbduxsigma.c b/drivers/staging/comedi/drivers/usbduxsigma.c
index 17e5a81..378c449 100644
--- a/drivers/staging/comedi/drivers/usbduxsigma.c
+++ b/drivers/staging/comedi/drivers/usbduxsigma.c
@@ -45,13 +45,12 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
#include <asm/unaligned.h>
#include "comedi_fc.h"
-#include "../comedidev.h"
+#include "../comedi_usb.h"
/* timeout for the USB-transfer in ms*/
#define BULK_TIMEOUT 1000
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 7/9] staging: comedi: usbduxsigma: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/usbduxsigma.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbduxsigma.c b/drivers/staging/comedi/drivers/usbduxsigma.c
index 17e5a81..378c449 100644
--- a/drivers/staging/comedi/drivers/usbduxsigma.c
+++ b/drivers/staging/comedi/drivers/usbduxsigma.c
@@ -45,13 +45,12 @@
#include <linux/module.h>
#include <linux/slab.h>
#include <linux/input.h>
-#include <linux/usb.h>
#include <linux/fcntl.h>
#include <linux/compiler.h>
#include <asm/unaligned.h>
#include "comedi_fc.h"
-#include "../comedidev.h"
+#include "../comedi_usb.h"
/* timeout for the USB-transfer in ms*/
#define BULK_TIMEOUT 1000
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 8/9] staging: comedi: vmk80xx: include new "comedi_usb.h" header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/drivers/vmk80xx.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/vmk80xx.c b/drivers/staging/comedi/drivers/vmk80xx.c
index eb76460..e371183 100644
--- a/drivers/staging/comedi/drivers/vmk80xx.c
+++ b/drivers/staging/comedi/drivers/vmk80xx.c
@@ -42,10 +42,9 @@
#include <linux/input.h>
#include <linux/slab.h>
#include <linux/poll.h>
-#include <linux/usb.h>
#include <linux/uaccess.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
enum {
DEVICE_VMK8055,
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 8/9] staging: comedi: vmk80xx: include new "comedi_usb.h" header
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Include the new "../comedi_usb.h" header instead of <linux/usb.h> and
"../comedidev.h", which will now get included indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/drivers/vmk80xx.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/vmk80xx.c b/drivers/staging/comedi/drivers/vmk80xx.c
index eb76460..e371183 100644
--- a/drivers/staging/comedi/drivers/vmk80xx.c
+++ b/drivers/staging/comedi/drivers/vmk80xx.c
@@ -42,10 +42,9 @@
#include <linux/input.h>
#include <linux/slab.h>
#include <linux/poll.h>
-#include <linux/usb.h>
#include <linux/uaccess.h>
-#include "../comedidev.h"
+#include "../comedi_usb.h"
enum {
DEVICE_VMK8055,
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 9/9] staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 17:49 ` Ian Abbott
-1 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel
Cc: Greg Kroah-Hartman, Ian Abbott, H Hartley Sweeten, linux-kernel
Move the USB-specific stuff out of "comedidev.h" into "comedi_usb.h".
Comedi USB drivers now include "comedi_usb.h" instead of "comedidev.h",
which now gets pulled in indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
---
drivers/staging/comedi/comedi_usb.h | 24 ++++++++++++++++++++++--
drivers/staging/comedi/comedidev.h | 29 -----------------------------
2 files changed, 22 insertions(+), 31 deletions(-)
diff --git a/drivers/staging/comedi/comedi_usb.h b/drivers/staging/comedi/comedi_usb.h
index a57025b..721128b 100644
--- a/drivers/staging/comedi/comedi_usb.h
+++ b/drivers/staging/comedi/comedi_usb.h
@@ -23,8 +23,28 @@
#include "comedidev.h"
-/*
- * TODO: Move USB-specific stuff into here from "comedidev.h".
+struct usb_interface *comedi_to_usb_interface(struct comedi_device *);
+struct usb_device *comedi_to_usb_dev(struct comedi_device *);
+
+int comedi_usb_auto_config(struct usb_interface *, struct comedi_driver *,
+ unsigned long context);
+void comedi_usb_auto_unconfig(struct usb_interface *);
+
+int comedi_usb_driver_register(struct comedi_driver *, struct usb_driver *);
+void comedi_usb_driver_unregister(struct comedi_driver *, struct usb_driver *);
+
+/**
+ * module_comedi_usb_driver() - Helper macro for registering a comedi USB driver
+ * @__comedi_driver: comedi_driver struct
+ * @__usb_driver: usb_driver struct
+ *
+ * Helper macro for comedi USB drivers which do not do anything special
+ * in module init/exit. This eliminates a lot of boilerplate. Each
+ * module may only use this macro once, and calling it replaces
+ * module_init() and module_exit()
*/
+#define module_comedi_usb_driver(__comedi_driver, __usb_driver) \
+ module_driver(__comedi_driver, comedi_usb_driver_register, \
+ comedi_usb_driver_unregister, &(__usb_driver))
#endif /* _COMEDI_USB_H */
diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h
index e5daaeb..bc1dfc1 100644
--- a/drivers/staging/comedi/comedidev.h
+++ b/drivers/staging/comedi/comedidev.h
@@ -629,33 +629,4 @@ void comedi_pcmcia_driver_unregister(struct comedi_driver *,
module_driver(__comedi_driver, comedi_pcmcia_driver_register, \
comedi_pcmcia_driver_unregister, &(__pcmcia_driver))
-/* comedi_usb.c - comedi USB driver specific functions */
-
-struct usb_driver;
-struct usb_interface;
-
-struct usb_interface *comedi_to_usb_interface(struct comedi_device *);
-struct usb_device *comedi_to_usb_dev(struct comedi_device *);
-
-int comedi_usb_auto_config(struct usb_interface *, struct comedi_driver *,
- unsigned long context);
-void comedi_usb_auto_unconfig(struct usb_interface *);
-
-int comedi_usb_driver_register(struct comedi_driver *, struct usb_driver *);
-void comedi_usb_driver_unregister(struct comedi_driver *, struct usb_driver *);
-
-/**
- * module_comedi_usb_driver() - Helper macro for registering a comedi USB driver
- * @__comedi_driver: comedi_driver struct
- * @__usb_driver: usb_driver struct
- *
- * Helper macro for comedi USB drivers which do not do anything special
- * in module init/exit. This eliminates a lot of boilerplate. Each
- * module may only use this macro once, and calling it replaces
- * module_init() and module_exit()
- */
-#define module_comedi_usb_driver(__comedi_driver, __usb_driver) \
- module_driver(__comedi_driver, comedi_usb_driver_register, \
- comedi_usb_driver_unregister, &(__usb_driver))
-
#endif /* _COMEDIDEV_H */
--
2.1.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 9/9] staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
@ 2015-01-27 17:49 ` Ian Abbott
0 siblings, 0 replies; 22+ messages in thread
From: Ian Abbott @ 2015-01-27 17:49 UTC (permalink / raw)
To: driverdev-devel; +Cc: Greg Kroah-Hartman, Ian Abbott, linux-kernel
Move the USB-specific stuff out of "comedidev.h" into "comedi_usb.h".
Comedi USB drivers now include "comedi_usb.h" instead of "comedidev.h",
which now gets pulled in indirectly.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
drivers/staging/comedi/comedi_usb.h | 24 ++++++++++++++++++++++--
drivers/staging/comedi/comedidev.h | 29 -----------------------------
2 files changed, 22 insertions(+), 31 deletions(-)
diff --git a/drivers/staging/comedi/comedi_usb.h b/drivers/staging/comedi/comedi_usb.h
index a57025b..721128b 100644
--- a/drivers/staging/comedi/comedi_usb.h
+++ b/drivers/staging/comedi/comedi_usb.h
@@ -23,8 +23,28 @@
#include "comedidev.h"
-/*
- * TODO: Move USB-specific stuff into here from "comedidev.h".
+struct usb_interface *comedi_to_usb_interface(struct comedi_device *);
+struct usb_device *comedi_to_usb_dev(struct comedi_device *);
+
+int comedi_usb_auto_config(struct usb_interface *, struct comedi_driver *,
+ unsigned long context);
+void comedi_usb_auto_unconfig(struct usb_interface *);
+
+int comedi_usb_driver_register(struct comedi_driver *, struct usb_driver *);
+void comedi_usb_driver_unregister(struct comedi_driver *, struct usb_driver *);
+
+/**
+ * module_comedi_usb_driver() - Helper macro for registering a comedi USB driver
+ * @__comedi_driver: comedi_driver struct
+ * @__usb_driver: usb_driver struct
+ *
+ * Helper macro for comedi USB drivers which do not do anything special
+ * in module init/exit. This eliminates a lot of boilerplate. Each
+ * module may only use this macro once, and calling it replaces
+ * module_init() and module_exit()
*/
+#define module_comedi_usb_driver(__comedi_driver, __usb_driver) \
+ module_driver(__comedi_driver, comedi_usb_driver_register, \
+ comedi_usb_driver_unregister, &(__usb_driver))
#endif /* _COMEDI_USB_H */
diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h
index e5daaeb..bc1dfc1 100644
--- a/drivers/staging/comedi/comedidev.h
+++ b/drivers/staging/comedi/comedidev.h
@@ -629,33 +629,4 @@ void comedi_pcmcia_driver_unregister(struct comedi_driver *,
module_driver(__comedi_driver, comedi_pcmcia_driver_register, \
comedi_pcmcia_driver_unregister, &(__pcmcia_driver))
-/* comedi_usb.c - comedi USB driver specific functions */
-
-struct usb_driver;
-struct usb_interface;
-
-struct usb_interface *comedi_to_usb_interface(struct comedi_device *);
-struct usb_device *comedi_to_usb_dev(struct comedi_device *);
-
-int comedi_usb_auto_config(struct usb_interface *, struct comedi_driver *,
- unsigned long context);
-void comedi_usb_auto_unconfig(struct usb_interface *);
-
-int comedi_usb_driver_register(struct comedi_driver *, struct usb_driver *);
-void comedi_usb_driver_unregister(struct comedi_driver *, struct usb_driver *);
-
-/**
- * module_comedi_usb_driver() - Helper macro for registering a comedi USB driver
- * @__comedi_driver: comedi_driver struct
- * @__usb_driver: usb_driver struct
- *
- * Helper macro for comedi USB drivers which do not do anything special
- * in module init/exit. This eliminates a lot of boilerplate. Each
- * module may only use this macro once, and calling it replaces
- * module_init() and module_exit()
- */
-#define module_comedi_usb_driver(__comedi_driver, __usb_driver) \
- module_driver(__comedi_driver, comedi_usb_driver_register, \
- comedi_usb_driver_unregister, &(__usb_driver))
-
#endif /* _COMEDIDEV_H */
--
2.1.4
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* RE: [PATCH 0/9] staging: comedi: introduce comedi_usb.h header
2015-01-27 17:49 ` Ian Abbott
@ 2015-01-27 18:34 ` Hartley Sweeten
-1 siblings, 0 replies; 22+ messages in thread
From: Hartley Sweeten @ 2015-01-27 18:34 UTC (permalink / raw)
To: Ian Abbott, driverdev-devel; +Cc: Greg Kroah-Hartman, linux-kernel
On Tuesday, January 27, 2015 10:49 AM, Ian Abbott wrote:
> "comedidev.h" includes USB-specific stuff that gets included by all
> comedi drivers including non-USB ones. Separate it out into its own
> header "comedi_usb.h". Make the new header include <linux/usb.h> and
> "comedidev.h" so that comedi USB drivers do not need to include them
> explicitly.
>
> 1) staging: comedi: add comedi_usb.h
> 2) staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
> 3) staging: comedi: dt9812: include new "comedi_usb.h" header
> 4) staging: comedi: ni_usb6501: include new "comedi_usb.h" header
> 5) staging: comedi: usbdux: include new "comedi_usb.h" header
> 6) staging: comedi: usbduxfast: include new "comedi_usb.h" header
> 7) staging: comedi: usbduxsigma: include new "comedi_usb.h" header
> 8) staging: comedi: vmk80xx: include new "comedi_usb.h" header
> 9) staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
>
> drivers/staging/comedi/comedi_usb.c | 3 +-
> drivers/staging/comedi/comedi_usb.h | 50 ++++++++++++++++++++++++++++
> drivers/staging/comedi/comedidev.h | 29 ----------------
> drivers/staging/comedi/drivers/dt9812.c | 3 +-
> drivers/staging/comedi/drivers/ni_usb6501.c | 3 +-
> drivers/staging/comedi/drivers/usbdux.c | 3 +-
> drivers/staging/comedi/drivers/usbduxfast.c | 3 +-
> drivers/staging/comedi/drivers/usbduxsigma.c | 3 +-
> drivers/staging/comedi/drivers/vmk80xx.c | 3 +-
> 9 files changed, 57 insertions(+), 43 deletions(-)
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH 0/9] staging: comedi: introduce comedi_usb.h header
@ 2015-01-27 18:34 ` Hartley Sweeten
0 siblings, 0 replies; 22+ messages in thread
From: Hartley Sweeten @ 2015-01-27 18:34 UTC (permalink / raw)
To: Ian Abbott, driverdev-devel; +Cc: Greg Kroah-Hartman, linux-kernel
On Tuesday, January 27, 2015 10:49 AM, Ian Abbott wrote:
> "comedidev.h" includes USB-specific stuff that gets included by all
> comedi drivers including non-USB ones. Separate it out into its own
> header "comedi_usb.h". Make the new header include <linux/usb.h> and
> "comedidev.h" so that comedi USB drivers do not need to include them
> explicitly.
>
> 1) staging: comedi: add comedi_usb.h
> 2) staging: comedi: comedi_usb.c: include new "comedi_usb.h" header
> 3) staging: comedi: dt9812: include new "comedi_usb.h" header
> 4) staging: comedi: ni_usb6501: include new "comedi_usb.h" header
> 5) staging: comedi: usbdux: include new "comedi_usb.h" header
> 6) staging: comedi: usbduxfast: include new "comedi_usb.h" header
> 7) staging: comedi: usbduxsigma: include new "comedi_usb.h" header
> 8) staging: comedi: vmk80xx: include new "comedi_usb.h" header
> 9) staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h
>
> drivers/staging/comedi/comedi_usb.c | 3 +-
> drivers/staging/comedi/comedi_usb.h | 50 ++++++++++++++++++++++++++++
> drivers/staging/comedi/comedidev.h | 29 ----------------
> drivers/staging/comedi/drivers/dt9812.c | 3 +-
> drivers/staging/comedi/drivers/ni_usb6501.c | 3 +-
> drivers/staging/comedi/drivers/usbdux.c | 3 +-
> drivers/staging/comedi/drivers/usbduxfast.c | 3 +-
> drivers/staging/comedi/drivers/usbduxsigma.c | 3 +-
> drivers/staging/comedi/drivers/vmk80xx.c | 3 +-
> 9 files changed, 57 insertions(+), 43 deletions(-)
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2015-01-27 18:35 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-27 17:49 [PATCH 0/9] staging: comedi: introduce comedi_usb.h header Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 1/9] staging: comedi: add comedi_usb.h Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 2/9] staging: comedi: comedi_usb.c: include new "comedi_usb.h" header Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 3/9] staging: comedi: dt9812: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 4/9] staging: comedi: ni_usb6501: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 5/9] staging: comedi: usbdux: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 6/9] staging: comedi: usbduxfast: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 7/9] staging: comedi: usbduxsigma: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 8/9] staging: comedi: vmk80xx: " Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 17:49 ` [PATCH 9/9] staging: comedi: comedi_usb.h: move USB stuff out of comedidev.h Ian Abbott
2015-01-27 17:49 ` Ian Abbott
2015-01-27 18:34 ` [PATCH 0/9] staging: comedi: introduce comedi_usb.h header Hartley Sweeten
2015-01-27 18:34 ` Hartley Sweeten
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.