Commit e63adecc authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Steven Price

drm/panfrost: Reduce the amount of logs on deferred probe

There is no point to print deferred probe (and its failures to get
resources) as an error.  Also there is no need to print regulator errors
twice.

In case of multiple probe tries this would pollute the dmesg.
Signed-off-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: default avatarSteven Price <steven.price@arm.com>
Signed-off-by: default avatarSteven Price <steven.price@arm.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200527200544.7849-1-krzk@kernel.org
parent 2aae8ed1
...@@ -101,7 +101,9 @@ static int panfrost_regulator_init(struct panfrost_device *pfdev) ...@@ -101,7 +101,9 @@ static int panfrost_regulator_init(struct panfrost_device *pfdev)
pfdev->comp->num_supplies, pfdev->comp->num_supplies,
pfdev->regulators); pfdev->regulators);
if (ret < 0) { if (ret < 0) {
dev_err(pfdev->dev, "failed to get regulators: %d\n", ret); if (ret != -EPROBE_DEFER)
dev_err(pfdev->dev, "failed to get regulators: %d\n",
ret);
return ret; return ret;
} }
...@@ -213,10 +215,8 @@ int panfrost_device_init(struct panfrost_device *pfdev) ...@@ -213,10 +215,8 @@ int panfrost_device_init(struct panfrost_device *pfdev)
} }
err = panfrost_regulator_init(pfdev); err = panfrost_regulator_init(pfdev);
if (err) { if (err)
dev_err(pfdev->dev, "regulator init failed %d\n", err);
goto err_out0; goto err_out0;
}
err = panfrost_reset_init(pfdev); err = panfrost_reset_init(pfdev);
if (err) { if (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