Author: mmel
Date: Sun Oct 20 10:48:27 2019
New Revision: 353773
URL: https://svnweb.freebsd.org/changeset/base/353773

Log:
  Update Armada 8k drivers to cover newly imported DT and latest changes
  in simple multifunction driver.
  - follow interrupt changes in DT. Split old ICU driver to function oriented
    parts and add drivers for newly defined parts (system error interrupts).
  - Many drivers are children of simple multifunction driver. But after r349596
    simple MF driver doesn't longer exports memory resources, and all children
    must use syscon interface to access their registers. Adapt affected
    drivers to this fact.
  
  MFC after:    3 weeks

Added:
  head/sys/arm/mv/mv_ap806_sei.c   (contents, props changed)
  head/sys/arm/mv/mv_cp110_icu_bus.c   (contents, props changed)
Modified:
  head/sys/arm/mv/mv_ap806_clock.c
  head/sys/arm/mv/mv_ap806_gicp.c
  head/sys/arm/mv/mv_cp110_clock.c
  head/sys/arm/mv/mv_cp110_icu.c
  head/sys/arm/mv/mv_thermal.c
  head/sys/arm/mv/mvebu_pinctrl.c
  head/sys/arm64/conf/GENERIC
  head/sys/conf/files.arm64

Modified: head/sys/arm/mv/mv_ap806_clock.c
==============================================================================
--- head/sys/arm/mv/mv_ap806_clock.c    Sun Oct 20 01:09:14 2019        
(r353772)
+++ head/sys/arm/mv/mv_ap806_clock.c    Sun Oct 20 10:48:27 2019        
(r353773)
@@ -44,13 +44,15 @@ __FBSDID("$FreeBSD$");
 #include <machine/resource.h>
 #include <machine/intr.h>
 
-#include <dev/fdt/simplebus.h>
+#include <dev/extres/clk/clk_fixed.h>
+#include <dev/extres/syscon/syscon.h>
 
 #include <dev/ofw/ofw_bus.h>
 #include <dev/ofw/ofw_bus_subr.h>
 
-#include <dev/extres/clk/clk_fixed.h>
 
+#include "syscon_if.h"
+
 static struct clk_fixed_def ap806_clk_cluster_0 = {
        .clkdef.id = 0,
        .clkdef.name = "ap806-cpu-cluster-0",
@@ -92,23 +94,19 @@ static struct clk_fixed_def ap806_clk_sdio = {
 };
 
 struct mv_ap806_clock_softc {
-       struct simplebus_softc  simplebus_sc;
        device_t                dev;
-       struct resource         *res;
+       struct syscon           *syscon;
 };
 
-static struct resource_spec mv_ap806_clock_res_spec[] = {
-       { SYS_RES_MEMORY,       0,      RF_ACTIVE | RF_SHAREABLE },
-       { -1, 0 }
-};
 
+
 static struct ofw_compat_data compat_data[] = {
-       {"marvell,ap806-clock", 1},
-       {NULL,             0}
+       {"marvell,ap806-clock", 1},
+       {NULL,                  0}
 };
 
-#define        RD4(sc, reg)            bus_read_4((sc)->res, (reg))
-#define        WR4(sc, reg, val)       bus_write_4((sc)->res, (reg), (val))
+#define        RD4(sc, reg)            SYSCON_READ_4((sc)->syscon, (reg))
+#define        WR4(sc, reg, val)       SYSCON_WRITE_4((sc)->syscon, (reg), 
(val))
 
 static int
 mv_ap806_clock_probe(device_t dev)
@@ -135,8 +133,9 @@ mv_ap806_clock_attach(device_t dev)
        sc = device_get_softc(dev);
        sc->dev = dev;
 
-       if (bus_alloc_resources(dev, mv_ap806_clock_res_spec, &sc->res) != 0) {
-               device_printf(dev, "cannot allocate resources for device\n");
+       if (SYSCON_GET_HANDLE(sc->dev, &sc->syscon) != 0 ||
+           sc->syscon == NULL) {
+               device_printf(dev, "cannot get syscon for device\n");
                return (ENXIO);
        }
 
@@ -160,7 +159,8 @@ mv_ap806_clock_attach(device_t dev)
                clock_freq = 1333000000;
                break;
        default:
-               device_printf(dev, "Cannot guess clock freq with reg %x\n", reg 
& 0x1f);
+               device_printf(dev, "Cannot guess clock freq with reg %x\n",
+                    reg & 0x1f);
                return (ENXIO);
                break;
        };

