new driver for wireless xbox receiver for review

From 61e650894787215e5939ad73d100d3fb3ab3ff17 Mon Sep 17 00:00:00 2001

From 4d0e6a5399b5c2bd4f4323f2717804ba8abb9b92 Mon Sep 17 00:00:00 2001

Hi,

This patch implements a driver for the "Microsoft Xbox 360 Wireless Receiver for Windows" that was recently released. It seems to mostly "just work" for the basics at this point and I'd like to get some feedback on it since this is my first attempt at driver development.

It borrows heavily from xpad.c since it's basically the same controller with new wireless magic. The difference being that the receiever can host multiple controllers through the same device and the driver manages them coming and going. In particular I moved the input device creation/deletion out of the probe function and into workqueues kicked off by usb interrupts. That would probably be a good place for prospective reviewers to start finding issues. :)

+ input_dev->private = xboxrcvr;

Thnk you for the patch. Looking at the code I do not see one device supporintg multiple controllers... You have one input device per usb device/interface and the properties of said device are known beforehand. So I would just create and register input device right there in xboxrcvr_probe() and get rid of your build and teardown works. The fact that is it a wireless device and at transmitter may at times be out of range is not important.

Overall I would like support for this wireless receiver to be incorporated into xpad.c driver.

I also have some more comments, see below.

Review

Quoted 2 lines

(quoted diff) +2 -0

+
+static unsigned long debug = 0;

You do not need to initialize statig variables to 0. It only increases size of the image.

drivers/input/joystick/xboxrcvr.c line 110

Quoted 7 lines

drivers/input/joystick/xboxrcvr.c new file +5 -0

+module_param(debug, ulong, 0644);
+MODULE_PARM_DESC(debug, "debug level");
+
 
+
+       if(debug) {

Please use spaces between statement (if,for, while) and opening paren. .

drivers/input/joystick/xboxrcvr.c line 154

Quoted 4 lines

drivers/input/joystick/xboxrcvr.c new file +4 -0

+
+       /* Valid pad data */
+       if(!(sdata[1] & 0x1) || !xboxrcvr->open)
+               return;

Why are we generating interrupts if device is not used? Have the ->open() method submit urbs and then you don't need to check it here.

Quoted 8 lines

drivers/input/joystick/xboxrcvr.c new file +4 -0

+
+       /* FIXME: Yuck.  Need to make sure this doesn't get truncated */
+       usb_make_path(xboxrcvr->udev, path, sizeof(xboxrcvr->phys));
+       snprintf(xboxrcvr->phys, sizeof(xboxrcvr->phys), "%s/input%d", 

path, xboxrcvr->id);
+ snprintf(xboxrcvr->uniq, sizeof(xboxrcvr->uniq), "xpad%d",
xboxrcvr->id >> 1);

Uniq is used to carry identificator unique for the device, not within a particular system. It is property of device itself and should not change if moved to a different box so do not try to come up with artificial data for it.

Review

Quoted 3 lines

(quoted diff) +2 -0

+
+       input_dev->name     = xboxrcvr_device[0].name;

Hmm....

drivers/input/joystick/xboxrcvr.c line 288

Quoted 3 lines

drivers/input/joystick/xboxrcvr.c new file +3 -0

+       input_dev->phys     = xboxrcvr->phys;
+       input_dev->uniq     = xboxrcvr->uniq;
+       input_dev->cdev.dev = &(xboxrcvr->intf->dev);

Please use

input_dev->dev.parent = &xboxrcvr->intf->dev;

- input devices are meving moved from class_device to struct device.

Quoted 1 line

+ input_dev->private = xboxrcvr;

input_set_drvdata(input_dev, xboxrcvr); And use input_get_drvdata() instead of accessing dev->private directly.

Dmitry

Hi Dimitri,

Thanks for taking the time to review. My reponses are inline.

-Brian

Quoted 8 lines · Dmitry Torokhov

Thnk you for the patch. Looking at the code I do not see one device supporintg multiple controllers... You have one input device per usb device/interface and the properties of said device are known beforehand. So I would just create and register input device right there in xboxrcvr_probe() and get rid of your build and teardown works. The fact that is it a wireless device and at transmitter may at times be out of range is not important.

Actually, I rather liked the fact that the input devices only show up when there is a controller connected to the receiver, since there can be anywhere from 0-4 controllers alive. This way there are no device nodes for non-existant controllers.

Quoted 3 lines

Overall I would like support for this wireless receiver to be incorporated into xpad.c driver.

Knew that was coming :). Should be able to manage it. Since the wireless model returns a expanded data format it needs to be handled specially. Another flag in the xpad_device struct?

