gpio-button-hotplug: fix 6.2 build failure
[openwrt/staging/stintel.git] / package / kernel / gpio-button-hotplug / src / gpio-button-hotplug.c
index e63d41428496bc230cc6e623736fbe06ed15324e..e36494c2fe20a23b307bdd85ad499da849e0b622 100644 (file)
 #define DRV_NAME       "gpio-keys"
 #define PFX    DRV_NAME ": "
 
-struct bh_priv {
-       unsigned long           seen;
-};
-
 struct bh_event {
        const char              *name;
        unsigned int            type;
@@ -56,7 +52,8 @@ struct bh_map {
 
 struct gpio_keys_button_data {
        struct delayed_work work;
-       struct bh_priv bh;
+       unsigned long seen;
+       int map_entry;
        int last_state;
        int count;
        int threshold;
@@ -98,6 +95,8 @@ static struct bh_map button_map[] = {
        BH_MAP(KEY_RESTART,             "reset"),
        BH_MAP(KEY_RFKILL,              "rfkill"),
        BH_MAP(KEY_VIDEO,               "video"),
+       BH_MAP(KEY_VOLUMEDOWN,          "volume_down"),
+       BH_MAP(KEY_VOLUMEUP,            "volume_up"),
        BH_MAP(KEY_WIMAX,               "wwan"),
        BH_MAP(KEY_WLAN,                "wlan"),
        BH_MAP(KEY_WPS_BUTTON,          "wps"),
@@ -108,7 +107,7 @@ static struct bh_map button_map[] = {
 static __printf(3, 4)
 int bh_event_add_var(struct bh_event *event, int argv, const char *format, ...)
 {
-       static char buf[128];
+       char buf[128];
        char *s;
        va_list args;
        int len;
@@ -238,72 +237,73 @@ static int button_get_index(unsigned int code)
        return -1;
 }
 
-static void button_hotplug_event(struct gpio_keys_button_data *data,
-                          unsigned int type, int value)
-{
-       struct bh_priv *priv = &data->bh;
-       unsigned long seen = jiffies;
-       int btn;
-
-       pr_debug(PFX "event type=%u, code=%u, value=%d\n", type, data->b->code, value);
-
-       if ((type != EV_KEY) && (type != EV_SW))
-               return;
-
-       btn = button_get_index(data->b->code);
-       if (btn < 0)
-               return;
-
-       if (priv->seen == 0)
-               priv->seen = seen;
-
-       button_hotplug_create_event(button_map[btn].name, type,
-                       (seen - priv->seen) / HZ, value);
-       priv->seen = seen;
-}
-
-struct gpio_keys_button_dev {
-       int polled;
-       struct delayed_work work;
-
-       struct device *dev;
-       struct gpio_keys_platform_data *pdata;
-       struct gpio_keys_button_data data[0];
-};
-
 static int gpio_button_get_value(struct gpio_keys_button_data *bdata)
 {
        int val;
 
        if (bdata->can_sleep)
-               val = !!gpio_get_value_cansleep(bdata->b->gpio);
+               val = !!gpiod_get_value_cansleep(bdata->gpiod);
        else
-               val = !!gpio_get_value(bdata->b->gpio);
+               val = !!gpiod_get_value(bdata->gpiod);
 
-       return val ^ bdata->b->active_low;
+       return val;
 }
 
-static void gpio_keys_polled_check_state(struct gpio_keys_button_data *bdata)
+static void gpio_keys_handle_button(struct gpio_keys_button_data *bdata)
 {
+       unsigned int type = bdata->b->type ?: EV_KEY;
        int state = gpio_button_get_value(bdata);
+       unsigned long seen = jiffies;
 
-       if (state != bdata->last_state) {
-               unsigned int type = bdata->b->type ?: EV_KEY;
+       pr_debug(PFX "event type=%u, code=%u, pressed=%d\n",
+                type, bdata->b->code, state);
+
+       /* is this the initialization state? */
+       if (bdata->last_state == -1) {
+               /*
+                * Don't advertise unpressed buttons on initialization.
+                * Just save their state and continue otherwise this
+                * can cause OpenWrt to enter failsafe.
+                */
+               if (type == EV_KEY && state == 0)
+                       goto set_state;
+               /*
+                * But we are very interested in pressed buttons and
+                * initial switch state. These will be reported to
+                * userland.
+                */
+       } else if (bdata->last_state == state) {
+               /* reset asserted counter (only relevant for polled keys) */
+               bdata->count = 0;
+               return;
+       }
 
-               if (bdata->count < bdata->threshold) {
-                       bdata->count++;
-                       return;
-               }
+       if (bdata->count < bdata->threshold) {
+               bdata->count++;
+               return;
+       }
 
-               if (bdata->last_state != -1 || type == EV_SW)
-                       button_hotplug_event(bdata, type, state);
+       if (bdata->seen == 0)
+               bdata->seen = seen;
 
-               bdata->last_state = state;
-       }
+       button_hotplug_create_event(button_map[bdata->map_entry].name, type,
+                                   (seen - bdata->seen) / HZ, state);
+       bdata->seen = seen;
 
+set_state:
+       bdata->last_state = state;
        bdata->count = 0;
 }
 
+struct gpio_keys_button_dev {
+       int polled;
+       struct delayed_work work;
+
+       struct device *dev;
+       struct gpio_keys_platform_data *pdata;
+       struct gpio_keys_button_data data[0];
+};
+
 static void gpio_keys_polled_queue_work(struct gpio_keys_button_dev *bdev)
 {
        struct gpio_keys_platform_data *pdata = bdev->pdata;
@@ -322,7 +322,9 @@ static void gpio_keys_polled_poll(struct work_struct *work)
 
        for (i = 0; i < bdev->pdata->nbuttons; i++) {
                struct gpio_keys_button_data *bdata = &bdev->data[i];
-               gpio_keys_polled_check_state(bdata);
+
+               if (bdata->gpiod)
+                       gpio_keys_handle_button(bdata);
        }
        gpio_keys_polled_queue_work(bdev);
 }
@@ -342,8 +344,7 @@ static void gpio_keys_irq_work_func(struct work_struct *work)
        struct gpio_keys_button_data *bdata = container_of(work,
                struct gpio_keys_button_data, work.work);
 
-       button_hotplug_event(bdata, bdata->b->type ?: EV_KEY,
-                            gpio_button_get_value(bdata));
+       gpio_keys_handle_button(bdata);
 }
 
 static irqreturn_t button_handle_irq(int irq, void *_bdata)
@@ -351,8 +352,8 @@ static irqreturn_t button_handle_irq(int irq, void *_bdata)
        struct gpio_keys_button_data *bdata =
                (struct gpio_keys_button_data *) _bdata;
 
-       schedule_delayed_work(&bdata->work,
-                             msecs_to_jiffies(bdata->software_debounce));
+       mod_delayed_work(system_wq, &bdata->work,
+                        msecs_to_jiffies(bdata->software_debounce));
 
        return IRQ_HANDLED;
 }
@@ -364,7 +365,6 @@ gpio_keys_get_devtree_pdata(struct device *dev)
        struct device_node *node, *pp;
        struct gpio_keys_platform_data *pdata;
        struct gpio_keys_button *button;
-       int error;
        int nbuttons;
        int i = 0;
 
@@ -374,14 +374,12 @@ gpio_keys_get_devtree_pdata(struct device *dev)
 
        nbuttons = of_get_child_count(node);
        if (nbuttons == 0)
-               return NULL;
+               return ERR_PTR(-EINVAL);
 
        pdata = devm_kzalloc(dev, sizeof(*pdata) + nbuttons * (sizeof *button),
                GFP_KERNEL);
-       if (!pdata) {
-               error = -ENOMEM;
-               goto err_out;
-       }
+       if (!pdata)
+               return ERR_PTR(-ENOMEM);
 
        pdata->buttons = (struct gpio_keys_button *)(pdata + 1);
        pdata->nbuttons = nbuttons;
@@ -390,37 +388,13 @@ gpio_keys_get_devtree_pdata(struct device *dev)
        of_property_read_u32(node, "poll-interval", &pdata->poll_interval);
 
        for_each_child_of_node(node, pp) {
-               enum of_gpio_flags flags;
-
-               if (!of_find_property(pp, "gpios", NULL)) {
-                       pdata->nbuttons--;
-                       dev_warn(dev, "Found button without gpios\n");
-                       continue;
-               }
-
                button = (struct gpio_keys_button *)(&pdata->buttons[i++]);
 
-               button->irq = irq_of_parse_and_map(pp, 0);
-
-               button->gpio = of_get_gpio_flags(pp, 0, &flags);
-               if (button->gpio < 0) {
-                       error = button->gpio;
-                       if (error != -ENOENT) {
-                               if (error != -EPROBE_DEFER)
-                                       dev_err(dev,
-                                               "Failed to get gpio flags, error: %d\n",
-                                               error);
-                               return ERR_PTR(error);
-                       }
-               } else {
-                       button->active_low = flags & OF_GPIO_ACTIVE_LOW;
-               }
-
                if (of_property_read_u32(pp, "linux,code", &button->code)) {
-                       dev_err(dev, "Button without keycode: 0x%x\n",
-                               button->gpio);
-                       error = -EINVAL;
-                       goto err_out;
+                       dev_err(dev, "Button node '%s' without keycode\n",
+                               pp->full_name);
+                       of_node_put(pp);
+                       return ERR_PTR(-EINVAL);
                }
 
                button->desc = of_get_property(pp, "label", NULL);
@@ -433,17 +407,12 @@ gpio_keys_get_devtree_pdata(struct device *dev)
                if (of_property_read_u32(pp, "debounce-interval",
                                        &button->debounce_interval))
                        button->debounce_interval = 5;
-       }
 
-       if (pdata->nbuttons == 0) {
-               error = -EINVAL;
-               goto err_out;
+               button->irq = irq_of_parse_and_map(pp, 0);
+               button->gpio = -ENOENT; /* mark this as device-tree */
        }
 
        return pdata;
-
-err_out:
-       return ERR_PTR(error);
 }
 
 static struct of_device_id gpio_keys_of_match[] = {
@@ -470,11 +439,12 @@ gpio_keys_get_devtree_pdata(struct device *dev)
 static int gpio_keys_button_probe(struct platform_device *pdev,
                struct gpio_keys_button_dev **_bdev, int polled)
 {
-       struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
        struct device *dev = &pdev->dev;
+       struct gpio_keys_platform_data *pdata = dev_get_platdata(dev);
        struct gpio_keys_button_dev *bdev;
        struct gpio_keys_button *buttons;
-       int error;
+       struct device_node *prev = NULL;
+       int error = 0;
        int i;
 
        if (!pdata) {
@@ -513,40 +483,82 @@ static int gpio_keys_button_probe(struct platform_device *pdev,
        for (i = 0; i < pdata->nbuttons; i++) {
                struct gpio_keys_button *button = &buttons[i];
                struct gpio_keys_button_data *bdata = &bdev->data[i];
-               unsigned int gpio = button->gpio;
+               const char *desc = button->desc ? button->desc : DRV_NAME;
 
                if (button->wakeup) {
-                       dev_err(dev, DRV_NAME "does not support wakeup\n");
-                       return -EINVAL;
+                       dev_err(dev, "does not support wakeup\n");
+                       error = -EINVAL;
+                       goto out;
                }
 
-               error = devm_gpio_request(dev, gpio,
-                                    button->desc ? button->desc : DRV_NAME);
-               if (error) {
-                       dev_err(dev, "unable to claim gpio %u, err=%d\n",
-                               gpio, error);
-                       return error;
+               bdata->map_entry = button_get_index(button->code);
+               if (bdata->map_entry < 0) {
+                       dev_err(dev, "does not support key code:%u\n",
+                               button->code);
+                       error = -EINVAL;
+                       goto out;
                }
-               bdata->gpiod = gpio_to_desc(gpio);
-               if (!bdata->gpiod)
-                       return -EINVAL;
 
-               error = gpio_direction_input(gpio);
-               if (error) {
-                       dev_err(dev,
-                               "unable to set direction on gpio %u, err=%d\n",
-                               gpio, error);
-                       return error;
+               if (!(button->type == 0 || button->type == EV_KEY ||
+                     button->type == EV_SW)) {
+                       dev_err(dev, "only supports buttons or switches\n");
+                       error = -EINVAL;
+                       goto out;
+               }
+
+               if (button->irq) {
+                       dev_err(dev, "skipping button %s (only gpio buttons supported)\n",
+                               button->desc);
+                       bdata->b = &pdata->buttons[i];
+                       continue;
+               }
+
+               if (gpio_is_valid(button->gpio)) {
+                       /* legacy platform data... but is it the lookup table? */
+                       bdata->gpiod = devm_gpiod_get_index(dev, desc, i,
+                                                           GPIOD_IN);
+                       if (IS_ERR(bdata->gpiod)) {
+                               /* or the legacy (button->gpio is good) way? */
+                               error = devm_gpio_request_one(dev,
+                                       button->gpio, GPIOF_IN | (
+                                       button->active_low ? GPIOF_ACTIVE_LOW :
+                                       0), desc);
+                               if (error) {
+                                       if (error != -EPROBE_DEFER) {
+                                               dev_err(dev, "unable to claim gpio %d, err=%d\n",
+                                                       button->gpio, error);
+                                       }
+                                       goto out;
+                               }
+
+                               bdata->gpiod = gpio_to_desc(button->gpio);
+                       }
+               } else {
+                       /* Device-tree */
+                       struct device_node *child =
+                               of_get_next_child(dev->of_node, prev);
+
+                       bdata->gpiod = devm_fwnode_gpiod_get(dev,
+                               of_fwnode_handle(child), NULL, GPIOD_IN,
+                               desc);
+
+                       prev = child;
+               }
+
+               if (IS_ERR_OR_NULL(bdata->gpiod)) {
+                       error = IS_ERR(bdata->gpiod) ? PTR_ERR(bdata->gpiod) :
+                               -EINVAL;
+                       goto out;
                }
 
-               bdata->can_sleep = gpio_cansleep(gpio);
-               bdata->last_state = -1;
+               bdata->can_sleep = gpiod_cansleep(bdata->gpiod);
+               bdata->last_state = -1; /* Unknown state on boot */
 
                if (bdev->polled) {
                        bdata->threshold = DIV_ROUND_UP(button->debounce_interval,
                                                        pdata->poll_interval);
                } else {
-                       bdata->threshold = 1;
+                       /* bdata->threshold = 0; already initialized */
 
                        if (button->debounce_interval) {
                                error = gpiod_set_debounce(bdata->gpiod,
@@ -570,8 +582,11 @@ static int gpio_keys_button_probe(struct platform_device *pdev,
        platform_set_drvdata(pdev, bdev);
 
        *_bdev = bdev;
+       error = 0;
 
-       return 0;
+out:
+       of_node_put(prev);
+       return error;
 }
 
 static int gpio_keys_probe(struct platform_device *pdev)
@@ -580,9 +595,7 @@ static int gpio_keys_probe(struct platform_device *pdev)
        struct gpio_keys_button_dev *bdev;
        int ret, i;
 
-
        ret = gpio_keys_button_probe(pdev, &bdev, 0);
-
        if (ret)
                return ret;
 
@@ -592,9 +605,10 @@ static int gpio_keys_probe(struct platform_device *pdev)
                struct gpio_keys_button_data *bdata = &bdev->data[i];
                unsigned long irqflags = IRQF_ONESHOT;
 
-               if (!button->irq) {
-                       bdata->irq = gpio_to_irq(button->gpio);
+               INIT_DELAYED_WORK(&bdata->work, gpio_keys_irq_work_func);
 
+               if (!button->irq) {
+                       bdata->irq = gpiod_to_irq(bdata->gpiod);
                        if (bdata->irq < 0) {
                                dev_err(&pdev->dev, "failed to get irq for gpio:%d\n",
                                        button->gpio);
@@ -606,12 +620,12 @@ static int gpio_keys_probe(struct platform_device *pdev)
                        bdata->irq = button->irq;
                }
 
-               INIT_DELAYED_WORK(&bdata->work, gpio_keys_irq_work_func);
+               schedule_delayed_work(&bdata->work,
+                                     msecs_to_jiffies(bdata->software_debounce));
 
                ret = devm_request_threaded_irq(&pdev->dev,
                        bdata->irq, NULL, button_handle_irq,
                        irqflags, dev_name(&pdev->dev), bdata);
-
                if (ret < 0) {
                        bdata->irq = 0;
                        dev_err(&pdev->dev, "failed to request irq:%d for gpio:%d\n",
@@ -621,9 +635,6 @@ static int gpio_keys_probe(struct platform_device *pdev)
                        dev_dbg(&pdev->dev, "gpio:%d has irq:%d\n",
                                button->gpio, bdata->irq);
                }
-
-               if (bdata->b->type == EV_SW)
-                       button_hotplug_event(bdata, EV_SW, gpio_button_get_value(bdata));
        }
 
        return 0;
@@ -634,23 +645,17 @@ static int gpio_keys_polled_probe(struct platform_device *pdev)
        struct gpio_keys_platform_data *pdata;
        struct gpio_keys_button_dev *bdev;
        int ret;
-       int i;
 
        ret = gpio_keys_button_probe(pdev, &bdev, 1);
-
        if (ret)
                return ret;
 
        INIT_DELAYED_WORK(&bdev->work, gpio_keys_polled_poll);
 
        pdata = bdev->pdata;
-
        if (pdata->enable)
                pdata->enable(bdev->dev);
 
-       for (i = 0; i < pdata->nbuttons; i++)
-               gpio_keys_polled_check_state(&bdev->data[i]);
-
        gpio_keys_polled_queue_work(bdev);
 
        return ret;