Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-07 Thread Benoît Thébaudeau
On Friday, September 7, 2012 9:41:38 AM, Sascha Hauer wrote: > On Thu, Sep 06, 2012 at 09:09:42PM +0200, Benoît Thébaudeau wrote: > > On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: > > > On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau > > > wrote: > > > > On Thursday,

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-07 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 09:09:42PM +0200, Benoît Thébaudeau wrote: > On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: > > On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: > > > On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: > > > > > > > > +

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-07 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 09:09:42PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: + int ret; +

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-07 Thread Benoît Thébaudeau
On Friday, September 7, 2012 9:41:38 AM, Sascha Hauer wrote: On Thu, Sep 06, 2012 at 09:09:42PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6,

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Benoît Thébaudeau
On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: > On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: > > On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: > > > > > > + int ret; > > > + > > > + ret = clk_prepare_enable(imx->clk_ipg); > > > + if (ret) >

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: > On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: > > > > + int ret; > > + > > + ret = clk_prepare_enable(imx->clk_ipg); > > + if (ret) > > + return ret; > > > > - return imx->config(chip, pwm,

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Benoît Thébaudeau
On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: > > From: Philipp Zabel > > The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq > (peripheral) clock. The ipg clock has to be enabled for this hardware > to work. The actual pwm output can either be driven by the

[PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Sascha Hauer
From: Philipp Zabel The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq (peripheral) clock. The ipg clock has to be enabled for this hardware to work. The actual pwm output can either be driven by the ipg clock or the ipg highfreq. The ipg highfreq has the advantage that it runs

[PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Sascha Hauer
From: Philipp Zabel p.za...@pengutronix.de The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq (peripheral) clock. The ipg clock has to be enabled for this hardware to work. The actual pwm output can either be driven by the ipg clock or the ipg highfreq. The ipg highfreq has the

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Benoît Thébaudeau
On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: From: Philipp Zabel p.za...@pengutronix.de The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq (peripheral) clock. The ipg clock has to be enabled for this hardware to work. The actual pwm output can either be

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: + int ret; + + ret = clk_prepare_enable(imx-clk_ipg); + if (ret) + return ret; - return imx-config(chip, pwm, duty_ns, period_ns);

Re: [PATCH 7/9] pwm i.MX: fix clock lookup

2012-09-06 Thread Benoît Thébaudeau
On Thursday, September 6, 2012 8:42:56 PM, Sascha Hauer wrote: On Thu, Sep 06, 2012 at 08:31:58PM +0200, Benoît Thébaudeau wrote: On Thursday, September 6, 2012 2:48:13 PM, Sascha Hauer wrote: + int ret; + + ret = clk_prepare_enable(imx-clk_ipg); + if (ret) + return

[PATCH 7/9] pwm i.MX: fix clock lookup

2012-08-28 Thread Sascha Hauer
From: Philipp Zabel The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq (peripheral) clock. The ipg clock has to be enabled for this hardware to work. The actual pwm output can either be driven by the ipg clock or the ipg highfreq. The ipg highfreq has the advantage that it runs

[PATCH 7/9] pwm i.MX: fix clock lookup

2012-08-28 Thread Sascha Hauer
From: Philipp Zabel p.za...@pengutronix.de The i.MX pwm core has two clocks: The ipg clock and the ipg highfreq (peripheral) clock. The ipg clock has to be enabled for this hardware to work. The actual pwm output can either be driven by the ipg clock or the ipg highfreq. The ipg highfreq has the