Hi David,
On Mon, 18 Aug 2008 00:24:32 -0700, David Brownell wrote:
> From: Marek Vasut <[EMAIL PROTECTED]>
>
> All the tps6501{0,1,2,3,4} chips have a signal for hooking up with
> a vibrator (for non-auditory cell phone "ring") ... expose that as
> one more (output-only) GPIO.
>
> [ [EMAIL PROTECTED]: comments; list tps65014 too ]
>
> Signed-off-by: David Brownell <[EMAIL PROTECTED]>
> ---
> drivers/i2c/chips/tps65010.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> --- a/drivers/i2c/chips/tps65010.c 2008-07-25 12:28:08.000000000 -0700
> +++ b/drivers/i2c/chips/tps65010.c 2008-08-18 00:22:48.000000000 -0700
> @@ -456,14 +456,17 @@ static irqreturn_t tps65010_irq(int irq,
>
> /* offsets 0..3 == GPIO1..GPIO4
> * offsets 4..5 == LED1/nPG, LED2 (we set one of the non-BLINK modes)
> + * offset 6 == vibrator motor driver
> */
> static void
> tps65010_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
> {
> if (offset < 4)
> tps65010_set_gpio_out_value(offset + 1, value);
> - else
> + else if (offset < 6)
> tps65010_set_led(offset - 3, value ? ON : OFF);
> + else
> + tps65010_set_vib(value);
> }
>
> static int
> @@ -477,8 +480,10 @@ tps65010_output(struct gpio_chip *chip,
> if (!(tps->outmask & (1 << offset)))
> return -EINVAL;
> tps65010_set_gpio_out_value(offset + 1, value);
> - } else
> + } else if (offset < 6)
> tps65010_set_led(offset - 3, value ? ON : OFF);
> + else
> + tps65010_set_vib(value);
>
> return 0;
> }
> @@ -646,7 +651,7 @@ static int tps65010_probe(struct i2c_cli
> tps->chip.get = tps65010_gpio_get;
>
> tps->chip.base = board->base;
> - tps->chip.ngpio = 6;
> + tps->chip.ngpio = 7;
> tps->chip.can_sleep = 1;
>
> status = gpiochip_add(&tps->chip);
> @@ -675,6 +680,7 @@ static const struct i2c_device_id tps650
> { "tps65011", TPS65011 },
> { "tps65012", TPS65012 },
> { "tps65013", TPS65013 },
> + { "tps65014", TPS65011 }, /* tps65011 charging at 6.5V max */
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tps65010_id);
Applied, thanks. I'm a bit worried that Marek's Signed-off-by is
missing though.
--
Jean Delvare
_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c
