lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110208103923.GA2009@polaris.bitmath.org>
Date:	Tue, 8 Feb 2011 11:39:23 +0100
From:	"Henrik Rydberg" <rydberg@...omail.se>
To:	Anton Chikin <kverlin@...il.com>
Cc:	jkosina@...e.cz, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Anton Chikin <anton.chikin@...aart.com>
Subject: Re: [PATCH] HID: added new driver for Panasonic Elite Panaboard
 UB-T780 and UB-T880

Hi Anton,

> From: Anton Chikin <anton.chikin@...aart.com>
> 
> Panasonic UB-T780 is a HID whiteboard with infrared detectors.
> UB-T880 is a multitouch-enabled HID whiteboard.
> Both of them use projector to present desktop, so they need calibration.
> I've used my own IOCTL's to switch board modes and feed calibration data from userspace.
> Sorry for previous ugly effort. I've missed some files in commit and forgot about copyrights.
> 
> Signed-off-by: Anton Chikin <kverlin@...il.com>
> ---

At large, it does look like the same patch, though. Please find some
initial comments below.

>  drivers/hid/Kconfig      |    6 +
>  drivers/hid/Makefile     |    1 +
>  drivers/hid/hid-ids.h    |    5 +
>  drivers/hid/hid-ubt880.c | 1205 ++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/hid/hid-ubt880.h |  162 +++++++
>  5 files changed, 1379 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/hid/hid-ubt880.c
>  create mode 100644 drivers/hid/hid-ubt880.h
> 
> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
> index 2560f01..8be6226 100644
> --- a/drivers/hid/Kconfig
> +++ b/drivers/hid/Kconfig
> @@ -312,6 +312,12 @@ config HID_MULTITOUCH
>  	  To compile this driver as a module, choose M here: the
>  	  module will be called hid-multitouch.
>  
> +config HID_PANASONIC
> +	tristate "Panasonic Elite Panaboards"
> +	depends on USB_HID
> +	---help---
> +	Support for Panasonic Elite Panaboard UB-T780 and UB-T880.
> +

I believe these should be ordered alphabetically.

