linux-kernel - Re: [PATCH 07/12] hwmon: lis3: New parameters to platform data

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: <4CC4423E.9010403@tremplin-utc.net>
Date:	Sun, 24 Oct 2010 16:27:10 +0200
From:	Éric Piel <eric.piel@...mplin-utc.net>
To:	Samu Onkalo <samu.p.onkalo@...ia.com>
CC:	khali@...ux-fr.org, guenter.roeck@...csson.com, jic23@....ac.uk,
	lm-sensors@...sensors.org, linux-i2c@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 07/12] hwmon: lis3: New parameters to platform data

Op 22-10-10 13:57, Samu Onkalo schreef:
> Added default output data rate setting to platform data.
> If default rate is 0, reset default value is used.
> Added control for duration via platform data.
> Added possibility to configure interrupts to trig on
> both rising and falling edge. The lis3 WU unit can be
> configured quite many ways and with some configurations it
> is quite handy to get coordinate refresh when some
> event trigs and when it reason goes away.
>
> Signed-off-by: Samu Onkalo<samu.p.onkalo@...ia.com>
Acked-by: Eric Piel <eric.piel@...mplin-utc.net>

> ---
>   drivers/hwmon/lis3lv02d.c |   21 ++++++++++++++-------
>   include/linux/lis3lv02d.h |    6 +++++-
>   2 files changed, 19 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/hwmon/lis3lv02d.c b/drivers/hwmon/lis3lv02d.c
> index b44d4c5..d66cbe1 100644
> --- a/drivers/hwmon/lis3lv02d.c
> +++ b/drivers/hwmon/lis3lv02d.c
> @@ -706,16 +706,16 @@ static void lis3lv02d_8b_configure(struct lis3lv02d *dev,
>   	if (p->wakeup_flags) {
>   		dev->write(dev, FF_WU_CFG_1, p->wakeup_flags);
>   		dev->write(dev, FF_WU_THS_1, p->wakeup_thresh&  0x7f);
> -		/* default to 2.5ms for now */
> -		dev->write(dev, FF_WU_DURATION_1, 1);
> +		/* pdata value + 1 to keep this backward compatible*/
> +		dev->write(dev, FF_WU_DURATION_1, p->duration1 + 1);
>   		ctrl2 ^= HP_FF_WU1; /* Xor to keep compatible with old pdata*/
>   	}
>
>   	if (p->wakeup_flags2) {
>   		dev->write(dev, FF_WU_CFG_2, p->wakeup_flags2);
>   		dev->write(dev, FF_WU_THS_2, p->wakeup_thresh2&  0x7f);
> -		/* default to 2.5ms for now */
> -		dev->write(dev, FF_WU_DURATION_2, 1);
> +		/* pdata value + 1 to keep this backward compatible*/
> +		dev->write(dev, FF_WU_DURATION_2, p->duration2 + 1);
>   		ctrl2 ^= HP_FF_WU2; /* Xor to keep compatible with old pdata*/
>   	}
>   	/* Configure hipass filters */
> @@ -725,8 +725,8 @@ static void lis3lv02d_8b_configure(struct lis3lv02d *dev,
>   		err = request_threaded_irq(p->irq2,
>   					NULL,
>   					lis302dl_interrupt_thread2_8b,
> -					IRQF_TRIGGER_RISING |
> -					IRQF_ONESHOT,
> +					IRQF_TRIGGER_RISING | IRQF_ONESHOT |
> +					(p->irq_flags2&  IRQF_TRIGGER_MASK),
>   					DRIVER_NAME,&lis3_dev);
>   		if (err<  0)
>   			printk(KERN_ERR DRIVER_NAME
> @@ -742,6 +742,7 @@ int lis3lv02d_init_device(struct lis3lv02d *dev)
>   {
>   	int err;
>   	irq_handler_t thread_fn;
> +	int irq_flags = 0;
>
>   	dev->whoami = lis3lv02d_read_8(dev, WHO_AM_I);
>
> @@ -804,9 +805,14 @@ int lis3lv02d_init_device(struct lis3lv02d *dev)
>   		if (dev->whoami == WAI_8B)
>   			lis3lv02d_8b_configure(dev, p);
>
> +		irq_flags = p->irq_flags1&  IRQF_TRIGGER_MASK;
> +
>   		dev->irq_cfg = p->irq_cfg;
>   		if (p->irq_cfg)
>   			dev->write(dev, CTRL_REG3, p->irq_cfg);
> +
> +		if (p->default_rate)
> +			lis3lv02d_set_odr(p->default_rate);
>   	}
>
>   	/* bail if we did not get an IRQ from the bus layer */
> @@ -834,7 +840,8 @@ int lis3lv02d_init_device(struct lis3lv02d *dev)
>
>   	err = request_threaded_irq(dev->irq, lis302dl_interrupt,
>   				thread_fn,
> -				IRQF_TRIGGER_RISING | IRQF_ONESHOT,
> +				IRQF_TRIGGER_RISING | IRQF_ONESHOT |
> +				irq_flags,
>   				DRIVER_NAME,&lis3_dev);
>
>   	if (err<  0) {
> diff --git a/include/linux/lis3lv02d.h b/include/linux/lis3lv02d.h
> index c4a4a52..18d578f 100644
> --- a/include/linux/lis3lv02d.h
> +++ b/include/linux/lis3lv02d.h
> @@ -36,7 +36,10 @@ struct lis3lv02d_platform_data {
>   #define LIS3_IRQ_OPEN_DRAIN	(1<<  6)
>   #define LIS3_IRQ_ACTIVE_LOW	(1<<  7)
>   	unsigned char irq_cfg;
> -
> +	unsigned char irq_flags1; /* Additional irq edge / level flags */
> +	unsigned char irq_flags2; /* Additional irq edge / level flags */
> +	unsigned char duration1;
> +	unsigned char duration2;
>   #define LIS3_WAKEUP_X_LO	(1<<  0)
>   #define LIS3_WAKEUP_X_HI	(1<<  1)
>   #define LIS3_WAKEUP_Y_LO	(1<<  2)
> @@ -66,6 +69,7 @@ struct lis3lv02d_platform_data {
>   	s8 axis_z;
>   #define LIS3_USE_REGULATOR_CTRL 0x01
>   	u16 driver_features;
> +	int default_rate;
>   	int (*setup_resources)(void);
>   	int (*release_resources)(void);
>   	/* Limits for selftest are specified in chip data sheet */

--
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/