Skip to content

Samsung PWM support cleanup

Sylwester Nawrocki linux-arm-kernel linux-pwm linux-samsung-soc 3 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest2 of 20 patches 0 repliesNew
  2. v20 of 18 patches 0 repliesSuperseded
  3. v10 of 15 patches 0 repliesSuperseded

2 of 20 patches are in this archive. The cover letter announces 20.

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
7/20ARM: SAMSUNG: Add new PWM platform device+0 -02 replies
13/20pwm: Add new pwm-samsung driver+0 -01 reply

Reviews trailers per patch

None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchSylwester NawrockiThierry Reding
SeriesR-b A-b
7/20
13/20

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [PATCH v4 00/20] Samsung PWM support cleanup
Quoted 21 lines · Tomasz Figa

Since we now have a proper Samsung PWM clocksource driver in place, we can proceed with further cleanup of PWM timers support on Samsung SoCs.

This series attempts to achieve this goal by:
1) fixing up few things in samsung_pwm_timer clocksource driver,
2) moving remaining Samsung platforms to the new clocksource driver,
3) removing old clocksource driver,
4) adding new multiplatform- and DT-aware PWM driver,
5) moving all Samsung platforms to use the new PWM driver,
6) removing old PWM driver,
7) removing all PWM-related code that is not used anymore.

Cleaning up the PWM driver is a bit tricky, because the design of current driver makes it completely unsuitable for DT and multiplatform and would require a heavy rework to make it usable, breaking any existing Samsung PWM users by the way. To avoid any breakage this series first renames the old driver, then adds new one using original name, migrates all platforms to use it and then finally removes the old driver.

See particular patches for more detailed descriptions.

Quoted 7 lines

[On S3C6410-based Tiny6410 (Mini6410-compatible) with pwm-beeper, SMDK6410 with PWM backlight and Exynos4210-based Origen board (with PWM0 attached to a scope)]

Tested-by Tomasz Figa<[email protected]>
[On S3C2440-based Mini2440 board]
Tested-by Sylwester Nawrocki<[email protected]>

I have retested this series on top of v3.11-rc1 with pwm-backlight (and buzzer
as the output :)) on Mini2440. It seems to work well - generated frequencies are correct. I'll check pulse widths with a scope tomorrow, as it's a bit late
now. FWIW you can add to this series my:

Reviewed-by Sylwester Nawrocki <[email protected]>
Quoted 9 lines
[On a s3c2416 based machine]
Tested-by Heiko Stuebner<[email protected]>
[Also not sure on what boards, but still]
Tested-by Mark Brown<[email protected]>
Acked-by Arnd Bergmann<[email protected]>
Acked-by Thierry Reding<[email protected]>
Acked-by Heiko Stuebner<[email protected]>

Thanks,
Sylwester

Read the cover letter in its thread