>  config HID_NTRIG
>  	tristate "N-Trig touch screen"
>  	depends on USB_HID
> diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile
> index 6efc2a0..85f3495 100644
> --- a/drivers/hid/Makefile
> +++ b/drivers/hid/Makefile
> @@ -73,6 +73,7 @@ obj-$(CONFIG_HID_ZEROPLUS)	+= hid-zpff.o
>  obj-$(CONFIG_HID_ZYDACRON)	+= hid-zydacron.o
>  obj-$(CONFIG_HID_WACOM)		+= hid-wacom.o
>  obj-$(CONFIG_HID_WALTOP)	+= hid-waltop.o
> +obj-$(CONFIG_HID_PANASONIC)     += hid-ubt880.o
>  
>  obj-$(CONFIG_USB_HID)		+= usbhid/
>  obj-$(CONFIG_USB_MOUSE)		+= usbhid/
> diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
> index 92a0d61..3ca7171 100644
> --- a/drivers/hid/hid-ids.h
> +++ b/drivers/hid/hid-ids.h
> @@ -468,6 +468,11 @@
>  #define USB_VENDOR_ID_ORTEK		0x05a4
>  #define USB_DEVICE_ID_ORTEK_WKB2000	0x2000
>  
> +#define USB_VENDOR_ID_PANASONIC		0x04da
> +#define USB_DEVICE_ID_PANABOARD_780	0x1044
> +#define USB_DEVICE_ID_PANABOARD_880	0x104d
> +#define USB_DEVICE_ID_PANABOARD_880_PEN 0x104e
> +
>  #define USB_VENDOR_ID_PANJIT		0x134c
>  
>  #define USB_VENDOR_ID_PANTHERLORD	0x0810
> diff --git a/drivers/hid/hid-ubt880.c b/drivers/hid/hid-ubt880.c
> new file mode 100644
> index 0000000..098864d
> --- /dev/null
> +++ b/drivers/hid/hid-ubt880.c
> @@ -0,0 +1,1205 @@
> +/*
> + *  USB HID driver for Panasonic elite Panaboard UTB780
> + *   Copyright (c) 2008 Igor Shakirov, Victor Grenke <comp.vision@...il.com>
> + *   Copyright (c) 2010-2011 Anton Chikin<anton.chikin@...aart.com> for Panasonic.
> + *
> + *  Information.
> + *  It's driver for supporting Panasonic Elite Panaboard HID USB device.
> + *  This code was inspired by hid-cando.c and hid-roccat.c(chrdev part)
> + *
> + *
> + *  This file is part of USB HID driver for Panasonic elite Panaboard UTB780.
> + *
> + *  This driver 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 3 of the License, or
> + *  (at your option) any later version.
> + *
> + *  This driver 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 driver.  If not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include <linux/device.h>
> +#include <linux/module.h>
> +#include <linux/slab.h>
> +#include <linux/usb.h>
> +#include <linux/fs.h>
> +#include <linux/cdev.h>
> +#include "hid-ids.h"
> +#include "hid-ubt880.h"
> +
> +#define UBT780_DEBUG
> +
> +#ifdef UBT780_DEBUG
> +#define UBT_DUMMY_DEBUG if (ubt_debug) printk(KERN_DEBUG "ubt880: %s:%s line %i\n", __FILE__, __func__ , __LINE__);

Check dev_dbg().

> +#else
> +#define UBT_DUMMY_DEBUG
> +#endif
> +
> +static int ubt_major;
> +static struct cdev ubt_cdev;
> +static struct class *ubt_class;
> +static struct ubt_chrdev *ubt_table[UBT780_MAX_DEVICES];
> +static DEFINE_MUTEX(minors_lock);
> +static int ubt_debug = 0;
> +module_param_named(debug_enabled, ubt_debug, int, 0600);
> +MODULE_PARM_DESC(debug_enabled, "Toggle huge and ugly UBT debugging messages.");

Perhaps the heavy debugging has been done already, and the mainline
code can be trimmed a bit, no?

> +
> +static int isCalibrated = 0;
> +
> +static int mode = 0;
> +
> +static int LTx = -1;
> +static int LTy = -1;
> +static int LBx = -1;
> +static int LBy = -1;
> +static int RTx = -1;
> +static int RTy = -1;
> +static int RBx = -1;
> +static int RBy = -1;
> +
> +static int LTX = -1;
> +static int LTY = -1;
> +static int LBX = -1;
> +static int LBY = -1;
> +static int RTX = -1;
> +static int RTY = -1;
> +static int RBX = -1;
> +static int RBY = -1;
> +
> +static int Xoff_A = -1;
> +static int Xmag_A = -1;
> +static int Xmag_B = -1;
> +static int Yoff_A = -1;
> +static int Ymag_A = -1;
> +static int Ymag_B = -1;

This is a lot of variables. Surely it can be simplified someway?

> +
> +module_param(mode, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(mode, "1 - digitizer mode, 0 - mouse mode");
> +
> +module_param(isCalibrated, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(isCalibrated, "Calibration data is set to driver.");
> +/*---------------------------------------------------------------------*/
> +
> +module_param(LTx, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LTx, "point x of left-up corner, screen coordinate");
> +
> +module_param(LTy, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LTy, "point y of left-up corner, screen coordinate");
> +
> +module_param(LBx, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LBx, "point x of left-bot corner, screen coordinate");
> +
> +module_param(LBy, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LBy, "point y of left-bot corner, screen coordinate");
> +
> +module_param(RTx, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RTx, "point x of rigth-up corner, screen coordinate");
> +
> +module_param(RTy, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RTy, "point y of rigth-up corner, screen coordinate");
> +
> +module_param(RBx, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RBx, "point x of rigth-bot corner, screen coordinate");
> +
> +module_param(RBy, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RBy, "point y of rigth-bot corner, screen coordinate");
> +
> +/*---------------------------------------------------------------------*/
> +
> +module_param(LTX, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LTX, "point x of left-up corner, board coordinate");
> +
> +module_param(LTY, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LTY, "point y of left-up corner, board coordinate");
> +
> +module_param(LBX, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LBX, "point x of left-bot corner, board coordinate");
> +
> +module_param(LBY, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(LBY, "point y of left-bot corner, board coordinate");
> +
> +module_param(RTX, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RTX, "point x of rigth-up corner, board coordinate");
> +
> +module_param(RTY, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RTY, "point y of rigth-up corner, board coordinate");
> +
> +module_param(RBX, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RBX, "point x of rigth-bot corner, board coordinate");
> +
> +module_param(RBY, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(RBY, "point y of rigth-bot corner, board coordinate");
> +/*-----------------------------------------------------------------*/
> +module_param(Xoff_A, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Xoff_A, "point x of rigth-up corner, board coordinate");
> +
> +module_param(Xmag_A, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Xmag_A, "point y of rigth-up corner, board coordinate");
> +
> +module_param(Xmag_B, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Xmag_B, "point x of rigth-bot corner, board coordinate");
> +
> +module_param(Ymag_A, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Ymag_A, "point y of rigth-bot corner, board coordinate");
> +
> +module_param(Ymag_B, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Ymag_B, "point y of rigth-bot corner, board coordinate");
> +
> +module_param(Yoff_A, int, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
> +MODULE_PARM_DESC(Yoff_A, "point x of rigth-up corner, board coordinate");

Uh-ho. I can see this either becoming a single coded parameter, or
better still, zero. We also have sysfs to consider for such things.

> +
> +/*Character device forward declarations*/
> +static int ubt880_open_io(struct inode *node, struct file *file);
> +static int ubt880_release_io(struct inode *node, struct file *file);
> +static long ubt880_ioctl(struct file *iofile, unsigned int command, unsigned long data);
> +static int ubt_chrdev_connect(struct hid_device *, struct ubt880_data *);
> +static void ubt_chrdev_disconnect(struct hid_device *hid);

Please reorganize your code to get rid of forward declarations.

> +#ifdef UBT780_DEBUG
> +static void send_test_packet(struct hid_device *hdev);
> +static void fake_calibration(struct hid_device *hdev, unsigned short *coords);
> +#endif

Such code should not be part of a mainline driver.

> +static struct input_dev *ubt880_get_input(struct hid_device *hdev);
> +static int ubt880_raw_event(struct hid_device *hdev, struct hid_report *report, u8 *data, int size);
> +/*We are using function from usbhid module*/
> +extern void usbhid_submit_report(struct hid_device *hid, struct hid_report *report, unsigned char dir);

An #include would be appropriate.

> +static void set_calib_to_device(struct ubt880_data *drvdata);
> +
> +/*switch_mode : Our device has two modes: mouse mode in which it is acting like mouse, producing
> + * absolute coordinates [0 - 4095]
> + * The second mode is digitizer mode, which produce two raw clock measurments from ultrasound
> + * recievers in the top-left corner of the board. This mode more flexible and also reports batterey
> + * status of digitizer pen and penID.
> + */
> +static int ubt880_switch_mode(struct hid_device *hid, unsigned char mode)
> +{
> +	int rc = 0;
> +
> +	struct hid_report *report = NULL;
> +	struct hid_report *report_cur = NULL;
> +	__s32 *val = NULL;
> +	/** Packet forming */
> +
> +	UBT_DUMMY_DEBUG
> +
> +	list_for_each_entry(report, &hid->report_enum[HID_FEATURE_REPORT].report_list, list)
> +	{
> +		if (hid->report_enum[HID_FEATURE_REPORT].numbered) {
> +			report_cur = report;
> +		}
> +	}
> +
> +	if (report_cur == NULL) {
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880 : switch mode : report_cur = 0");
> +		return rc;
> +	}
> +
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt880: switch mode: reportid = %d", report_cur->id);
> +	val = report_cur->field[0]->value;
> +
> +	switch (mode) {
> +	case MODE_MOUSE: {
> +		mode = 0x0;
> +		break;
> +	}
> +	case MODE_SINGLETOUCH: {
> +		mode = 0x01;
> +		break;
> +	}
> +	case MODE_DGTZR: {
> +		mode = 0x02;
> +		break;
> +	}
> +	default: {
> +		rc = -EIO;
> +		return rc;
> +	}
> +	}
> +	val[0] = mode;
> +	UBT_DUMMY_DEBUG
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt880 :switch_mode: usbhid_submit_report drv=%p p=%p mode=%d", hid, report_cur, mode);
> +	/*Send report to device*/
> +
> +	usbhid_submit_report(hid, report_cur, USB_DIR_OUT);
> +	UBT_DUMMY_DEBUG
> +	return rc;
> +}

There ought to be a function in hid to help you achieve this.

> +
> +static int ubt780_switch_mode(struct hid_device *hid, unsigned char mode)
> +{
> +	int rc = 0;
> +
> +	struct hid_report *report = NULL;
> +	struct hid_report *report_cur = NULL;
> +	__s32 *val = NULL;
> +	/** Packet forming */
> +	UBT_DUMMY_DEBUG
> +
> +	list_for_each_entry(report,
> +			&hid->report_enum[HID_OUTPUT_REPORT].report_list,
> +			list)
> +	{
> +		if (hid->report_enum[HID_OUTPUT_REPORT].numbered)
> +			report_cur = report;
> +	}
> +
> +	switch (mode) {
> +	case MODE_MOUSE: {
> +		mode = 0x00;
> +		break;
> +	}
> +	case MODE_DGTZR: {
> +		mode = 0x01;
> +		break;
> +	}
> +	default: {
> +		rc = -EIO;
> +		return rc;
> +	}
> +	}
> +	val = report_cur->field[0]->value;
> +	val[0] = 0x7e;
> +	val[1] = 0x04;
> +	val[2] = 0x4d;
> +	val[3] = mode;
> +	val[4] = 0x00;
> +	val[5] = 0x0a;
> +	val[6] = 0x00;
> +	val[7] = 0x00;
> +
> +	UBT_DUMMY_DEBUG
> +
> +	if (report_cur == NULL) {
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt780 : switch mode : report_cur = 0");
> +		return rc;
> +	}
> +
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt780 :switch_mode:\
> +				usbhid_submit_report\
> +				drv=%p p=%p mode=%d",
> +				hid, report_cur, mode);
> +	/*Send report to device*/
> +
> +	usbhid_submit_report(hid, report_cur, USB_DIR_OUT);
> +	UBT_DUMMY_DEBUG
> +	return rc;
> +}

Similar code as above, so simplifications are definitely possible.

> +
> +/* PenToInt : converts ultrasound clock measurements to screen coordinates.*/
> +/*Apply calibration to ultrasound clock measurements*/

Please use kernel-style comments.

> +int xold = 0, yold = 0;

static

> +bool ubt_pen_to_int(int *pLeft, int *pRight)

static

> +{
> +	int left2, right2 , n, w_n, sqr, xx, yy;
> +	bool bRet = false;
> +	static int w = 1164;
> +
> +	left2  = (*pLeft)*(*pLeft);
> +	right2 = (*pRight)*(*pRight);

This is style violation - please check your patches with checkpatch before submission.

> +	UBT_DUMMY_DEBUG
> +	if (left2 == 0 && right2 == 0) {
> +		*pLeft = (int)xold;
> +		*pRight = (int)yold;
> +		if (ubt_debug)
> +			printk(KERN_DEBUG
> +				"ubt780_XY old values: %d,%d", xold, yold);
> +	    return true;
> +	}

This seems to rely on a special state of the hardware and should
probably we lifted out of this function.

> +
> +	n = (right2 - left2) / (2*w);
> +	w_n = w-n;
> +
> +	sqr = (2*left2 - (w_n * w_n));
> +
> +	if (sqr < 0)
> +		return bRet;
> +
> +	xx = (w_n + int_sqrt(sqr)) / 2;
> +	yy = xx + n;
> +
> +	if (xx < 0 || yy < 0)
> +		return bRet;
> +
> +	*pLeft = (int)xx;
> +	*pRight = (int)yy;
> +	bRet = true;
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt780_XY: %d,%d", xx, yy);
> +
> +	return bRet;
> +}

Without even trying to understand what this code does, the return
values look odd. Just skip bRet altogether.

> +
> +static bool ubt_calibrate(int inX, int inY, int *outX, int *outY, struct ubt_calib *calib)
> +{
> +	int Y_LTY, X_LTX, Xoff, Xmag, Yoff, Ymag;
> +	unsigned int x, y;
> +	xold = inX;
> +	yold = inY;
> +
> +	Y_LTY = inY - calib->LTY;
> +	X_LTX = inX - calib->LTX;
> +
> +	Xoff = Y_LTY * calib->Xoff_A + calib->LTX * 10000;
> +	Xmag = (Y_LTY * calib->Xmag_A) / 10000;
> +	Xmag += calib->Xmag_B;
> +	Yoff = X_LTX * calib->Yoff_A + calib->LTY * 10000;
> +	Ymag = X_LTX * calib->Ymag_A / 10000;
> +	Ymag += calib->Ymag_B;
> +
> +	if (!Xmag)
> +		Xmag = 1;

Why?

> +	x  = (inX * 10000 - Xoff) / Xmag;
> +	x  += calib->LTx;
> +
> +	if (!Ymag)
> +		Ymag = 1;
> +	y  = (inY * 10000 - Yoff) / Ymag;
> +	y  += calib->LTy;
> +
> +	if (x < 0)
> +		x = 0;
> +	if (x > UBT880_MAX_AXIS_X)
> +		x = UBT880_MAX_AXIS_X;
> +	if (y < 0)
> +		y = 0;
> +	if (y > UBT880_MAX_AXIS_Y)
> +		y = UBT880_MAX_AXIS_Y;

clamp_val()

> +
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "%d - %d", x, y);
> +	*outX = x;
> +	*outY = y;
> +	return true;
> +}

Seems the above function could be void.

> +
> +/*
> +input_mapping : Set appropriate bits for input device according to current field, usage and usage page.
> +For info about valid usage pages and usages see include/linux/hid.h
> +For more info on input see Documentation/input/input.txt and Documentation/input/input-programming.txt
> +and of course include/linux/input.h
> +What to return?
> +return < 0 - field was ignored
> +return > 0 - field was mapped
> +return == 0 - let hid driver map this field for you
> +*/
> +static int ubt880_input_mapping(struct hid_device *hdev, struct hid_input *hi,
> +		struct hid_field *field, struct hid_usage *usage,
> +		unsigned long **bit, int *max)
> +{
> +	/*Just ignore all fields*/
> +	return -1;
> +}

This function is not doing anything extraordinary, so it has a
disproportional amount of comments.

> +/*
> + * input_mapped : called from hidinput_configure_usage after input is mapped.
> + * Never called if you have returned negative value from input_mapping.
> + * Return values:
> + * return < 0 - ignore field
> + * return >= 0 - do further processing
> + */
> +/*static int ubt880_input_mapped(struct hid_device *hdev, struct hid_input *hi,
> +		struct hid_field *field, struct hid_usage *usage,
> +		unsigned long **bit, int *max)
> +{
> +	return -1;
> +}*/

Ditto

> +
> +
> +/* event : Called on every single report field.
> + * Return :
> + * return == 0 - do generic input hidinput and hiddev processing. See hid_process_event in hid-core.c.
> + * return > 0 - finish processing.
> + */
> +static int ubt880_event(struct hid_device *hid, struct hid_field *field,
> +				struct hid_usage *usage, __s32 value)
> +{
> +	/*Just prevent further processing by hid*/
> +	return 1;
> +}

Remove altogether

> +static void ubt880_set_input(struct input_dev *input)
> +{
> +	/* Basics */
> +	/*We have a key*/
> +	__set_bit(EV_KEY, input->evbit);
> +	__set_bit(BTN_LEFT, input->keybit);
> +	__set_bit(BTN_RIGHT, input->keybit);
> +	/*two absolute axis*/
> +	__set_bit(EV_ABS, input->evbit);
> +	__set_bit(ABS_X, input->absbit);
> +	__set_bit(ABS_Y, input->absbit);
> +	/*two absolute MT axis and tracking IDs*/
> +	input_set_abs_params(input, ABS_MT_POSITION_X, 0, UBT880_MAX_AXIS_X, 0, 0);
> +	input_set_abs_params(input, ABS_MT_POSITION_Y, 0, UBT880_MAX_AXIS_Y, 0, 0);
> +	input_set_abs_params(input, ABS_MT_TRACKING_ID, 0, 3, 0, 0);
> +	input_set_abs_params(input, ABS_X, 0, UBT880_MAX_AXIS_X, 0, 0);
> +	input_set_abs_params(input, ABS_Y, 0, UBT880_MAX_AXIS_Y, 0, 0);
> +}

Please use slotted protocol instead (see 2.6.38 Documentation/input/).

> +
> +static void ubt780_set_input(struct input_dev *input)
> +{
> +	input->evbit[0] |= BIT(EV_KEY) | BIT(EV_ABS);
> +	input->absbit[0] |= BIT(ABS_X) | BIT(ABS_Y);
> +	set_bit(BTN_LEFT, input->keybit);
> +	set_bit(BTN_RIGHT, input->keybit);
> +
> +	input->absmax[ABS_X] = UBT780_MAX_AXIS_X;
> +	input->absmax[ABS_Y] = UBT780_MAX_AXIS_Y;
> +	input->absmin[ABS_X] = 0;
> +	input->absmin[ABS_Y] = 0;
> +}

This looks like standard HID stuff, and could probably be setup automatically.

> +
> +int ubt_set_device(struct ubt880_data *data, __u32 devid)
> +{
> +	int ret;
> +	data->calib.calibrated = 0;
> +	data->ubt_packet.report = 0;
> +	set_calib_to_device(data);
> +	switch (devid) {
> +	case USB_DEVICE_ID_PANABOARD_UBT780: {
> +		data->switch_mode = ubt780_switch_mode;
> +		data->set_input = ubt780_set_input;
> +		break;
> +	}
> +	case USB_DEVICE_ID_PANABOARD_UBT880: {
> +		data->switch_mode = ubt880_switch_mode;
> +		data->set_input = ubt880_set_input;
> +		break;
> +	}
> +	default: {
> +		ret = -1;
> +	}
> +	}
> +	return 0;
> +}

Static table would work too.

> +static int ubt880_probe(struct hid_device *hdev, const struct hid_device_id *id)
> +{
> +	int ret;
> +	struct ubt880_data *drvdata;
> +	struct hid_input *hidinput = NULL;
> +	struct input_dev *input;
> +
> +	UBT_DUMMY_DEBUG

remove

> +
> +	drvdata = kmalloc(sizeof(struct ubt880_data), GFP_KERNEL);
> +	if (!drvdata) {
> +		dev_err(&hdev->dev, "cannot allocate UB-T880 data\n");
> +		return -ENOMEM;
> +	}
> +
> +	hid_set_drvdata(hdev, drvdata);
> +
> +	ret = hid_parse(hdev);
> +	if (!ret) {
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "Trying hid_hw_start");

please use simpler debug function or remove

> +		/*We need HIDINPUT dev only. We have our own char device instead of hidraw*/
> +		ret = hid_hw_start(hdev, HID_CONNECT_HIDINPUT);
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "hid_hw_start() returned : %d", ret);
> +	}
> +
> +	if (ret) {
> +		kfree(drvdata);
> +		goto skip_input;
> +	}
> +	ubt_chrdev_connect(hdev, drvdata);
> +	ubt_set_device(drvdata, hdev->product);
> +
> +	drvdata->switch_mode(hdev, MODE_MOUSE);
> +	drvdata->current_mode = MODE_MOUSE;

