| Message ID | 1312553358-26280-3-git-send-email-ospite@studenti.unina.it (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 08/05/2011 10:09 AM, Antonio Ospite wrote: > Add a plugin which handles the connection of a Sixaxis device, when a > new hidraw device is connected the plugin: > - Filters udev events, and select the Sixaxis device > - Sets LEDs to match the joystick system number (for USB and BT) > - Sets the Master bluetooth address in the Sixaxis (USB pairing) > - Adds the device to the database of the current default > adapter (BT association) > > Signed-off-by: Bastien Nocera <hadess@hadess.net> > Signed-off-by: Antonio Ospite <ospite@studenti.unina.it> > --- > Makefile.am | 9 +- > acinclude.m4 | 10 + > plugins/sixaxis.c | 593 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 610 insertions(+), 2 deletions(-) > create mode 100644 plugins/sixaxis.c > > diff --git a/Makefile.am b/Makefile.am > index 68380d9..dbe0170 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -224,6 +224,11 @@ builtin_sources += thermometer/main.c \ > thermometer/thermometer.h thermometer/thermometer.c > endif > > +if SIXAXISPLUGIN > +builtin_modules += sixaxis > +builtin_sources += plugins/sixaxis.c > +endif > + > builtin_modules += hciops mgmtops > builtin_sources += plugins/hciops.c plugins/mgmtops.c > > @@ -283,7 +288,7 @@ src_bluetoothd_SOURCES = $(gdbus_sources) $(builtin_sources) \ > src/event.h src/event.c \ > src/oob.h src/oob.c src/eir.h src/eir.c > src_bluetoothd_LDADD = lib/libbluetooth.la @GLIB_LIBS@ @DBUS_LIBS@ \ > - @CAPNG_LIBS@ -ldl -lrt > + @CAPNG_LIBS@ @UDEV_LIBS@ -ldl -lrt > src_bluetoothd_LDFLAGS = -Wl,--export-dynamic \ > -Wl,--version-script=$(srcdir)/src/bluetooth.ver > > @@ -398,7 +403,7 @@ EXTRA_DIST += doc/manager-api.txt \ > > AM_YFLAGS = -d > > -AM_CFLAGS = @DBUS_CFLAGS@ @GLIB_CFLAGS@ @CAPNG_CFLAGS@ \ > +AM_CFLAGS = @DBUS_CFLAGS@ @GLIB_CFLAGS@ @CAPNG_CFLAGS@ @UDEV_CFLAGS@ \ > -DBLUETOOTH_PLUGIN_BUILTIN -DPLUGINDIR=\""$(plugindir)"\" > > INCLUDES = -I$(builddir)/lib -I$(builddir)/src -I$(srcdir)/src \ > diff --git a/acinclude.m4 b/acinclude.m4 > index 3cb9459..6176483 100644 > --- a/acinclude.m4 > +++ b/acinclude.m4 > @@ -186,6 +186,7 @@ AC_DEFUN([AC_ARG_BLUEZ], [ > sndfile_enable=${sndfile_found} > hal_enable=no > usb_enable=${usb_found} > + sixaxis_enable=${udev_found} > alsa_enable=${alsa_found} > gstreamer_enable=${gstreamer_found} > audio_enable=yes > @@ -286,6 +287,10 @@ AC_DEFUN([AC_ARG_BLUEZ], [ > usb_enable=${enableval} > ]) > > + AC_ARG_ENABLE(sixaxis, AC_HELP_STRING([--enable-sixaxis], [enable Sixaxis plugin]), [ > + sixaxis_enable=${enableval} > + ]) > + > AC_ARG_ENABLE(tracer, AC_HELP_STRING([--enable-tracer], [install Tracing daemon]), [ > tracer_enable=${enableval} > ]) > @@ -385,6 +390,10 @@ AC_DEFUN([AC_ARG_BLUEZ], [ > AC_DEFINE(HAVE_LIBUSB, 1, [Define to 1 if you have USB library.]) > fi > > + if (test "${sixaxis_enable}" = "yes" && test "${udev_found}" = "yes"); then > + AC_DEFINE(HAVE_SIXAXIS_PLUGIN, 1, [Define to 1 if you have sixaxis plugin.]) > + fi > + > AM_CONDITIONAL(SNDFILE, test "${sndfile_enable}" = "yes" && test "${sndfile_found}" = "yes") > AM_CONDITIONAL(USB, test "${usb_enable}" = "yes" && test "${usb_found}" = "yes") > AM_CONDITIONAL(SBC, test "${alsa_enable}" = "yes" || test "${gstreamer_enable}" = "yes" || > @@ -421,4 +430,5 @@ AC_DEFUN([AC_ARG_BLUEZ], [ > AM_CONDITIONAL(DBUSOOBPLUGIN, test "${dbusoob_enable}" = "yes") > AM_CONDITIONAL(WIIMOTEPLUGIN, test "${wiimote_enable}" = "yes") > AM_CONDITIONAL(THERMOMETERPLUGIN, test "${thermometer_enable}" = "yes") > + AM_CONDITIONAL(SIXAXISPLUGIN, test "${sixaxis_enable}" = "yes" && test "${udev_found}" = "yes") > ]) > diff --git a/plugins/sixaxis.c b/plugins/sixaxis.c > new file mode 100644 > index 0000000..2b0616a > --- /dev/null > +++ b/plugins/sixaxis.c > @@ -0,0 +1,593 @@ > +/* > + * sixaxis plugin: do cable association for Sixaxis controller > + * > + * Copyright (C) 2009 Bastien Nocera <hadess@hadess.net> > + * Copyright (C) 2011 Antonio Ospite <ospite@studenti.unina.it> > + * > + * > + * 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. > + * > + * 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., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA > + * > + */ > + > +/* > + * In the following this terminology is used: > + * > + * - controller: a Sixaxis joypad. > + * - adapter: the bluetooth dongle on the host system. > + * - adapter_bdaddr: the bdaddr of the bluetooth adapter. > + * - device_bdaddr: the bdaddr of the Sixaxis controller. > + * - master_bdaddr: the bdaddr of the adapter to be configured into the > + * Sixaxis controller > + */ > + > +#ifdef HAVE_CONFIG_H > +#include <config.h> > +#endif > + > +#include <stdio.h> > +#include <stdint.h> > +#include <stdlib.h> > +#include <sys/ioctl.h> > +#include <sys/types.h> > +#include <sys/stat.h> > +#include <fcntl.h> > +#include <unistd.h> > +#include <errno.h> > +#include <glib.h> > +#include <linux/hidraw.h> > + > +#define LIBUDEV_I_KNOW_THE_API_IS_SUBJECT_TO_CHANGE 1 > +#include <libudev.h> > + > +#include "plugin.h" > +#include "log.h" > +#include "adapter.h" > +#include "device.h" > +#include "manager.h" > +#include "storage.h" > +#include "sdp_lib.h" > + > +/* Fallback definitions to compile with older headers */ > +#ifndef HIDIOCGFEATURE > +#define HIDIOCGFEATURE(len) _IOC(_IOC_WRITE|_IOC_READ, 'H', 0x07, len) > +#endif > + > +#ifndef HIDIOCSFEATURE > +#define HIDIOCSFEATURE(len) _IOC(_IOC_WRITE|_IOC_READ, 'H', 0x06, len) > +#endif > + > +#define BDADDR_STR_SIZE 18 /* strlen("00:00:00:00:00:00") + 1 */ > + > +/* Vendor and product ID for the Sixaxis PS3 controller */ > +#define VENDOR 0x054c > +#define PRODUCT 0x0268 > +#define SIXAXIS_NAME "PLAYSTATION(R)3 Controller" > +#define SIXAXIS_PNP_RECORD "3601920900000A000100000900013503191124090004350D35061901000900113503190011090006350909656E09006A0901000900093508350619112409010009000D350F350D350619010009001335031900110901002513576972656C65737320436F6E74726F6C6C65720901012513576972656C65737320436F6E74726F6C6C6572090102251B536F6E7920436F6D707574657220456E7465727461696E6D656E740902000901000902010901000902020800090203082109020428010902052801090206359A35980822259405010904A101A102850175089501150026FF00810375019513150025013500450105091901291381027501950D0600FF8103150026FF0005010901A10075089504350046FF0009300931093209358102C0050175089527090181027508953009019102750895300901B102C0A1028502750895300901B102C0A10285EE750895300901B102C0A10285EF750895300901B102C0C0090207350835060904090901000902082800090209280109020A280109020B09010009020C093E8009020D280009020E2800" > +#define HID_UUID "00001124-0000-1000-8000-00805f9b34fb" > + > +#define LED_1 (0x01 << 1) > +#define LED_2 (0x01 << 2) > +#define LED_3 (0x01 << 3) > +#define LED_4 (0x01 << 4) > + > +#define LED_STATUS_OFF 0 > +#define LED_STATUS_ON 1 > + > +static struct udev *ctx; > +static struct udev_monitor *monitor; > +static guint watch_id; > + > + > +static int create_sixaxis_association(struct btd_adapter *adapter, > + const char *name, > + const char *address, > + guint32 vendor_id, > + guint32 product_id, > + const char *pnp_record) > +{ > + DBusConnection *conn; > + sdp_record_t *rec; > + struct btd_device *device; > + bdaddr_t src, dst; > + char srcaddr[18]; > + int ret = 0; > + > + str2ba(address, &dst); > + adapter_get_address(adapter, &src); > + ba2str(&src, srcaddr); > + > + write_device_name(&dst, &src, (char *) name); > + > + /* Store the device's SDP record */ > + rec = record_from_string(pnp_record); > + store_record(srcaddr, address, rec); > + sdp_record_free(rec); > + > + /* Set the device id */ > + store_device_id(srcaddr, address, 0xffff, vendor_id, product_id, 0); > + /* Don't write a profile here, > + * it will be updated when the device connects */ > + > + write_trust(srcaddr, address, "[all]", TRUE); > + > + conn = dbus_bus_get(DBUS_BUS_SYSTEM, NULL); > + if (conn == NULL) { > + DBG("Failed to get on the bus"); > + ret = -EPERM; > + goto fail_dbus; > + } > + > + device = adapter_get_device(conn, adapter, address); > + if (device == NULL) { > + DBG("Failed to get the device"); > + ret = -ENODEV; > + goto fail_device; > + } > + > + device_set_temporary(device, FALSE); > + device_set_name(device, name); > + btd_device_add_uuid(device, HID_UUID); > + > +fail_device: > + dbus_connection_unref(conn); > +fail_dbus: > + return ret; > +} > + > +/* Usb cable pairing section */ > +static unsigned char *get_feature_report(int fd, uint8_t report_number, > + unsigned int len) > +{ > + unsigned char *buf; > + int ret; > + > + buf = calloc(len, sizeof(*buf)); > + if (buf == NULL) { > + error("%s:%s() calloc failed", __FILE__, __func__); > + return NULL; > + } > + > + buf[0] = report_number; > + > + ret = ioctl(fd, HIDIOCGFEATURE(len), buf); > + if (ret < 0) { > + error("%s:%s() HIDIOCGFEATURE ret = %d", > + __FILE__, __func__, ret); > + free(buf); > + return NULL; > + } > + > + return buf; > +} > + > +static int set_feature_report(int fd, uint8_t *report, int len) > +{ > + int ret; > + > + ret = ioctl(fd, HIDIOCSFEATURE(len), report); > + if (ret < 0) > + error("%s:%s() HIDIOCSFEATURE failed, ret = %d", > + __FILE__, __func__, ret); > + > + return ret; > +} > + > +static char *get_device_bdaddr(int fd) > +{ > + unsigned char *buf; > + char *address; > + > + buf = get_feature_report(fd, 0xf2, 18); > + if (buf == NULL) { > + error("%s:%s() cannot get feature report", __FILE__, __func__); > + return NULL; > + } > + > + address = calloc(BDADDR_STR_SIZE, sizeof(*address)); > + if (address == NULL) { > + error("%s:%s() calloc failed", __FILE__, __func__); > + free(buf); > + return NULL; > + } > + > + snprintf(address, BDADDR_STR_SIZE, > + "%02X:%02X:%02X:%02X:%02X:%02X", > + buf[4], buf[5], buf[6], buf[7], buf[8], buf[9]); > + > + free(buf); > + return address; > +} > + > +static char *get_master_bdaddr(int fd) > +{ > + unsigned char *buf; > + char *address; > + > + buf = get_feature_report(fd, 0xf5, 8); > + if (buf == NULL) { > + error("%s:%s() cannot get feature report", __FILE__, __func__); > + return NULL; > + } > + > + address = calloc(BDADDR_STR_SIZE, sizeof(*address)); > + if (address == NULL) { > + error("%s:%s() calloc failed", __FILE__, __func__); > + free(buf); > + return NULL; > + } > + > + snprintf(address, BDADDR_STR_SIZE, > + "%02X:%02X:%02X:%02X:%02X:%02X", > + buf[2], buf[3], buf[4], buf[5], buf[6], buf[7]); > + > + free(buf); > + return address; > +} > + > +static int set_master_bdaddr(int fd, char *adapter_bdaddr) > +{ > + uint8_t *report; > + uint8_t addr[6]; > + int ret; > + > + ret = sscanf(adapter_bdaddr, > + "%02hhx:%02hhx:%02hhx:%02hhx:%02hhx:%02hhx", > + &addr[0], &addr[1], &addr[2], > + &addr[3], &addr[4], &addr[5]); > + if (ret != 6) { > + > + error("%s:%s() Parsing the bt address failed", > + __FILE__, __func__); > + return -EINVAL; > + } > + > + report = malloc(8); > + if (report == NULL) { > + error("%s:%s() malloc failed", __FILE__, __func__); > + return -ENOMEM; > + } > + > + report[0] = 0xf5; > + report[1] = 0x01; > + > + report[2] = addr[0]; > + report[3] = addr[1]; > + report[4] = addr[2]; > + report[5] = addr[3]; > + report[6] = addr[4]; > + report[7] = addr[5]; > + > + ret = set_feature_report(fd, report, 8); > + if (ret < 0) { > + error("%s:%s() cannot set feature report", > + __FILE__, __func__); > + goto out; > + } > + > + DBG("New Master Bluetooth address: %s", adapter_bdaddr); > + > +out: > + free(report); > + return ret; > +} > + > +static int sixpair(int fd, struct btd_adapter *adapter) > +{ > + char *device_bdaddr; > + char *master_bdaddr; > + char adapter_bdaddr[18]; > + bdaddr_t dst; > + int ret = 0; > + > + adapter_get_address(adapter, &dst); > + ba2str(&dst, adapter_bdaddr); > + DBG("Adapter bdaddr %s", adapter_bdaddr); > + > + master_bdaddr = get_master_bdaddr(fd); > + if (master_bdaddr == NULL) { > + DBG("Failed to get the Old master Bluetooth address from the device"); > + return -EPERM; > + } > + > + /* Only set the master bdaddr when needed, this is how the PS3 does > + * it, perhaps to avoid unnecessary writes to some eeprom. > + */ > + if (g_strcmp0(master_bdaddr, adapter_bdaddr) != 0) { > + DBG("Old master Bluetooth address was: %s", master_bdaddr); > + ret = set_master_bdaddr(fd, adapter_bdaddr); > + if (ret < 0) { > + DBG("Failed to set the master Bluetooth address"); > + free(master_bdaddr); > + return ret; > + } > + } > + > + device_bdaddr = get_device_bdaddr(fd); > + if (device_bdaddr == NULL) { > + DBG("Failed to get the Bluetooth address from the device"); > + free(master_bdaddr); > + return -EPERM; > + } > + > + DBG("Device bdaddr %s", device_bdaddr); > + > + ret = create_sixaxis_association(adapter, > + SIXAXIS_NAME, > + device_bdaddr, > + VENDOR, PRODUCT, SIXAXIS_PNP_RECORD); You create the association with a hard-coded VID/PID. You match the device on string name below (not VID/PID), so what if you used the actual VID/PID of the device for the association instead? > + free(device_bdaddr); > + free(master_bdaddr); > + return ret; > +} > + > +/* Led setting section */ > +static int set_leds(int fd, unsigned char leds_status[4]) > +{ > + int ret; > + > + /* > + * the total time the led is active (0xff means forever) > + * | duty_length: how long a cycle is in deciseconds: > + * | | (0 means "blink very fast") > + * | | ??? (Maybe a phase shift or duty_length multiplier?) > + * | | | % of duty_length led is off (0xff means 100%) > + * | | | | % of duty_length led is on (0xff is 100%) > + * | | | | | > + * 0xff, 0x27, 0x10, 0x00, 0x32, > + */ > + unsigned char leds_report[] = { > + 0x01, > + 0x00, 0x00, 0x00, 0x00, 0x00, /* rumble values TBD */ > + 0x00, 0x00, 0x00, 0x00, 0x1e, /* LED_1=0x02, LED_2=0x04 ... */ > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_4 */ > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_3 */ > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_2 */ > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_1 */ > + 0x00, 0x00, 0x00, 0x00, 0x00, > + }; > + > + int leds = 0; > + if (leds_status[0]) > + leds |= LED_1; > + if (leds_status[1]) > + leds |= LED_2; > + if (leds_status[2]) > + leds |= LED_3; > + if (leds_status[3]) > + leds |= LED_4; > + > + leds_report[10] = leds; If you are overwriting leds_report[10] unconditionally, why is it initialized to 0x1e in the leds_report initializer (unless I counted wrong)? > + > + ret = write(fd, leds_report, sizeof(leds_report)); > + if (ret < (ssize_t) sizeof(leds_report)) > + error("%s:%s() Unable to write to hidraw device", > + __FILE__, __func__); > + > + return ret; > +} > + > +static int set_controller_number(int fd, unsigned int n) > +{ > + unsigned char leds_status[4] = {0, 0, 0, 0}; > + > + switch (n) { > + case 0: > + break; > + case 1: > + case 2: > + case 3: > + case 4: > + leds_status[n - 1] = LED_STATUS_ON; > + break; > + case 5: > + case 6: > + case 7: > + leds_status[4 - 1] = LED_STATUS_ON; > + leds_status[n - 4 - 1] = LED_STATUS_ON; > + break; > + default: > + error("%s:%s() Only 7 controllers supported for now", > + __FILE__, __func__); > + return -1; > + } > + > + return set_leds(fd, leds_status); > +} > + > + > +static inline gboolean is_sixaxis(const char *hid_name) > +{ > + return g_str_has_suffix(hid_name, "PLAYSTATION(R)3 Controller") || > + g_str_has_suffix(hid_name, > + "Sony Computer Entertainment Wireless Controller"); > +} Why do you key on the string name here. This seems problematic, especially if you want to support things like the PSMove. I'd expect a table of VID/PIDs or something. > + > +static void handle_device_plug(struct udev_device *udevice) > +{ > + struct udev_device *hid_parent; > + struct udev_enumerate *enumerate; > + struct udev_list_entry *devices, *dev_list_entry; > + const char *hid_name; > + const char *hid_phys; > + const char *hidraw_node; > + unsigned char is_usb = FALSE; > + int js_num = 0; > + int fd; > + > + hid_parent = udev_device_get_parent_with_subsystem_devtype(udevice, > + "hid", NULL); > + if (!hid_parent) { > + error("%s:%s() cannot get parent hid device", > + __FILE__, __func__); > + return; > + } > + > + hid_name = udev_device_get_property_value(hid_parent, "HID_NAME"); > + DBG("name: %s", hid_name); > + > + if (!is_sixaxis(hid_name)) > + return; > + > + DBG("Found a Sixaxis device"); > + > + hidraw_node = udev_device_get_devnode(udevice); > + > + hid_phys = udev_device_get_property_value(hid_parent, "HID_PHYS"); > + > + /* looking for joysticks */ > + enumerate = udev_enumerate_new(udev_device_get_udev(udevice)); > + udev_enumerate_add_match_sysname(enumerate, "js*"); > + udev_enumerate_scan_devices(enumerate); > + > + devices = udev_enumerate_get_list_entry(enumerate); > + udev_list_entry_foreach(dev_list_entry, devices) { > + const char *devname; > + struct udev_device *js_dev; > + struct udev_device *input_parent; > + const char *input_phys; > + > + devname = udev_list_entry_get_name(dev_list_entry); > + js_dev = udev_device_new_from_syspath(udev_device_get_udev(udevice), > + devname); > + > + input_parent = udev_device_get_parent_with_subsystem_devtype(js_dev, > + "input", NULL); > + if (!input_parent) { > + error("%s:%s() cannot get parent input device.", > + __FILE__, __func__); > + continue; > + } > + > + /* check this is the joystick relative to > + * the hidraw device above */ > + input_phys = udev_device_get_sysattr_value(input_parent, > + "phys"); > + if (g_strcmp0(input_phys, hid_phys) == 0) { > + const char *usb_driver; > + > + usb_driver = udev_device_get_property_value(js_dev, > + "ID_USB_DRIVER"); > + js_num = atoi(udev_device_get_sysnum(js_dev)) + 1; > + DBG("joypad device_num: %d", js_num); > + DBG("hidraw_node: %s", hidraw_node); > + DBG("driver: %s", usb_driver); > + > + if (g_strcmp0(usb_driver, "usbhid") == 0) > + is_usb = TRUE; > + } > + > + udev_device_unref(js_dev); > + } > + > + udev_enumerate_unref(enumerate); > + > + fd = open(hidraw_node, O_RDWR); > + if (fd < 0) { > + error("%s:%s() hidraw open", __FILE__, __func__); > + return; > + } > + > + if (is_usb) { > + struct btd_adapter *adapter; > + > + /* Look for the default adapter */ > + adapter = manager_get_default_adapter(); > + if (adapter == NULL) { > + DBG("No adapters, exiting"); > + return; > + } > + sixpair(fd, adapter); > + } > + > + if (js_num > 0) > + set_controller_number(fd, js_num); > + > + close(fd); > +} > + > +static gboolean device_event_idle(struct udev_device *udevice) > +{ > + handle_device_plug(udevice); > + udev_device_unref(udevice); > + return FALSE; > +} I'm a little confused why this is called "idle." Thanks for your work on this, Antonio. Linux working out of the box with these controllers will be really cool. Alan. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, 09 Aug 2011 22:24:37 -0400 Alan Ott <alan@signal11.us> wrote: [...] > > + ret = create_sixaxis_association(adapter, > > + SIXAXIS_NAME, > > + device_bdaddr, > > + VENDOR, PRODUCT, SIXAXIS_PNP_RECORD); > > You create the association with a hard-coded VID/PID. You match the > device on string name below (not VID/PID), so what if you used the > actual VID/PID of the device for the association instead? > Or use statically defined VID/PID to match the device, see below. [...] > > + /* > > + * the total time the led is active (0xff means forever) > > + * | duty_length: how long a cycle is in deciseconds: > > + * | | (0 means "blink very fast") > > + * | | ??? (Maybe a phase shift or duty_length multiplier?) > > + * | | | % of duty_length led is off (0xff means 100%) > > + * | | | | % of duty_length led is on (0xff is 100%) > > + * | | | | | > > + * 0xff, 0x27, 0x10, 0x00, 0x32, > > + */ > > + unsigned char leds_report[] = { > > + 0x01, > > + 0x00, 0x00, 0x00, 0x00, 0x00, /* rumble values TBD */ > > + 0x00, 0x00, 0x00, 0x00, 0x1e, /* LED_1=0x02, LED_2=0x04 ... */ > > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_4 */ > > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_3 */ > > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_2 */ > > + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_1 */ > > + 0x00, 0x00, 0x00, 0x00, 0x00, > > + }; > > > > + int leds = 0; > > + if (leds_status[0]) > > + leds |= LED_1; > > + if (leds_status[1]) > > + leds |= LED_2; > > + if (leds_status[2]) > > + leds |= LED_3; > > + if (leds_status[3]) > > + leds |= LED_4; > > + > > + leds_report[10] = leds; > > If you are overwriting leds_report[10] unconditionally, why is it > initialized to 0x1e in the leds_report initializer (unless I counted wrong)? > You are right having it initialized to 0x1e is not necessary at all, but I'd still keep it to 0x1e in the initializer because this is the "hardware default", meaning "all leds blinking", I see it as a form of documentation-inside-code, but if it is confusing I will avoid that. [...] > > + > > +static inline gboolean is_sixaxis(const char *hid_name) > > +{ > > + return g_str_has_suffix(hid_name, "PLAYSTATION(R)3 Controller") || > > + g_str_has_suffix(hid_name, > > + "Sony Computer Entertainment Wireless Controller"); > > +} > > Why do you key on the string name here. This seems problematic, > especially if you want to support things like the PSMove. I'd expect a > table of VID/PIDs or something. > Right, I like this suggestion, I generalized the code a little bit to handle different devices, see the two follow-up incremental patches, in the first one I remove static #defines in favor of a structure representing a device type, in the second one I match devices using VID/PID from a table of device types. If the idea looks OK I'll fold those patches in v5, along with the plugin renaming to "sony-controller", and maybe I'll split the plugin in several modules too: sony-controller.c /* udev and bluez stuff */ sony-controller-hid.c /* hidraw and device specific stuff */ sony-controlled-hid.h [...] > > + > > +static gboolean device_event_idle(struct udev_device *udevice) > > +{ > > + handle_device_plug(udevice); > > + udev_device_unref(udevice); > > + return FALSE; > > +} > > I'm a little confused why this is called "idle." > I don't know either, maybe trying to communicate that it is called asynchronously by glib (via g_timeout_add_seconds())? But even if so this is not a property of the function itself, so I think we can just call it device_event(). Ah Bastien, I noticed we are using g_timeout_add_seconds() with the callback always returning FALSE, so the latter is called only once, can't we in this case just sleep and then call the function directly? Isn't the main loop in sixaxis_init() enough to handle multiple overlapping udev events? Maybe I asked that already... :P > Thanks for your work on this, Antonio. Linux working out of the box with > these controllers will be really cool. > > Alan. > Regards, Antonio
On Fri, 19 Aug 2011 21:57:16 +0200 Antonio Ospite <ospite@studenti.unina.it> wrote: > On Thu, 18 Aug 2011 16:13:57 +0200 > Antonio Ospite <ospite@studenti.unina.it> wrote: > > [...] > > If the idea looks OK I'll fold those patches in v5, along with the > > plugin renaming to "sony-controller", and maybe I'll split the plugin in > > several modules too: > > sony-controller.c /* udev and bluez stuff */ > > sony-controller-hid.c /* hidraw and device specific stuff */ > > sony-controlled-hid.h > > > > Do you think "controller" fits the PS3 headset as well? > OK, given that: - all the possibly supported devices at this time are PS3 peripherals; - some of them are not input controllers; - the cable pairing/association mechanism generally does not apply to other sony BT hardware; - someone imagined a future PS4 controller using the same pairing mechanism; with Alan we came up with the name: playstation-peripheral for the plugin; do you people like that, or does the variant playstation-device sound better to you? Thanks, Antonio
diff --git a/Makefile.am b/Makefile.am index 68380d9..dbe0170 100644 --- a/Makefile.am +++ b/Makefile.am @@ -224,6 +224,11 @@ builtin_sources += thermometer/main.c \ thermometer/thermometer.h thermometer/thermometer.c endif +if SIXAXISPLUGIN +builtin_modules += sixaxis +builtin_sources += plugins/sixaxis.c +endif + builtin_modules += hciops mgmtops builtin_sources += plugins/hciops.c plugins/mgmtops.c @@ -283,7 +288,7 @@ src_bluetoothd_SOURCES = $(gdbus_sources) $(builtin_sources) \ src/event.h src/event.c \ src/oob.h src/oob.c src/eir.h src/eir.c src_bluetoothd_LDADD = lib/libbluetooth.la @GLIB_LIBS@ @DBUS_LIBS@ \ - @CAPNG_LIBS@ -ldl -lrt + @CAPNG_LIBS@ @UDEV_LIBS@ -ldl -lrt src_bluetoothd_LDFLAGS = -Wl,--export-dynamic \ -Wl,--version-script=$(srcdir)/src/bluetooth.ver @@ -398,7 +403,7 @@ EXTRA_DIST += doc/manager-api.txt \ AM_YFLAGS = -d -AM_CFLAGS = @DBUS_CFLAGS@ @GLIB_CFLAGS@ @CAPNG_CFLAGS@ \ +AM_CFLAGS = @DBUS_CFLAGS@ @GLIB_CFLAGS@ @CAPNG_CFLAGS@ @UDEV_CFLAGS@ \ -DBLUETOOTH_PLUGIN_BUILTIN -DPLUGINDIR=\""$(plugindir)"\" INCLUDES = -I$(builddir)/lib -I$(builddir)/src -I$(srcdir)/src \ diff --git a/acinclude.m4 b/acinclude.m4 index 3cb9459..6176483 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -186,6 +186,7 @@ AC_DEFUN([AC_ARG_BLUEZ], [ sndfile_enable=${sndfile_found} hal_enable=no usb_enable=${usb_found} + sixaxis_enable=${udev_found} alsa_enable=${alsa_found} gstreamer_enable=${gstreamer_found} audio_enable=yes @@ -286,6 +287,10 @@ AC_DEFUN([AC_ARG_BLUEZ], [ usb_enable=${enableval} ]) + AC_ARG_ENABLE(sixaxis, AC_HELP_STRING([--enable-sixaxis], [enable Sixaxis plugin]), [ + sixaxis_enable=${enableval} + ]) + AC_ARG_ENABLE(tracer, AC_HELP_STRING([--enable-tracer], [install Tracing daemon]), [ tracer_enable=${enableval} ]) @@ -385,6 +390,10 @@ AC_DEFUN([AC_ARG_BLUEZ], [ AC_DEFINE(HAVE_LIBUSB, 1, [Define to 1 if you have USB library.]) fi + if (test "${sixaxis_enable}" = "yes" && test "${udev_found}" = "yes"); then + AC_DEFINE(HAVE_SIXAXIS_PLUGIN, 1, [Define to 1 if you have sixaxis plugin.]) + fi + AM_CONDITIONAL(SNDFILE, test "${sndfile_enable}" = "yes" && test "${sndfile_found}" = "yes") AM_CONDITIONAL(USB, test "${usb_enable}" = "yes" && test "${usb_found}" = "yes") AM_CONDITIONAL(SBC, test "${alsa_enable}" = "yes" || test "${gstreamer_enable}" = "yes" || @@ -421,4 +430,5 @@ AC_DEFUN([AC_ARG_BLUEZ], [ AM_CONDITIONAL(DBUSOOBPLUGIN, test "${dbusoob_enable}" = "yes") AM_CONDITIONAL(WIIMOTEPLUGIN, test "${wiimote_enable}" = "yes") AM_CONDITIONAL(THERMOMETERPLUGIN, test "${thermometer_enable}" = "yes") + AM_CONDITIONAL(SIXAXISPLUGIN, test "${sixaxis_enable}" = "yes" && test "${udev_found}" = "yes") ]) diff --git a/plugins/sixaxis.c b/plugins/sixaxis.c new file mode 100644 index 0000000..2b0616a --- /dev/null +++ b/plugins/sixaxis.c @@ -0,0 +1,593 @@ +/* + * sixaxis plugin: do cable association for Sixaxis controller + * + * Copyright (C) 2009 Bastien Nocera <hadess@hadess.net> + * Copyright (C) 2011 Antonio Ospite <ospite@studenti.unina.it> + * + * + * 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. + * + * 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., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + * + */ + +/* + * In the following this terminology is used: + * + * - controller: a Sixaxis joypad. + * - adapter: the bluetooth dongle on the host system. + * - adapter_bdaddr: the bdaddr of the bluetooth adapter. + * - device_bdaddr: the bdaddr of the Sixaxis controller. + * - master_bdaddr: the bdaddr of the adapter to be configured into the + * Sixaxis controller + */ + +#ifdef HAVE_CONFIG_H +#include <config.h> +#endif + +#include <stdio.h> +#include <stdint.h> +#include <stdlib.h> +#include <sys/ioctl.h> +#include <sys/types.h> +#include <sys/stat.h> +#include <fcntl.h> +#include <unistd.h> +#include <errno.h> +#include <glib.h> +#include <linux/hidraw.h> + +#define LIBUDEV_I_KNOW_THE_API_IS_SUBJECT_TO_CHANGE 1 +#include <libudev.h> + +#include "plugin.h" +#include "log.h" +#include "adapter.h" +#include "device.h" +#include "manager.h" +#include "storage.h" +#include "sdp_lib.h" + +/* Fallback definitions to compile with older headers */ +#ifndef HIDIOCGFEATURE +#define HIDIOCGFEATURE(len) _IOC(_IOC_WRITE|_IOC_READ, 'H', 0x07, len) +#endif + +#ifndef HIDIOCSFEATURE +#define HIDIOCSFEATURE(len) _IOC(_IOC_WRITE|_IOC_READ, 'H', 0x06, len) +#endif + +#define BDADDR_STR_SIZE 18 /* strlen("00:00:00:00:00:00") + 1 */ + +/* Vendor and product ID for the Sixaxis PS3 controller */ +#define VENDOR 0x054c +#define PRODUCT 0x0268 +#define SIXAXIS_NAME "PLAYSTATION(R)3 Controller" +#define SIXAXIS_PNP_RECORD "3601920900000A000100000900013503191124090004350D35061901000900113503190011090006350909656E09006A0901000900093508350619112409010009000D350F350D350619010009001335031900110901002513576972656C65737320436F6E74726F6C6C65720901012513576972656C65737320436F6E74726F6C6C6572090102251B536F6E7920436F6D707574657220456E7465727461696E6D656E740902000901000902010901000902020800090203082109020428010902052801090206359A35980822259405010904A101A102850175089501150026FF00810375019513150025013500450105091901291381027501950D0600FF8103150026FF0005010901A10075089504350046FF0009300931093209358102C0050175089527090181027508953009019102750895300901B102C0A1028502750895300901B102C0A10285EE750895300901B102C0A10285EF750895300901B102C0C0090207350835060904090901000902082800090209280109020A280109020B09010009020C093E8009020D280009020E2800" +#define HID_UUID "00001124-0000-1000-8000-00805f9b34fb" + +#define LED_1 (0x01 << 1) +#define LED_2 (0x01 << 2) +#define LED_3 (0x01 << 3) +#define LED_4 (0x01 << 4) + +#define LED_STATUS_OFF 0 +#define LED_STATUS_ON 1 + +static struct udev *ctx; +static struct udev_monitor *monitor; +static guint watch_id; + + +static int create_sixaxis_association(struct btd_adapter *adapter, + const char *name, + const char *address, + guint32 vendor_id, + guint32 product_id, + const char *pnp_record) +{ + DBusConnection *conn; + sdp_record_t *rec; + struct btd_device *device; + bdaddr_t src, dst; + char srcaddr[18]; + int ret = 0; + + str2ba(address, &dst); + adapter_get_address(adapter, &src); + ba2str(&src, srcaddr); + + write_device_name(&dst, &src, (char *) name); + + /* Store the device's SDP record */ + rec = record_from_string(pnp_record); + store_record(srcaddr, address, rec); + sdp_record_free(rec); + + /* Set the device id */ + store_device_id(srcaddr, address, 0xffff, vendor_id, product_id, 0); + /* Don't write a profile here, + * it will be updated when the device connects */ + + write_trust(srcaddr, address, "[all]", TRUE); + + conn = dbus_bus_get(DBUS_BUS_SYSTEM, NULL); + if (conn == NULL) { + DBG("Failed to get on the bus"); + ret = -EPERM; + goto fail_dbus; + } + + device = adapter_get_device(conn, adapter, address); + if (device == NULL) { + DBG("Failed to get the device"); + ret = -ENODEV; + goto fail_device; + } + + device_set_temporary(device, FALSE); + device_set_name(device, name); + btd_device_add_uuid(device, HID_UUID); + +fail_device: + dbus_connection_unref(conn); +fail_dbus: + return ret; +} + +/* Usb cable pairing section */ +static unsigned char *get_feature_report(int fd, uint8_t report_number, + unsigned int len) +{ + unsigned char *buf; + int ret; + + buf = calloc(len, sizeof(*buf)); + if (buf == NULL) { + error("%s:%s() calloc failed", __FILE__, __func__); + return NULL; + } + + buf[0] = report_number; + + ret = ioctl(fd, HIDIOCGFEATURE(len), buf); + if (ret < 0) { + error("%s:%s() HIDIOCGFEATURE ret = %d", + __FILE__, __func__, ret); + free(buf); + return NULL; + } + + return buf; +} + +static int set_feature_report(int fd, uint8_t *report, int len) +{ + int ret; + + ret = ioctl(fd, HIDIOCSFEATURE(len), report); + if (ret < 0) + error("%s:%s() HIDIOCSFEATURE failed, ret = %d", + __FILE__, __func__, ret); + + return ret; +} + +static char *get_device_bdaddr(int fd) +{ + unsigned char *buf; + char *address; + + buf = get_feature_report(fd, 0xf2, 18); + if (buf == NULL) { + error("%s:%s() cannot get feature report", __FILE__, __func__); + return NULL; + } + + address = calloc(BDADDR_STR_SIZE, sizeof(*address)); + if (address == NULL) { + error("%s:%s() calloc failed", __FILE__, __func__); + free(buf); + return NULL; + } + + snprintf(address, BDADDR_STR_SIZE, + "%02X:%02X:%02X:%02X:%02X:%02X", + buf[4], buf[5], buf[6], buf[7], buf[8], buf[9]); + + free(buf); + return address; +} + +static char *get_master_bdaddr(int fd) +{ + unsigned char *buf; + char *address; + + buf = get_feature_report(fd, 0xf5, 8); + if (buf == NULL) { + error("%s:%s() cannot get feature report", __FILE__, __func__); + return NULL; + } + + address = calloc(BDADDR_STR_SIZE, sizeof(*address)); + if (address == NULL) { + error("%s:%s() calloc failed", __FILE__, __func__); + free(buf); + return NULL; + } + + snprintf(address, BDADDR_STR_SIZE, + "%02X:%02X:%02X:%02X:%02X:%02X", + buf[2], buf[3], buf[4], buf[5], buf[6], buf[7]); + + free(buf); + return address; +} + +static int set_master_bdaddr(int fd, char *adapter_bdaddr) +{ + uint8_t *report; + uint8_t addr[6]; + int ret; + + ret = sscanf(adapter_bdaddr, + "%02hhx:%02hhx:%02hhx:%02hhx:%02hhx:%02hhx", + &addr[0], &addr[1], &addr[2], + &addr[3], &addr[4], &addr[5]); + if (ret != 6) { + + error("%s:%s() Parsing the bt address failed", + __FILE__, __func__); + return -EINVAL; + } + + report = malloc(8); + if (report == NULL) { + error("%s:%s() malloc failed", __FILE__, __func__); + return -ENOMEM; + } + + report[0] = 0xf5; + report[1] = 0x01; + + report[2] = addr[0]; + report[3] = addr[1]; + report[4] = addr[2]; + report[5] = addr[3]; + report[6] = addr[4]; + report[7] = addr[5]; + + ret = set_feature_report(fd, report, 8); + if (ret < 0) { + error("%s:%s() cannot set feature report", + __FILE__, __func__); + goto out; + } + + DBG("New Master Bluetooth address: %s", adapter_bdaddr); + +out: + free(report); + return ret; +} + +static int sixpair(int fd, struct btd_adapter *adapter) +{ + char *device_bdaddr; + char *master_bdaddr; + char adapter_bdaddr[18]; + bdaddr_t dst; + int ret = 0; + + adapter_get_address(adapter, &dst); + ba2str(&dst, adapter_bdaddr); + DBG("Adapter bdaddr %s", adapter_bdaddr); + + master_bdaddr = get_master_bdaddr(fd); + if (master_bdaddr == NULL) { + DBG("Failed to get the Old master Bluetooth address from the device"); + return -EPERM; + } + + /* Only set the master bdaddr when needed, this is how the PS3 does + * it, perhaps to avoid unnecessary writes to some eeprom. + */ + if (g_strcmp0(master_bdaddr, adapter_bdaddr) != 0) { + DBG("Old master Bluetooth address was: %s", master_bdaddr); + ret = set_master_bdaddr(fd, adapter_bdaddr); + if (ret < 0) { + DBG("Failed to set the master Bluetooth address"); + free(master_bdaddr); + return ret; + } + } + + device_bdaddr = get_device_bdaddr(fd); + if (device_bdaddr == NULL) { + DBG("Failed to get the Bluetooth address from the device"); + free(master_bdaddr); + return -EPERM; + } + + DBG("Device bdaddr %s", device_bdaddr); + + ret = create_sixaxis_association(adapter, + SIXAXIS_NAME, + device_bdaddr, + VENDOR, PRODUCT, SIXAXIS_PNP_RECORD); + free(device_bdaddr); + free(master_bdaddr); + return ret; +} + +/* Led setting section */ +static int set_leds(int fd, unsigned char leds_status[4]) +{ + int ret; + + /* + * the total time the led is active (0xff means forever) + * | duty_length: how long a cycle is in deciseconds: + * | | (0 means "blink very fast") + * | | ??? (Maybe a phase shift or duty_length multiplier?) + * | | | % of duty_length led is off (0xff means 100%) + * | | | | % of duty_length led is on (0xff is 100%) + * | | | | | + * 0xff, 0x27, 0x10, 0x00, 0x32, + */ + unsigned char leds_report[] = { + 0x01, + 0x00, 0x00, 0x00, 0x00, 0x00, /* rumble values TBD */ + 0x00, 0x00, 0x00, 0x00, 0x1e, /* LED_1=0x02, LED_2=0x04 ... */ + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_4 */ + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_3 */ + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_2 */ + 0xff, 0x27, 0x10, 0x00, 0x32, /* LED_1 */ + 0x00, 0x00, 0x00, 0x00, 0x00, + }; + + int leds = 0; + if (leds_status[0]) + leds |= LED_1; + if (leds_status[1]) + leds |= LED_2; + if (leds_status[2]) + leds |= LED_3; + if (leds_status[3]) + leds |= LED_4; + + leds_report[10] = leds; + + ret = write(fd, leds_report, sizeof(leds_report)); + if (ret < (ssize_t) sizeof(leds_report)) + error("%s:%s() Unable to write to hidraw device", + __FILE__, __func__); + + return ret; +} + +static int set_controller_number(int fd, unsigned int n) +{ + unsigned char leds_status[4] = {0, 0, 0, 0}; + + switch (n) { + case 0: + break; + case 1: + case 2: + case 3: + case 4: + leds_status[n - 1] = LED_STATUS_ON; + break; + case 5: + case 6: + case 7: + leds_status[4 - 1] = LED_STATUS_ON; + leds_status[n - 4 - 1] = LED_STATUS_ON; + break; + default: + error("%s:%s() Only 7 controllers supported for now", + __FILE__, __func__); + return -1; + } + + return set_leds(fd, leds_status); +} + + +static inline gboolean is_sixaxis(const char *hid_name) +{ + return g_str_has_suffix(hid_name, "PLAYSTATION(R)3 Controller") || + g_str_has_suffix(hid_name, + "Sony Computer Entertainment Wireless Controller"); +} + +static void handle_device_plug(struct udev_device *udevice) +{ + struct udev_device *hid_parent; + struct udev_enumerate *enumerate; + struct udev_list_entry *devices, *dev_list_entry; + const char *hid_name; + const char *hid_phys; + const char *hidraw_node; + unsigned char is_usb = FALSE; + int js_num = 0; + int fd; + + hid_parent = udev_device_get_parent_with_subsystem_devtype(udevice, + "hid", NULL); + if (!hid_parent) { + error("%s:%s() cannot get parent hid device", + __FILE__, __func__); + return; + } + + hid_name = udev_device_get_property_value(hid_parent, "HID_NAME"); + DBG("name: %s", hid_name); + + if (!is_sixaxis(hid_name)) + return; + + DBG("Found a Sixaxis device"); + + hidraw_node = udev_device_get_devnode(udevice); + + hid_phys = udev_device_get_property_value(hid_parent, "HID_PHYS"); + + /* looking for joysticks */ + enumerate = udev_enumerate_new(udev_device_get_udev(udevice)); + udev_enumerate_add_match_sysname(enumerate, "js*"); + udev_enumerate_scan_devices(enumerate); + + devices = udev_enumerate_get_list_entry(enumerate); + udev_list_entry_foreach(dev_list_entry, devices) { + const char *devname; + struct udev_device *js_dev; + struct udev_device *input_parent; + const char *input_phys; + + devname = udev_list_entry_get_name(dev_list_entry); + js_dev = udev_device_new_from_syspath(udev_device_get_udev(udevice), + devname); + + input_parent = udev_device_get_parent_with_subsystem_devtype(js_dev, + "input", NULL); + if (!input_parent) { + error("%s:%s() cannot get parent input device.", + __FILE__, __func__); + continue; + } + + /* check this is the joystick relative to + * the hidraw device above */ + input_phys = udev_device_get_sysattr_value(input_parent, + "phys"); + if (g_strcmp0(input_phys, hid_phys) == 0) { + const char *usb_driver; + + usb_driver = udev_device_get_property_value(js_dev, + "ID_USB_DRIVER"); + js_num = atoi(udev_device_get_sysnum(js_dev)) + 1; + DBG("joypad device_num: %d", js_num); + DBG("hidraw_node: %s", hidraw_node); + DBG("driver: %s", usb_driver); + + if (g_strcmp0(usb_driver, "usbhid") == 0) + is_usb = TRUE; + } + + udev_device_unref(js_dev); + } + + udev_enumerate_unref(enumerate); + + fd = open(hidraw_node, O_RDWR); + if (fd < 0) { + error("%s:%s() hidraw open", __FILE__, __func__); + return; + } + + if (is_usb) { + struct btd_adapter *adapter; + + /* Look for the default adapter */ + adapter = manager_get_default_adapter(); + if (adapter == NULL) { + DBG("No adapters, exiting"); + return; + } + sixpair(fd, adapter); + } + + if (js_num > 0) + set_controller_number(fd, js_num); + + close(fd); +} + +static gboolean device_event_idle(struct udev_device *udevice) +{ + handle_device_plug(udevice); + udev_device_unref(udevice); + return FALSE; +} + +static gboolean monitor_event(GIOChannel *source, GIOCondition condition, + gpointer data) +{ + struct udev_device *udevice; + + udevice = udev_monitor_receive_device(monitor); + if (udevice == NULL) + goto out; + if (g_strcmp0(udev_device_get_action(udevice), "add") != 0) { + udev_device_unref(udevice); + goto out; + } + + /* Give UDEV some time to load kernel modules */ + g_timeout_add_seconds(1, (GSourceFunc) device_event_idle, udevice); + +out: + return TRUE; +} + +static int sixaxis_init(void) +{ + GIOChannel *channel; + + DBG("Setup Sixaxis cable plugin"); + + ctx = udev_new(); + monitor = udev_monitor_new_from_netlink(ctx, "udev"); + if (monitor == NULL) { + error("%s:%s() Could not get udev monitor", + __FILE__, __func__); + return -1; + } + + /* Listen for newly connected hidraw interfaces */ + udev_monitor_filter_add_match_subsystem_devtype(monitor, + "hidraw", NULL); + udev_monitor_enable_receiving(monitor); + + channel = g_io_channel_unix_new(udev_monitor_get_fd(monitor)); + watch_id = g_io_add_watch(channel, G_IO_IN, monitor_event, NULL); + g_io_channel_unref(channel); + + return 0; +} + +static void sixaxis_exit(void) +{ + DBG("Cleanup Sixaxis cable plugin"); + + if (watch_id != 0) { + g_source_remove(watch_id); + watch_id = 0; + } + if (monitor != NULL) { + udev_monitor_unref(monitor); + monitor = NULL; + } + if (ctx != NULL) { + udev_unref(ctx); + ctx = NULL; + } +} + +BLUETOOTH_PLUGIN_DEFINE(sixaxis, VERSION, + BLUETOOTH_PLUGIN_PRIORITY_DEFAULT, + sixaxis_init, sixaxis_exit)