Hi Brain,

[Added Jan to CC list]

Quoted 16 lines · Brian Magnuson

Hi Dimitri,

Thanks for taking the time to review. My reponses are inline.

-Brian

Quoted 8 lines · Dmitry Torokhov

Thnk you for the patch. Looking at the code I do not see one device supporintg multiple controllers... You have one input device per usb device/interface and the properties of said device are known beforehand. So I would just create and register input device right there in xboxrcvr_probe() and get rid of your build and teardown works. The fact that is it a wireless device and at transmitter may at times be out of range is not important.

Quoted 1 line

Actually, I rather liked the fact that the input devices only show up

when

Quoted 1 line

there is a controller connected to the receiver, since there can be

anywhere

Quoted 2 lines

from 0-4 controllers alive. This way there are no device nodes for non-existant controllers.

Let's keep it simple. This way we don't have to worry about races between build and teardown work pieces and the driver is much simplier. After all we don't do dynamic device creation for wireless mice (non blue-tooth). And if we did it would make some programs unhappy if they did not see that device all the time... Even with controller - imagine you are playing and somebody calls you - you walk away with your controller in hand and when you come back it stops working because we tore down one device and created the a one but application is still latched to the old device - not nice.

Quoted 5 lines

Quoted 3 lines

Overall I would like support for this wireless receiver to be incorporated into xpad.c driver.

Knew that was coming :). Should be able to manage it. Since the

wireless

Quoted 2 lines

model returns a expanded data format it needs to be handled specially. Another flag in the xpad_device struct?

We have xtype field, I think it should be used.

Quoted 6 lines

Quoted 1 line

Knew that was coming :). Should be able to manage it. Since the

wireless

Quoted 2 lines

model returns a expanded data format it needs to be handled specially. Another flag in the xpad_device struct?

We have xtype field, I think it should be used.

I don't see this field in xpad_device in your tree or Linus'.

In any case I'll take these changes and your earlier ones, merge it into xpad.c and
resubmit.

Thanks,
Brian

Hello,

Quoted 3 lines

Quoted 3 lines

Quoted 1 line

Knew that was coming :). Should be able to manage it. Since the

wireless

Quoted 1 line

model returns a expanded data format it needs to be handled

specially.

Quoted 2 lines

Quoted 2 lines

Quoted 1 line

Another flag in the xpad_device struct?

Quoted 4 lines

Quoted 1 line

We have xtype field, I think it should be used.

I don't see this field in xpad_device in your tree or Linus'.

It is not yet in Linus' tree, but it definitely is in Dmitry input tree.

http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=commit;h=bad058541d848500b5f2cc049c1082d0b2935b7d

see that in dmitry tree there is support for rumble as well as other minor changes. (And I submited support for controlling leds yesterday) Btw you are stating that it process packet is based on information from
http://euc.jp/periphs/xbox-controller.en.html, which I doubt, because this page
is actually about xbox (1) controller. Not xbox 360. But on the other hand you
are interpreting the report correctly. :)

Quoted 4 lines

In any case I'll take these changes and your earlier ones, merge it into xpad.c and
resubmit.

Good luck,
Jan

Quoted 3 lines · Jan Kratochvil

It is not yet in Linus' tree, but it definitely is in Dmitry input tree.

http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=commit;h=bad058541d848500b5f2cc049c1082d0b2935b7d

Indeed it is. I think this is just some basic git ignorance on my part. I clearly see it in
the link you provided but if I descend from the tree link starting from top commit at
http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=summary
I get some older
version of xpad.c. Am I just being dense (likely) or is there something wrong?

The topmost commit (time-wise) is on "for-linus" branch of my tree. It contains bug-fixes and tiny cleanups that I plan to push to Linus shortly. The main branch is "master" which contains all the stuff that I want to merge once 2.6.23 opens (that includes xbox, aiptek and more).

