Commit 21240eb9 authored by Philipp Zabel's avatar Philipp Zabel

reset: make (de)assert report success for self-deasserting reset drivers

By now there are drivers using shared reset controls and (de)assert
calls on platforms with self-deasserting reset lines and thus reset
drivers that do not implement .assert() and .deassert().
As long as the initial state of the reset line is deasserted, there
is no reason for a reset_control_assert call to return an error for
shared reset controls, or for a reset_control_deassert call to return
an error for either shared or exclusive reset controls: after a call
to reset_control_deassert the reset line is guaranteed to be deasserted,
and after a call to reset_control_assert it is valid for the reset
line to stay deasserted for shared reset controls.
Signed-off-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 17c82e20
...@@ -280,9 +280,6 @@ int reset_control_assert(struct reset_control *rstc) ...@@ -280,9 +280,6 @@ int reset_control_assert(struct reset_control *rstc)
if (reset_control_is_array(rstc)) if (reset_control_is_array(rstc))
return reset_control_array_assert(rstc_to_array(rstc)); return reset_control_array_assert(rstc_to_array(rstc));
if (!rstc->rcdev->ops->assert)
return -ENOTSUPP;
if (rstc->shared) { if (rstc->shared) {
if (WARN_ON(atomic_read(&rstc->triggered_count) != 0)) if (WARN_ON(atomic_read(&rstc->triggered_count) != 0))
return -EINVAL; return -EINVAL;
...@@ -292,6 +289,21 @@ int reset_control_assert(struct reset_control *rstc) ...@@ -292,6 +289,21 @@ int reset_control_assert(struct reset_control *rstc)
if (atomic_dec_return(&rstc->deassert_count) != 0) if (atomic_dec_return(&rstc->deassert_count) != 0)
return 0; return 0;
/*
* Shared reset controls allow the reset line to be in any state
* after this call, so doing nothing is a valid option.
*/
if (!rstc->rcdev->ops->assert)
return 0;
} else {
/*
* If the reset controller does not implement .assert(), there
* is no way to guarantee that the reset line is asserted after
* this call.
*/
if (!rstc->rcdev->ops->assert)
return -ENOTSUPP;
} }
return rstc->rcdev->ops->assert(rstc->rcdev, rstc->id); return rstc->rcdev->ops->assert(rstc->rcdev, rstc->id);
...@@ -321,9 +333,6 @@ int reset_control_deassert(struct reset_control *rstc) ...@@ -321,9 +333,6 @@ int reset_control_deassert(struct reset_control *rstc)
if (reset_control_is_array(rstc)) if (reset_control_is_array(rstc))
return reset_control_array_deassert(rstc_to_array(rstc)); return reset_control_array_deassert(rstc_to_array(rstc));
if (!rstc->rcdev->ops->deassert)
return -ENOTSUPP;
if (rstc->shared) { if (rstc->shared) {
if (WARN_ON(atomic_read(&rstc->triggered_count) != 0)) if (WARN_ON(atomic_read(&rstc->triggered_count) != 0))
return -EINVAL; return -EINVAL;
...@@ -332,6 +341,16 @@ int reset_control_deassert(struct reset_control *rstc) ...@@ -332,6 +341,16 @@ int reset_control_deassert(struct reset_control *rstc)
return 0; return 0;
} }
/*
* If the reset controller does not implement .deassert(), we assume
* that it handles self-deasserting reset lines via .reset(). In that
* case, the reset lines are deasserted by default. If that is not the
* case, the reset controller driver should implement .deassert() and
* return -ENOTSUPP.
*/
if (!rstc->rcdev->ops->deassert)
return 0;
return rstc->rcdev->ops->deassert(rstc->rcdev, rstc->id); return rstc->rcdev->ops->deassert(rstc->rcdev, rstc->id);
} }
EXPORT_SYMBOL_GPL(reset_control_deassert); EXPORT_SYMBOL_GPL(reset_control_deassert);
......
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