diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2024-04-15 07:31:30 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2024-04-16 12:27:07 +0200 |
commit | 8c29354b8025c0be1737c43e0f1cee6428ae5ba3 (patch) | |
tree | b2dd4adb1ee688399527855fbf7cb0cfb7518d74 /drivers/soc | |
parent | 6f369372f147ea95f1a3b7b90e96fabe1df3fe1d (diff) | |
download | barebox-8c29354b8025c0be1737c43e0f1cee6428ae5ba3.tar.gz barebox-8c29354b8025c0be1737c43e0f1cee6428ae5ba3.tar.xz |
soc: rockchip: io-domain: handle missing supply correctly
The original Linux code calls regulator_get_optional and handles its
-ENODEV return code, which morphed into regulator_get when ported to
barebox. The error handling stayed as-is leading to causing supplies to
trigger errors instead of being silently ignored.
As we use NULL to describe the dummy regulator in barebox, we can add a
trivial regulator_get_optional implementation and use it to fix the
I/O domain driver.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20240415053130.368278-1-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/soc')
-rw-r--r-- | drivers/soc/rockchip/io-domain.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/soc/rockchip/io-domain.c b/drivers/soc/rockchip/io-domain.c index 9f60b6eb4c..453d393cb0 100644 --- a/drivers/soc/rockchip/io-domain.c +++ b/drivers/soc/rockchip/io-domain.c @@ -165,7 +165,7 @@ static int rockchip_iodomain_probe(struct device *dev) if (!supply_name) continue; - reg = regulator_get(dev, supply_name); + reg = regulator_get_optional(dev, supply_name); if (IS_ERR(reg)) { ret = PTR_ERR(reg); |