Modified: head/sys/arm/mv/mv_ap806_gicp.c
==============================================================================
--- head/sys/arm/mv/mv_ap806_gicp.c     Sun Oct 20 01:09:14 2019        
(r353772)
+++ head/sys/arm/mv/mv_ap806_gicp.c     Sun Oct 20 10:48:27 2019        
(r353773)
@@ -60,6 +60,7 @@ struct mv_ap806_gicp_softc {
 
        ssize_t                 spi_ranges_cnt;
        uint32_t                *spi_ranges;
+       struct intr_map_data_fdt *parent_map_data;
 };
 
 static struct ofw_compat_data compat_data[] = {
@@ -96,11 +97,13 @@ mv_ap806_gicp_attach(device_t dev)
 
        /* Look for our parent */
        if ((intr_parent = ofw_bus_find_iparent(node)) == 0) {
-               device_printf(dev, "Cannot find our parent interrupt 
controller\n");
+               device_printf(dev,
+                    "Cannot find our parent interrupt controller\n");
                return (ENXIO);
        }
        if ((sc->parent = OF_device_from_xref(intr_parent)) == NULL) {
-               device_printf(dev, "cannot find parent interrupt controller 
device\n");
+               device_printf(dev,
+                    "cannot find parent interrupt controller device\n");
                return (ENXIO);
        }
 
@@ -112,7 +115,10 @@ mv_ap806_gicp_attach(device_t dev)
                device_printf(dev, "Cannot register GICP\n");
                return (ENXIO);
        }
-
+       /* Allocate GIC compatible mapping entry (3 cells) */
+       sc->parent_map_data = (struct intr_map_data_fdt *)intr_alloc_map_data(
+           INTR_MAP_DATA_FDT, sizeof(struct intr_map_data_fdt) +
+           + 3 * sizeof(phandle_t), M_WAITOK | M_ZERO);
        OF_device_register_xref(xref, dev);
 
        return (0);
@@ -125,6 +131,45 @@ mv_ap806_gicp_detach(device_t dev)
        return (EBUSY);
 }
 
+static struct intr_map_data *
+mv_ap806_gicp_convert_map_data(struct mv_ap806_gicp_softc *sc,
+    struct intr_map_data *data)
+{
+       struct intr_map_data_fdt *daf;
+       uint32_t i, irq_num, irq_type;
+
+       daf = (struct intr_map_data_fdt *)data;
+       if (daf->ncells != 2)
+               return (NULL);
+
+       irq_num = daf->cells[0];
+       irq_type = daf->cells[1];
+       if (irq_num >= MV_AP806_GICP_MAX_NIRQS)
+               return (NULL);
+
+       /* Construct GIC compatible mapping. */
+       sc->parent_map_data->ncells = 3;
+       sc->parent_map_data->cells[0] = 0; /* SPI */
+       sc->parent_map_data->cells[2] = irq_type;
+
+       /* Map the interrupt number to SPI number */
+       for (i = 0; i < sc->spi_ranges_cnt / 2; i += 2) {
+               if (irq_num < sc->spi_ranges[i + 1]) {
+                       irq_num += sc->spi_ranges[i];
+                       break;
+               }
+
+               irq_num -= sc->spi_ranges[i];
+       }
+
+       sc->parent_map_data->cells[1] = irq_num - 32;
+
+       return ((struct intr_map_data *)sc->parent_map_data);
+}
+
+
+
+
 static int
 mv_ap806_gicp_activate_intr(device_t dev, struct intr_irqsrc *isrc,
     struct resource *res, struct intr_map_data *data)
@@ -132,6 +177,9 @@ mv_ap806_gicp_activate_intr(device_t dev, struct intr_
        struct mv_ap806_gicp_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_ap806_gicp_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_ACTIVATE_INTR(sc->parent, isrc, res, data));
 }
@@ -161,36 +209,20 @@ mv_ap806_gicp_map_intr(device_t dev, struct intr_map_d
     struct intr_irqsrc **isrcp)
 {
        struct mv_ap806_gicp_softc *sc;
-       struct intr_map_data_fdt *daf;
-       uint32_t group, irq_num, irq_type;
-       int i;
+       int ret;
 
        sc = device_get_softc(dev);
 
        if (data->type != INTR_MAP_DATA_FDT)
                return (ENOTSUP);
 
-       daf = (struct intr_map_data_fdt *)data;
-       if (daf->ncells != 3 || daf->cells[0] >= MV_AP806_GICP_MAX_NIRQS)
+       data = mv_ap806_gicp_convert_map_data(sc, data);
+       if (data == NULL)
                return (EINVAL);
 
-       group = daf->cells[0];
-       irq_num = daf->cells[1];
-       irq_type = daf->cells[2];
-
-       /* Map the interrupt number to spi number */
-       for (i = 0; i < sc->spi_ranges_cnt / 2; i += 2) {
-               if (irq_num < sc->spi_ranges[i + 1]) {
-                       irq_num += sc->spi_ranges[i];
-                       break;
-               }
-
-               irq_num -= sc->spi_ranges[i];
-       }
-
-       daf->cells[1] = irq_num - 32;
-
-       return (PIC_MAP_INTR(sc->parent, data, isrcp));
+       ret = PIC_MAP_INTR(sc->parent, data, isrcp);
+       (*isrcp)->isrc_dev = sc->dev;
+       return(ret);
 }
 
 static int
@@ -201,6 +233,10 @@ mv_ap806_gicp_deactivate_intr(device_t dev, struct int
 
        sc = device_get_softc(dev);
 
+       data = mv_ap806_gicp_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
+
        return (PIC_DEACTIVATE_INTR(sc->parent, isrc, res, data));
 }
 
@@ -211,6 +247,9 @@ mv_ap806_gicp_setup_intr(device_t dev, struct intr_irq
        struct mv_ap806_gicp_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_ap806_gicp_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_SETUP_INTR(sc->parent, isrc, res, data));
 }
@@ -222,6 +261,9 @@ mv_ap806_gicp_teardown_intr(device_t dev, struct intr_
        struct mv_ap806_gicp_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_ap806_gicp_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_TEARDOWN_INTR(sc->parent, isrc, res, data));
 }

Added: head/sys/arm/mv/mv_ap806_sei.c
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/sys/arm/mv/mv_ap806_sei.c      Sun Oct 20 10:48:27 2019        
(r353773)
@@ -0,0 +1,419 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause-FreeBSD
+ *
+ * Copyright (c) 2019 Michal Meloun <m...@freebsd.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ *
+ */
+
+#include <sys/cdefs.h>
+__FBSDID("$FreeBSD$");
+
+#include <sys/param.h>
+#include <sys/systm.h>
+#include <sys/bus.h>
+#include <sys/kernel.h>
+#include <sys/proc.h>
+#include <sys/rman.h>
+#include <sys/lock.h>
+#include <sys/module.h>
+#include <sys/mutex.h>
+
+#include <machine/bus.h>
+#include <machine/intr.h>
+#include <machine/resource.h>
+
+#include <dev/fdt/simplebus.h>
+
+#include <dev/ofw/ofw_bus.h>
+#include <dev/ofw/ofw_bus_subr.h>
+
+#include "pic_if.h"
+
+#define        MV_AP806_SEI_LOCK(_sc)          mtx_lock(&(_sc)->mtx)
+#define        MV_AP806_SEI_UNLOCK(_sc)        mtx_unlock(&(_sc)->mtx)
+#define        MV_AP806_SEI_LOCK_INIT(_sc)     mtx_init(&_sc->mtx,             
        \
