From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 0/2] omap_vout: remove cpu_is_* uses Date: Thu, 29 Nov 2012 08:29:27 -0800 Message-ID: <20121129162927.GF5312@atomide.com> References: <1352727220-22540-1-git-send-email-tomi.valkeinen@ti.com> <1421983.jJNXU7RvjW@avalon> <50B72B34.6080808@ti.com> <4208124.v72gFsjH2D@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <4208124.v72gFsjH2D@avalon> Sender: linux-media-owner@vger.kernel.org To: Laurent Pinchart Cc: Tomi Valkeinen , hvaibhav@ti.com, linux-media@vger.kernel.org, linux-omap@vger.kernel.org, Archit Taneja List-Id: linux-omap@vger.kernel.org * Laurent Pinchart [121129 01:37]: > Hi Tomi, > > On Thursday 29 November 2012 11:30:28 Tomi Valkeinen wrote: > > On 2012-11-28 17:13, Laurent Pinchart wrote: > > > On Monday 12 November 2012 15:33:38 Tomi Valkeinen wrote: > > >> Hi, > > >> > > >> This patch removes use of cpu_is_* funcs from omap_vout, and uses > > >> omapdss's version instead. The other patch removes an unneeded plat/dma.h > > >> include. > > >> > > >> These are based on current omapdss master branch, which has the omapdss > > >> version code. The omapdss version code is queued for v3.8. I'm not sure > > >> which is the best way to handle these patches due to the dependency to > > >> omapdss. The easiest option is to merge these for 3.9. > > >> > > >> There's still the OMAP DMA use in omap_vout_vrfb.c, which is the last > > >> OMAP dependency in the omap_vout driver. I'm not going to touch that, as > > >> it doesn't look as trivial as this cpu_is_* removal, and I don't have > > >> much knowledge of the omap_vout driver. > > >> > > >> Compiled, but not tested. > > > > > > Tested on a Beagleboard-xM. > > > > > > Tested-by: Laurent Pinchart > > > Acked-by: Laurent Pinchart > > > > Thanks. > > > > > The patches depend on unmerged OMAP DSS patches. Would you like to push > > > this series through linuxtv or through your DSS tree ? The later might be > > > easier, depending on when the required DSS patches will hit mainline. > > > > The DSS patches will be merged for 3.8. I can take this via the omapdss > > tree, as there probably won't be any conflicts with other v4l2 stuff. > > > > Or, we can just delay these until 3.9. These patches remove omap > > platform dependencies, helping the effort to get common ARM kernel. > > However, as there's still the VRFB code in the omap_vout driver, the > > dependency remains. Thus, in way, these patches alone don't help > > anything, and we could delay these for 3.9 and hope that > > omap_vout_vrfb.c gets converted also for that merge window. > > OK, I'll queue them for v3.9 then. Please rather queue the cpu_is_omap removal to v3.8 so I can remove plat/cpu.h for omap2+. Regards, Tony