SPEAr320: Add pwm support
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 3 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 3 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/3] SPEAr320: Add pwm support
Quoted 13 lines · Arnd Bergmann
Quoted 12 lines · Sascha Hauer
Quoted 2 lines
Sascha sent a patchset for something similar earlier. But don't know what happened to it.
Nothing yet, unfortunately. It was a straight forward implementation which basically implements the existing pwm functions and adds register/unregister functions.
Bill Gatliff posts a pwm framework since 2008. I wasn't aware of his patches by the time I wrote my pwm layer. Bills patches target more the motor controller use case and are (in my opinion) too complicated. I can't agree on his approach, but didn't want to go with competing patches either.
Quoted 7 lines
Quoted 2 lines
As Bills patches still didn't make it into the kernel Maybe it's time to respin my version.
Yes, I think that would be good, then we can add the new SPEAr320 pwm driver to the new subsystem as a first example, and move the other drivers over after that.
Obviously this can be done, but as it has already taken too long for that framework to get added in mainline, and still we don't know how much more time it will take. I would suggest to push this patchset now. And i will update this accordingly once that framework comes up.
Signature · 1 line
viresh