Re: [PATCH] pwm: tegra: fail probe on reset deassert errors

[Thierry Reding <thierry.reding@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1174 bytes --]

On Sun, Aug 30, 2026 at 08:58:43PM +0800, Pengpeng Hou wrote:
> tegra_pwm_probe() registers the PWM provider after deasserting the reset
> control without checking whether the reset controller accepted the request.
> 
> Propagate the failure through the existing runtime-PM cleanup path instead
> of publishing a PWM controller whose hardware may still be reset.
> 
> Fixes: 5dfbd2bd5439 ("pwm: tegra: Add support for reset control")
> Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
> ---
>  drivers/pwm/pwm-tegra.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

This is probably purely cosmetic since I've never seen a reset fail to
assert or deassert on Tegra (old generations can only fail if the reset
ID is invalid, meaning it's a bug in the framework, and new generations
may fail if for some reason the BPMP is not operational, in which case
the boot will fail much earlier).

Anyway, not saying NAK because from an API point of view it's correct to
check this. However, the Fixes: tag is a bit overkill. This does not
need to be backported to 10 years' worth of stable kernels.

Acked-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]