surely there is a HID way to do this

> +
> +	UBT_DUMMY_DEBUG
> +	hidinput = list_entry(hdev->inputs.next, struct hid_input, list);
> +	if (hidinput == NULL) {
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880: probe : Failed to get valid hidinput");
> +		goto skip_input;
> +	}
> +	UBT_DUMMY_DEBUG
> +	input = hidinput->input;
> +
> +	UBT_DUMMY_DEBUG
> +	drvdata->set_input(input);
> +skip_input:
> +	return ret;
> +}
> +/* Our own char device for ioctls */
> +/*---------------------------------------------------------------------------------------------*/
> +

Why??

> +
> +static const struct file_operations ubt880_fops = {
> +	.owner = THIS_MODULE,
> +	.open = ubt880_open_io,
> +	.release = ubt880_release_io,
> +	.unlocked_ioctl = ubt880_ioctl,
> +};

And this is used solely to communicate with the device for calibration
and such? Why is it even part of this driver?

> +
> +static int ubt_chrdev_init(void)
> +{
> +	int result;
> +	dev_t dev_id;
> +
> +	result = alloc_chrdev_region(&dev_id, UBT780_FIRST_MINOR,
> +		UBT780_MAX_DEVICES, "ubt_chr");
> +
> +	ubt_major = MAJOR(dev_id);
> +
> +	if (result < 0) {
> +		printk(KERN_WARNING "ubt: can't get major number\n");
> +		result = 0;
> +		goto out;
> +	}
> +
> +	ubt_class = class_create(THIS_MODULE, "ubt_chr");
> +	if (IS_ERR(ubt_class)) {
> +		result = PTR_ERR(ubt_class);
> +		unregister_chrdev(ubt_major, "ubt_chr");
> +		goto out;
> +	}
> +
> +	cdev_init(&ubt_cdev, &ubt880_fops);
> +	cdev_add(&ubt_cdev, dev_id, UBT780_MAX_DEVICES);
> +out:
> +	return result;
> +}
> +
> +static void ubt_chrdev_cleanup(void)
> +{
> +	dev_t dev_id = MKDEV(ubt_major, 0);
> +	cdev_del(&ubt_cdev);
> +	class_destroy(ubt_class);
> +	unregister_chrdev_region(dev_id, UBT780_MAX_DEVICES);
> +}
> +static int ubt_chrdev_connect(struct hid_device *hid, struct ubt880_data *drvdata)
> +{
> +	int minor, result;
> +	struct ubt_chrdev *dev;
> +
> +	dev = kzalloc(sizeof(struct ubt_chrdev), GFP_KERNEL);
> +	if (!dev)
> +		return -ENOMEM;
> +
> +	result = -EINVAL;
> +
> +	mutex_lock(&minors_lock);
> +
> +	for (minor = 0; minor < UBT780_MAX_DEVICES; minor++) {
> +		if (ubt_table[minor])
> +			continue;
> +		ubt_table[minor] = dev;
> +		result = 0;
> +		break;
> +	}
> +	UBT_DUMMY_DEBUG
> +	printk(KERN_DEBUG "ubt_chrdev_connect : result = %X", result);
> +
> +	if (result) {
> +		mutex_unlock(&minors_lock);
> +		kfree(dev);
> +		goto out;
> +	}
> +	printk(KERN_DEBUG "ubt : chrdev_connect : ubt_majo=%d, minor=%d", ubt_major, minor);
> +	dev->dev = device_create(ubt_class, &hid->dev, MKDEV(ubt_major, minor),
> +				 NULL, "%s%d", "ubt_", minor);
> +
> +	if (IS_ERR(dev->dev)) {
> +		ubt_table[minor] = NULL;
> +		mutex_unlock(&minors_lock);
> +		result = PTR_ERR(dev->dev);
> +		kfree(dev);
> +		drvdata->chrdev = NULL;
> +		goto out;
> +	}
> +
> +	UBT_DUMMY_DEBUG
> +	printk(KERN_DEBUG "ubt_chrdev_connect : result = %X", result);
> +
> +	mutex_unlock(&minors_lock);
> +
> +	dev->hid = hid;
> +	drvdata->chrdev = dev;
> +	dev->minor = minor;
> +
> +	dev->exist = 1;
> +
> +out:
> +	printk(KERN_DEBUG "ubt_chrdev_connect : result = %X", result);
> +	return result;
> +
> +}
> +
> +static void ubt_chrdev_disconnect(struct hid_device *hid)
> +{
> +
> +	struct ubt880_data *ubt880_drv = hid_get_drvdata(hid);
> +	struct ubt_chrdev *ubt880 = ubt880_drv->chrdev;
> +
> +	if (!ubt880)
> +		return;
> +
> +	ubt880->exist = 0;
> +
> +	mutex_lock(&minors_lock);
> +	ubt_table[ubt880->minor] = NULL;
> +	mutex_unlock(&minors_lock);
> +
> +	device_destroy(ubt_class, MKDEV(ubt_major, ubt880->minor));
> +
> +	if (ubt880->open) {
> +		hid->ll_driver->close(hid);
> +	} else {
> +		kfree(ubt880);
> +	}
> +}
> +
> +
> +static int ubt880_open_io(struct inode *node, struct file *file)
> +{
> +	unsigned int minor = iminor(node);
> +	struct ubt_chrdev *device;
> +	int error = 0;
> +
> +	mutex_lock(&minors_lock);
> +	UBT_DUMMY_DEBUG
> +	device = ubt_table[minor];
> +
> +	if (!device) {
> +		printk(KERN_EMERG "ubt880 device with minor %d doesn't exist\n",
> +				minor);
> +		error = -ENODEV;
> +		goto exit_unlock;
> +	}
> +
> +	if (!device->open) {
> +		device->open++;
> +		/* power on device */
> +		if (device->hid->ll_driver->power) {
> +			error = device->hid->ll_driver->power(device->hid,
> +					PM_HINT_FULLON);
> +			if (error < 0) {
> +				--device->open;
> +				goto exit_unlock;
> +			}
> +		}
> +		error = device->hid->ll_driver->open(device->hid);
> +		if (error < 0) {
> +			if (device->hid->ll_driver->power)
> +				device->hid->ll_driver->power(device->hid,
> +						PM_HINT_NORMAL);
> +			--device->open;
> +			goto exit_unlock;
> +		}
> +	} else {
> +		device->open++;
> +		/*error = -EBUSY;*/
> +	}
> +
> +exit_unlock:
> +	mutex_unlock(&minors_lock);
> +	return error;
> +}
> +
> +static int ubt880_release_io(struct inode *node, struct file *file)
> +{
> +	unsigned int minor = iminor(node);
> +	struct ubt_chrdev *dev;
> +	int ret = 0;
> +	UBT_DUMMY_DEBUG
> +	mutex_lock(&minors_lock);
> +	if (!ubt_table[minor]) {
> +		ret = -ENODEV;
> +		goto unlock;
> +	}
> +
> +	dev = ubt_table[minor];
> +	if (dev->open) {
> +		dev->open--;
> +		if (dev->open)
> +			goto unlock;
> +		if (dev->hid->ll_driver->power)
> +				dev->hid->ll_driver->power(dev->hid, PM_HINT_NORMAL);
> +		dev->hid->ll_driver->close(dev->hid);
> +	}
> +
> +	ret = 0;
> +unlock:
> +	mutex_unlock(&minors_lock);
> +
> +	return ret;
> +}
> +static void set_calib_to_driver(struct ubt_calib *calib)
> +{
> +
> +	if (!calib)
> +		return;
> +	LTx = calib->LTx;
> +	LTy = calib->LTy;
> +	LBx = calib->LBx;
> +	LBy = calib->LBy;
> +	RTx = calib->RTx;
> +	RTy = calib->RTy;
> +	RBx = calib->RBx;
> +	RBy = calib->RBy;
> +
> +	LTX = calib->LTX;
> +	LTY = calib->LTY;
> +	LBX = calib->LBX;
> +	LBY = calib->LBY;
> +	RTX = calib->RTX;
> +	RTY = calib->RTY;
> +	RBX = calib->RBX;
> +	RBY = calib->RBY;
> +
> +	Xoff_A = calib->Xoff_A;
> +	Xmag_A = calib->Xmag_A;
> +	Xmag_B = calib->Xmag_B;
> +	Yoff_A = calib->Yoff_A;
> +	Ymag_A = calib->Ymag_A;
> +	Ymag_B = calib->Ymag_B;
> +
> +	isCalibrated = 1;
> +}
> +
> +
> +static void set_calib_to_device(struct ubt880_data *drvdata)
> +{
> +
> +	if (!drvdata)
> +		return;
> +	if (!isCalibrated)
> +		return;
> +	drvdata->calib.LTx = LTx;
> +	drvdata->calib.LTy = LTy;
> +	drvdata->calib.LBx = LBx;
> +	drvdata->calib.LBy = LBy;
> +	drvdata->calib.RTx = RTx;
> +	drvdata->calib.RTy = RTy;
> +	drvdata->calib.RBx = RBx;
> +	drvdata->calib.RBy = RBy;
> +
> +	drvdata->calib.LTX = LTX;
> +	drvdata->calib.LTY = LTY;
> +	drvdata->calib.LBX = LBX;
> +	drvdata->calib.LBY = LBY;
> +	drvdata->calib.RTX = RTX;
> +	drvdata->calib.RTY = RTY;
> +	drvdata->calib.RBX = RBX;
> +	drvdata->calib.RBY = RBY;
> +
> +	drvdata->calib.Xoff_A = Xoff_A;
> +	drvdata->calib.Xmag_A = Xmag_A;
> +	drvdata->calib.Xmag_B = Xmag_B;
> +	drvdata->calib.Yoff_A = Yoff_A;
> +	drvdata->calib.Ymag_A = Ymag_A;
> +	drvdata->calib.Ymag_B = Ymag_B;
> +
> +	drvdata->calib.calibrated = 1;
> +}
> +
> +static long ubt880_ioctl(struct file *iofile, unsigned int command, unsigned long data)
> +{
> +
> +	int retval = 0;
> +	struct inode *inode = iofile->f_path.dentry->d_inode;
> +	unsigned int minor = iminor(inode);
> +	struct ubt_chrdev *dev;
> +	struct ubt880_data *drvdata;
> +	mutex_lock(&minors_lock);
> +	dev = ubt_table[minor];
> +	if (!dev) {
> +		retval = -ENODEV;
> +		goto out;
> +	}
> +	drvdata = hid_get_drvdata(dev->hid);
> +	switch (command) {
> +	case GET_BATTERY_STATUS:
> +	{
> +		int *in = (int *) data;
> +		*in = 0;
> +		retval = 0;
> +		break;
> +	}
> +	case GET_LAST_PACKET_OLD:
> +	case GET_LAST_PACKET:
> +	{
> +		struct ubt880_dgtzr *packsrc = &drvdata->ubt_packet;
> +		struct ubt880_dgtzr *packdst = (struct ubt880_dgtzr *) data;
> +
> +		if (packsrc->report != 0x00) {
> +			memcpy((void *)packdst, (void *)packsrc, sizeof(struct ubt880_dgtzr));
> +			packsrc->report = 0;
> +			if (ubt_debug)
> +				printk(KERN_DEBUG "ubt880_ioctl_io : GET_LAST_PACKET2");
> +		}
> +
> +		retval = 0;
> +		break;
> +	}
> +	case GET_MODE:
> +	{
> +		int *in = (int *) data;
> +		printk(KERN_DEBUG "ubt880_ioctl_io : GET_MODE");
> +		*in = drvdata->current_mode;
> +		retval = 0;
> +		break;
> +	}
> +	case SET_MODE:
> +	{
> +		int *dt = (int *)data;
> +		printk(KERN_DEBUG "ubt_ioctl_io : SET_MODE");
> +		drvdata->ubt_packet.report = 0;
> +
> +		if (drvdata->switch_mode(dev->hid, *dt) != 0) {
> +				retval = -EIO;
> +				if (ubt_debug)
> +					printk(KERN_DEBUG "ubt_ioctl_io : SET_MODE error");
> +		} else {
> +			drvdata->current_mode = *dt;
> +			if (ubt_debug)
> +				printk(KERN_DEBUG "ubt_ioctl_io : SET_MODE ok, mode = %d", drvdata->current_mode);
> +			retval = 0;
> +		}
> +		break;
> +	}
> +	case SET_CALIB: {
> +		struct ubt_calib *calib = (struct ubt_calib *) data;
> +
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880_ioctl_io : SET_CALIB");
> +
> +		set_calib_to_driver(calib);
> +		set_calib_to_device(drvdata);
> +
> +		retval = 0;
> +		break;
> +	}
> +	case GET_DEVICE_ID: {
> +		int *in = (int *) data;
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt_ioctl_io : get device id");
> +		*in = dev->hid->product;
> +		retval = 0;
> +		break;
> +	}
> +#ifdef UBT780_DEBUG
> +	case TEST_MT_PACKET: {
> +
> +		send_test_packet(dev->hid);
> +		retval = 0;
> +		break;
> +	}
> +	case TEST_CALIBRATION: {
> +
> +		unsigned short *coord = (unsigned short *)data;
> +		fake_calibration(dev->hid, coord);
> +		retval = 0;
> +		break;
> +	}
> +#endif
> +	default:
> +	{
> +		retval = -EIO;
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880_ioctl_io : unknown command %d", command);
> +		break;
> +	}
> +	}
> +out:
> +	mutex_unlock(&minors_lock);
> +	return retval;
> +}