Apparently you are descending into the tree for "for-linus" branch.

--
Dmitry

Hi,

This patch implements a driver for the "Microsoft Xbox 360 Wireless Receiver for Windows" that was recently released. It seems to mostly "just work" for the basics at this point and I'd like to get some feedback on it since this is my first attempt at driver development.

It borrows heavily from xpad.c since it's basically the same controller with new wireless magic. The difference being that the receiever can host multiple controllers through the same device and the driver manages them coming and going. In particular I moved the input device creation/deletion out of the probe function and into workqueues kicked off by usb interrupts. That would probably be a good place for prospective reviewers to start finding issues. :)

Thanks for looking.

-Brian

Please CC me on replies since I'm not subscribed. Thanks.

Quoted 1 line

From 61e650894787215e5939ad73d100d3fb3ab3ff17 Mon Sep 17 00:00:00 2001

From: Brian Magnuson <magnuson@ferrari.(none)>
Date: Mon, 28 May 2007 23:10:18 -0400
Subject: [PATCH] New xboxrcvr driver

Signed-off-by Brian Magnuson <[email protected]>

---
 drivers/input/joystick/xboxrcvr.c |  463 +++++++++++++++++++++++++++++++++++++
 drivers/input/joystick/xboxrcvr.h |   72 ++++++
 2 files changed, 535 insertions(+), 0 deletions(-)

drivers/input/joystick/xboxrcvr.c new file +463 -0

