Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 7 Sep 2022 07:59:40 GMT
From:      =?utf-8?Q?Kornel=20Dul=C4=99ba?= <kd@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 07c5be33f145 - main - Add ARMADA38X clkgen driver
Message-ID:  <202209070759.2877xeZ9081988@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by kd:

URL: https://cgit.FreeBSD.org/src/commit/?id=07c5be33f145906c8d80a86cbfa94e5847c513f9

commit 07c5be33f145906c8d80a86cbfa94e5847c513f9
Author:     Albert Jakieła <aja@semihalf.com>
AuthorDate: 2022-07-05 15:01:02 +0000
Commit:     Kornel Dulęba <kd@FreeBSD.org>
CommitDate: 2022-09-07 07:44:58 +0000

    Add ARMADA38X clkgen driver
    
    This patch introduces clkgen driver for Armada38x SoCs.
    Clock topology consists of single coreclk which supplies
    clock signal to CPU cores and peripherials.
    
    Reviewed by:    manu
    Obtained from:  Semihalf
    Differential Revision: https://reviews.freebsd.org/D36453
---
 sys/arm/mv/clk/armada38x_coreclk.c | 222 +++++++++++++++++++++++++++++++++++++
 sys/arm/mv/clk/armada38x_gen.c     | 100 +++++++++++++++++
 sys/arm/mv/clk/armada38x_gen.h     |  40 +++++++
 sys/arm/mv/files.arm7              |   3 +
 4 files changed, 365 insertions(+)

