Introduce a new domain for wakeup capable GPIOs. The domain can be
requested using the bus token DOMAIN_BUS_WAKEUP. In the following
patches, we will specify PDC as the wakeup-parent for the TLMM GPIO
irqchip. Requesting a wakeup GPIO will setup the GPIO and the
corresponding PDC interrupt as its parent.

Co-developed-by: Stephen Boyd <swb...@chromium.org>
Signed-off-by: Lina Iyer <il...@codeaurora.org>
---
Changes in v4:
        - Remove vestigial changes from v2
Changes in v3:
        - Remove PDC GPIO map data (moved to DT)
        - hwirq passed in .alloc() is a PDC pin now
Changes in v2:
        - Remove separate file for PDC GPIO map data
        - Error checks and return
        - Whitespace fixes
---
 drivers/irqchip/qcom-pdc.c   | 72 ++++++++++++++++++++++++++++++++++--
 include/linux/soc/qcom/irq.h | 23 ++++++++++++
 2 files changed, 92 insertions(+), 3 deletions(-)
 create mode 100644 include/linux/soc/qcom/irq.h

diff --git a/drivers/irqchip/qcom-pdc.c b/drivers/irqchip/qcom-pdc.c
index faa7d61b9d6c..29118f6d84c3 100644
--- a/drivers/irqchip/qcom-pdc.c
+++ b/drivers/irqchip/qcom-pdc.c
@@ -13,12 +13,13 @@
 #include <linux/of.h>
 #include <linux/of_address.h>
 #include <linux/of_device.h>
+#include <linux/soc/qcom/irq.h>
 #include <linux/spinlock.h>
-#include <linux/platform_device.h>
 #include <linux/slab.h>
 #include <linux/types.h>
 
 #define PDC_MAX_IRQS           126
+#define PDC_MAX_GPIO_IRQS      256
 
 #define CLEAR_INTR(reg, intr)  (reg & ~(1 << intr))
 #define ENABLE_INTR(reg, intr) (reg | (1 << intr))
@@ -169,7 +170,6 @@ static irq_hw_number_t get_parent_hwirq(int pin)
                        return (region->parent_base + pin - region->pin_base);
        }
 
-       WARN_ON(1);
        return ~0UL;
 }
 
@@ -232,6 +232,60 @@ static const struct irq_domain_ops qcom_pdc_ops = {
        .free           = irq_domain_free_irqs_common,
 };
 
+static int qcom_pdc_gpio_alloc(struct irq_domain *domain, unsigned int virq,
+                              unsigned int nr_irqs, void *data)
+{
+       struct qcom_irq_fwspec *qcom_fwspec = data;
+       struct irq_fwspec *fwspec = &qcom_fwspec->fwspec;
+       struct irq_fwspec parent_fwspec;
+       irq_hw_number_t hwirq, parent_hwirq;
+       unsigned int type;
+       int ret;
+
+       ret = qcom_pdc_translate(domain, fwspec, &hwirq, &type);
+       if (ret)
+               return -EINVAL;
+
+       parent_hwirq = get_parent_hwirq(hwirq);
+       if (parent_hwirq == ~0UL)
+               return -EINVAL;
+
+       ret = irq_domain_set_hwirq_and_chip(domain, virq, hwirq,
+                                           &qcom_pdc_gic_chip, NULL);
+       if (ret)
+               return ret;
+
+       qcom_fwspec->mask = true;
+
+       if (type & IRQ_TYPE_EDGE_BOTH)
+               type = IRQ_TYPE_EDGE_RISING;
+
+       if (type & IRQ_TYPE_LEVEL_MASK)
+               type = IRQ_TYPE_LEVEL_HIGH;
+
+       parent_fwspec.fwnode      = domain->parent->fwnode;
+       parent_fwspec.param_count = 3;
+       parent_fwspec.param[0]    = 0;
+       parent_fwspec.param[1]    = parent_hwirq;
+       parent_fwspec.param[2]    = type;
+
+       return irq_domain_alloc_irqs_parent(domain, virq, nr_irqs,
+                                           &parent_fwspec);
+}
+
+static int qcom_pdc_gpio_domain_select(struct irq_domain *d,
+                                      struct irq_fwspec *fwspec,
+                                      enum irq_domain_bus_token bus_token)
+{
+       return (bus_token == DOMAIN_BUS_WAKEUP);
+}
+
+static const struct irq_domain_ops qcom_pdc_gpio_ops = {
+       .select         = qcom_pdc_gpio_domain_select,
+       .alloc          = qcom_pdc_gpio_alloc,
+       .free           = irq_domain_free_irqs_common,
+};
+
 static int pdc_setup_pin_mapping(struct device_node *np)
 {
        int ret, n;
@@ -270,7 +324,7 @@ static int pdc_setup_pin_mapping(struct device_node *np)
 
 static int qcom_pdc_init(struct device_node *node, struct device_node *parent)
 {
-       struct irq_domain *parent_domain, *pdc_domain;
+       struct irq_domain *parent_domain, *pdc_domain, *pdc_gpio_domain;
        int ret;
 
        pdc_base = of_iomap(node, 0);
@@ -301,6 +355,18 @@ static int qcom_pdc_init(struct device_node *node, struct 
device_node *parent)
                goto fail;
        }
 
+       pdc_gpio_domain = irq_domain_create_hierarchy(parent_domain, 0,
+                                                     PDC_MAX_GPIO_IRQS,
+                                                     of_fwnode_handle(node),
+                                                     &qcom_pdc_gpio_ops, NULL);
+       if (!pdc_gpio_domain) {
+               pr_err("%pOF: GIC domain add failed for GPIO domain\n", node);
+               ret = -ENOMEM;
+               goto fail;
+       }
+
+       irq_domain_update_bus_token(pdc_gpio_domain, DOMAIN_BUS_WAKEUP);
+
        return 0;
 
 fail:
diff --git a/include/linux/soc/qcom/irq.h b/include/linux/soc/qcom/irq.h
new file mode 100644
index 000000000000..bacc9edbce0d
--- /dev/null
+++ b/include/linux/soc/qcom/irq.h
@@ -0,0 +1,23 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+
+#ifndef __QCOM_IRQ_H
+#define __QCOM_IRQ_H
+
+#include <linux/irqdomain.h>
+
+/**
+ * struct qcom_irq_fwspec - qcom specific irq fwspec wrapper
+ * @fwspec: irq fwspec
+ * @mask: if true, keep the irq masked in the gpio controller
+ *
+ * Use this structure to communicate between the parent irq chip, MPM or PDC,
+ * to the gpio chip, TLMM, about the gpio being allocated in the parent
+ * and if the gpio chip should keep the line masked because the parent irq
+ * chip is handling everything about the irq line.
+ */
+struct qcom_irq_fwspec {
+       struct irq_fwspec fwspec;
+       bool mask;
+};
+
+#endif
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

Reply via email to