[PATCH 1/3] mx25: add watchdog clock

[Wolfram Sang]

On Thu, Jun 10, 2010 at 11:18:06AM +0300, Baruch Siach wrote:
> Hi Wolfram,
> 
> On Thu, Jun 10, 2010 at 10:10:42AM +0200, Wolfram Sang wrote:
> > On Wed, Jun 09, 2010 at 02:07:00PM +0300, Baruch Siach wrote:
> > > Signed-off-by: Baruch Siach <baruch at tkos.co.il>
> > > ---
> > >  arch/arm/mach-mx25/clock.c |    2 ++
> > >  1 files changed, 2 insertions(+), 0 deletions(-)
> > > 
> > > diff --git a/arch/arm/mach-mx25/clock.c b/arch/arm/mach-mx25/clock.c
> > > index 1550149..f9662d9 100644
> > > --- a/arch/arm/mach-mx25/clock.c
> > > +++ b/arch/arm/mach-mx25/clock.c
> > > @@ -191,6 +191,7 @@ DEFINE_CLOCK(i2c_clk,	 0, CCM_CGCR0,  6, get_rate_i2c, NULL, NULL);
> > >  DEFINE_CLOCK(fec_clk,	 0, CCM_CGCR1, 15, get_rate_ipg, NULL, &fec_ahb_clk);
> > >  DEFINE_CLOCK(dryice_clk, 0, CCM_CGCR1,  8, get_rate_ipg, NULL, NULL);
> > >  DEFINE_CLOCK(lcdc_clk,	 0, CCM_CGCR1, 29, get_rate_lcdc, NULL, &lcdc_per_clk);
> > > +DEFINE_CLOCK(wdt_clk,    0, CCM_CGCR1, 19, get_rate_ipg, NULL,  NULL);
> > 
> > That is clk_gpt1 according to my RM (Rev 1)?
> 
> Right. It should be CCM_CGCR2. I'll fix this.
> 
> Note that this bit is marked "Reserved" in the RM. But a look at the Freescale 
> supplied BSP reveals the following line in crm_regs.h:
> 
> #define MXC_CCM_CGCR2_WDOG_OFFSET        (51-32)

OK. Please add this information also in a short comment above the clock
definition.

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20100610/819481d9/attachment.sig>