[PATCH] arm/mach-pxa: add pulse width modulator controller

Sascha Hauer s.hauer at pengutronix.de
Tue Jan 31 04:37:06 EST 2012


On Fri, Jan 27, 2012 at 05:07:04PM +0100, Robert Jarzmik wrote:
> Sascha Hauer <s.hauer at pengutronix.de> writes:
> 
> > On Thu, Jan 26, 2012 at 08:55:28PM +0100, Robert Jarzmik wrote:
> >> Add PXA embedded pulse width modulator support. The PWM can
> >> generate signals from 49.6kHz to 1.625MHz.
> >> 
> >> Signed-off-by: Robert Jarzmik <robert.jarzmik at free.fr>
> >> ---
> >
> > [...]
> >
> >> +
> >> +	/* NOTE: the clock to PWM has to be enabled first
> >> +	 * before writing to the registers
> >> +	 */
> >
> > Does this mean that pwm_enable() has to be called prior to pxm_config()?
> Yes.
> 
> > I think we should rather have a real pwm API.
> So do I.
> 
> > The following Kernel patch could be easily adopted and the overhead is
> > acceptable I think.
> Would you adapt it or do you want me to do it ?

I just sent a patch, compile tested only.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the barebox mailing list