+           device_get_nameunit(_sc->dev), "mv_ap806_sei", MTX_DEF)
+#define        MV_AP806_SEI_LOCK_DESTROY(_sc)  mtx_destroy(&_sc->mtx);
+#define        MV_AP806_SEI_ASSERT_LOCKED(_sc) mtx_assert(&_sc->mtx, MA_OWNED);
+#define        MV_AP806_SEI_ASSERT_UNLOCKED(_sc) mtx_assert(&_sc->mtx, 
MA_NOTOWNED);
+
+#define        MV_AP806_SEI_MAX_NIRQS  64
+#define GICP_SECR0             0x00
+#define GICP_SECR1             0x04
+#define GICP_SECR(i)           (0x00  + (((i)/32) * 0x4))
+#define GICP_SECR_BIT(i)       ((i) % 32)
+#define GICP_SEMR0             0x20
+#define GICP_SEMR1             0x24
+#define GICP_SEMR(i)           (0x20  + (((i)/32) * 0x4))
+#define GICP_SEMR_BIT(i)       ((i) % 32)
+
+
+
+struct mv_ap806_sei_irqsrc {
+       struct intr_irqsrc      isrc;
+       u_int                   irq;
+};
+
+struct mv_ap806_sei_softc {
+       device_t                dev;
+       struct resource         *mem_res;
+       struct resource         *irq_res;
+       void                    *irq_ih;
+       struct mtx              mtx;
+
+       struct mv_ap806_sei_irqsrc *isrcs;
+};
+
+static struct ofw_compat_data compat_data[] = {
+       {"marvell,ap806-sei", 1},
+       {NULL,             0}
+};
+
+#define        RD4(sc, reg)            bus_read_4((sc)->mem_res, (reg))
+#define        WR4(sc, reg, val)       bus_write_4((sc)->mem_res, (reg), (val))
+
+static inline void
+mv_ap806_sei_isrc_mask(struct mv_ap806_sei_softc *sc,
+     struct mv_ap806_sei_irqsrc *sisrc, uint32_t val)
+{
+       uint32_t tmp;
+       int bit;
+
+       bit = GICP_SEMR_BIT(sisrc->irq);
+       MV_AP806_SEI_LOCK(sc);
+       tmp = RD4(sc, GICP_SEMR(sisrc->irq));
+       if (val != 0)
+               tmp |= 1 << bit;
+       else
+               tmp &= ~(1 << bit);
+       WR4(sc, GICP_SEMR(sisrc->irq), tmp);
+       MV_AP806_SEI_UNLOCK(sc);
+}
+
+static inline void
+mv_ap806_sei_isrc_eoi(struct mv_ap806_sei_softc *sc,
+     struct mv_ap806_sei_irqsrc *sisrc)
+{
+
+       WR4(sc, GICP_SECR(sisrc->irq), GICP_SECR_BIT(sisrc->irq));
+}
+
+static void
+mv_ap806_sei_enable_intr(device_t dev, struct intr_irqsrc *isrc)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+       mv_ap806_sei_isrc_mask(sc, sisrc, 0);
+}
+
+static void
+mv_ap806_sei_disable_intr(device_t dev, struct intr_irqsrc *isrc)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+       mv_ap806_sei_isrc_mask(sc, sisrc, 1);
+}
+
+static int
+mv_ap806_sei_map(device_t dev, struct intr_map_data *data, u_int *irqp)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct intr_map_data_fdt *daf;
+       u_int irq;
+
+       sc = device_get_softc(dev);
+
+       if (data->type != INTR_MAP_DATA_FDT)
+               return (ENOTSUP);
+
+       daf = (struct intr_map_data_fdt *)data;
+       if (daf->ncells != 1 || daf->cells[0] >= MV_AP806_SEI_MAX_NIRQS)
+               return (EINVAL);
+       irq = daf->cells[0];
+       if (irqp != NULL)
+               *irqp = irq;
+
+       return(0);
+}
+
+static int
+mv_ap806_sei_map_intr(device_t dev, struct intr_map_data *data,
+    struct intr_irqsrc **isrcp)
+{
+       struct mv_ap806_sei_softc *sc;
+       u_int irq;
+       int rv;
+
+       sc = device_get_softc(dev);
+       rv = mv_ap806_sei_map(dev, data, &irq);
+       if (rv == 0)
+               *isrcp = &sc->isrcs[irq].isrc;
+
+       return (rv);
+}
+
+
+
+static int
+mv_ap806_sei_setup_intr(device_t dev, struct intr_irqsrc *isrc,
+    struct resource *res, struct intr_map_data *data)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+       u_int irq;
+       int rv;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+       if (data == NULL)
+               return (ENOTSUP);
+       rv = mv_ap806_sei_map(dev, data, &irq);
+       if (rv != 0)
+               return (rv);
+       if (irq != sisrc->irq)
+               return (EINVAL);
+       mv_ap806_sei_isrc_mask(sc, sisrc, 0);
+       return (0);
+}
+
+static int
+mv_ap806_sei_teardown_intr(device_t dev, struct intr_irqsrc *isrc,
+    struct resource *res, struct intr_map_data *data)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+
+       mv_ap806_sei_isrc_mask(sc, sisrc, 1);
+       return (0);
+}
+
+static void
+mv_ap806_sei_pre_ithread(device_t dev, struct intr_irqsrc *isrc)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+
+       mv_ap806_sei_isrc_mask(sc, sisrc, 1);
+       mv_ap806_sei_isrc_eoi(sc, sisrc);
+}
+
+static void
+mv_ap806_sei_post_ithread(device_t dev, struct intr_irqsrc *isrc)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+
+       mv_ap806_sei_isrc_mask(sc, sisrc, 1);
+}
+
+static void
+mv_ap806_sei_post_filter(device_t dev, struct intr_irqsrc *isrc)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sisrc;
+
+       sc = device_get_softc(dev);
+       sisrc = (struct mv_ap806_sei_irqsrc *)isrc;
+
+       mv_ap806_sei_isrc_mask(sc, sisrc, 1);
+       mv_ap806_sei_isrc_eoi(sc, sisrc);
+}
+
+/* ----------------------------------------------------------------------------
+ *
+ *             B u s    i n t e r f a c e
+ */
+static int
+mv_ap806_sei_intr(void *arg)
+{
+       struct mv_ap806_sei_softc *sc;
+       struct mv_ap806_sei_irqsrc *sirq;
+       struct trapframe *tf;
+       uint64_t cause;
+       u_int irq;
+
+       sc = (struct mv_ap806_sei_softc *)arg;
+       tf = curthread->td_intr_frame;
+       while (1) {
+               cause = RD4(sc, GICP_SECR1);
+               cause <<= 32;
+               cause |= RD4(sc, GICP_SECR0);
+
+               irq = ffsll(cause);
+               if (irq == 0) break;
+               irq--;
+               sirq = &sc->isrcs[irq];
+               if (intr_isrc_dispatch(&sirq->isrc, tf) != 0) {
+                       mv_ap806_sei_isrc_mask(sc, sirq, 0);
+                       mv_ap806_sei_isrc_eoi(sc, sirq);
+                       device_printf(sc->dev,
+                           "Stray irq %u disabled\n", irq);
+               }
+       }
+
+       return (FILTER_HANDLED);
+}
+
+
+static int
+mv_ap806_sei_probe(device_t dev)
+{
+
+       if (!ofw_bus_status_okay(dev))
+               return (ENXIO);
+
+       if (ofw_bus_search_compatible(dev, compat_data)->ocd_data == 0)
+               return (ENXIO);
+
+       device_set_desc(dev, "Marvell SEI");
+       return (BUS_PROBE_DEFAULT);
+}
+
+static int
+mv_ap806_sei_attach(device_t dev)
+{
+       struct mv_ap806_sei_softc *sc;
+       phandle_t xref, node;
+       uint32_t irq;
+       const char *name;
+       int rv, rid;
+
+       sc = device_get_softc(dev);
+       sc->dev = dev;
+       node = ofw_bus_get_node(dev);
+       MV_AP806_SEI_LOCK_INIT(sc);
+
+       /* Allocate resources. */
+       rid = 0;
+       sc->mem_res = bus_alloc_resource_any(dev, SYS_RES_MEMORY, &rid,
+           RF_ACTIVE);
+       if (sc->mem_res == NULL) {
+               device_printf(dev, "Cannot allocate memory resources\n");
+               rv = ENXIO;
+               goto fail;
+       }
+
+       rid = 0;
+       sc->irq_res = bus_alloc_resource_any(dev, SYS_RES_IRQ, &rid, RF_ACTIVE);
+       if (sc->irq_res == NULL) {
+               device_printf(dev, "Cannot allocate IRQ resources\n");
+               rv = ENXIO;
+               goto fail;
+       }
+
+       /* Mask all interrupts) */
+       WR4(sc, GICP_SEMR0, 0xFFFFFFFF);
+       WR4(sc, GICP_SEMR1, 0xFFFFFFFF);
+
+       /* Create all interrupt sources */
+       sc->isrcs = malloc(sizeof(*sc->isrcs) * MV_AP806_SEI_MAX_NIRQS,
+           M_DEVBUF, M_WAITOK | M_ZERO);
+       name = device_get_nameunit(sc->dev);
+       for (irq = 0; irq < MV_AP806_SEI_MAX_NIRQS; irq++) {
+               sc->isrcs[irq].irq = irq;
+               rv = intr_isrc_register(&sc->isrcs[irq].isrc,
+                   sc->dev, 0, "%s,%u", name, irq);
+               if (rv != 0)
+                       goto fail; /* XXX deregister ISRCs */
+       }
+       xref = OF_xref_from_node(node);;
+       if (intr_pic_register(dev, xref) == NULL) {
+               device_printf(dev, "Cannot register SEI\n");
+               rv = ENXIO;
+               goto fail;
+       }
+       if (bus_setup_intr(dev, sc->irq_res,INTR_TYPE_MISC | INTR_MPSAFE,
+           mv_ap806_sei_intr, NULL, sc, &sc->irq_ih)) {
+               device_printf(dev,
+                   "Unable to register interrupt handler\n");
+               rv = ENXIO;
+               goto fail;
+       }
+       
+       OF_device_register_xref(xref, dev);
+       return (0);
+
+fail:
+       if (sc->irq_ih != NULL)
+               bus_teardown_intr(dev, sc->irq_res, sc->irq_ih);
+       if (sc->irq_res != NULL)
+               bus_release_resource(dev, SYS_RES_IRQ, 0, sc->irq_res);
+       if (sc->mem_res != NULL)
+               bus_release_resource(dev, SYS_RES_MEMORY, 0, sc->mem_res);
+       MV_AP806_SEI_LOCK_DESTROY(sc);
+       return (ENXIO);
+}
+
+static int
+mv_ap806_sei_detach(device_t dev)
+{
+
+       return (EBUSY);
+}
+
+
+static device_method_t mv_ap806_sei_methods[] = {
+       /* Device interface */
+       DEVMETHOD(device_probe,         mv_ap806_sei_probe),
+       DEVMETHOD(device_attach,        mv_ap806_sei_attach),
+       DEVMETHOD(device_detach,        mv_ap806_sei_detach),
+
+       /* Interrupt controller interface */
+       DEVMETHOD(pic_disable_intr,     mv_ap806_sei_disable_intr),
+       DEVMETHOD(pic_enable_intr,      mv_ap806_sei_enable_intr),
+       DEVMETHOD(pic_map_intr,         mv_ap806_sei_map_intr),
+       DEVMETHOD(pic_setup_intr,       mv_ap806_sei_setup_intr),
+       DEVMETHOD(pic_teardown_intr,    mv_ap806_sei_teardown_intr),
+       DEVMETHOD(pic_post_filter,      mv_ap806_sei_post_filter),
+       DEVMETHOD(pic_post_ithread,     mv_ap806_sei_post_ithread),
+       DEVMETHOD(pic_pre_ithread,      mv_ap806_sei_pre_ithread),
+
+       DEVMETHOD_END
+};
+
+static devclass_t mv_ap806_sei_devclass;
+
+static driver_t mv_ap806_sei_driver = {
+       "mv_ap806_sei",
+       mv_ap806_sei_methods,
+       sizeof(struct mv_ap806_sei_softc),
+};
+
+EARLY_DRIVER_MODULE(mv_ap806_sei, simplebus, mv_ap806_sei_driver,
+    mv_ap806_sei_devclass, 0, 0, BUS_PASS_INTERRUPT + BUS_PASS_ORDER_MIDDLE);

