From bce3a70ec5c9896bd29df3a29d4c3b0ee4623f6c Mon Sep 17 00:00:00 2001 From: Peter Bigot Date: Mon, 2 Nov 2020 05:36:38 -0600 Subject: [PATCH] regulator: check manager init for failure Use the return value to verify success when assertions are enabled. Signed-off-by: Peter Bigot --- drivers/regulator/regulator_fixed.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/regulator/regulator_fixed.c b/drivers/regulator/regulator_fixed.c index 89e091715fc..5424a67df15 100644 --- a/drivers/regulator/regulator_fixed.c +++ b/drivers/regulator/regulator_fixed.c @@ -264,14 +264,17 @@ static const struct regulator_driver_api api_onoff = { static int regulator_fixed_init_onoff(const struct device *dev) { struct driver_data_onoff *data = dev->data; - int rc = common_init(dev, &data->gpio); + int rc; data->dev = dev; - onoff_manager_init(&data->mgr, &transitions); + rc = onoff_manager_init(&data->mgr, &transitions); + __ASSERT_NO_MSG(rc == 0); + #ifdef CONFIG_MULTITHREADING k_delayed_work_init(&data->delayed_work, onoff_worker); #endif /* CONFIG_MULTITHREADING */ + rc = common_init(dev, &data->gpio); if (rc >= 0) { rc = 0; }