summaryrefslogtreecommitdiffstats
path: root/sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch
diff options
context:
space:
mode:
Diffstat (limited to 'sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch')
-rw-r--r--sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch77
1 files changed, 77 insertions, 0 deletions
diff --git a/sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch b/sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch
new file mode 100644
index 00000000..70bc2d3c
--- /dev/null
+++ b/sys-kernel/boest-v5.0.15/raspberrypi/0135-hwmon-raspberrypi-Prevent-voltage-low-warnings-from-.patch
@@ -0,0 +1,77 @@
+From 1d988f96947cd9e0d6132175a4a2c782906a5cb2 Mon Sep 17 00:00:00 2001
+From: Stefan Wahren <stefan.wahren@i2se.com>
+Date: Sat, 6 Oct 2018 16:46:18 +0200
+Subject: [PATCH 135/194] hwmon: raspberrypi: Prevent voltage low warnings from
+ filling log
+
+Although the correct fix for low voltage warnings is to
+improve the power supply, the current implementation
+of the detection can fill the log if the warning
+happens freqently. This replaces the logging with
+slightly custom ratelimited logging.
+
+Signed-off-by: James Hughes <james.hughes@raspberrypi.org>
+Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
+---
+ drivers/hwmon/raspberrypi-hwmon.c | 41 ++++++++++++++++++++++++++++---
+ 1 file changed, 37 insertions(+), 4 deletions(-)
+
+diff --git a/drivers/hwmon/raspberrypi-hwmon.c b/drivers/hwmon/raspberrypi-hwmon.c
+index 0d0457245e7d..d64392882208 100644
+--- a/drivers/hwmon/raspberrypi-hwmon.c
++++ b/drivers/hwmon/raspberrypi-hwmon.c
+@@ -15,6 +15,36 @@
+ #include <linux/workqueue.h>
+ #include <soc/bcm2835/raspberrypi-firmware.h>
+
++/*
++ * This section defines some rate limited logging that prevent
++ * repeated messages at much lower Hz than the default kernel settings.
++ * It's usually 5s, this is 5 minutes.
++ * Burst 3 means you may get three messages 'quickly', before
++ * the ratelimiting kicks in.
++ */
++#define LOCAL_RATELIMIT_INTERVAL (5 * 60 * HZ)
++#define LOCAL_RATELIMIT_BURST 3
++
++#ifdef CONFIG_PRINTK
++#define printk_ratelimited_local(fmt, ...) \
++({ \
++ static DEFINE_RATELIMIT_STATE(_rs, \
++ LOCAL_RATELIMIT_INTERVAL, \
++ LOCAL_RATELIMIT_BURST); \
++ \
++ if (__ratelimit(&_rs)) \
++ printk(fmt, ##__VA_ARGS__); \
++})
++#else
++#define printk_ratelimited_local(fmt, ...) \
++ no_printk(fmt, ##__VA_ARGS__)
++#endif
++
++#define pr_crit_ratelimited_local(fmt, ...) \
++ printk_ratelimited_local(KERN_CRIT pr_fmt(fmt), ##__VA_ARGS__)
++#define pr_info_ratelimited_local(fmt, ...) \
++printk_ratelimited_local(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
++
+ #define UNDERVOLTAGE_STICKY_BIT BIT(16)
+
+ struct rpi_hwmon_data {
+@@ -47,10 +77,13 @@ static void rpi_firmware_get_throttled(struct rpi_hwmon_data *data)
+ if (new_uv == old_uv)
+ return;
+
+- if (new_uv)
+- dev_crit(data->hwmon_dev, "Undervoltage detected!\n");
+- else
+- dev_info(data->hwmon_dev, "Voltage normalised\n");
++ if (new_uv) {
++ pr_crit_ratelimited_local("Under-voltage detected! (0x%08x)\n",
++ value);
++ } else {
++ pr_info_ratelimited_local("Voltage normalised (0x%08x)\n",
++ value);
++ }
+
+ sysfs_notify(&data->hwmon_dev->kobj, NULL, "in0_lcrit_alarm");
+ }