From: baruch@tkos.co.il (Baruch Siach) Date: Thu, 10 Jun 2010 11:18:06 +0300 Subject: [PATCH 1/3] mx25: add watchdog clock In-Reply-To: <20100610081042.GB5333@pengutronix.de> References: <20100610081042.GB5333@pengutronix.de> Message-ID: <20100610081806.GC31840@jasper.tkos.co.il> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 > > --- > > 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) baruch > > > > #define _REGISTER_CLOCK(d, n, c) \ > > { \ > > @@ -225,6 +226,7 @@ static struct clk_lookup lookups[] = { > > _REGISTER_CLOCK("fec.0", NULL, fec_clk) > > _REGISTER_CLOCK("imxdi_rtc.0", NULL, dryice_clk) > > _REGISTER_CLOCK("imx-fb.0", NULL, lcdc_clk) > > + _REGISTER_CLOCK("imx-wdt.0", NULL, wdt_clk) > > }; > > > > int __init mx25_clocks_init(void) > > -- > > 1.7.1 -- ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -