Input: gpio-beeper - simplify GPIO handling
authorAlexander Shiyan <shc_work@mail.ru>
Sat, 26 Apr 2014 06:45:05 +0000 (23:45 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Wed, 14 May 2014 23:39:51 +0000 (16:39 -0700)
This patch simplifies GPIO handling in the driver by using GPIO functions
based on descriptors. As a result this driver now can be used for boards
without DT support.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/misc/Kconfig
drivers/input/misc/gpio-beeper.c

index f772981bdcdbef5d46d14461eb2cc04fd223fae6..05ae7e7cdddebe2febe6a2e9cb5a22dd337030a8 100644 (file)
@@ -224,7 +224,7 @@ config INPUT_GP2A
 
 config INPUT_GPIO_BEEPER
        tristate "Generic GPIO Beeper support"
-       depends on OF_GPIO
+       depends on GPIOLIB
        help
          Say Y here if you have a beeper connected to a GPIO pin.
 
index b757435e2b3d0eda193f5d1265bd4a730ddcf255..5ed50ddfacfc21321c7dd39988032153a1c8b6cc 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Generic GPIO beeper driver
  *
- * Copyright (C) 2013 Alexander Shiyan <shc_work@mail.ru>
+ * Copyright (C) 2013-2014 Alexander Shiyan <shc_work@mail.ru>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -11,7 +11,8 @@
 
 #include <linux/input.h>
 #include <linux/module.h>
-#include <linux/of_gpio.h>
+#include <linux/gpio/consumer.h>
+#include <linux/of.h>
 #include <linux/workqueue.h>
 #include <linux/platform_device.h>
 
 
 struct gpio_beeper {
        struct work_struct      work;
-       int                     gpio;
-       bool                    active_low;
+       struct gpio_desc        *desc;
        bool                    beeping;
 };
 
 static void gpio_beeper_toggle(struct gpio_beeper *beep, bool on)
 {
-       gpio_set_value_cansleep(beep->gpio, on ^ beep->active_low);
+       gpiod_set_value_cansleep(beep->desc, on);
 }
 
 static void gpio_beeper_work(struct work_struct *work)
@@ -65,18 +65,16 @@ static void gpio_beeper_close(struct input_dev *input)
 static int gpio_beeper_probe(struct platform_device *pdev)
 {
        struct gpio_beeper *beep;
-       enum of_gpio_flags flags;
        struct input_dev *input;
-       unsigned long gflags;
        int err;
 
        beep = devm_kzalloc(&pdev->dev, sizeof(*beep), GFP_KERNEL);
        if (!beep)
                return -ENOMEM;
 
-       beep->gpio = of_get_gpio_flags(pdev->dev.of_node, 0, &flags);
-       if (!gpio_is_valid(beep->gpio))
-               return beep->gpio;
+       beep->desc = devm_gpiod_get(&pdev->dev, NULL);
+       if (IS_ERR(beep->desc))
+               return PTR_ERR(beep->desc);
 
        input = devm_input_allocate_device(&pdev->dev);
        if (!input)
@@ -94,10 +92,7 @@ static int gpio_beeper_probe(struct platform_device *pdev)
 
        input_set_capability(input, EV_SND, SND_BELL);
 
-       beep->active_low = flags & OF_GPIO_ACTIVE_LOW;
-       gflags = beep->active_low ? GPIOF_OUT_INIT_HIGH : GPIOF_OUT_INIT_LOW;
-
-       err = devm_gpio_request_one(&pdev->dev, beep->gpio, gflags, pdev->name);
+       err = gpiod_direction_output(beep->desc, 0);
        if (err)
                return err;
 
@@ -106,17 +101,19 @@ static int gpio_beeper_probe(struct platform_device *pdev)
        return input_register_device(input);
 }
 
+#ifdef CONFIG_OF
 static struct of_device_id gpio_beeper_of_match[] = {
        { .compatible = BEEPER_MODNAME, },
        { }
 };
 MODULE_DEVICE_TABLE(of, gpio_beeper_of_match);
+#endif
 
 static struct platform_driver gpio_beeper_platform_driver = {
        .driver = {
                .name           = BEEPER_MODNAME,
                .owner          = THIS_MODULE,
-               .of_match_table = gpio_beeper_of_match,
+               .of_match_table = of_match_ptr(gpio_beeper_of_match),
        },
        .probe  = gpio_beeper_probe,
 };