new file mode 100644
index 0000000..e132726
@@ -0,0 +1,463 @@
+/*
+ * Xbox wireless receiver input device driver for Linux - v0.0.1
+ *
+ * Copyright (c)  2007 Brian Magnuson <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ *
+ * This driver is based on:
+ *  - information from     http://euc.jp/periphs/xbox-controller.en.html
+ *  - the iForce driver    drivers/char/joystick/iforce.c
+ *  - the skeleton-driver  drivers/usb/usb-skeleton.c
+ *  - the xpad driver      drivers/usb/input/xpad.c
+ *
+ * Thanks to:
+ *  - ITO Takayuki for providing essential xpad information on his website
+ *  - Vojtech Pavlik     - iforce driver / input subsystem
+ *  - Greg Kroah-Hartman - usb-skeleton driver
+ *  - The xpad.c developers on which this driver is based in large part
+ *        Vojtech Pavlik <[email protected]>,
+ *        Oliver Schwartz <[email protected]>,
+ *        Thomas Pedley <[email protected]>,
+ *        Steven Toth <[email protected]>,
+ *        Franz Lehner <[email protected]>,
+ *        Ivan Hawkes <[email protected]>
+ *        Edgar Hucek <[email protected]>
+ *        Niklas Lundberg <[email protected]>
+ *
+ * TODO:
+ *  - Der blinken-lights remain blinking.  The xpad method of turning it off
+ *    doesn't appear to work.
+ *  - Rumble is also not functional
+ *  - There's always the headset. :)  Probably belongs in a different driver.
+ *
+ */
+#include <linux/kernel.h>
+#include <linux/init.h>
+#include <linux/slab.h>
+#include <linux/module.h>
+#include <linux/smp_lock.h>
+#include <linux/usb.h>
+#include <linux/version.h>
+#include <linux/usb/input.h>
+#include <linux/usbdevice_fs.h>
+#include <linux/timer.h>
+#include <asm/uaccess.h>
+
+#include "xboxrcvr.h"
+
+static unsigned long debug = 0;
+module_param(debug, ulong, 0644);
+MODULE_PARM_DESC(debug, "debug level");
+
+static const struct xboxrcvr_device xboxrcvr_device[] = {
+	{ 0x045e, 0x0719, "Xbox 360 Wireless Receiver for Windows"}, /* Ha! */
+	{ 0x0000, 0x0000, "nothing detected - FAIL"}
+};
+
+static struct usb_device_id xboxrcvr_table [] = {
+	{ USB_DEVICE(0x045e, 0x0719) },
+	{ }
+};
+
+static const signed short xboxrcvr_btn[] = {
+	BTN_A, BTN_B, BTN_X, BTN_Y,                  /* analogue buttons */
+	BTN_START, BTN_BACK, BTN_THUMBL, BTN_THUMBR, /* start/back/sticks */
+	BTN_0, BTN_1, BTN_2, BTN_3,                  /* d-pad as buttons */
+	BTN_TL, BTN_TR,                              /* Button LB/RB */
+	BTN_MODE,                                    /* The big X */
+	-1                                           /* terminating entry */
+};
+
+static const signed short xboxrcvr_abs[] = {
+	ABS_X, ABS_Y,   /* left stick */
+	ABS_RX, ABS_RY, /* right stick */
+	ABS_Z, ABS_RZ,  /* triggers left/right */
+	-1              /* terminating entry */
+};
+
+MODULE_DEVICE_TABLE(usb, xboxrcvr_table);
+
+/**
+ * xboxrcvr_process_packet
+ *
+ * Completes a request by converting the data into events
+ * for the input subsystem.
+ *
+ * The report descriptor was taken from ITO Takayukis website:
+ *  http://euc.jp/periphs/xbox-controller.en.html
+ */
+static void xboxrcvr_process_packet(struct usb_xboxrcvr *xboxrcvr, unsigned char *sdata)
+{
+	struct input_dev *dev = xboxrcvr->dev;
+	int i;
+	int last;
+
+	/* Four byte shift over so we can just use the code from xpad.c */
+	unsigned char *data = &sdata[4];
+
+	if(debug) {
+		printk(KERN_INFO "xboxrcvr: id: %d, data:", xboxrcvr->id);
+		/* Just the 4 bytes of header info or everything */
+		if(debug == 1)
+			last = 4;
+		else
+			last = 29;
+		for(i = 0; i < last; i++) {
+			printk("0x%02x ", sdata[i]);
+		}
+		printk("\n");
+		return;
+	}
+
+	/* The 1st four bytes appear to be some kind of status word */
+
+	/* Presence change - Create/Delete an input device */
+	/* Well, we're getting something.  Create a device */
+	if(!xboxrcvr->gamepad_present) {
+		xboxrcvr->gamepad_present = 1;
+		schedule_work(&xboxrcvr->idev_build);
+		return;
+	}
+
+	if(!xboxrcvr->gamepad_registered)
+		return;
+
+	if(sdata[0] & 0x8) {
+		if(!(sdata[1] & 0x80)) {
+			/* xboxrcvr->gamepad_present cleared in teardown */
+			xboxrcvr->gamepad_registered = 0;
+			schedule_work(&xboxrcvr->idev_teardown);
+		}
+
+		/* Doesn't do anything yet.  Thanks for noticing */
+		if(sdata[1] & 0x40) {
+			xboxrcvr->headset_present = 1;
+		} else {
+			xboxrcvr->headset_present = 0;
+		}
+	}
+
+	/* Valid pad data */
+	if(!(sdata[1] & 0x1) || !xboxrcvr->open)
+		return;
+
+	/* digital pad (button mode) bits (3 2 1 0) (right left down up) */
+	input_report_key(dev, BTN_0, (data[2] & 0x01));
+	input_report_key(dev, BTN_1, (data[2] & 0x08) >> 3);
+	input_report_key(dev, BTN_2, (data[2] & 0x02) >> 1);
+	input_report_key(dev, BTN_3, (data[2] & 0x04) >> 2);
+
+	/* start and back buttons */
+	input_report_key(dev, BTN_START, (data[2] & 0x10) >> 4);
+	input_report_key(dev, BTN_BACK,  (data[2] & 0x20) >> 5);
+
+	/* stick press left/right */
+	input_report_key(dev, BTN_THUMBL, (data[2] & 0x40) >> 6);
+	input_report_key(dev, BTN_THUMBR,  data[2] >> 7);
+
+	/* buttons A, B, X, Y digital mode */
+	input_report_key(dev, BTN_A,    (data[3] & 0x10) >> 4);
+	input_report_key(dev, BTN_B,    (data[3] & 0x20) >> 5);
+	input_report_key(dev, BTN_X,    (data[3] & 0x80) >> 7);
+	input_report_key(dev, BTN_Y,    (data[3] & 0x40) >> 6);
+	input_report_key(dev, BTN_TL,    data[3] & 0x01);
+	input_report_key(dev, BTN_TR,   (data[3] & 0x02) >> 1);
+	input_report_key(dev, BTN_MODE, (data[3] & 0x04) >> 2);
+
+	/* left stick */
+	input_report_abs(dev, ABS_X,  (__s16)(((__s16)data[7] << 8) | (__s16)data[6]));
+	input_report_abs(dev, ABS_Y,  (__s16)(((__s16)data[9] << 8) | (__s16)data[8]));
+
+	/* right stick */
+	input_report_abs(dev, ABS_RX, (__s16)(((__s16)data[13] << 8) | (__s16)data[12]));
+	input_report_abs(dev, ABS_RY, (__s16)(((__s16)data[11] << 8) | (__s16)data[10]));
+
+	/* triggers left/right */
+	input_report_abs(dev, ABS_Z,  data[4]);
+	input_report_abs(dev, ABS_RZ, data[5]);
+
+	input_sync(dev);
+}
+
+/**
+ * xboxrcvr_open
+ *
+ * Called when a an application opens the device.
+ */
+static int xboxrcvr_open(struct input_dev *dev)
+{
+	struct usb_xboxrcvr *xboxrcvr = dev->private;
+
+	if(debug)
+		info("opening device");
+	xboxrcvr->open = 1;
+
+	return 0;
+}
+
+/**
+ * xboxrcvr_close
+ *
+ * Called when an application closes the device.
+ */
+static void xboxrcvr_close(struct input_dev *dev)
+{
+	struct usb_xboxrcvr *xboxrcvr = dev->private;
+
+	if(debug)
+		info("closing device");
+	xboxrcvr->open = 0;
+}
+
+/**
+ * xboxrcvr_irq_in
+ *
+ * Completion handler for interrupt in transfers (user input).
+ * Just calls xboxrcvr_process_packet which does then emit input events.
+ */
+static void xboxrcvr_irq_in(struct urb *urb)
+{
+	struct usb_xboxrcvr *xboxrcvr = urb->context;
+	int retval;
+
+	switch (urb->status) {
+	case 0:
+		/* success */
+		break;
+	case -ECONNRESET:
+	case -ENOENT:
+	case -ESHUTDOWN:
+		/* this urb is terminated, clean up */
+		dbg("%s - urb shutting down with status: %d",
+		    __FUNCTION__, urb->status);
+		return;
+	default:
+		dbg("%s - nonzero urb status received: %d",
+		    __FUNCTION__, urb->status);
+		goto exit;
+	}
+
+	xboxrcvr_process_packet(xboxrcvr, xboxrcvr->idata);
+
+exit:
+	retval = usb_submit_urb(urb, GFP_ATOMIC);
+	if (retval)
+		err("%s - usb_submit_urb failed with result %d",
+		    __FUNCTION__, retval);
+}
+
+/*
+ * Create an input device when a controller shows up
+*/
+void idev_build_func(struct work_struct *work)
+{
+	struct usb_xboxrcvr *xboxrcvr = container_of(work, struct usb_xboxrcvr,
+	                                             idev_build);
+
+	struct input_dev *input_dev;
+	char path[65];
+	int i;
+
+	input_dev = input_allocate_device();
+	if(!input_dev) {
+		info("Could not allocate input device");
+		return;
+	}
+
+
+	/* FIXME: Yuck.  Need to make sure this doesn't get truncated */
+	usb_make_path(xboxrcvr->udev, path, sizeof(xboxrcvr->phys));
+	snprintf(xboxrcvr->phys, sizeof(xboxrcvr->phys), "%s/input%d", path, xboxrcvr->id);
+	snprintf(xboxrcvr->uniq, sizeof(xboxrcvr->uniq), "xpad%d", xboxrcvr->id >> 1);
+
+	input_dev->name     = xboxrcvr_device[0].name;
+	input_dev->phys     = xboxrcvr->phys;
+	input_dev->uniq     = xboxrcvr->uniq;
+	input_dev->cdev.dev = &(xboxrcvr->intf->dev);
+	input_dev->private  = xboxrcvr;
+	input_dev->open     = xboxrcvr_open;
+	input_dev->close    = xboxrcvr_close;
+	usb_to_input_id(xboxrcvr->udev, &input_dev->id);
+
+	input_dev->evbit[0] = BIT(EV_KEY) | BIT(EV_ABS);
+
+	for (i = 0; xboxrcvr_btn[i] >= 0; ++i)
+		set_bit(xboxrcvr_btn[i], input_dev->keybit);
+
+	for (i = 0; xboxrcvr_abs[i] >= 0; ++i) {
+
+		signed short t = xboxrcvr_abs[i];
+
+		set_bit(t, input_dev->absbit);
+
+		switch (t) {
+			case ABS_X:
+			case ABS_Y:
+			case ABS_RX:
+			case ABS_RY: /* the two sticks */
+				input_set_abs_params(input_dev, t,
+				  -32768, 32767, 16, 12000);
+				break;
+			case ABS_Z:  /* left trigger */
+			case ABS_RZ: /* right trigger */
+				input_set_abs_params(input_dev, t,
+				   0, 255, 0, 0);
+				break;
+		}
+	}
+
+	xboxrcvr->dev = input_dev;
+	input_register_device(xboxrcvr->dev);
+	xboxrcvr->gamepad_registered = 1;
+}
+
+void xboxrcvr_unregister_idev(struct input_dev **input_dev)
+{
+	if(*input_dev)
+		input_unregister_device(*input_dev);
+	*input_dev = NULL;
+}
+
+void idev_teardown_func(struct work_struct *work)
+{
+	struct usb_xboxrcvr *xboxrcvr = container_of(work, struct usb_xboxrcvr,
+	                                             idev_teardown);
+	xboxrcvr_unregister_idev(&xboxrcvr->dev);
+	xboxrcvr->gamepad_present = 0;
+}
+
+/**
+ * xboxrcvr_probe
+ *
+ * Called upon device detection to find a suitable driver.
+ * Must return NULL when no xboxrcvr is found, else setup everything.
+ */
+static int xboxrcvr_probe(struct usb_interface *intf,
+                          const struct usb_device_id *id)
+{
+	struct usb_device *udev = interface_to_usbdev(intf);
+	struct usb_xboxrcvr *xboxrcvr;
+
+	struct usb_endpoint_descriptor *ep_irq_in;
+	int status;
+
+	/* Odd numbered interfaces appear to be audio/microphone */
+	if(intf->cur_altsetting->desc.bInterfaceNumber % 2)
+		return -ENODEV;
+
+	xboxrcvr = kzalloc(sizeof(struct usb_xboxrcvr), GFP_KERNEL);
+	if (!xboxrcvr)
+		goto fail1;
+
+	xboxrcvr->id = intf->cur_altsetting->desc.bInterfaceNumber;
+	xboxrcvr->idata = usb_buffer_alloc(udev, XBOXRCVR_PKT_LEN,
+	                                   GFP_ATOMIC, &xboxrcvr->idata_dma);
+	xboxrcvr->udev = udev;
+
+	if (!xboxrcvr->idata)
+		goto fail1;
+
+	/* setup input interrupt pipe (button and axis state) */
+	xboxrcvr->irq_in = usb_alloc_urb(0, GFP_KERNEL);
+	if (!xboxrcvr->irq_in)
+		goto fail2;
+
+	ep_irq_in = &intf->cur_altsetting->endpoint[0].desc;
+	usb_fill_int_urb(xboxrcvr->irq_in, udev,
+	                 usb_rcvintpipe(udev, ep_irq_in->bEndpointAddress),
+	                 xboxrcvr->idata, XBOXRCVR_PKT_LEN, xboxrcvr_irq_in,
+	                 xboxrcvr, ep_irq_in->bInterval);
+	xboxrcvr->irq_in->transfer_dma = xboxrcvr->idata_dma;
+	xboxrcvr->irq_in->transfer_flags |= URB_NO_TRANSFER_DMA_MAP;
+
+	usb_set_intfdata(intf, xboxrcvr);
+
+	xboxrcvr->irq_in->dev = xboxrcvr->udev;
+	if ((status = usb_submit_urb(xboxrcvr->irq_in, GFP_KERNEL))) {
+		err("submit int urb failed: %d", status);
+		goto fail2;
+	}
+
+	xboxrcvr->intf = intf;
+	INIT_WORK(&xboxrcvr->idev_build, idev_build_func);
+	INIT_WORK(&xboxrcvr->idev_teardown, idev_teardown_func);
+
+	return 0;
+
+fail2:  usb_buffer_free(udev, XBOXRCVR_PKT_LEN, xboxrcvr->idata,
+                        xboxrcvr->idata_dma);
+fail1:  return -ENOMEM;
+}
+
+/**
+ * xboxrcvr_disconnect
+ *
+ * Called upon device disconnect to dispose of the structures and
+ * close the USB connections.
+ */
+static void xboxrcvr_disconnect(struct usb_interface *intf)
+{
+	struct usb_xboxrcvr *xboxrcvr = usb_get_intfdata(intf);
+
+	usb_set_intfdata(intf, NULL);
+	if (xboxrcvr) {
+		usb_kill_urb(xboxrcvr->irq_in);
+		usb_free_urb(xboxrcvr->irq_in);
+
+		xboxrcvr_unregister_idev(&xboxrcvr->dev);
+
+		usb_buffer_free(interface_to_usbdev(intf), XBOXRCVR_PKT_LEN,
+		                xboxrcvr->idata, xboxrcvr->idata_dma);
+
+		kfree(xboxrcvr);
+	}
+
+}
+
+/******************* Linux driver framework specific stuff ************/
+
+static struct usb_driver xboxrcvr_driver = {
+	.name       = "xboxrcvr",
+	.probe      = xboxrcvr_probe,
+	.disconnect = xboxrcvr_disconnect,
+	.id_table   = xboxrcvr_table,
+};
+
+/**
+ * driver init entry point
+ */
+static int __init usb_xboxrcvr_init(void)
+{
+	int result = usb_register(&xboxrcvr_driver);
+	if (result == 0)
+		info(DRIVER_DESC " " DRIVER_VERSION);
+	return result;
+}
+
+/**
+ * driver exit entry point
+ */
+static void __exit usb_xboxrcvr_exit(void)
+{
+	usb_deregister(&xboxrcvr_driver);
+}
+
+module_init(usb_xboxrcvr_init);
+module_exit(usb_xboxrcvr_exit);
+
+MODULE_AUTHOR(DRIVER_AUTHOR);
+MODULE_DESCRIPTION(DRIVER_DESC);
+MODULE_LICENSE("GPL");
+

