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

[Tomi Valkeinen <tomi.valkeinen@ti.com>]

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

On 2012-11-28 17:13, Laurent Pinchart wrote:
> Hi Tomi,
> 
> 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.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 899 bytes --]