RE: [PATCH 08/10] OMAP: split plat-omap/common.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Balbi, Felipe
> Sent: Monday, October 04, 2010 12:31 PM
> To: Balbi, Felipe
> Cc: Shilimkar, Santosh; Paul Walmsley; linux-omap@xxxxxxxxxxxxxxx; linux-
> arm-kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 08/10] OMAP: split plat-omap/common.c
> 
> On Mon, Oct 04, 2010 at 01:56:24AM -0500, Balbi, Felipe wrote:
> >On Mon, Oct 04, 2010 at 12:34:48AM -0500, Shilimkar, Santosh wrote:
> >>Felipe,
> >>
> >>> -----Original Message-----
> >>> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> >>> owner@xxxxxxxxxxxxxxx] On Behalf Of Paul Walmsley
> >>> Sent: Saturday, October 02, 2010 3:06 AM
> >>> To: linux-omap@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> >>> Subject: [PATCH 08/10] OMAP: split plat-omap/common.c
> >>>
> >>> Split plat-omap/common.c into three pieces:
> >>>
> >>> 1. the 32KiHz sync timer and clocksource code, which now lives in
> >>>    plat-omap/32ksynctimer.c;
> >>>
> >>I remember you have converted the 32k sync time code to a platform
> driver.
> >>That was also addressing the multi-omap build and was removing all
> >>the compile time switches.
> >>Do you plan to rebase it and post it again ?
> >
> >I still have that sitting on my gitorious tree. The patch is attached
> >to this mail. I compile tested (only) for omap3_defconfig, it probably
> >needs to start using pm_runtime functions at some point and probably
> >move to drivers/clocksource instead of arch/arm/plat-omap.
> >
> >patch is also available at [1]
> >
> >[1] git://gitorious.org/usb/usb.git clksource
> 
> I made a few changes to make checkpatch.pl --strict happy. Attached
> 
Good !!
This patch addresses splitting as well as multi-omap builds and probably
also a better candidate in case we need to convert sync timer to hwmod
driver.

Paul, what's your take?

Regards,
Santosh

--
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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux