@@ -9,6 +9,7 @@
#include <linux/clk.h>
#include <linux/module.h>
#include <linux/devfreq-event.h>
+#include <linux/jiffies.h>
#include <linux/kernel.h>
#include <linux/of_address.h>
#include <linux/platform_device.h>
@@ -60,19 +61,6 @@ static int exynos_nocp_set_event(struct devfreq_event_dev *edev)
if (ret < 0)
goto out;
- ret = regmap_update_bits(nocp->regmap, NOCP_COUNTERS_2_SRC,
- NOCP_CNT_SRC_INTEVENT_MASK,
- NOCP_CNT_SRC_INTEVENT_CYCLE_MASK);
- if (ret < 0)
- goto out;
-
- ret = regmap_update_bits(nocp->regmap, NOCP_COUNTERS_3_SRC,
- NOCP_CNT_SRC_INTEVENT_MASK,
- NOCP_CNT_SRC_INTEVENT_CHAIN_MASK);
- if (ret < 0)
- goto out;
-
-
/* Set an alarm with a max/min value of 0 to generate StatALARM */
ret = regmap_write(nocp->regmap, NOCP_STAT_ALARM_MIN, 0x0);
if (ret < 0)
@@ -95,18 +83,6 @@ static int exynos_nocp_set_event(struct devfreq_event_dev *edev)
if (ret < 0)
goto out;
- ret = regmap_update_bits(nocp->regmap, NOCP_COUNTERS_2_ALARM_MODE,
- NOCP_CNT_ALARM_MODE_MASK,
- NOCP_CNT_ALARM_MODE_MIN_MAX_MASK);
- if (ret < 0)
- goto out;
-
- ret = regmap_update_bits(nocp->regmap, NOCP_COUNTERS_3_ALARM_MODE,
- NOCP_CNT_ALARM_MODE_MASK,
- NOCP_CNT_ALARM_MODE_MIN_MAX_MASK);
- if (ret < 0)
- goto out;
-
/* Enable the measurements by setting AlarmEn and StatEn */
ret = regmap_update_bits(nocp->regmap, NOCP_MAIN_CTL,
NOCP_MAIN_CTL_STATEN_MASK | NOCP_MAIN_CTL_ALARMEN_MASK,
@@ -145,6 +121,7 @@ static int exynos_nocp_get_event(struct devfreq_event_dev *edev,
{
struct exynos_nocp *nocp = devfreq_event_get_drvdata(edev);
unsigned int counter[4];
+ unsigned long dt;
int ret;
/* Read cycle count */
@@ -156,19 +133,29 @@ static int exynos_nocp_get_event(struct devfreq_event_dev *edev,
if (ret < 0)
goto out;
- ret = regmap_read(nocp->regmap, NOCP_COUNTERS_2_VAL, &counter[2]);
- if (ret < 0)
- goto out;
-
- ret = regmap_read(nocp->regmap, NOCP_COUNTERS_3_VAL, &counter[3]);
- if (ret < 0)
- goto out;
-
edata->load_count = ((counter[1] << 16) | counter[0]);
- edata->total_count = ((counter[3] << 16) | counter[2]);
-
- dev_dbg(&edev->dev, "%s (event: %lu/%lu)\n", edev->desc->name,
- edata->load_count, edata->total_count);
+ dt = jiffies_to_msecs((long)jiffies - (long)edata->prev_time);
+ if (dt == 0)
+ dt = 1;
+
+ /*
+ * count load in kB/s
+ * load = load / 1000 * 1000 / dt
+ */
+ edata->load_count = edata->load_count / dt;
+
+ /*
+ * count max bandwidth in kB/s:
+ * bw = freq * ((bus width in bits / 8) / nr of mem channels) / 1000
+ * where 128 bits / 8 bits per byte / 4 channels = 4
+ * so it is: bw = freq * 4 / 1000
+ */
+ edata->total_count = edata->curr_freq / 250;
+
+ dev_dbg(&edev->dev, "%s (event: %lu/%lu) %lums %lu%% 0x%08x\n",
+ edev->desc->name, edata->load_count, edata->total_count, dt,
+ edata->load_count * 100 / edata->total_count,
+ (counter[1] << 16) | counter[0]);
return 0;
There are two problems with exynos-nocp driver. First one is reading bytes and cycle counters and comparing them one to one without taking into account wcore bus width. Second one is that cycle counter depends on DREX DRAM clock, not on wcore clock. The latter problem was exposed by commit 6e7674c3c6df5 ("memory: Add DMC driver for Exynos5422"), which changes DRAM clock depending on memory read/write pressure and when it gets at higher freqency, NoCP cycle counter also increases. Note that both of these problems was there before this commit. The proposed solution is to use byte counter from NoCP h/w registers for current wcore bandwidth calculations and derive max bandwidth from current frequency. While at it, change from bytes to kilobytes to avoid overflow in later calculations in exynos-bus and devfreq drivers. Also while at it, remove cycle counters setting and reading and extend dev_dbg with time interval in miliseconds, percent of load, raw counter value in hex. Signed-off-by: Kamil Konieczny <k.konieczny@samsung.com> --- drivers/devfreq/event/exynos-nocp.c | 61 ++++++++++++----------------- 1 file changed, 24 insertions(+), 37 deletions(-)