Modified: head/sys/arm/mv/mv_cp110_clock.c
==============================================================================
--- head/sys/arm/mv/mv_cp110_clock.c    Sun Oct 20 01:09:14 2019        
(r353772)
+++ head/sys/arm/mv/mv_cp110_clock.c    Sun Oct 20 10:48:27 2019        
(r353773)
@@ -44,17 +44,17 @@ __FBSDID("$FreeBSD$");
 #include <machine/resource.h>
 #include <machine/intr.h>
 
-#include <dev/fdt/simplebus.h>
+#include <dev/extres/clk/clk_fixed.h>
+#include <dev/extres/clk/clk_gate.h>
+#include <dev/extres/syscon/syscon.h>
 
 #include <dev/ofw/ofw_bus.h>
 #include <dev/ofw/ofw_bus_subr.h>
 
-#include <dev/extres/clk/clk_fixed.h>
-#include <dev/extres/clk/clk_gate.h>
-
 #include <arm/mv/mv_cp110_clock.h>
 
 #include "clkdev_if.h"
+#include "syscon_if.h"
 
 /* Clocks */
 static struct clk_fixed_def cp110_clk_pll_0 = {
@@ -126,24 +126,19 @@ static struct cp110_gate cp110_gates[] = {
 };
 
 struct mv_cp110_clock_softc {
-       struct simplebus_softc  simplebus_sc;
        device_t                dev;
-       struct resource         *res;
+       struct syscon           *syscon;
        struct mtx              mtx;
 };
 
-static struct resource_spec mv_cp110_clock_res_spec[] = {
-       { SYS_RES_MEMORY,       0,      RF_ACTIVE | RF_SHAREABLE },
-       { -1, 0 }
-};
 
 static struct ofw_compat_data compat_data[] = {
        {"marvell,cp110-clock", 1},
        {NULL,             0}
 };
 
-#define        RD4(sc, reg)            bus_read_4((sc)->res, (reg))
-#define        WR4(sc, reg, val)       bus_write_4((sc)->res, (reg), (val))
+#define        RD4(sc, reg)            SYSCON_READ_4((sc)->syscon, (reg))
+#define        WR4(sc, reg, val)       SYSCON_WRITE_4((sc)->syscon, (reg), 
(val))
 
 static char *
 mv_cp110_clock_name(device_t dev, const char *name)
