RE: [PATCH 00/14] misc/ep93xx_pwm: cleanup driver for conversion toPWM framework

From: H Hartley Sweeten
Date: Mon May 27 2013 - 12:28:12 EST


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@xxxxxxxx

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--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/