The above seems like a different driver altogether, and possibly
better implemented in userland.

> +/*-----------------------------------------------------------------------------------------------*/
> +static void ubt_report_input(struct input_dev *input, unsigned int x, unsigned int y, int btn_left, int btn_right)
> +{
> +	/* To prevent situation when new coords arrive before unpess */
> +	/* and a line is drawn istead of 2 dots */
> +	input_report_key(input, BTN_LEFT, btn_left);
> +	input_report_key(input, BTN_RIGHT, btn_right);
> +	/*if (btn)*/
> +		input_report_abs(input, ABS_X, x);
> +		input_report_abs(input, ABS_Y, y);
> +
> +	input_sync(input);
> +}
> +/* Debug output of raw MT packet from device */
> +static void ubt880_print_multitouch(struct ubt_mt_contact *pack, int size)
> +{
> +	int i;
> +	printk(KERN_DEBUG "-------------------------------------");
> +	for (i = 0; i < size; i++) {
> +		printk(KERN_DEBUG "-----------%d------------", i);
> +		printk(KERN_DEBUG "ubt880: mt packet: flags : inrange %X | tipswitch %X", pack[i].flags & 0x02, pack[i].flags & 0x01);
> +		printk(KERN_DEBUG "ubt880: mt packet: id | %d", pack[i].id);
> +		printk(KERN_DEBUG "ubt880: mt packet: X | %d", pack[i].x);
> +		printk(KERN_DEBUG "ubt880: mt packet: Y | %d", pack[i].y);
> +		printk(KERN_DEBUG "-------------------------");
> +	}
> +	printk(KERN_DEBUG "ubt880: mt packet: count: %d", size);
> +}

