[RFC,v1,6/7] regulator: set Power State Change Reason before hw_protection_shutdown()

Commit Message

Oleksij Rempel Jan. 19, 2024, 1:25 p.m. UTC

Store the state change reason to some black box, for later
investigation.

Signed-off-by: Oleksij Rempel <[e-mail]>
---
 drivers/regulator/core.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Mark Brown Jan. 19, 2024, 2:07 p.m. UTC | #1

On Fri, Jan 19, 2024 at 02:25:20PM +0100, Oleksij Rempel wrote:

> Store the state change reason to some black box, for later
> investigation.

Reviewed-by: Mark Brown <[e-mail]>

Patch

diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
index a6cb84af989e..d8a45c89f14a 100644
--- a/drivers/regulator/core.c
+++ b/drivers/regulator/core.c
@@ -19,6 +19,7 @@ 
 #include <linux/delay.h>
 #include <linux/gpio/consumer.h>
 #include <linux/of.h>
+#include <linux/pscr.h>
 #include <linux/reboot.h>
 #include <linux/regmap.h>
 #include <linux/regulator/of_regulator.h>
@@ -5074,6 +5075,7 @@  EXPORT_SYMBOL_GPL(regulator_bulk_free);
 static void regulator_handle_critical(struct regulator_dev *rdev,
 				      unsigned long event)
 {
+	enum power_state_change_reason pscr;
 	const char *reason = NULL;
 
 	if (!rdev->constraints->system_critical)
@@ -5082,17 +5084,21 @@  static void regulator_handle_critical(struct regulator_dev *rdev,
 	switch (event) {
 	case REGULATOR_EVENT_UNDER_VOLTAGE:
 		reason = "System critical regulator: voltage drop detected";
+		pscr = PSCR_UNDER_VOLTAGE;
 		break;
 	case REGULATOR_EVENT_OVER_CURRENT:
 		reason = "System critical regulator: over-current detected";
+		pscr = PSCR_OVER_CURRENT;
 		break;
 	case REGULATOR_EVENT_FAIL:
 		reason = "System critical regulator: unknown error";
+		pscr = PSCR_REGULATOR_FAILURE;
 	}
 
 	if (!reason)
 		return;
 
+	set_power_state_change_reason(pscr);
 	hw_protection_shutdown(reason,
 			       rdev->constraints->uv_less_critical_window_ms);
 }