Commit 52282163 authored by Steven Price's avatar Steven Price Committed by Rob Herring

drm/panfrost: Add missing check for pfdev->regulator

When modifying panfrost_devfreq_target() to support a device without a
regulator defined I missed the check on the error path. Let's add it.
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Fixes: e21dd290 ("drm/panfrost: Enable devfreq to work without regulator")
Signed-off-by: default avatarSteven Price <steven.price@arm.com>
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20190822093218.26014-1-steven.price@arm.com
parent f2521f77
...@@ -53,8 +53,10 @@ static int panfrost_devfreq_target(struct device *dev, unsigned long *freq, ...@@ -53,8 +53,10 @@ static int panfrost_devfreq_target(struct device *dev, unsigned long *freq,
if (err) { if (err) {
dev_err(dev, "Cannot set frequency %lu (%d)\n", target_rate, dev_err(dev, "Cannot set frequency %lu (%d)\n", target_rate,
err); err);
regulator_set_voltage(pfdev->regulator, pfdev->devfreq.cur_volt, if (pfdev->regulator)
pfdev->devfreq.cur_volt); regulator_set_voltage(pfdev->regulator,
pfdev->devfreq.cur_volt,
pfdev->devfreq.cur_volt);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment