[PATCH v2 0/8] pwm: mediatek: Convert to waveform API

Uwe Kleine-König u.kleine-koenig at baylibre.com
Wed Aug 6 07:43:42 PDT 2025


Hello,

On Fri, Jul 25, 2025 at 05:45:04PM +0200, Uwe Kleine-König wrote:
> here comes v2 of my effort to convert the pwm-mediatek driver to the new
> waveform API. Changes since (implicit) v1 (available at
> https://lore.kernel.org/linux-pwm/cover.1751994509.git.u.kleine-koenig@baylibre.com/):
> 
>  - Rebase on top of the latest fix for period and duty setting
>    (https://lore.kernel.org/linux-pwm/20250724210041.2513590-2-u.kleine-koenig@baylibre.com/)
> 
>  - Don't report a disabled PWM when the PWM is disabled. Though that
>    seems counter intuitive, this is the only way to allow the consumer
>    to query the minimal period. This wasn't necessary in v1 because back
>    then the driver claimed to provide a non-disabled setting for
>    duty_cycle=0 .
> 
> As before the patches in the middle don't serve a functional purpose
> because the last patch removes the changes again. Still I consider it
> useful because it reduces the last patch to what is needed in such a
> conversion.
> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (8):
>   pwm: mediatek: Simplify representation of channel offsets
>   pwm: mediatek: Introduce and use a few more register defines
>   pwm: mediatek: Rework parameters for clk helper function
>   pwm: mediatek: Initialize clks when the hardware is enabled at probe
>     time
>   pwm: mediatek: Implement .get_state() callback
>   pwm: mediatek: Fix various issues in the .apply() callback
>   pwm: mediatek: Lock and cache clock rate
>   pwm: mediatek: Convert to waveform API

i pushed the first 7 patches to pwm/for-nexxt now. I found a twist in
the conversion to the waveform APIs that I first want to think about a
bit more. The problem is that a setting with a small duty cycle now
reports the PWM as disabled. While this is technically correct,
this isn't helpful to let the consumer find the smallest possible
period.

Will followup when I left the current rabbit hole.

Best regards
Uwe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20250806/715446cd/attachment.sig>


More information about the linux-arm-kernel mailing list