Re: Re: Re: [PATCH v13 5/5] clocksource: dw_apb_timer: Use reset array API to handle all resets
From: Philipp Zabel
Date: Tue Jul 28 2026 - 07:46:52 EST
On Di, 2026-07-28 at 18:59 +0800, Xuyang Dong wrote:
[...]
> Will drop "reset_control_put(rstc);" and "rstc = NULL;".
> Keep only the content below:
>
> - rstc = of_reset_control_get(np, NULL);
> + rstc = of_reset_control_array_get_optional_exclusive(np);
> if (!IS_ERR(rstc)) {
> reset_control_assert(rstc);
> reset_control_deassert(rstc);
> }
>
> Do you think this change is correct?
If each timer has their own exclusive resets (or none at all), and they
have to be triggered at about the same time (if multiple), without any
ordering requirements, then yes, this looks correct.
regards
Philipp