> -----Original Message----- > From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx] > Sent: Thursday, September 30, 2010 1:11 AM > To: Nayak, Rajendra > Cc: Raja, Govindraj; Tony Lindgren; linux-omap@xxxxxxxxxxxxxxx; Varadarajan, Charulatha > Subject: Re: [GIT PULL] for testing: OMAP hwmod driver conversions: watchdog, UART, i2c > > "Nayak, Rajendra" <rnayak@xxxxxx> writes: > > >> -----Original Message----- > >> From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx] > >> Sent: Thursday, September 30, 2010 12:49 AM > >> To: Nayak, Rajendra; Raja, Govindraj > >> Cc: Tony Lindgren; linux-omap@xxxxxxxxxxxxxxx; Varadarajan, Charulatha > >> Subject: Re: [GIT PULL] for testing: OMAP hwmod driver conversions: watchdog, UART, i2c > >> > >> "Nayak, Rajendra" <rnayak@xxxxxx> writes: > >> > >> [...] > >> > >> > > >> > I forgot to mention, there are similar WARN's that I see on 2430sdp even with UART > >> > > >> > >> Govidraj, can you look into these UART issues please? > >> > >> Looks like you might need to double check the bitfields and the > >> module_offs for both of these. > > > > Btw, the WARN's come only for UART3 and the error at hwmod init shows that > > maybe _wait_target_ready fails. > > > > [ 0.000000] NR_IRQS:402 > > [ 0.000000] Clocking rate (Crystal/DPLL/MPU): 13.0/660/330 MHz > > [ 0.000000] omap_hwmod: uart3: cannot be enabled (3) > > > > UART3 on 24xx is in CM_IDLEST2_CORE, and the 2430 hwmod data has it in > CM_IDLEST1_CORE. > > Can you test the change below? I don't have a 2430, and it appears that > this is already correct in the 2420 hwmod data. > > Kevin > > > diff --git a/arch/arm/mach-omap2/omap_hwmod_2430_data.c b/arch/arm/mach-omap2/omap_hwmod_2430_data.c > index 92f4ec0..5b61f7c 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_2430_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_2430_data.c > @@ -449,9 +449,9 @@ static struct omap_hwmod omap2430_uart3_hwmod = { > .prcm = { > .omap2 = { > .module_offs = CORE_MOD, > - .prcm_reg_id = 1, > + .prcm_reg_id = 2, > .module_bit = OMAP24XX_EN_UART3_SHIFT, > - .idlest_reg_id = 1, > + .idlest_reg_id = 2, > .idlest_idle_bit = OMAP24XX_EN_UART3_SHIFT, > }, > }, This does seem to fix the issue on 2430. I don't see the error at hwmod init or the WARN's later. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html