Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-11-19 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 01:48:59PM -0300, Fabio Estevam wrote: > With the new i.mx clock framework the mxc_w1 clock is registered as: > > clk_register_clkdev(clk[owire_gate], NULL, "mxc_w1.0" > > So we do not need to pass "owire" string and can use NULL instead. > > While at it, also fix the clo

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-11-18 Thread GregKH
On Sun, Nov 18, 2012 at 11:56:13PM -0200, Fabio Estevam wrote: > Hi Sascha, > > On Wed, Oct 3, 2012 at 4:28 AM, Evgeniy Polyakov wrote: > > On Tue, Oct 02, 2012 at 10:04:19PM -0300, Fabio Estevam > > (feste...@gmail.com) wrote: > >> > I have no objections per se, but I'm hardly an expert in imx

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-11-18 Thread Fabio Estevam
Hi Sascha, On Wed, Oct 3, 2012 at 4:28 AM, Evgeniy Polyakov wrote: > On Tue, Oct 02, 2012 at 10:04:19PM -0300, Fabio Estevam (feste...@gmail.com) > wrote: >> > I have no objections per se, but I'm hardly an expert in imx clock >> > framework :) >> > >> > Since it is only one patch in set of 3, I

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-10-03 Thread Sascha Hauer
On Thu, Sep 06, 2012 at 01:48:59PM -0300, Fabio Estevam wrote: > With the new i.mx clock framework the mxc_w1 clock is registered as: > > clk_register_clkdev(clk[owire_gate], NULL, "mxc_w1.0" > > So we do not need to pass "owire" string and can use NULL instead. > > While at it, also fix the clo

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-10-03 Thread Evgeniy Polyakov
On Tue, Oct 02, 2012 at 10:04:19PM -0300, Fabio Estevam (feste...@gmail.com) wrote: > > I have no objections per se, but I'm hardly an expert in imx clock > > framework :) > > > > Since it is only one patch in set of 3, I suppose it will be pushed > > through different tree than w1. Feel free to a

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-10-02 Thread Fabio Estevam
Sascha, On Tue, Oct 2, 2012 at 2:32 AM, Evgeniy Polyakov wrote: > On Mon, Oct 01, 2012 at 02:51:44PM -0300, Fabio Estevam > (fabio.este...@freescale.com) wrote: >> Evgeny, >> >> Any comments, please? > > I have no objections per se, but I'm hardly an expert in imx clock > framework :) > > Since

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-10-01 Thread Evgeniy Polyakov
On Mon, Oct 01, 2012 at 02:51:44PM -0300, Fabio Estevam (fabio.este...@freescale.com) wrote: > Evgeny, > > Any comments, please? I have no objections per se, but I'm hardly an expert in imx clock framework :) Since it is only one patch in set of 3, I suppose it will be pushed through different

Re: [PATCH v2 1/3] w1: mxc_w1: Adapt the clock name to the new clock framework

2012-10-01 Thread Fabio Estevam
Evgeny, Any comments, please? Fabio Estevam wrote: > With the new i.mx clock framework the mxc_w1 clock is registered as: > > clk_register_clkdev(clk[owire_gate], NULL, "mxc_w1.0" > > So we do not need to pass "owire" string and can use NULL instead. > > While at it, also fix the clock error