Update the SRAB driver to manage per-port interrupts. Since we cannot
sleep during b53_io_ops, schedule a workqueue whenever we get a port
specific interrupt. We will later make use of this to call back into
PHYLINK when there is e.g: a link state change.

Signed-off-by: Florian Fainelli <f.faine...@gmail.com>
---
 drivers/net/dsa/b53/b53_srab.c | 108 +++++++++++++++++++++++++++++++++
 1 file changed, 108 insertions(+)

diff --git a/drivers/net/dsa/b53/b53_srab.c b/drivers/net/dsa/b53/b53_srab.c
index 91de2ba99ad1..411b84f61903 100644
--- a/drivers/net/dsa/b53/b53_srab.c
+++ b/drivers/net/dsa/b53/b53_srab.c
@@ -22,6 +22,7 @@
 #include <linux/platform_device.h>
 #include <linux/platform_data/b53.h>
 #include <linux/of.h>
+#include <linux/workqueue.h>
 
 #include "b53_priv.h"
 
@@ -47,6 +48,7 @@
 
 /* command and status register of the SRAB */
 #define B53_SRAB_CTRLS                 0x40
+#define  B53_SRAB_CTRLS_HOST_INTR      BIT(1)
 #define  B53_SRAB_CTRLS_RCAREQ         BIT(3)
 #define  B53_SRAB_CTRLS_RCAGNT         BIT(4)
 #define  B53_SRAB_CTRLS_SW_INIT_DONE   BIT(6)
@@ -60,8 +62,17 @@
 #define  B53_SRAB_P7_SLEEP_TIMER       BIT(11)
 #define  B53_SRAB_IMP0_SLEEP_TIMER     BIT(12)
 
+struct b53_srab_port_priv {
+       struct work_struct irq_work;
+       int irq;
+       bool irq_enabled;
+       struct b53_device *dev;
+       unsigned int num;
+};
+
 struct b53_srab_priv {
        void __iomem *regs;
+       struct b53_srab_port_priv port_intrs[B53_N_PORTS];
 };
 
 static int b53_srab_request_grant(struct b53_device *dev)
@@ -344,6 +355,50 @@ static int b53_srab_write64(struct b53_device *dev, u8 
page, u8 reg,
        return ret;
 }
 
+static void b53_srab_port_defer(struct work_struct *work)
+{
+}
+
+static irqreturn_t b53_srab_port_isr(int irq, void *dev_id)
+{
+       struct b53_srab_port_priv *port = dev_id;
+       struct b53_device *dev = port->dev;
+       struct b53_srab_priv *priv = dev->priv;
+
+       /* Acknowledge the interrupt */
+       writel(BIT(port->num), priv->regs + B53_SRAB_INTR);
+
+       schedule_work(&port->irq_work);
+
+       return IRQ_HANDLED;
+}
+
+static int b53_srab_irq_enable(struct b53_device *dev, int port)
+{
+       struct b53_srab_priv *priv = dev->priv;
+       struct b53_srab_port_priv *p = &priv->port_intrs[port];
+       int ret;
+
+       ret = request_irq(p->irq, b53_srab_port_isr, 0,
+                         dev_name(dev->dev), p);
+       if (!ret)
+               p->irq_enabled = true;
+
+       return ret;
+}
+
+static void b53_srab_irq_disable(struct b53_device *dev, int port)
+{
+       struct b53_srab_priv *priv = dev->priv;
+       struct b53_srab_port_priv *p = &priv->port_intrs[port];
+
+       if (p->irq_enabled) {
+               free_irq(p->irq, p);
+               cancel_work_sync(&p->irq_work);
+               p->irq_enabled = false;
+       }
+}
+
 static const struct b53_io_ops b53_srab_ops = {
        .read8 = b53_srab_read8,
        .read16 = b53_srab_read16,
@@ -355,6 +410,8 @@ static const struct b53_io_ops b53_srab_ops = {
        .write32 = b53_srab_write32,
        .write48 = b53_srab_write48,
        .write64 = b53_srab_write64,
+       .irq_enable = b53_srab_irq_enable,
+       .irq_disable = b53_srab_irq_disable,
 };
 
 static const struct of_device_id b53_srab_of_match[] = {
@@ -379,6 +436,53 @@ static const struct of_device_id b53_srab_of_match[] = {
 };
 MODULE_DEVICE_TABLE(of, b53_srab_of_match);
 
+static void b53_srab_intr_set(struct b53_srab_priv *priv, bool set)
+{
+       u32 reg;
+
+       reg = readl(priv->regs + B53_SRAB_CTRLS);
+       if (set)
+               reg |= B53_SRAB_CTRLS_HOST_INTR;
+       else
+               reg &= ~B53_SRAB_CTRLS_HOST_INTR;
+       writel(reg, priv->regs + B53_SRAB_CTRLS);
+}
+
+static void b53_srab_prepare_irq(struct platform_device *pdev)
+{
+       struct b53_device *dev = platform_get_drvdata(pdev);
+       struct b53_srab_priv *priv = dev->priv;
+       struct b53_srab_port_priv *port;
+       unsigned int i;
+       char *name;
+
+       /* Clear all pending interrupts */
+       writel(0xffffffff, priv->regs + B53_SRAB_INTR);
+
+       if (dev->pdata && dev->pdata->chip_id != BCM58XX_DEVICE_ID)
+               return;
+
+       for (i = 0; i < B53_N_PORTS; i++) {
+               port = &priv->port_intrs[i];
+
+               /* There is no port 6 */
+               if (i == 6)
+                       continue;
+
+               name = kasprintf(GFP_KERNEL, "link_state_p%d", i);
+               if (!name)
+                       return;
+
+               port->num = i;
+               port->dev = dev;
+               INIT_WORK(&port->irq_work, b53_srab_port_defer);
+               port->irq = platform_get_irq_byname(pdev, name);
+               kfree(name);
+       }
+
+       b53_srab_intr_set(priv, true);
+}
+
 static int b53_srab_probe(struct platform_device *pdev)
 {
        struct b53_platform_data *pdata = pdev->dev.platform_data;
@@ -417,13 +521,17 @@ static int b53_srab_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, dev);
 
+       b53_srab_prepare_irq(pdev);
+
        return b53_switch_register(dev);
 }
 
 static int b53_srab_remove(struct platform_device *pdev)
 {
        struct b53_device *dev = platform_get_drvdata(pdev);
+       struct b53_srab_priv *priv = dev->priv;
 
+       b53_srab_intr_set(priv, false);
        if (dev)
                b53_switch_remove(dev);
 
-- 
2.17.1

Reply via email to