@@ -201,8 +196,9 @@ mv_cp110_clock_attach(device_t dev)
        sc = device_get_softc(dev);
        sc->dev = dev;
 
-       if (bus_alloc_resources(dev, mv_cp110_clock_res_spec, &sc->res) != 0) {
-               device_printf(dev, "cannot allocate resources for device\n");
+       if (SYSCON_GET_HANDLE(sc->dev, &sc->syscon) != 0 ||
+           sc->syscon == NULL) {
+               device_printf(dev, "cannot get syscon for device\n");
                return (ENXIO);
        }
 

Modified: head/sys/arm/mv/mv_cp110_icu.c
==============================================================================
--- head/sys/arm/mv/mv_cp110_icu.c      Sun Oct 20 01:09:14 2019        
(r353772)
+++ head/sys/arm/mv/mv_cp110_icu.c      Sun Oct 20 10:48:27 2019        
(r353773)
@@ -49,6 +49,7 @@ __FBSDID("$FreeBSD$");
 #include <dev/ofw/ofw_bus.h>
 #include <dev/ofw/ofw_bus_subr.h>
 
+#include <gnu/dts/include/dt-bindings/interrupt-controller/irq.h>
 #include "pic_if.h"
 
 #define        ICU_GRP_NSR             0x0
@@ -72,6 +73,7 @@ struct mv_cp110_icu_softc {
        device_t                dev;
        device_t                parent;
        struct resource         *res;
+       struct intr_map_data_fdt *parent_map_data;
 };
 
 static struct resource_spec mv_cp110_icu_res_spec[] = {
@@ -80,8 +82,9 @@ static struct resource_spec mv_cp110_icu_res_spec[] = 
 };
 
 static struct ofw_compat_data compat_data[] = {
-       {"marvell,cp110-icu", 1},
-       {NULL,             0}
+       {"marvell,cp110-icu-nsr",       1},
+       {"marvell,cp110-icu-sei",       2},
+       {NULL,                          0}
 };
 
 #define        RD4(sc, reg)            bus_read_4((sc)->res, (reg))
@@ -130,6 +133,11 @@ mv_cp110_icu_attach(device_t dev)
                device_printf(dev, "Cannot register ICU\n");
                goto fail;
        }
