Re: [PATCH 0/2] omap_vout: remove cpu_is_* uses

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

[-- Attachment #1: Type: text/plain, Size: 2007 bytes --]

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 <laurent.pinchart@ideasonboard.com>
> > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> 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.

-- 
Regards,

Laurent Pinchart

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 490 bytes --]