linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [Bluez-devel] Renaming USB HID driver
@ 2004-03-22 13:54 Marcel Holtmann
  2004-03-23 10:22 ` Nils Faerber
  2004-03-29  8:47 ` dirk husemann
  0 siblings, 2 replies; 6+ messages in thread
From: Marcel Holtmann @ 2004-03-22 13:54 UTC (permalink / raw)
  To: BlueZ Mailing List; +Cc: Vojtech Pavlik, Greg Kroah-Hartman

[-- Attachment #1: Type: text/plain, Size: 930 bytes --]

Hi Folks,

my plan is to move the HID parser out of the USB subsystem and provide a
general HID parser implementation under drivers/hid/ which could then be
used by the USB and the Bluetooth subsystem drivers. To avoid any naming
conflict due and after the development I suggest to rename the USB HID
module from hid.ko to usbhid.ko (see attached patch).

I know that renaming a kernel module in a stable series is not a good
idea, but doing it now will help us in the long term. However people
which are using hotplug won't even notice it, because the kernel knows
the right module name. And if the transition is finished we can simply
remove the HID parser code from usbhid and everything will still work.

In the end it should look like this:

	drivers/hid/hid.ko		HID parser (input+hiddev+ff)
	drivers/usb/input/usbhid.ko	USB HID transport layer
	net/bluetooth/hidp/hidp.ko	Bluetooth HID protocol

Comments?

Regards

Marcel


[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 1583 bytes --]

===== drivers/usb/input/Kconfig 1.13 vs edited =====
--- 1.13/drivers/usb/input/Kconfig	Fri Feb 27 00:32:51 2004
+++ edited/drivers/usb/input/Kconfig	Mon Mar 22 14:42:37 2004
@@ -21,7 +21,7 @@
 	  If unsure, say Y.
 
 	  To compile this driver as a module, choose M here: the
-	  module will be called hid.
+	  module will be called usbhid.
 
 comment "Input core support is needed for USB HID input layer or HIDBP support"
 	depends on USB && INPUT=n
===== drivers/usb/input/Makefile 1.17 vs edited =====
--- 1.17/drivers/usb/input/Makefile	Fri Feb 27 13:51:51 2004
+++ edited/drivers/usb/input/Makefile	Mon Mar 22 14:32:15 2004
@@ -3,32 +3,32 @@
 #
 
 # Multipart objects.
-hid-objs	:= hid-core.o
+usbhid-objs	:= hid-core.o
 
 # Optional parts of multipart objects.
 
 ifeq ($(CONFIG_USB_HIDDEV),y)
-	hid-objs        += hiddev.o
+	usbhid-objs	+= hiddev.o
 endif
 ifeq ($(CONFIG_USB_HIDINPUT),y)
-	hid-objs        += hid-input.o
+	usbhid-objs	+= hid-input.o
 endif
 ifeq ($(CONFIG_HID_PID),y)
-	hid-objs	+= pid.o
+	usbhid-objs	+= pid.o
 endif
 ifeq ($(CONFIG_LOGITECH_FF),y)
-	hid-objs	+= hid-lgff.o
+	usbhid-objs	+= hid-lgff.o
 endif
 ifeq ($(CONFIG_THRUSTMASTER_FF),y)
-	hid-objs	+= hid-tmff.o
+	usbhid-objs	+= hid-tmff.o
 endif
 ifeq ($(CONFIG_HID_FF),y)
-	hid-objs	+= hid-ff.o
+	usbhid-objs	+= hid-ff.o
 endif
 
 obj-$(CONFIG_USB_AIPTEK)	+= aiptek.o
 obj-$(CONFIG_USB_ATI_REMOTE)	+= ati_remote.o
-obj-$(CONFIG_USB_HID)		+= hid.o
+obj-$(CONFIG_USB_HID)		+= usbhid.o
 obj-$(CONFIG_USB_KBD)		+= usbkbd.o
 obj-$(CONFIG_USB_KBTAB)		+= kbtab.o
 obj-$(CONFIG_USB_MOUSE)		+= usbmouse.o

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2004-03-29  8:47 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-03-22 13:54 [Bluez-devel] Renaming USB HID driver Marcel Holtmann
2004-03-23 10:22 ` Nils Faerber
2004-03-23 10:30   ` Vojtech Pavlik
2004-03-24 15:27   ` Philip Blundell
2004-03-25 14:45     ` Nils Faerber
2004-03-29  8:47 ` dirk husemann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).