please remove excessive debugging

> +/* Report MT to the input subsystem*/
> +static void ubt880_report_contact(struct input_dev *input, struct ubt_mt_contact *contact)
> +{
> +	if (!contact || !input)
> +		return;
> +	input_report_abs(input, ABS_MT_TRACKING_ID, contact->id);
> +	input_report_abs(input, ABS_MT_POSITION_X, contact->x);
> +	input_report_abs(input, ABS_MT_POSITION_Y, contact->y);
> +	input_mt_sync(input);
> +}

please use slotted protocol instead

> +static void ubt880_report_mt(struct input_dev *input, struct ubt_mt_contact *pack, int size, struct ubt_calib *calib)
> +{
> +	int i = 0;
> +	/* Report MT contacts */
> +	if (size < 1)
> +		return;
> +	if (!input || !pack || !calib)
> +		return;

what do these error paths mean?

> +
> +	for (i = 0; i < size; i++) {
> +		/* If Tip Switch bit is set */
> +		if (pack[i].flags & 0x01) {
> +			if (calib->calibrated) {
> +				ubt_calibrate(pack[i].x,
> +						pack[i].y,
> +						(int *)&pack[i].x,
> +						(int *)&pack[i].y,
> +						calib);
> +			}
> +			ubt880_report_contact(input, &pack[i]);
> +		}
> +	}
> +	input_sync(input);
> +	/* Emulate single-touch device. Only first contact is used. */
> +	/* Note that is already calibrated */
> +	ubt_report_input(input, pack[0].x, pack[0].y, pack[0].flags & 0x01, 0);
> +}
> +
> +#ifdef UBT780_DEBUG
> +static void fake_calibration(struct hid_device *hdev, unsigned short *coord)
> +{
> +	char pack[6];
> +	unsigned short *pack_coord = (unsigned short *)&pack[2];
> +	pack[0] = 0x01;
> +	pack[1] = coord[0];
> +	pack_coord[0] = coord[1];
> +	pack_coord[1] = coord[2];
> +	ubt880_raw_event(hdev, 0, (u8 *)&pack, 6);
> +}
> +static void send_test_packet(struct hid_device *hdev)
> +{
> +	char testdata[20];
> +	struct ubt_mt_contact *contacts;
> +	struct ubt_calib calib;
> +
> +	calib.calibrated = 0;
> +
> +	testdata[0] = 0x03;
> +	contacts = (struct ubt_mt_contact *)&testdata[1];
> +	contacts[0].flags = 0x01;
> +	contacts[0].id = 0x01;
> +	contacts[0].x = 205;
> +	contacts[0].y = 1023;
> +
> +	contacts[1].flags = 0x01;
> +	contacts[1].id = 0x02;
> +	contacts[1].x = 1283;
> +	contacts[1].y = 1048;
> +
> +	contacts[2].flags = 0x01;
> +	contacts[2].id = 0x03;
> +	contacts[2].x = 508;
> +	contacts[2].y = 2473;
> +
> +	testdata[19] = 3;
> +
> +	ubt880_raw_event(hdev, 0, (u8 *)&testdata, 20);
> +}
> +#endif

