<div dir="ltr">2013/7/1 Tzafrir Cohen <span dir="ltr">&lt;<a href="mailto:tzafrir.cohen@xorcom.com" target="_blank">tzafrir.cohen@xorcom.com</a>&gt;</span><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5">On Mon, May 27, 2013 at 09:35:21PM +0200, Lorenzo Miniero wrote:<br>
&gt; 2013/5/27 Tzafrir Cohen &lt;<a href="mailto:tzafrir.cohen@xorcom.com">tzafrir.cohen@xorcom.com</a>&gt;<br>
&gt;<br>
&gt; &gt; On Mon, May 27, 2013 at 04:09:08PM +0200, Lorenzo Miniero wrote:<br>
&gt; &gt; &gt; Dear all,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; I&#39;ve just published the patch on github:<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; <a href="https://github.com/meetecho/asterisk-opus" target="_blank">https://github.com/meetecho/asterisk-opus</a><br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; The README should be quite self explainatory, but if you need any<br>
&gt; &gt; &gt; additional info feel free to ask me.<br>
&gt; &gt; &gt; Any feedback will be more than welcome!<br>
&gt; &gt;<br>
&gt; &gt; $ diffstat asterisk-opus/asterisk_opus+vp8.diff<br>
&gt; &gt;  build_tools/<a href="http://menuselect-deps.in" target="_blank">menuselect-deps.in</a> |    1<br>
&gt; &gt;  channels/chan_sip.c            |   91 ++++++-<br>
&gt; &gt;  codecs/codec_opus.c            |  529<br>
&gt; &gt; +++++++++++++++++++++++++++++++++++++++++<br>
&gt; &gt;  codecs/ex_opus.h               |   35 ++<br>
&gt; &gt;  <a href="http://configure.ac" target="_blank">configure.ac</a>                   |    3<br>
&gt; &gt;  formats/format_vp8.c           |  195 +++++++++++++++<br>
&gt; &gt;  include/asterisk/format.h      |    4<br>
&gt; &gt;  main/channel.c                 |    2<br>
&gt; &gt;  main/format.c                  |   16 +<br>
&gt; &gt;  main/frame.c                   |   38 ++<br>
&gt; &gt;  main/rtp_engine.c              |    6<br>
&gt; &gt;  <a href="http://makeopts.in" target="_blank">makeopts.in</a>                    |    3<br>
&gt; &gt;  res/res_rtp_asterisk.c         |   42 +++<br>
&gt; &gt;  13 files changed, 960 insertions(+), 5 deletions(-)<br>
&gt; &gt;<br>
&gt; &gt; Any problem with including the format parts of this patch into Asterisk?<br>
&gt; &gt; Asterisk has limited support for H.264 video. I don&#39;t suppose Asterisk<br>
&gt; &gt; comes with a license for a H.264 video playback (let alone encoding).<br>
&gt; &gt;<br>
&gt; &gt; At first glance, the following parts of the patch seem to be related to<br>
&gt; &gt; formats, rather than codecs:<br>
&gt; &gt;<br>
&gt; &gt;  channels/chan_sip.c            |   91 ++++++-<br>
&gt; &gt;  formats/format_vp8.c           |  195 +++++++++++++++<br>
&gt; &gt;  include/asterisk/format.h      |    4<br>
&gt; &gt;  main/channel.c                 |    2<br>
&gt; &gt;  main/format.c                  |   16 +<br>
&gt; &gt;  main/rtp_engine.c              |    6<br>
&gt; &gt;  res/res_rtp_asterisk.c         |   42 +++<br>
&gt; &gt;<br>
&gt; &gt; Would a patch / review of those parts by Lorenzo be welcomed?<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt;<br>
&gt; Actually all the files have stuff related to both codecs: so the code<br>
&gt; related to Opus should be stripped from chan_sip, format, rtp_engine, etc.<br>
&gt; first in order to have them refer to VP8 only. The format_vp8.c file itself<br>
&gt; is pretty much a clone of the H.264 one, so if this one&#39;s ok I guess that<br>
&gt; one will be too.<br>
<br>
</div></div>In case I was not clear enough: I believe (and I suppose Matt&#39;s answer<br>
clarified it) that those parts of the patch can be merged into Asterisk<br>
as there&#39;s no reasonable fear of abusing any patents with them.<br>
<br>
If you agree with me, please follow up with the bug I opened:<br>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-21981" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-21981</a><br>
and attach the partial patch.<br>
<br>
The nice thing about it is that it includes almost all of the patching<br>
needed to actual Asterisk code (besides the changes to frame.c . I<br>
wonder if those are actually needed). Apart from that you only have<br>
codec modules and changes to the build system required to build them.<br>
<br>
(if this bug is a duplicate: sorry for the noise. I didn&#39;t see any<br>
progress here)<br>
<br>
If this is to be merged before the Asterisk 12 freeze, we don&#39;t have<br>
much time.<br>
<div class="im"><br></div></blockquote><div><br></div><div><br></div>The code in frame.c is only needed if Asterisk needs to be aware of how many samples an Opus RTP packet actually includes. Considering the patch would be targeted to only add passthrough support for the codec, I guess the changes in frame.c can be safely ignored.<div>
<br></div><div>I think format_vp8.c can be removed as well: in fact, VP8 passthrough support does not strictly require that file, which is only needed in case you also want to provide ability to read/write VP8 files (e.g., for announcements), something that code doesn&#39;t currently provide anyway (it&#39;s just a copy of format_h264.c acting as a placeholder).</div>
<div><br></div><div>I&#39;m out of the lab today so I won&#39;t be able to upload the partial patch until tomorrow, though, is it OK anyway considering the Asterisk 12 freeze deadline you mentioned?</div><div><br></div><div>
Lorenzo</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
--<br>
               Tzafrir Cohen<br>
icq#16849755              <a href="mailto:jabber%3Atzafrir.cohen@xorcom.com">jabber:tzafrir.cohen@xorcom.com</a><br>
<a href="tel:%2B972-50-7952406" value="+972507952406">+972-50-7952406</a>           mailto:<a href="mailto:tzafrir.cohen@xorcom.com">tzafrir.cohen@xorcom.com</a><br>
<a href="http://www.xorcom.com" target="_blank">http://www.xorcom.com</a><br>
<br>
</div><div class=""><div class="h5">--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>
</div></div></blockquote></div><br></div></div>