Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Tue, 20 May 2014 20:10:28 +0200, Pali Rohár said: > > Hmm.. what tree are you building against? I wonder if your > > checkpatch is a different version than mine (next-20140519). > > I'm using up-to-date linus tree, git rev > 60b5f90d0fac7585f1a43ccdad06787b97eda0ab Ah, OK. Linux-next tree

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Pali Rohár
On Tuesday 20 May 2014 20:02:41 valdis.kletni...@vt.edu wrote: > On Tue, 20 May 2014 10:56:47 +0200, Pali Rohár said: > > Hm? Which errors? Here is output from checkpacth: > > > > total: 0 errors, 0 warnings, 276 lines checked > > > > 0001-platform-x86-dell-smo8800-Dell-Latitude-freefall-dri.pa

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Tue, 20 May 2014 10:56:47 +0200, Pali Rohár said: > Hm? Which errors? Here is output from checkpacth: > > total: 0 errors, 0 warnings, 276 lines checked > > 0001-platform-x86-dell-smo8800-Dell-Latitude-freefall-dri.patch > has no obvious style problems and is ready for submission. > > total: 0

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Pali Rohár
On Tuesday 20 May 2014 09:00:05 valdis.kletni...@vt.edu wrote: > On Sat, 03 May 2014 12:47:56 +0200, Pali Rohár said: > > This acpi driver provide supports for freefall sensors > > SMO8800/SMO8810 which can be found on Dell Latitude > > laptops. Driver register /dev/freefall misc device which > >

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Sat, 03 May 2014 12:47:56 +0200, Pali Rohár said: > This acpi driver provide supports for freefall sensors SMO8800/SMO8810 which > can be found on Dell Latitude laptops. Driver register /dev/freefall misc > device which has same interface as driver hp_accel freefall driver. So any > existing

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Sat, 03 May 2014 12:47:56 +0200, Pali Rohár said: This acpi driver provide supports for freefall sensors SMO8800/SMO8810 which can be found on Dell Latitude laptops. Driver register /dev/freefall misc device which has same interface as driver hp_accel freefall driver. So any existing

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Pali Rohár
On Tuesday 20 May 2014 09:00:05 valdis.kletni...@vt.edu wrote: On Sat, 03 May 2014 12:47:56 +0200, Pali Rohár said: This acpi driver provide supports for freefall sensors SMO8800/SMO8810 which can be found on Dell Latitude laptops. Driver register /dev/freefall misc device which has same

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Tue, 20 May 2014 10:56:47 +0200, Pali Rohár said: Hm? Which errors? Here is output from checkpacth: total: 0 errors, 0 warnings, 276 lines checked 0001-platform-x86-dell-smo8800-Dell-Latitude-freefall-dri.patch has no obvious style problems and is ready for submission. total: 0 errors,

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Pali Rohár
On Tuesday 20 May 2014 20:02:41 valdis.kletni...@vt.edu wrote: On Tue, 20 May 2014 10:56:47 +0200, Pali Rohár said: Hm? Which errors? Here is output from checkpacth: total: 0 errors, 0 warnings, 276 lines checked 0001-platform-x86-dell-smo8800-Dell-Latitude-freefall-dri.pa tch has

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-20 Thread Valdis . Kletnieks
On Tue, 20 May 2014 20:10:28 +0200, Pali Rohár said: Hmm.. what tree are you building against? I wonder if your checkpatch is a different version than mine (next-20140519). I'm using up-to-date linus tree, git rev 60b5f90d0fac7585f1a43ccdad06787b97eda0ab Ah, OK. Linux-next tree has a

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-07 Thread One Thousand Gnomes
> +static irqreturn_t smo8800_interrupt_quick(int irq, void *data) > +{ > + struct smo8800_device *smo8800 = data; > + atomic_inc(>count); > + wake_up_interruptible(>misc_wait); > + return IRQ_WAKE_THREAD; > +} > + > +static irqreturn_t smo8800_interrupt_thread(int irq, void *data)

Re: [PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-07 Thread One Thousand Gnomes
+static irqreturn_t smo8800_interrupt_quick(int irq, void *data) +{ + struct smo8800_device *smo8800 = data; + atomic_inc(smo8800-count); + wake_up_interruptible(smo8800-misc_wait); + return IRQ_WAKE_THREAD; +} + +static irqreturn_t smo8800_interrupt_thread(int irq, void

[PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-03 Thread Pali Rohár
This acpi driver provide supports for freefall sensors SMO8800/SMO8810 which can be found on Dell Latitude laptops. Driver register /dev/freefall misc device which has same interface as driver hp_accel freefall driver. So any existing applications for HP freefall sensor /dev/freefall will work for

[PATCH 1/2] platform: x86: dell-smo8800: Dell Latitude freefall driver (ACPI SMO8800/SMO8810)

2014-05-03 Thread Pali Rohár
This acpi driver provide supports for freefall sensors SMO8800/SMO8810 which can be found on Dell Latitude laptops. Driver register /dev/freefall misc device which has same interface as driver hp_accel freefall driver. So any existing applications for HP freefall sensor /dev/freefall will work for