Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 27 Jul 2022 00:14:01 GMT
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: b3189b3295d6 - stable/13 - arm64, qoriq_therm: configure the number of sites base don SoC
Message-ID:  <202207270014.26R0E1eA015287@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by bz:

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

commit b3189b3295d6bd2342001c35dacb24ac6365dc2b
Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2022-07-09 00:30:17 +0000
Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2022-07-26 22:41:38 +0000

    arm64, qoriq_therm: configure the number of sites base don SoC
    
    Configure the number of sites (sensors) based on SoC.
    This avoids timeouts reading non-existent sensors.
    
    The changes are based on mmel's initial work at:
    https://github.com/strejda/freebsd/commit/914e3f0098b090cb5c1492b0d24992012c5c553b
    
    Sponsored by:   Traverse Technologies (providing Ten64 HW for testing)
    Reviewed by:    mmel
    Differential Revision: https://reviews.freebsd.org/D35759
    
    (cherry picked from commit 394453302bca7a5147db126ce01a3497eb3415f7)
---
 sys/arm64/qoriq/qoriq_therm.c | 131 ++++++++++++++++++++++++++++++++++++++----
 1 file changed, 120 insertions(+), 11 deletions(-)

diff --git a/sys/arm64/qoriq/qoriq_therm.c b/sys/arm64/qoriq/qoriq_therm.c
index 33237fc94c07..1374a6d96558 100644
--- a/sys/arm64/qoriq/qoriq_therm.c
+++ b/sys/arm64/qoriq/qoriq_therm.c
@@ -88,13 +88,104 @@ static struct sysctl_ctx_list qoriq_therm_sysctl_ctx;
 
 struct tsensor default_sensors[] =
 {
-	{ 0, "site0", 0},
-	{ 1, "site1", 1},
-	{ 2, "site2", 2},
-	{ 3, "site3", 3},
-	{ 4, "site4", 4},
-	{ 5, "site5", 5},
-	{ 6, "site6", 6},
+	{ 0,	"site0",		0 },
+	{ 1,	"site1",		1 },
+	{ 2,	"site2",		2 },
+	{ 3,	"site3",		3 },
+	{ 4,	"site4",		4 },
+	{ 5,	"site5",		5 },
+	{ 6,	"site6",		6 },
+	{ 7,	"site7",		7 },
+	{ 8,	"site8",		8 },
+	{ 9,	"site9",		9 },
+	{ 10,	"site10",		10 },
+	{ 11,	"site11",		11 },
+	{ 12,	"site12",		12 },
+	{ 13,	"site13",		13 },
+	{ 14,	"site14",		14 },
+	{ 15,	"site15",		15 },
+};
+
+static struct tsensor imx8mq_sensors[] =
+{
+	{ 0,	"cpu",			0 },
+	{ 1,	"gpu",			1 },
+	{ 2,	"vpu",			2 },
+};
+
+static struct tsensor ls1012_sensors[] =
+{
+	{ 0,	"cpu-thermal",		0 },
+};
+
+static struct tsensor ls1028_sensors[] =
+{
+	{ 0,	"ddr-controller",	0 },
+	{ 1,	"core-cluster",		1 },
+};
+
+static struct tsensor ls1043_sensors[] =
+{
+	{ 0,	"ddr-controller",	0 },
+	{ 1,	"serdes",		1 },
+	{ 2,	"fman",			2 },
+	{ 3,	"core-cluster",		3 },
+};
+
+static struct tsensor ls1046_sensors[] =
+{
+	{ 0,	"ddr-controller",	0 },
+	{ 1,	"serdes",		1 },
+	{ 2,	"fman",			2 },
+	{ 3,	"core-cluster",		3 },
+	{ 4,	"sec",			4 },
+};
+
+static struct tsensor ls1088_sensors[] =
+{
+	{ 0,	"core-cluster",		0 },
+	{ 1,	"soc",			1 },
+};
+
+/* Note: tmu[1..7] not [0..6]. */
+static struct tsensor lx2080_sensors[] =
+{
+	{ 1,	"ddr-controller1",	0 },
+	{ 2,	"ddr-controller2",	1 },
+	{ 3,	"ddr-controller3",	2 },
+	{ 4,	"core-cluster1",	3 },
+	{ 5,	"core-cluster2",	4 },
+	{ 6,	"core-cluster3",	5 },
+	{ 7,	"core-cluster4",	6 },
+};
+
+static struct tsensor lx2160_sensors[] =
+{
+	{ 0,	"cluster6-7",		0 },
+	{ 1,	"ddr-cluster5",		1 },
+	{ 2,	"wriop",		2 },
+	{ 3,	"dce-qbman-hsio2",	3 },
+	{ 4,	"ccn-dpaa-tbu",		4 },
+	{ 5,	"cluster4-hsio3",	5 },
+	{ 6,	"cluster2-3",		6 },
+};
+
+struct qoriq_therm_socs {
+	const char		*name;
+	struct tsensor		*tsensors;
+	int			ntsensors;
+} qoriq_therm_socs[] = {
+#define	_SOC(_n, _a)	{ _n, _a, nitems(_a) }
+	_SOC("fsl,imx8mq",	imx8mq_sensors),
+	_SOC("fsl,ls1012a",	ls1012_sensors),
+	_SOC("fsl,ls1028a",	ls1028_sensors),
+	_SOC("fsl,ls1043a",	ls1043_sensors),
+	_SOC("fsl,ls1046a",	ls1046_sensors),
+	_SOC("fsl,ls1088a",	ls1088_sensors),
+	_SOC("fsl,ls2080a",	lx2080_sensors),
+	_SOC("fsl,lx2160a",	lx2160_sensors),
+	{ NULL,	NULL, 0 }
+#undef _SOC
 };
 
 static struct ofw_compat_data compat_data[] = {
@@ -260,7 +351,8 @@ static int
 qoriq_therm_attach(device_t dev)
 {
 	struct qoriq_therm_softc *sc;
-	phandle_t node;
+	struct qoriq_therm_socs *soc;
+	phandle_t node, root;
 	uint32_t sites;
 	int rid, rv;
 
@@ -307,9 +399,26 @@ qoriq_therm_attach(device_t dev)
 
 	sc->ver = (RD4(sc, TMU_VERSION) >> 8) & 0xFF;
 
-	/* XXX add per SoC customization */
-	sc->ntsensors = nitems(default_sensors);
-	sc->tsensors = default_sensors;
+	/* Select per SoC configuration. */
+	root = OF_finddevice("/");
+	if (root < 0) {
+		device_printf(dev, "Cannot get root node: %d\n", root);
+		goto fail;
+	}
+	soc = qoriq_therm_socs;
+	while (soc != NULL && soc->name != NULL) {
+		if (ofw_bus_node_is_compatible(root, soc->name))
+			break;
+		soc++;
+	}
+	if (soc == NULL) {
+		device_printf(dev, "Unsupported SoC, using default sites.\n");
+		sc->tsensors = default_sensors;
+		sc->ntsensors = nitems(default_sensors);
+	} else {
+		sc->tsensors = soc->tsensors;
+		sc->ntsensors = soc->ntsensors;
+	}
 
 	/* stop monitoring */
 	WR4(sc, TMU_TMR, 0);



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