drivers/input/joystick/xboxrcvr.h new file +72 -0

new file mode 100644
index 0000000..685dbff
@@ -0,0 +1,72 @@
+/*
+ * Xbox wireless receiver input device driver for Linux - v0.0.1
+ *
+ * Copyright (c)  2007 Brian Magnuson <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ *
+ */
+
+#ifndef __XBOXRCVR_h
+#define __XBOXRCVR_h
+
+/************************* driver internals ***************************/
+#ifdef __KERNEL__
+
+#include <linux/input.h>
+#include <linux/circ_buf.h>
+
+/****************** driver description and version ********************/
+#define DRIVER_VERSION "v0.0.1"
+#define DRIVER_AUTHOR  "Brian Magnuson"
+#define DRIVER_DESC    "Driver for wireless xbox360 controllers"
+
+#define XBOXRCVR_PKT_LEN 32
+
+/************************* the device struct **************************/
+struct usb_xboxrcvr {
+	struct input_dev *dev;      /* input device interface */
+	struct usb_device *udev;    /* usb device */
+
+	struct urb *irq_in;         /* urb for int. in report */
+
+	int id;
+	int gamepad_present;
+	int gamepad_registered;
+	int headset_present;
+
+	struct usb_interface *intf;
+
+	struct work_struct idev_build;
+	struct work_struct idev_teardown;
+
+	unsigned char *idata;   /* input data */
+	dma_addr_t idata_dma;
+
+	int open;               /* open to the input layer */
+
+	char phys[65];          /* physical input dev path */
+	char uniq[5];           /* unique identifier */
+};
+
+/* for the list of know devices */
+struct xboxrcvr_device {
+	u16 idVendor;
+	u16 idProduct;
+	char *name;
+};
+
+#endif /* __KERNEL__ */
+
+#endif /* __XBOXRCVR_h */