<div dir="ltr"><div>Hi Martin,</div><div><br></div><div>Thanks for all your comments, I'm OK with all of them, I'll propose a new set </div><div>of patches tomorrow.</div><div><br></div><div>For the makefile part, it's in vlc-android git (iomx_hwbuffer.c is build in </div>
<div>libiomx.so). Patches are ready but I didn't send them since I was not sure to</div><div>propose the android headers patches that are about 6MB.</div><div><br></div><div>For iomx Graphics Buffers, I need to build libiomx against ics, jb, jbmr2 and</div>
<div>kitkat. I'm not 100% sure that a specific jbmr2 build is needed. I did it for</div><div>Archos Video Player, but maybe it was for an other private API (sigh...).</div><div><br></div><div>Thomas.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">2014-06-24 18:15 GMT+02:00 Martin Storsjö <span dir="ltr"><<a href="mailto:martin@martin.st" target="_blank">martin@martin.st</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<div class=""><br>
<br>
On Tue, 24 Jun 2014, Thomas Guillem wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This set of patches is my first attempt to backport android hardware buffers in<br>
OMX from the  Archos Video Player application that I'm working on.<br>
</blockquote>
<br></div>
Nice! I did a similar proof of concept some time ago, but it felt quite hacky - this does seem a little less bad.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I did the development on a Nexus 10 with last android 4.4.3, and I have a<br>
performance issue using HW buffers. I need to investigate more since I know<br>
that I don't have any performances issue with my other player. I also tested it<br>
on a RK30 and a QCOM device.<br>
<br>
By the way, I don't know if you want the patches to enable libiomx build on jb<br>
and kitkat (mostly android headers patch).<br>
</blockquote>
<br></div>
Yes, even though we might not want to build the iomx code specifically for those versions, it'd be useful to have your patches for testing.<br>
<br>
So far we've been able to use one single set of headers (from ICS) for IOMX for everything up to 4.3 - does some of the GraphicBuffer APIs change ABI inbetween to require building separate versions for 4.0 and 4.1-4.3?<br>

<br>
Also, the patches seem to be missing the makefile parts (both for the vlc-android and the vlc project itself) - these would be useful for understanding better how it all fits together.<br>
<br>
I'll follow up with comments on the individual patches...<br>
<br>
// Martin<br>
______________________________<u></u>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>listinfo/vlc-devel</a><br>
</blockquote></div><br></div>

<br>
<p><span style="color:rgb(136,136,136);font-family:arial,sans-serif;line-height:18.1875px;background-color:rgb(255,255,255)">This email and any files transmitted with it are confidential and are intended solely for the use of the individual or entity to which they are addressed. Access to this e-mail by anyone else is unauthorised. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited. E-mail messages are not necessarily secure. Archos does not accept responsibility for any changes made to this message after it was sent.</span></p>