From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 0/2] omap_vout: remove cpu_is_* uses Date: Thu, 29 Nov 2012 17:39:45 +0100 Message-ID: <2148719.6v36XORg4e@avalon> References: <1352727220-22540-1-git-send-email-tomi.valkeinen@ti.com> <4208124.v72gFsjH2D@avalon> <20121129162927.GF5312@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <20121129162927.GF5312@atomide.com> Sender: linux-media-owner@vger.kernel.org To: Tony Lindgren Cc: Tomi Valkeinen , hvaibhav@ti.com, linux-media@vger.kernel.org, linux-omap@vger.kernel.org, Archit Taneja , mchehab@redhat.com List-Id: linux-omap@vger.kernel.org Hi Tony, On Thursday 29 November 2012 08:29:27 Tony Lindgren wrote: > * Laurent Pinchart [121129 01:37]: > > 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+. In that case the patches should go through the DSS tree. Mauro, are you fine with that ? -- Regards, Laurent Pinchart