diff options
| author | Primoz Fiser <primoz.fiser@norik.com> | 2024-06-06 09:01:27 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-08-03 08:48:58 +0200 |
| commit | caba0bec56232b414ed200bea41f05c398f2249c (patch) | |
| tree | fe8d5d51bbeab9977d610cc6613ded857a9a55a0 /drivers | |
| parent | 7c61fd9eab640b7caafce125f70f4162973a9a0c (diff) | |
| download | linux-caba0bec56232b414ed200bea41f05c398f2249c.tar.gz linux-caba0bec56232b414ed200bea41f05c398f2249c.tar.bz2 linux-caba0bec56232b414ed200bea41f05c398f2249c.zip | |
OPP: ti: Fix ti_opp_supply_probe wrong return values
[ Upstream commit 3a1ac6b8f603a9310274990a0ad563a5fb709f59 ]
Function ti_opp_supply_probe() since commit 6baee034cb55 ("OPP: ti:
Migrate to dev_pm_opp_set_config_regulators()") returns wrong values
when all goes well and hence driver probing eventually fails.
Fixes: 6baee034cb55 ("OPP: ti: Migrate to dev_pm_opp_set_config_regulators()")
Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/opp/ti-opp-supply.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/opp/ti-opp-supply.c b/drivers/opp/ti-opp-supply.c index 8f3f13fbbb25..a8a696d2e03a 100644 --- a/drivers/opp/ti-opp-supply.c +++ b/drivers/opp/ti-opp-supply.c @@ -400,10 +400,12 @@ static int ti_opp_supply_probe(struct platform_device *pdev) } ret = dev_pm_opp_set_config_regulators(cpu_dev, ti_opp_config_regulators); - if (ret < 0) + if (ret < 0) { _free_optimized_voltages(dev, &opp_data); + return ret; + } - return ret; + return 0; } static struct platform_driver ti_opp_supply_driver = { |