+
+       /* Allocate GICP compatible mapping entry (2 cells) */
+       sc->parent_map_data = (struct intr_map_data_fdt *)intr_alloc_map_data(
+           INTR_MAP_DATA_FDT, sizeof(struct intr_map_data_fdt) +
+           + 3 * sizeof(phandle_t), M_WAITOK | M_ZERO);
        return (0);
 
 fail:
@@ -137,6 +145,35 @@ fail:
        return (ENXIO);
 }
 
+static struct intr_map_data *
+mv_cp110_icu_convert_map_data(struct mv_cp110_icu_softc *sc, struct 
intr_map_data *data)
+{
+       struct intr_map_data_fdt *daf;
+       uint32_t reg, irq_no, irq_type;
+
+       daf = (struct intr_map_data_fdt *)data;
+       if (daf->ncells != 2)
+               return (NULL);
+       irq_no = daf->cells[0];
+       irq_type = daf->cells[1];
+       if (irq_no >= MV_CP110_ICU_MAX_NIRQS)
+               return (NULL);
+       if (irq_type != IRQ_TYPE_LEVEL_HIGH &&
+           irq_type != IRQ_TYPE_EDGE_RISING)
+               return (NULL);
+
+       /* We rely on fact that ICU->GIC mapping is preset by bootstrap. */
+       reg = RD4(sc, ICU_INT_CFG(irq_no));
+
+       /* Construct GICP compatible mapping. */
+       sc->parent_map_data->ncells = 2;
+       sc->parent_map_data->cells[0] = reg & ICU_INT_MASK;
+       sc->parent_map_data->cells[1] = irq_type;
+
+       return ((struct intr_map_data *)sc->parent_map_data);
+}
+
+
 static int
 mv_cp110_icu_detach(device_t dev)
 {
@@ -151,7 +188,9 @@ mv_cp110_icu_activate_intr(device_t dev, struct intr_i
        struct mv_cp110_icu_softc *sc;
 
        sc = device_get_softc(dev);
-
+       data = mv_cp110_icu_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
        return (PIC_ACTIVATE_INTR(sc->parent, isrc, res, data));
 }
 
@@ -159,7 +198,6 @@ static void
 mv_cp110_icu_enable_intr(device_t dev, struct intr_irqsrc *isrc)
 {
        struct mv_cp110_icu_softc *sc;
-
        sc = device_get_softc(dev);
 
        PIC_ENABLE_INTR(sc->parent, isrc);
@@ -175,32 +213,43 @@ mv_cp110_icu_disable_intr(device_t dev, struct intr_ir
        PIC_DISABLE_INTR(sc->parent, isrc);
 }
 
+
+
 static int
 mv_cp110_icu_map_intr(device_t dev, struct intr_map_data *data,
     struct intr_irqsrc **isrcp)
 {
        struct mv_cp110_icu_softc *sc;
        struct intr_map_data_fdt *daf;
-       uint32_t reg;
+       uint32_t reg, irq_no, irq_type;
+       int ret;
 
        sc = device_get_softc(dev);
 
        if (data->type != INTR_MAP_DATA_FDT)
                return (ENOTSUP);
 
+       /* Parse original */
        daf = (struct intr_map_data_fdt *)data;
-       if (daf->ncells != 3 || daf->cells[0] >= MV_CP110_ICU_MAX_NIRQS)
+       if (daf->ncells != 2)
                return (EINVAL);
+       irq_no = daf->cells[0];
+       irq_type = daf->cells[1];
+       data = mv_cp110_icu_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
-       reg = RD4(sc, ICU_INT_CFG(daf->cells[1]));
+       reg = RD4(sc, ICU_INT_CFG(irq_no));
+       reg |= ICU_INT_ENABLE;
+       if (irq_type == IRQ_TYPE_LEVEL_HIGH)
+               reg &= ~ICU_INT_EDGE;
+       else
+               reg |= ICU_INT_EDGE;
+       WR4(sc, ICU_INT_CFG(irq_no), reg);
 
-       if ((reg & ICU_INT_ENABLE) == 0) {
-               reg |= ICU_INT_ENABLE;
-               WR4(sc, ICU_INT_CFG(daf->cells[1]), reg);
-       }
-
-       daf->cells[1] = reg & ICU_INT_MASK;
-       return (PIC_MAP_INTR(sc->parent, data, isrcp));
+       ret = PIC_MAP_INTR(sc->parent, data, isrcp);
+       (*isrcp)->isrc_dev = sc->dev;
+       return (ret);
 }
 
 static int
@@ -210,6 +259,9 @@ mv_cp110_icu_deactivate_intr(device_t dev, struct intr
        struct mv_cp110_icu_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_cp110_icu_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_DEACTIVATE_INTR(sc->parent, isrc, res, data));
 }
@@ -221,6 +273,9 @@ mv_cp110_icu_setup_intr(device_t dev, struct intr_irqs
        struct mv_cp110_icu_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_cp110_icu_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_SETUP_INTR(sc->parent, isrc, res, data));
 }
@@ -232,6 +287,9 @@ mv_cp110_icu_teardown_intr(device_t dev, struct intr_i
        struct mv_cp110_icu_softc *sc;
 
        sc = device_get_softc(dev);
+       data = mv_cp110_icu_convert_map_data(sc, data);
+       if (data == NULL)
+               return (EINVAL);
 
        return (PIC_TEARDOWN_INTR(sc->parent, isrc, res, data));
 }
@@ -295,5 +353,5 @@ static driver_t mv_cp110_icu_driver = {
        sizeof(struct mv_cp110_icu_softc),
 };
 
-EARLY_DRIVER_MODULE(mv_cp110_icu, simplebus, mv_cp110_icu_driver,
+EARLY_DRIVER_MODULE(mv_cp110_icu, mv_cp110_icu_bus, mv_cp110_icu_driver,
     mv_cp110_icu_devclass, 0, 0, BUS_PASS_INTERRUPT + BUS_PASS_ORDER_LAST);

Added: head/sys/arm/mv/mv_cp110_icu_bus.c
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/sys/arm/mv/mv_cp110_icu_bus.c  Sun Oct 20 10:48:27 2019        
(r353773)
@@ -0,0 +1,78 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause-FreeBSD
+ *
+ * Copyright (c) 2019 Michal Meloun <m...@freebsd.org>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+#include <sys/cdefs.h>
+__FBSDID("$FreeBSD$");
+
+#include <sys/param.h>
+#include <sys/bus.h>
+#include <sys/kernel.h>
+#include <sys/module.h>
+#include <sys/mutex.h>
+#include <sys/rman.h>
+#include <machine/bus.h>
+
+#include <dev/ofw/openfirm.h>
+#include <dev/ofw/ofw_bus.h>
+#include <dev/ofw/ofw_bus_subr.h>
+
+#include <dev/extres/syscon/syscon.h>
+#include <dev/fdt/simple_mfd.h>
+
+static struct ofw_compat_data compat_data[] = {
+       {"marvell,cp110-icu",   1},
+       {NULL,                  0}
+};
+
+static int
+mv_cp110_icu_bus_probe(device_t dev)
+{
+
+       if (!ofw_bus_status_okay(dev))
+               return (ENXIO);
+       if (ofw_bus_search_compatible(dev, compat_data)->ocd_data == 0)
+               return (ENXIO);
+
+       device_set_desc(dev, "Marvell Interrupt Consolidation Unit Bus");
+       return (BUS_PROBE_DEFAULT);
+}
+
+static device_method_t mv_cp110_icu_bus_methods[] = {
+       DEVMETHOD(device_probe, mv_cp110_icu_bus_probe),
+
+       DEVMETHOD_END
+};
+
+DEFINE_CLASS_1(mv_cp110_icu_bus, mv_cp110_icu_bus_driver,
+    mv_cp110_icu_bus_methods, sizeof(struct simple_mfd_softc),
+    simple_mfd_driver);
+
+static devclass_t mv_cp110_icu_bus_devclass;
+EARLY_DRIVER_MODULE(mv_cp110_icu_bus, simplebus, mv_cp110_icu_bus_driver,
+    mv_cp110_icu_bus_devclass, 0, 0, BUS_PASS_INTERRUPT);
+MODULE_VERSION(mv_cp110_icu_bus, 1);

Modified: head/sys/arm/mv/mv_thermal.c
==============================================================================
--- head/sys/arm/mv/mv_thermal.c        Sun Oct 20 01:09:14 2019        
(r353772)
+++ head/sys/arm/mv/mv_thermal.c        Sun Oct 20 10:48:27 2019        
(r353773)
@@ -44,13 +44,14 @@ __FBSDID("$FreeBSD$");
 #include <machine/bus.h>
 #include <machine/resource.h>

*** DIFF OUTPUT TRUNCATED AT 1000 LINES ***
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to