Module Name:    src
Committed By:   jmcneill
Date:           Tue May 21 22:15:26 UTC 2019

Modified Files:
        src/sys/dev/fdt: cpufreq_dt.c

Log Message:
opp-v2 table seems to list opps from lowest to highest, so swap the order to 
ensure the highest frequency is first


To generate a diff of this commit:
cvs rdiff -u -r1.7 -r1.8 src/sys/dev/fdt/cpufreq_dt.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/fdt/cpufreq_dt.c
diff -u src/sys/dev/fdt/cpufreq_dt.c:1.7 src/sys/dev/fdt/cpufreq_dt.c:1.8
--- src/sys/dev/fdt/cpufreq_dt.c:1.7	Thu Nov  1 14:47:36 2018
+++ src/sys/dev/fdt/cpufreq_dt.c	Tue May 21 22:15:26 2019
@@ -1,4 +1,4 @@
-/* $NetBSD: cpufreq_dt.c,v 1.7 2018/11/01 14:47:36 jmcneill Exp $ */
+/* $NetBSD: cpufreq_dt.c,v 1.8 2019/05/21 22:15:26 jmcneill Exp $ */
 
 /*-
  * Copyright (c) 2015-2017 Jared McNeill <jmcne...@invisible.ca>
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpufreq_dt.c,v 1.7 2018/11/01 14:47:36 jmcneill Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpufreq_dt.c,v 1.8 2019/05/21 22:15:26 jmcneill Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -377,9 +377,11 @@ cpufreq_dt_parse_opp_v2(struct cpufreq_d
 		opp_uv = fdtbus_get_prop(opp_node, "opp-microvolt", &len);
 		if (opp_uv == NULL || len < 1)
 			return EINVAL;
-		sc->sc_opp[i].freq_khz = (u_int)(opp_hz / 1000);
-		sc->sc_opp[i].voltage_uv = be32toh(opp_uv[0]);
-		of_getprop_uint32(opp_node, "clock-latency-ns", &sc->sc_opp[i].latency_ns);
+		/* Table is in reverse order */
+		const int index = sc->sc_nopp - i - 1;
+		sc->sc_opp[index].freq_khz = (u_int)(opp_hz / 1000);
+		sc->sc_opp[index].voltage_uv = be32toh(opp_uv[0]);
+		of_getprop_uint32(opp_node, "clock-latency-ns", &sc->sc_opp[index].latency_ns);
 	}
 
 	return 0;

Reply via email to