On Fri, May 28, 2010 at 2:35 AM, Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> wrote: > Hi Govindraj, > > "Govindraj.R" <govindraj.raja@xxxxxx> writes: > >> Patch series is based on remotes/origin/pm-wip/govindraj >> branch from Kevin's PM tree. >> >> Patches are tested with 3430SDP. >> Have updated 2420/2430 hwmod data files >> it would be great if some one can test the same. > > OK, I've updated pm-wip/uart to include most of this series as well as > some of the other fixes I posted yesterday to get things working on > OMAP2. > >> Govindraj.R (6): >> Serial: Clean up set_uart_globals. >> Serial: Add UART4 hwmod data. >> Serial: Modify OMAP3 hwmod data file with uart data. >> Serial: Add UART hwmod data for 2420. >> Serial: Add UART hwmod data for 2430. >> Serial: Update UART HWMOD layer. > > I've kept patch 1 as is, but changed the subject slightly. > > Patches 2, 4 and 5 I've folded into the patch that added hwmod data > for OMAP3. > > Patch 6 I folded into the initial UART conversion patch and also > changed the authorship for that patch to you, since you've done most > of that work. > Thanks. Looks great now. > Patch 3, I've dropped as that shouldn't be needed after you switch > to platform_get_resource_byname(). > One comment here, In that patch other than swapping rx and tx fields. I had modified name field from uart*_hwmod to uart* - .name = "uart[1-3]_hwmod", + .name = "uart[1-3]", I think this might be necessary. To be in sync with auto generated data file. I think it can be incorporated with, OMAP2/3: UART: add omap_hwmod data for UARTs 1-4 > Please update your omap-serial driver on top of the new pm-wip/uart > branch and then we'll be basically ready for upstream. > Sure and here is the update: http://marc.info/?l=linux-serial&m=127504585628001&w=2 Meanwhile I took wip-uart patches to pm branch and tested enabling sleep_while_idle and enable_off_mode on 3430/3630SDP with omap3_pm_defconfig. Things worked fine with ret/off count getting incremented. --- Regards, Govindraj.R > Kevin > >> arch/arm/mach-omap2/omap_hwmod_2420_data.c | 184 ++++++++++++++++++++++++++++ >> arch/arm/mach-omap2/omap_hwmod_2430_data.c | 184 ++++++++++++++++++++++++++++ >> arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 66 +++++++++- >> arch/arm/mach-omap2/prcm-common.h | 2 + >> arch/arm/mach-omap2/serial.c | 38 +++---- >> arch/arm/plat-omap/common.c | 17 --- >> arch/arm/plat-omap/include/plat/common.h | 1 - >> arch/arm/plat-omap/include/plat/dma.h | 2 + >> arch/arm/plat-omap/include/plat/irqs.h | 2 + >> 9 files changed, 449 insertions(+), 47 deletions(-) > -- > 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 > -- 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