please remove

> +
> +static struct input_dev *ubt880_get_input(struct hid_device *hdev)
> +{
> +	struct hid_input *hidinput = list_entry(hdev->inputs.next, struct hid_input, list);
> +	struct input_dev *input;
> +
> +	if (!hidinput)
> +		return 0;
> +	input = hidinput->input;
> +	return input;
> +}
> +static int ubt880_raw_event(struct hid_device *hdev, struct hid_report *report, u8 *data, int size)
> +{
> +	struct ubt880_data *driver_data = hid_get_drvdata(hdev);
> +	struct ubt880_dgtzr *pack = &driver_data->ubt_packet;
> +
> +	struct input_dev *input = ubt880_get_input(hdev);
> +	if (!input)
> +		return -1;
> +
> +	if (!driver_data)
> +		return -1;
> +	/*Save the packet for userspace processing*/
> +	/*Mouse mode packet*/
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt880:raw_event: report = %X", data[0]);
> +	switch (data[0]) {
> +	case 0x01: {
> +		int X, Y;
> +		memcpy((void *)pack, (void *)data, size);
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880_mode: Mouse");
> +
> +		if (driver_data->current_mode == MODE_UKN)
> +			driver_data->current_mode = MODE_MOUSE;
> +
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880: mouse packet: X=%d,Y=%d,Button=%d",
> +				pack->data[0], pack->data[1], pack->command);
> +		X = pack->data[0];
> +		Y = pack->data[1];
> +		if (driver_data->calib.calibrated) {
> +			ubt_calibrate(X, Y, &X, &Y, &driver_data->calib);
> +			if (ubt_debug)
> +				printk(KERN_DEBUG "ubt880: mouse packet calibrated: X=%d,Y=%d",
> +					X, Y);
> +		}
> +
> +		ubt_report_input(input, X, Y, pack->command & 0x01, pack->command & 0x02);

Transformation seems to appear both inside and outside of this function.

> +		/*Switch to digitizer*/
> +		if (driver_data->current_mode != MODE_MOUSE)
> +			ubt880_switch_mode(hdev, MODE_MOUSE);
> +		break;
> +	}
> +	case 0x02: {
> +		struct ubt780_dgtzr *pack = (struct ubt780_dgtzr *)data;
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt780_mode: DIG");
> +		if (driver_data->current_mode == MODE_UKN)
> +			driver_data->current_mode = MODE_DGTZR;
> +
> +		if (driver_data->current_mode == MODE_MOUSE) {
> +			pack->report = 0;
> +			if (ubt_debug)
> +				printk(KERN_DEBUG "ubt780_current_mode: MOUSE");
> +			/*Switch to mouse*/
> +			driver_data->switch_mode(hdev, MODE_MOUSE);
> +		} else if (driver_data->current_mode == MODE_DGTZR) {
> +			unsigned short *pCoord = (unsigned short *)(&pack->data[3]);
> +			struct ubt_calib *calib = &driver_data->calib;
> +			int X = (int) pCoord[0];
> +			int Y = (int) pCoord[1];
> +			int leftBtn = pack->data[2] >> 5 & 0x01;
> +			int rightBtn = pack->data[2] >> 4 & 0x01;
> +
> +			ubt_pen_to_int(&X, &Y);
> +
> +			driver_data->ubt_packet.report = 0x02;
> +			driver_data->ubt_packet.command = pack->data[2];
> +			driver_data->ubt_packet.data[0] = X;
> +			driver_data->ubt_packet.data[1] = Y;
> +
> +			if (driver_data->calib.calibrated) {
> +				if (ubt_calibrate(X, Y, &X, &Y, calib)) {
> +					if (ubt_debug)
> +						printk(KERN_DEBUG "ubt780_mode:\
> +							Calibrated;\
> +							x=%d y=%d,left_btn=%d\n",
> +							X, Y, leftBtn);
> +					ubt_report_input(input, X, Y, leftBtn, rightBtn);
> +				}
> +			} else {
> +				ubt_report_input(input, X, Y, leftBtn, rightBtn);
> +				if (ubt_debug)
> +					printk(KERN_DEBUG "ubt780_mode:\
> +						Not calibrated;\
> +						x=%d y=%d,left_btn=%d\n",
> +						X, Y, leftBtn);
> +			}
> +		}
> +		break;
> +	}
> +	case 0x03: {
> +		struct ubt_mt_packet *packet = (struct ubt_mt_packet *)data;
> +		if (ubt_debug) {
> +			printk(KERN_DEBUG "ubt880_mode: Multitouch");
> +			printk(KERN_DEBUG "ubt880_driver_mode : %d", driver_data->current_mode);
> +		}
> +		if (driver_data->current_mode == MODE_UKN)
> +			driver_data->current_mode = MODE_DGTZR;
> +
> +		if (ubt_debug)
> +			ubt880_print_multitouch((struct ubt_mt_contact *)&packet->data[1], packet->data[19]);
> +
> +		if (driver_data->current_mode == MODE_MOUSE) {
> +			driver_data->switch_mode(hdev, MODE_MOUSE);
> +		} else if (driver_data->current_mode == MODE_DGTZR) {
> +			ubt880_report_mt(input, (struct ubt_mt_contact *)&packet->data[1], packet->data[19], &driver_data->calib);
> +		}
> +		break;
> +	}
> +	default:
> +		if (ubt_debug)
> +			printk(KERN_DEBUG "ubt880: raw_report: Unknown report %X", pack->report);
> +	}
> +
> +
> +	UBT_DUMMY_DEBUG
> +	return 0;
> +}
> +static void ubt880_remove(struct hid_device *hdev)
> +{
> +	UBT_DUMMY_DEBUG
> +	ubt_chrdev_disconnect(hdev);
> +	hid_hw_stop(hdev);
> +	kfree(hid_get_drvdata(hdev));
> +	hid_set_drvdata(hdev, NULL);
> +}
> +
> +static struct hid_device_id ubt880_devices[] = {
> +	{ HID_USB_DEVICE(0x04da, 0x104d) },
> +	{ HID_USB_DEVICE(0x04da, 0x1044) },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(hid, ubt880_devices);
> +
> +static struct hid_driver ubt880_driver = {
> +	.name = "ubt880",
> +	.id_table = ubt880_devices,
> +	.probe = ubt880_probe,
> +	.remove = ubt880_remove,
> +	.input_mapping = ubt880_input_mapping,
> +	.raw_event = ubt880_raw_event,
> +	.event = ubt880_event,
> +};
> +
> +static int __init ubt880_init(void)
> +{
> +	int retval = ubt_chrdev_init();
> +	retval = hid_register_driver(&ubt880_driver);
> +	UBT_DUMMY_DEBUG
> +	if (ubt_debug)
> +		printk(KERN_DEBUG "ubt880 : init : chrdev_init : retval = %X", retval);
> +	return retval;
> +}
> +
> +static void __exit ubt880_exit(void)
> +{
> +	UBT_DUMMY_DEBUG
> +	hid_unregister_driver(&ubt880_driver);
> +	ubt_chrdev_cleanup();
> +}
> +
> +module_init(ubt880_init);
> +module_exit(ubt880_exit);
> +
> +MODULE_AUTHOR("Anton Chikin <kverlin@...il.com>");
> +MODULE_DESCRIPTION("Panasonic UB-T780 driver");
> +MODULE_LICENSE("GPL");
> diff --git a/drivers/hid/hid-ubt880.h b/drivers/hid/hid-ubt880.h
> new file mode 100644
> index 0000000..961b03f
> --- /dev/null
> +++ b/drivers/hid/hid-ubt880.h
> @@ -0,0 +1,162 @@
> +/*
> + *  USB HID driver for Panasonic elite Panaboard UTB780
> + *   Copyright (c) 2008 Igor Shakirov, Victor Grenke <comp.vision@...il.com>
> + *   Copyright (c) 2010-2011 Anton Chikin<anton.chikin@...aart.com> for Panasonic.
> + *
> + *  Information.
> + *  It's driver for supporting Panasonic Elite Panaboard HID USB device.
> + *
> + *  This file is part of USB HID driver for Panasonic elite Panaboard UTB780.
> + *
> + *  This driver 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 3 of the License, or
> + *  (at your option) any later version.
> + *
> + *  This driver 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 driver.  If not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include <linux/hid.h>
> +#ifndef UBT780CTRL
> +#define UBT780CTRL
> +
> +/*IOCTRL codes*/
> +/** Battary status IOCTL defines */
> +#define GET_BATTERY_STATUS		0x01 /*get battery status, return BATTERY_STATUS_UKN, BATTERY_STATUS_FINE, BATTERY_STATUS_WEAK*/
> +#define GET_LAST_PACKET_OLD		0x02 /*get last packet (ubt780_dgtzr). It is necessary for calibration*/
> +
> +/** Modes of work IOCTL */
> +#define GET_MODE			0x10 /*get current mode, return MODE_MOUSE, MODE_DGTZR*/
> +#define SET_MODE			0x11 /*switch to mouse mode, MODE_MOUSE, MODE_DGTZR*/
> +#define SET_CALIB			0x20 /*set calibration mode*/
> +#define GET_LAST_PACKET			0x21 /*get last packet (ubt780_dgtzr). It is necessary for calibration*/
> +#define GET_DEVICE_ID			0x22 /*get device id to recognize the device version*/
> +#define TEST_MT_PACKET			0x23
> +#define TEST_CALIBRATION		0x24
> +/*return values for ioctrl*/
> +/** Battary status defines */
> +#define BATTERY_STATUS_FINE		0x00 /*battery is fine*/
> +#define BATTERY_STATUS_WEAK		0x01 /*battery is weak*/
> +#define BATTERY_STATUS_UKN		0x02 /*unknown status. Status will be known after the first pen touch*/
> +
> +/** Mode status defines */
> +#define MODE_MOUSE			0x10 /*current mode is mouse*/
> +#define MODE_DGTZR			0x11 /*current mode is digitizer*/
> +#define MODE_UKN			0x12 /*current mode is unknown. It will be known after the first pen touch*/
> +#define MODE_SINGLETOUCH		0x13
> +
> +/*Char device declarations*/
> +#define UBT780_MAX_DEVICES		0x05
> +#define UBT780_FIRST_MINOR		0
> +
> +#define UBT780_MAX_AXIS_X	4095
> +#define UBT780_MAX_AXIS_Y	4095
> +#define UBT880_MAX_AXIS_X	32767
> +#define UBT880_MAX_AXIS_Y	32767
> +#define NUM_CONTACTS		0x03
> +/** Digitizer mode stucture: contains packet data */
> +struct ubt880_dgtzr {
> +	/** Report contains the type of packet: 0 - mouse, 1 - digitize */
> +	unsigned char report;
> +	/** Command is coming from device */
> +	unsigned char command;
> +	/** Packet size */
> +	unsigned short data[2];
> +};
> +
> +struct ubt780_dgtzr {
> +	/** Report contains the type of packet: 0 - mouse, 1 - digitize */
> +	unsigned char report;
> +	/** Command is coming from device */
> +	unsigned char command;
> +	/** Packet size */
> +	unsigned char number;
> +	/** Data part of packet */
> +	unsigned char data[17];
> +};
> +/** Calibration stucture: contains calibration data */
> +struct ubt_calib {
> +	/** Screen coordinates: Left Top X */
> +	int LTx;
> +	/** Screen coordinates: Left Top Y */
> +	int LTy;
> +	/** Screen coordinates: Left Bottom X */
> +	int LBx;
> +	/** Screen coordinates: Left Bottom Y */
> +	int LBy;
> +	/** Screen coordinates: Right Top X */
> +	int RTx;
> +	/** Screen coordinates: Right Top Y */
> +	int RTy;
> +	/** Screen coordinates: Right Bottom X */
> +	int RBx;
> +	/** Screen coordinates: Right Bottom Y */
> +	int RBy;
> +
> +	/** Board coordinates: Left Top X */
> +	int LTX;
> +	/** Board coordinates: Left Top Y */
> +	int LTY;
> +	/** Board coordinates: Left Bottom X */
> +	int LBX;
> +	/** Board coordinates: Left Bottom Y */
> +	int LBY;
> +	/** Board coordinates: Right Top X */
> +	int RTX;
> +	/** Board coordinates: Right Top Y */
> +	int RTY;
> +	/** Board coordinates: Right Bottom X */
> +	int RBX;
> +	/** Board coordinates: Right Bottom Y */
> +	int RBY;
> +
> +	/** Calculated values */
> +	int Xoff_A;
> +	int Xmag_A;
> +	int Xmag_B;
> +	int Yoff_A;
> +	int Ymag_A;
> +	int Ymag_B;
> +	int calibrated;
> +};
> +/*Character device data structure*/
> +struct ubt_chrdev {
> +	unsigned int minor;
> +	int exist;
> +	int open;
> +	struct hid_device *hid;
> +	struct device *dev;
> +};
> +/*Driver data structure*/
> +struct ubt780_data {
> +	struct ubt_calib calib;
> +	struct ubt780_dgtzr ubt_packet;
> +	struct ubt_chrdev *chrdev;
> +	unsigned char current_mode;
> +};
> +
> +struct ubt880_data {
> +	struct ubt_calib calib;
> +	struct ubt880_dgtzr ubt_packet;
> +	struct ubt_chrdev *chrdev;
> +	unsigned char current_mode;
> +	int (*switch_mode) (struct hid_device *hid, unsigned char mode);
> +	void (*set_input) (struct input_dev *input);
> +};
> +/*ubt880 multitouch structures*/
> +struct ubt_mt_contact {
> +	unsigned char flags;
> +	unsigned char id;
> +	unsigned short x;
> +	unsigned short y;
> +};
> +struct ubt_mt_packet {
> +	unsigned char data[20];
> +};
> +#endif /* UBT780CTRL */
> -- 
> 1.7.1
> 

Thanks,
Henrik
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