diff --git a/sys/arm/mv/clk/armada38x_coreclk.c b/sys/arm/mv/clk/armada38x_coreclk.c
new file mode 100644
index 000000000000..242be9f355a4
--- /dev/null
+++ b/sys/arm/mv/clk/armada38x_coreclk.c
@@ -0,0 +1,222 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause-FreeBSD
+ *
+ * Copyright (c) 2022 Semihalf.
+ *
+ * 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 ``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 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("$FressBSD$");
+
+#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 <machine/fdt.h>
+
+#include <dev/fdt/simplebus.h>
+#include <dev/ofw/ofw_bus.h>
+#include <dev/ofw/ofw_bus_subr.h>
+#include <dev/extres/clk/clk.h>
+
+#include <arm/mv/mvwin.h>
+#include <arm/mv/mvreg.h>
+#include <arm/mv/mvvar.h>
+
+#include <arm/mv/clk/armada38x_gen.h>
+
+#include "clkdev_if.h"
+
+#define ARMADA38X_CORECLK_MAXREG 0
+
+static struct resource_spec armada38x_coreclk_specs[] = {
+	{ SYS_RES_MEMORY,	0,	RF_ACTIVE },
+	{ -1, 0 }
+};
+
+struct armada38x_coreclk_softc {
+	struct resource	*res;
+	struct clkdom	*clkdom;
+	struct mtx	mtx;
+};
+
+static int armada38x_coreclk_attach(device_t dev);
+static int armada38x_coreclk_probe(device_t dev);
+
+static struct armada38x_gen_clknode_def gen_nodes[] =
+{
+	{
+		.def = {
+			.name = "coreclk_0",
+			.id = 0,
+			.parent_cnt = 0,
+		},
+	},
+	{
+		.def = {
+			.name = "coreclk_2",
+			.id = 1,
+			.parent_cnt = 0,
+		},
+	}
+};
+
+static int
+armada38x_coreclk_read_4(device_t dev, bus_addr_t addr, uint32_t *val)
+{
+	struct armada38x_coreclk_softc *sc;
+
+	sc = device_get_softc(dev);
+
+	if (addr > ARMADA38X_CORECLK_MAXREG)
+		return (EINVAL);
+
+	*val = bus_read_4(sc->res, addr);
+
+	return (0);
+}
+
+static int
+armada38x_coreclk_write_4(device_t dev, bus_addr_t addr, uint32_t val)
+{
+	struct armada38x_coreclk_softc *sc;
+
+	sc = device_get_softc(dev);
+
+	if (addr > ARMADA38X_CORECLK_MAXREG)
+		return (EINVAL);
+
+	bus_write_4(sc->res, addr, val);
+
+	return (0);
+}
+
+static void
+armada38x_coreclk_device_lock(device_t dev)
+{
+	struct armada38x_coreclk_softc *sc;
+
+	sc = device_get_softc(dev);
+	mtx_lock(&sc->mtx);
+}
+
+static void
+armada38x_coreclk_device_unlock(device_t dev)
+{
+	struct armada38x_coreclk_softc *sc;
+
+	sc = device_get_softc(dev);
+	mtx_unlock(&sc->mtx);
+}
+
+static int
+armada38x_coreclk_probe(device_t dev)
+{
+
+	if (!ofw_bus_status_okay(dev))
+		return (ENXIO);
+
+	if (!ofw_bus_is_compatible(dev, "marvell,armada-380-core-clock"))
+		return (ENXIO);
+
+	device_set_desc(dev, "ARMADA38X core-clock");
+
+	return (BUS_PROBE_DEFAULT);
+}
+
+static int
+armada38x_coreclk_create_coreclk(device_t dev)
+{
+	struct armada38x_coreclk_softc *sc;
+	int rv, i;
+
+	sc = device_get_softc(dev);
+
+	for (i = 0; i < nitems(gen_nodes); ++i) {
+		rv = armada38x_gen_register(sc->clkdom, &gen_nodes[i]);
+		if (rv)
+			return (rv);
+	}
+
+	return (rv);
+}
+
+static int
+armada38x_coreclk_attach(device_t dev)
+{
+	struct armada38x_coreclk_softc *sc;
+	int error;
+
+	sc = device_get_softc(dev);
+
+	if (bus_alloc_resources(dev, armada38x_coreclk_specs, &sc->res) != 0) {
+		device_printf(dev, "Cannot allocate resources.\n");
+		return (ENXIO);
+	}
+
+	mtx_init(&sc->mtx, device_get_nameunit(dev), NULL, MTX_DEF);
+
+	sc->clkdom = clkdom_create(dev);
+	if (NULL == sc->clkdom) {
+		device_printf(dev, "Cannot create clkdom\n");
+		return (ENXIO);
+	}
+
+	error = armada38x_coreclk_create_coreclk(dev);
+	if (0 != error) {
+		device_printf(dev, "Cannot create coreclk.\n");
+		return (error);
+	}
+
+	if (clkdom_finit(sc->clkdom) != 0)
+		panic("Cannot finalize clock domain initialization.\n");
+
+	if (bootverbose)
+		clkdom_dump(sc->clkdom);
+
+	return (0);
+}
+
+static device_method_t amada38x_coreclk_methods[] = {
+	DEVMETHOD(clkdev_write_4,	armada38x_coreclk_write_4),
+	DEVMETHOD(clkdev_read_4,	armada38x_coreclk_read_4),
+	DEVMETHOD(clkdev_device_lock,	armada38x_coreclk_device_lock),
+	DEVMETHOD(clkdev_device_unlock,	armada38x_coreclk_device_unlock),
+
+	DEVMETHOD(device_attach,	armada38x_coreclk_attach),
+	DEVMETHOD(device_probe,		armada38x_coreclk_probe),
+
+	DEVMETHOD_END
+};
+
+static driver_t armada38x_coreclk_driver = {
+	"armada38x_coreclk",
+	amada38x_coreclk_methods,
+	sizeof(struct armada38x_coreclk_softc),
+};
+
+EARLY_DRIVER_MODULE(armada38x_coreclk, simplebus, armada38x_coreclk_driver, 0, 0,
+    BUS_PASS_BUS + BUS_PASS_ORDER_MIDDLE);
diff --git a/sys/arm/mv/clk/armada38x_gen.c b/sys/arm/mv/clk/armada38x_gen.c
new file mode 100644
index 000000000000..7b7896fa8517
--- /dev/null
+++ b/sys/arm/mv/clk/armada38x_gen.c
@@ -0,0 +1,100 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause-FreeBSD
+ *
+ * Copyright (c) 2022 Semihalf.
+ *
+ * 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 ``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 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 <machine/bus.h>
+
+#include <dev/extres/clk/clk.h>
+
+#include <arm/mv/clk/armada38x_gen.h>
+
+#include "clkdev_if.h"
+
+#define SAR_A38X_TCLK_FREQ_SHIFT	15
+#define SAR_A38X_TCLK_FREQ_MASK		0x00008000
+
+#define TCLK_250MHZ					250 * 1000 * 1000
+#define TCLK_200MHZ					200 * 1000 * 1000
+
+#define	WR4(_clk, offset, val)					\
+	CLKDEV_WRITE_4(clknode_get_device(_clk), offset, val)
+#define	RD4(_clk, offset, val)					\
+	CLKDEV_READ_4(clknode_get_device(_clk), offset, val)
+#define	DEVICE_LOCK(_clk)					\
+	CLKDEV_DEVICE_LOCK(clknode_get_device(_clk))
+#define	DEVICE_UNLOCK(_clk)					\
+	CLKDEV_DEVICE_UNLOCK(clknode_get_device(_clk))
+
+static int
+armada38x_gen_recalc(struct clknode *clk, uint64_t *freq)
+{
+	uint32_t reg;
+
+	DEVICE_LOCK(clk);
+	RD4(clk, 0, &reg);
+	DEVICE_UNLOCK(clk);
+
+	reg = (reg & SAR_A38X_TCLK_FREQ_MASK) >> SAR_A38X_TCLK_FREQ_SHIFT;
+	*freq = reg ? TCLK_200MHZ : TCLK_250MHZ;
+
+	return (0);
+}
+
+static int
+armada38x_gen_init(struct clknode *clk, device_t dev)
+{
+	return (0);
+}
+
+static clknode_method_t armada38x_gen_clknode_methods[] = {
+	/* Device interface */
+	CLKNODEMETHOD(clknode_init,		armada38x_gen_init),
+	CLKNODEMETHOD(clknode_recalc_freq,	armada38x_gen_recalc),
+	CLKNODEMETHOD_END
+};
+
+DEFINE_CLASS_1(armada38x_gen_clknode, armada38x_gen_clknode_class,
+    armada38x_gen_clknode_methods, 0, clknode_class);
+
+int
+armada38x_gen_register(struct clkdom *clkdom, const struct armada38x_gen_clknode_def *clkdef)
+{
+	struct clknode *clk;
+
+	clk = clknode_create(clkdom, &armada38x_gen_clknode_class, &clkdef->def);
+	if (clk == NULL)
+		return (1);
+
+	clknode_register(clkdom, clk);
+
+	return(0);
+}
diff --git a/sys/arm/mv/clk/armada38x_gen.h b/sys/arm/mv/clk/armada38x_gen.h
new file mode 100644
index 000000000000..fecb23b08785
--- /dev/null
+++ b/sys/arm/mv/clk/armada38x_gen.h
@@ -0,0 +1,40 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause-FreeBSD
+ *
+ * Copyright (c) 2022 Semihalf.
+ *
+ * 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 ``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 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.
+ */
+
+#ifndef _ARMADA38X_GEN_H_
+#define _ARMADA38X_GEN_H_
+
+#include <dev/extres/clk/clk.h>
+
+struct armada38x_gen_clknode_def
+{
+	struct clknode_init_def		def;
+};
+
+int armada38x_gen_register(struct clkdom *clkdom,
+    const struct armada38x_gen_clknode_def *clkdef);
+
+#endif
diff --git a/sys/arm/mv/files.arm7 b/sys/arm/mv/files.arm7
index 6f93fd6edbc5..47354c68733a 100644
--- a/sys/arm/mv/files.arm7
+++ b/sys/arm/mv/files.arm7
@@ -34,3 +34,6 @@ dev/usb/controller/ehci_mv.c	optional	ehci
 dev/usb/controller/generic_xhci.c	optional	xhci
 dev/usb/controller/generic_xhci_fdt.c	optional	xhci
 dev/ahci/ahci_mv_fdt.c		optional	ahci
+
+arm/mv/clk/armada38x_coreclk.c	standard
+arm/mv/clk/armada38x_gen.c	standard



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202209070759.2877xeZ9081988>