Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-24 Thread Yan Burman
Arjan van de Ven wrote: Hi, your code looks very nice and clean, only few comments, see below Thanks +static int mdps_joystick_kthread(void *data) +{ + int x = 0, y = 0, z = 0; + + while (!kthread_should_stop()) { + if (input_3d) { +

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-24 Thread Yan Burman
Arjan van de Ven wrote: Hi, your code looks very nice and clean, only few comments, see below Thanks +static int mdps_joystick_kthread(void *data) +{ + int x = 0, y = 0, z = 0; + + while (!kthread_should_stop()) { + if (input_3d) { +

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Arjan van de Ven
Hi, your code looks very nice and clean, only few comments, see below > +static int mdps_joystick_kthread(void *data) > +{ > + int x = 0, y = 0, z = 0; > + > + while (!kthread_should_stop()) { > + if (input_3d) { > + mdps_get_xyz(mdps.device->handle, , ,

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Yan Burman
Dmitry Torokhov wrote: Hi Yan, On 3/23/07, Yan Burman <[EMAIL PROTECTED]> wrote: + +static unsigned int input_3d; +module_param(input_3d, bool, S_IRUGO); +MODULE_PARM_DESC(input_3d, "Operate as a 3D joystick instead of 2D"); Why do you need that? Just have the driver always report all 3

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Dmitry Torokhov
Hi Yan, On 3/23/07, Yan Burman <[EMAIL PROTECTED]> wrote: + +static unsigned int input_3d; +module_param(input_3d, bool, S_IRUGO); +MODULE_PARM_DESC(input_3d, "Operate as a 3D joystick instead of 2D"); Why do you need that? Just have the driver always report all 3 events and have applications

[PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Yan Burman
HP Mobile Data Protection System 3D ACPI driver. Similar to hdaps in functionality. This driver provides 4 kinds of functionality: 1) Creates a misc device /dev/accel that acts similar to /dev/rtc and unblocks the process reading from it when the device detects free-fall interrupt 2) Functions as

[PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Yan Burman
HP Mobile Data Protection System 3D ACPI driver. Similar to hdaps in functionality. This driver provides 4 kinds of functionality: 1) Creates a misc device /dev/accel that acts similar to /dev/rtc and unblocks the process reading from it when the device detects free-fall interrupt 2) Functions as

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Dmitry Torokhov
Hi Yan, On 3/23/07, Yan Burman [EMAIL PROTECTED] wrote: + +static unsigned int input_3d; +module_param(input_3d, bool, S_IRUGO); +MODULE_PARM_DESC(input_3d, Operate as a 3D joystick instead of 2D); Why do you need that? Just have the driver always report all 3 events and have applications

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Yan Burman
Dmitry Torokhov wrote: Hi Yan, On 3/23/07, Yan Burman [EMAIL PROTECTED] wrote: + +static unsigned int input_3d; +module_param(input_3d, bool, S_IRUGO); +MODULE_PARM_DESC(input_3d, Operate as a 3D joystick instead of 2D); Why do you need that? Just have the driver always report all 3 events

Re: [PATCH 2.6.21-rc4] hwmon: HP Mobile Data Protection System 3D ACPI driver

2007-03-23 Thread Arjan van de Ven
Hi, your code looks very nice and clean, only few comments, see below +static int mdps_joystick_kthread(void *data) +{ + int x = 0, y = 0, z = 0; + + while (!kthread_should_stop()) { + if (input_3d) { + mdps_get_xyz(mdps.device-handle, x, y, z); +