lkml.org 
[lkml]   [2013]   [May]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRE: [PATCH 00/14] misc/ep93xx_pwm: cleanup driver for conversion to PWM framework
On Monday, May 27, 2013 8:20 AM, Arnd Bergmann wrote:
> On Saturday 25 May 2013, H Hartley Sweeten wrote:
>> This driver needs to be converted to the new PWM framework.
>>
>> Before converting it clean up all the cruft,
>>
>> H Hartley Sweeten (14):
>> misc/ep93xx_pwm: use managed device resources
>> misc/ep93xx_pwm: use {read,write}* instead of __raw_* versions for io
>> misc/ep93xx_pwm: remove ep93xx_pwm_{write,read}l() inline functions
>> misc/ep93xx_pwm: remove ep93xx_pwm_write_tc() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_write_dc() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_enable() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_disable() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_invert() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_normal() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_read_tc() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_is_enabled() inline function
>> misc/ep93xx_pwm: remove ep93xx_pwm_is_inverted() inline function
>> misc/ep93xx_pwm: use module_platform_driver()
>> misc/ep93xx_pwm: use kstrtol instead of strict_strtol
>
> Whole series:
>
> Acked-by: Arnd Bergmann arnd@arndb.de

Arnd,

Ryan Mallon has also provided a Reviewed-by for this series.

Will you be the one that merges this? I would like it to be in linux-next
before I convert it to the PWM framework.

Also, I have a question about the conversion.

If I strip the sysfs support out of this driver the conversion is quite simple.
But, my use for this driver requires user space control of the PWM.

Should I:
1) convert the driver to the PWM framework and leave the sysfs stuff in it
2) work out a generic sysfs support for the PWM framework and then
convert the driver
3) other...

I've been looking at 2) by doing something like how gpiolib does it. Do
you think that would be acceptable?

Thanks,
Hartley

\
 
 \ /
  Last update: 2013-05-27 19:01    [W:0.115 / U:0.364 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site