Re: [PATCH 6.6.y] pwm: tiehrpwm: Clear period on disable to allow reconfiguration

["Uwe Kleine-König" <ukleinek@kernel.org>]

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

Hello again,

On Mon, Nov 09, 2026 at 09:17:29AM +0100, Uwe Kleine-König wrote:
> From: Alexander Sverdlin <alexander.sverdlin@siemens.com>
> 
> [ Upstream commit e74b9a7ee50071aad25d3989bf985292f90b0c6e ]
> 
> Both channels share one period register, so ehrpwm_pwm_config() rejects a
> period differing from the sibling's period_cycles[]. That value was cleared
> only in .free(), not in .disable(), so a stopped channel kept blocking the
> sibling from switching to a new frequency. Clear it in .disable() too.
> 
> Testcase:
> 
>   cd /sys/class/pwm/pwmchip0; echo 0 > export; echo 1 > export
>   for p in pwm0 pwm1; do
>     echo 1000000 > $p/period; echo 500000 > $p/duty_cycle
>     echo 1 > $p/enable; echo 0 > $p/enable; echo 2000000 > $p/period
>   done
>   echo 1 > pwm0/enable	# before: -EINVAL, after: succeeds
> 
> Fixes: 01b2d4536f02 ("pwm: pwm-tiehrpwm: Fix conflicting channel period setting")
> Cc: stable@vger.kernel.org
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@siemens.com>
> Link: https://patch.msgid.link/20260929120806.2790355-1-alexander.sverdlin@siemens.com
> Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>

I forgot to mention, this patch also applies to v5.10.x, so I assume
this patch is fine also for 6.1 and 5.15.

Best regards
Uwe

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