<div dir="ltr">Hi Gurkirpal, I am glad to see you continue working on this out of the GSoC project. <div>I have reviewed it already during that period so just giving my official "the series is:"<br></div><div><div><br></div><div>Reviewed-by: Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 30 November 2017 at 00:23, Dylan Baker <span dir="ltr"><<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Quoting Eric Engestrom (2017-11-29 07:19:02)<br>
<span class="">> On Wednesday, 2017-11-29 09:32:09 +0530, Gurkirpal Singh wrote:<br>
> > These are the refactored commits related to the GSoC project involving<br>
> > adding a st/omx state tracker using tizonia.<br>
> > There are still some parts of code that i didn't refactor yet as<br>
> > explained below:<br>
> > 1) I wasn't sure if it's okay to use #if-#else declaratives for function<br>
> > declarations. For eg: One function accepts omx_base_PortType and the other<br>
> > one vid_dec_PrivateType<br>
> > 2) Because of the argument type differences there is excessive amounts of<br>
> > #if-#else pairs will be needed<br>
> > So I decided to wait for review before making those changes.<br>
><br>
> I notice you left the meson build system out; could you give it a stab?<br>
> Feel free to ask me or Dylan for help if you get stuck :)<br>
<br>
</span>Do note that the meson omx code hasn't landed yet (hopefully that will happen<br>
in the next day or two though).<br>
<br>
One thing I'm not sure about there is how to handle the command line option to<br>
enable the build. Currently it's `gallium-omx`, and accepts, 'true', 'false',<br>
and 'auto'. It might make sense to make it more like the glx option, and accept<br>
'auto', 'bellagio', 'disabled', and when this lands 'tizonia'.<br>
<span class="HOEnZb"><font color="#888888"><br>
Dylan<br>
</font></span><br>______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
<br></blockquote></div><br></div>