Re: [PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

Re: [PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 662 bytes --]

Hi Tony,

On Fri, Mar 27, 2015 at 03:39:46PM +0100, Sebastian Reichel wrote:
> This adds support for the N900's accelerometer to
> the Nokia N900 DTS file.
> 
> Signed-off-by: Sebastian Reichel <sre@kernel.org>
> Acked-by: Tony Lindgren <tony@atomide.com>
> ---
> Quote from Tony:
> 
> This at least currently does not conflict with anything I have
> queued, so I suggest you try to get Greg to take the whole set:
> 
> http://article.gmane.org/gmane.linux.ports.arm.omap/125020

It seems Greg did not see this message (this patch did not make it
into 4.1-rc and the driver changes did), so could you queue this
patch for 4.2?

-- Sebastian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

[Tony Lindgren]

* Sebastian Reichel <sre@kernel.org> [150514 15:41]:
> Hi Tony,
> 
> On Fri, Mar 27, 2015 at 03:39:46PM +0100, Sebastian Reichel wrote:
> > This adds support for the N900's accelerometer to
> > the Nokia N900 DTS file.
> > 
> > Signed-off-by: Sebastian Reichel <sre@kernel.org>
> > Acked-by: Tony Lindgren <tony@atomide.com>
> > ---
> > Quote from Tony:
> > 
> > This at least currently does not conflict with anything I have
> > queued, so I suggest you try to get Greg to take the whole set:
> > 
> > http://article.gmane.org/gmane.linux.ports.arm.omap/125020
> 
> It seems Greg did not see this message (this patch did not make it
> into 4.1-rc and the driver changes did), so could you queue this
> patch for 4.2?

Hmm seems to be there as commit 1ac4e6fee41d6.

Tony

Re: [PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 1036 bytes --]

Hi,

On Wed, May 20, 2015 at 10:21:22AM -0700, Tony Lindgren wrote:
> * Sebastian Reichel <sre@kernel.org> [150514 15:41]:
> > On Fri, Mar 27, 2015 at 03:39:46PM +0100, Sebastian Reichel wrote:
> > > This adds support for the N900's accelerometer to
> > > the Nokia N900 DTS file.
> > > 
> > > Signed-off-by: Sebastian Reichel <sre@kernel.org>
> > > Acked-by: Tony Lindgren <tony@atomide.com>
> > > ---
> > > Quote from Tony:
> > > 
> > > This at least currently does not conflict with anything I have
> > > queued, so I suggest you try to get Greg to take the whole set:
> > > 
> > > http://article.gmane.org/gmane.linux.ports.arm.omap/125020
> > 
> > It seems Greg did not see this message (this patch did not make it
> > into 4.1-rc and the driver changes did), so could you queue this
> > patch for 4.2?
> 
> Hmm seems to be there as commit 1ac4e6fee41d6.

uhm right, my mistake. No clue how I missed it (especially since it's
the last change in the n900 dts file).

Sorry for the noise.

-- Sebastian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[PATCHv2 0/4] lis3lv02d: update DT binding for use with Nokia N900

[Sebastian Reichel]

Hi,

The lis302 has already a DT binding described in [0],
which descibes misc. hardware properties. The current
binding does not support all values needed to convert
the Nokia N900's platform data to DT, though.

This patchset introduces support for describing inverted
axis, configuration of second wakeup unit and wakeup
threshold support.

The series is based on top of v4.0-rc1 and has been
tested on my Nokia N900.

[0] Documentation/devicetree/bindings/misc/lis302.txt

-- Sebastian

Sebastian Reichel (4):
  lis3lv02d: DT: use s32 to support negative values
  lis3lv02d: DT: add wakeup unit 2 and wakeup threshold
  Documentation: DT: lis302: update wakeup binding
  DTS: ARM: OMAP3-N900: Add lis3lv02d support

 Documentation/devicetree/bindings/misc/lis302.txt |  9 +++-
 arch/arm/boot/dts/omap3-n900.dts                  | 52 +++++++++++++++++++++
 drivers/misc/lis3lv02d/lis3lv02d.c                | 56 +++++++++++++++--------
 3 files changed, 97 insertions(+), 20 deletions(-)

-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

[Sebastian Reichel]

From: Sebastian Reichel <sre@kernel.or>

This adds support for the N900's accelerometer to
the Nokia N900 DTS file.

Signed-off-by: Sebastian Reichel <sre@kernel.org>
---
 arch/arm/boot/dts/omap3-n900.dts | 52 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 52 insertions(+)

diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index 6040327..b02a717 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -602,6 +602,58 @@
 	pinctrl-0 = <&i2c3_pins>;
 
 	clock-frequency = <400000>;
+
+	lis302dl: lis3lv02d@1d {
+		compatible = "st,lis3lv02d";
+		reg = <0x1d>;
+
+		Vdd-supply = <&vaux1>;
+		Vdd_IO-supply = <&vio>;
+
+		interrupt-parent = <&gpio6>;
+		interrupts = <21 20>; /* 181 and 180 */
+
+		/* click flags */
+		st,click-single-x;
+		st,click-single-y;
+		st,click-single-z;
+
+		/* Limits are 0.5g * value */
+		st,click-threshold-x = <8>;
+		st,click-threshold-y = <8>;
+		st,click-threshold-z = <10>;
+
+		/* Click must be longer than time limit */
+		st,click-time-limit = <9>;
+
+		/* Kind of debounce filter */
+		st,click-latency = <50>;
+
+		/* Interrupt line 2 for click detection */
+		st,irq2-click;
+
+		st,wakeup-x-hi;
+		st,wakeup-y-hi;
+		st,wakeup-threshold = <(800/18)>; /* millig-value / 18 to get HW values */
+
+		st,wakeup2-z-hi;
+		st,wakeup2-threshold = <(900/18)>; /* millig-value / 18 to get HW values */
+
+		st,hipass1-disable;
+		st,hipass2-disable;
+
+		st,axis-x = <1>;    /* LIS3_DEV_X */
+		st,axis-y = <(-2)>; /* LIS3_INV_DEV_Y */
+		st,axis-z = <(-3)>; /* LIS3_INV_DEV_Z */
+
+		st,min-limit-x = <(-32)>;
+		st,min-limit-y = <3>;
+		st,min-limit-z = <3>;
+
+		st,max-limit-x = <(-3)>;
+		st,max-limit-y = <32>;
+		st,max-limit-z = <32>;
+	};
 };
 
 &mmc1 {
-- 
2.1.4

Re: [PATCHv2 4/4] DTS: ARM: OMAP3-N900: Add lis3lv02d support

[Tony Lindgren]

* Sebastian Reichel <sre@kernel.org> [150321 13:20]:
> From: Sebastian Reichel <sre@kernel.or>
> 
> This adds support for the N900's accelerometer to
> the Nokia N900 DTS file.
> 
> Signed-off-by: Sebastian Reichel <sre@kernel.org>

This at least currently does not conflict with anything I have
queued, so I suggest you try to get Greg to take the whole set:

Acked-by: Tony Lindgren <tony@atomide.com>

> ---
>  arch/arm/boot/dts/omap3-n900.dts | 52 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 52 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
> index 6040327..b02a717 100644
> --- a/arch/arm/boot/dts/omap3-n900.dts
> +++ b/arch/arm/boot/dts/omap3-n900.dts
> @@ -602,6 +602,58 @@
>  	pinctrl-0 = <&i2c3_pins>;
>  
>  	clock-frequency = <400000>;
> +
> +	lis302dl: lis3lv02d@1d {
> +		compatible = "st,lis3lv02d";
> +		reg = <0x1d>;
> +
> +		Vdd-supply = <&vaux1>;
> +		Vdd_IO-supply = <&vio>;
> +
> +		interrupt-parent = <&gpio6>;
> +		interrupts = <21 20>; /* 181 and 180 */
> +
> +		/* click flags */
> +		st,click-single-x;
> +		st,click-single-y;
> +		st,click-single-z;
> +
> +		/* Limits are 0.5g * value */
> +		st,click-threshold-x = <8>;
> +		st,click-threshold-y = <8>;
> +		st,click-threshold-z = <10>;
> +
> +		/* Click must be longer than time limit */
> +		st,click-time-limit = <9>;
> +
> +		/* Kind of debounce filter */
> +		st,click-latency = <50>;
> +
> +		/* Interrupt line 2 for click detection */
> +		st,irq2-click;
> +
> +		st,wakeup-x-hi;
> +		st,wakeup-y-hi;
> +		st,wakeup-threshold = <(800/18)>; /* millig-value / 18 to get HW values */
> +
> +		st,wakeup2-z-hi;
> +		st,wakeup2-threshold = <(900/18)>; /* millig-value / 18 to get HW values */
> +
> +		st,hipass1-disable;
> +		st,hipass2-disable;
> +
> +		st,axis-x = <1>;    /* LIS3_DEV_X */
> +		st,axis-y = <(-2)>; /* LIS3_INV_DEV_Y */
> +		st,axis-z = <(-3)>; /* LIS3_INV_DEV_Z */
> +
> +		st,min-limit-x = <(-32)>;
> +		st,min-limit-y = <3>;
> +		st,min-limit-z = <3>;
> +
> +		st,max-limit-x = <(-3)>;
> +		st,max-limit-y = <32>;
> +		st,max-limit-z = <32>;
> +	};
>  };
>  
>  &mmc1 {
> -- 
> 2.1.4
>