[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