<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 14, 2017 at 8:55 PM, Leo Liu <span dir="ltr"><<a href="mailto:leo.liu@amd.com" target="_blank">leo.liu@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF"><span class="gmail-">
    <p><br>
    </p>
    <br>
    <div class="gmail-m_-6451156942813313588moz-cite-prefix">On 08/14/2017 11:19 AM, Gurkirpal Singh
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr"><br>
        <div class="gmail_extra"><br>
          <div class="gmail_quote">On Mon, Aug 14, 2017 at 8:05 PM, Leo
            Liu <span dir="ltr"><<a href="mailto:leo.liu@amd.com" target="_blank">leo.liu@amd.com</a>></span>
            wrote:<br>
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div bgcolor="#FFFFFF"><span class="gmail-m_-6451156942813313588gmail-">
                  <p><br>
                  </p>
                  <br>
                  <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230moz-cite-prefix">On
                    08/14/2017 05:46 AM, Julien Isorce wrote:<br>
                  </div>
                  <blockquote type="cite">
                    <div dir="ltr">Hi Leo,
                      <div><br>
                      </div>
                      <div>><span style="font-size:12.8px">It would
                          be better if you can extract the common code
                          between bellagio and tizonia to avoid the
                          duplication.</span><br>
                        <div><br>
                        </div>
                        <div>This is something Gurkirpal and me
                          discussed, like having
                          state_trackers/omx/common,
                          state_trackers/omx/bellagio,
                          state_trackers/omx/tizonia. To anticipate that
                          Gurkirpal sent an email <a href="https://www.mail-archive.com/mesa-dev@lists.freedesktop.org/msg153562.html" target="_blank">https://www.mail-archive<wbr>.com/mesa-dev@lists.freedeskto<wbr>p.org/msg153562.html</a>
                          during the Community Bonding period in May (a
                          few weeks from the date (early May) students
                          are officially accepted to the date (end May)
                          where student actually starts working)</div>
                        <div><br>
                        </div>
                        <div>The problem with the directory layout above
                          is that it would look like it will be built
                          together in the same shared library. This is
                          not good because it is suppose to
                          export OMX_ComponentInit for each target. Of
                          course this would still be possible to have 2
                          shared libs but I believe in state_trackers
                          dir, all sub dir are for one target. Also we
                          were afraid that there would be other
                          limitations so we decided to go for a separate
                          directory.</div>
                        <div>And since there were no objections in
                          Gurkirpal's mail above we went this way.</div>
                        <div><br>
                        </div>
                        <div>Now if I look into
                          state_trackers/omx_tizonia, in fact the common
                          code with state_trackers/omx_bellagio does not
                          have anything to do with omx. For example
                          "slice_header". Maybe it can be moved to
                          gallium/auxiliary/vl/ like it is done already
                          for vl_rbsp_init. Same for omx_get_screen.</div>
                        <div><br>
                        </div>
                        <div>So I suggest this 'factorization' to be not
                          a blocker for merging
                          state_trackers/omx_tizonia into usptream. But
                          later on we can move gradually bits from
                          state_trackers/omx_bellagio to
                          gallium/auxiliary. And then make
                          state_trackers/omx_tizonia use it as well.</div>
                        <div>This way you will only bother about
                          maintaining state_trackers/omx_bellagio the
                          time being. This also allows to not slowly get
                          its work lost.</div>
                        <div><br>
                        </div>
                        <div>Of course we would have done differently if
                          we knew advance. But as today Gurkirpal won't
                          have enough time to do this factorization/move
                          as the project ends in 1 week. Having all of
                          this merged in upstream is not mandatory to
                          succeed the project but Gurkirpal will need
                          some rest after these 3 months of hard work.
                          And who knows what happens after, whether he
                          will still be around after sometimes or not.
                          And this is entirely up to him.</div>
                        <div><br>
                        </div>
                        <div>So again I suggest this factorization/move
                          not to be a blocker for the reasons above.
                          What do you think?</div>
                      </div>
                    </div>
                  </blockquote>
                  <br>
                </span> When the whole project will be completed, as
                least same status as bellagio? Is there anyone keep
                working on this project after this step 1? <br>
              </div>
            </blockquote>
            <div>Hi, I plan on continuing working on this after the GSoC
              project ends. <br>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br></span>
    Is the GSoC project done with your current patches?<br>
    <br></div></blockquote><div>Apart from these patches, at least two more will be needed for the project. One for adding H.264 enc and other for adding EGLImage support to H.264 dec.</div><div>Since I keep revising the commits quite frequently I'll give the link to the branch <a href="https://github.com/gpalsingh/mesa/commits/gsoc">https://github.com/gpalsingh/mesa/commits/gsoc</a></div><div><br></div><div>Cheers</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="#FFFFFF">
    <br>
    Regards,<br>
    Leo<div><div class="gmail-h5"><br>
    <br>
    <blockquote type="cite">
      <div dir="ltr">
        <div class="gmail_extra">
          <div class="gmail_quote">
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div bgcolor="#FFFFFF"> <br>
                The left stuff compared to bellagio are codecs of mpeg2
                and hevc, and encoding that is more important, since the
                most usage for omx currently is to use transcoding from
                one codec to another.<br>
                <br>
                Other is the transcoding performance,  at least meet the
                performance of bellagio's.<br>
                <br>
                If the above could be addressed at early time,  we could
                remove Bellagio completely, then there is no need to do
                any refactor now. 
              </div>
            </blockquote>
            <div>The original plan was this but making gst-omx  OMX IL
              1.2 compatible has been a bit time consuming. </div>
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div bgcolor="#FFFFFF"> <br>
                <br>
                BTW what's the plan for new feature like EGL Images?<br>
              </div>
            </blockquote>
            <div>We've been able to make it work but with wrong colours
              in output. I've written more about it and the performance
              comparison in my blog post here <a href="https://singhcodes.wordpress.com/2017/08/04/gsoc-2017-third-phase-starts/" target="_blank">https://singhcodes.wordpress.<wbr>com/2017/08/04/gsoc-2017-<wbr>third-phase-starts/</a></div>
            <div>But recently the EGLImage hook stopped working and I've
              opened the issue <a href="https://github.com/gpalsingh/mesa/issues/15" target="_blank">https://github.com/gpalsingh/<wbr>mesa/issues/15</a>
              to keep track of it.</div>
            <div>This happened after we fixed another issue which made
              changes to gst-omx which might be the cause <a href="https://github.com/gpalsingh/mesa/issues/2#issuecomment-321839195" target="_blank">https://github.com/<wbr>gpalsingh/mesa/issues/2#<wbr>issuecomment-321839195</a></div>
            <div>I'm still looking into this.</div>
            <div><br>
            </div>
            <div>Cheers</div>
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div bgcolor="#FFFFFF"> <br>
                Regards,<br>
                Leo
                <div>
                  <div class="gmail-m_-6451156942813313588gmail-h5"><br>
                    <br>
                    <blockquote type="cite">
                      <div dir="ltr">
                        <div>
                          <div><br>
                          </div>
                          <div>Thx</div>
                          <div>Julien<br>
                            <div class="gmail_extra"><br>
                              <div class="gmail_quote">On 13 August 2017
                                at 16:52, Gurkirpal Singh <span dir="ltr"><<a href="mailto:gurkirpal204@gmail.com" target="_blank">gurkirpal204@gmail.com</a>></span>
                                wrote:<br>
                                <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                  <div dir="ltr"><br>
                                    <div class="gmail_extra"><br>
                                      <div class="gmail_quote"><span class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-">On
                                          Sun, Aug 13, 2017 at 8:47 AM,
                                          Leo Liu <span dir="ltr"><<a href="mailto:leo.liu@amd.com" target="_blank">leo.liu@amd.com</a>></span>
                                          wrote:<br>
                                          <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Where is the patch 1?</blockquote>
                                        </span>
                                        <div>Sorry for the patches got
                                          messed up somehow while
                                          sending, I could only see two
                                          patches on mail-archive but
                                          three on patchwork.</div>
                                        <div>Tried two times and same
                                          result.</div>
                                        <div>About the first one I got a
                                          mail saying that it was too
                                          large has been put aside for
                                          mod approval.</div>
                                        <div>The changes I made were to
                                          just rename the st/omx
                                          directory to st/omx_bellagio
                                          (the reason it became large)</div>
                                        <div>and renaming bits in the <a href="http://configure.ac" target="_blank">configure.ac</a>
                                          and Makefiles. </div>
                                        <div>
                                          <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-h5">
                                            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                              <div>
                                                <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-h5"><br>
                                                  <br>
                                                  <br>
                                                  On 08/12/2017 12:07
                                                  PM, Gurkirpal Singh
                                                  wrote:<br>
                                                  <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                                    Coexist with
                                                    --enable-omx so they
                                                    can be built
                                                    independently<br>
                                                    Detect tizonia
                                                    package config file<br>
                                                    Generate
                                                    libomxtiz_mesa.so
                                                    and install it to
                                                    libtizcore.pc::pluginsdir<br>
                                                    Only compile empty
                                                    source (target.c)
                                                    for now.<br>
                                                    <br>
                                                    v2: Show error
                                                    message when
                                                    --enable-omx is used
                                                    (Christian)<br>
                                                         Use single
                                                    PKG_CHECK_MODULES
                                                    for omx_tizonia
                                                    checks (Emil)<br>
                                                         Use spaces
                                                    instead of tabs<br>
                                                         Add checks
                                                    around omx-tizonia<br>
                                                    <br>
                                                    GSoC Project link: <a href="https://summerofcode.withgoogle.com/projects/#4737166321123328" rel="noreferrer" target="_blank">https://summerofcode.withgoogl<wbr>e.com/projects/#47371663211233<wbr>28</a><br>
                                                    <br>
                                                    Signed-off-by:
                                                    Gurkirpal Singh <<a href="mailto:gurkirpal204@gmail.com" target="_blank">gurkirpal204@gmail.com</a>><br>
Reviewed-and-Tested-by: Julien Isorce <<a href="mailto:julien.iso...@gmail.com" target="_blank">julien.iso...@gmail.com</a>><br>
                                                    ---<br>
                                                      <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> 
                                                                       
                                                              | 40
                                                    ++++++++++++++-<br>
                                                     
                                                    src/gallium/Makefile.am 
                                                                     
                                                     |  4 ++<br>
                                                     
                                                    src/gallium/targets/omx-tizoni<wbr>a/Makefile.am
                                                    | 77
                                                    +++++++++++++++++++++++++++++<br>
                                                     
                                                    src/gallium/targets/omx-tizoni<wbr>a/omx.sym 
                                                       | 11 +++++<br>
                                                     
                                                    src/gallium/targets/omx-tizoni<wbr>a/target.c 
                                                      |  2 +<br>
                                                      5 files changed,
                                                    132 insertions(+), 2
                                                    deletions(-)<br>
                                                      create mode 100644
src/gallium/targets/omx-tizoni<wbr>a/Makefile.am<br>
                                                      create mode 100644
src/gallium/targets/omx-tizoni<wbr>a/omx.sym<br>
                                                      create mode 100644
src/gallium/targets/omx-tizoni<wbr>a/target.c<br>
                                                    <br>
                                                    diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>
                                                    b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
                                                    index
                                                    38af96a..5669695
                                                    100644<br>
                                                    --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
                                                    +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
                                                    @@ -85,6 +85,7 @@
                                                    dnl Versions for
                                                    external
                                                    dependencies<br>
                                                     
                                                    DRI2PROTO_REQUIRED=2.8<br>
                                                     
                                                    GLPROTO_REQUIRED=1.4.14<br>
                                                     
                                                    LIBOMXIL_BELLAGIO_REQUIRED=0.0<br>
+LIBOMXIL_TIZONIA_REQUIRED=0.9<wbr>.0<br>
                                                     
                                                    LIBVA_REQUIRED=0.38.0<br>
                                                      VDPAU_REQUIRED=1.1<br>
                                                     
                                                    WAYLAND_REQUIRED=1.11<br>
                                                    @@ -1216,14 +1217,19
                                                    @@
                                                    AC_ARG_ENABLE([vdpau],<br>
                                                       
                                                     [enable_vdpau=auto])<br>
                                                     
                                                    AC_ARG_ENABLE([omx],<br>
                                                       
                                                     [AS_HELP_STRING([--enable-omx<wbr>],<br>
                                                    -       
                                                     [DEPRECATED: Use
                                                    --enable-omx-bellagio
                                                    instead
                                                    @<:@default=auto@:>@])],<br>
                                                    - 
                                                     [AC_MSG_ERROR([--enable-omx
                                                    is deprecated. Use
                                                    --enable-omx-bellagio
                                                    instead.])],<br>
                                                  </blockquote>
                                                  <br>
                                                </div>
                                              </div>
                                              Is this in patch 1?</blockquote>
                                          </div>
                                        </div>
                                        <div>Yes, it is so.</div>
                                        <div>
                                          <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-h5">
                                            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                              <div>
                                                <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-h5"><br>
                                                  <br>
                                                  <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                                    +       
                                                     [DEPRECATED: Use
                                                    --enable-omx-bellagio
                                                    or
                                                    --enable-omx-tizonia
                                                    instead
                                                    @<:@default=auto@:>@])],<br>
                                                    + 
                                                     [AC_MSG_ERROR([--enable-omx
                                                    is deprecated. Use
                                                    --enable-omx-bellagio
                                                    or
                                                    --enable-omx-tizonia
                                                    instead.])],<br>
                                                         [])<br>
                                                     
                                                    AC_ARG_ENABLE([omx-bellagio],<br>
                                                       
                                                     [AS_HELP_STRING([--enable-omx<wbr>-bellagio],<br>
                                                               [enable
                                                    OpenMAX Bellagio
                                                    library
                                                    @<:@default=disabled@:>@])],<br>
                                                       
                                                     [enable_omx_bellagio="$enable<wbr>val"],<br>
                                                       
                                                     [enable_omx_bellagio=no])<br>
+AC_ARG_ENABLE([omx-tizonia],<br>
                                                    + 
                                                     [AS_HELP_STRING([--enable-omx<wbr>-tizonia],<br>
                                                    +         [enable
                                                    OpenMAX Tizonia
                                                    library
                                                    @<:@default=disabled@:>@])],<br>
                                                    + 
                                                     [enable_omx_tizonia="$enablev<wbr>al"],<br>
                                                    + 
                                                     [enable_omx_tizonia=no])<br>
                                                     
                                                    AC_ARG_ENABLE([va],<br>
                                                       
                                                     [AS_HELP_STRING([--enable-va]<wbr>,<br>
                                                               [enable
                                                    va library
                                                    @<:@default=auto@:>@])],<br>
                                                    @@ -1275,6 +1281,7
                                                    @@ if test
                                                    "x$enable_opengl" =
                                                    xno -a \<br>
                                                             
                                                    "x$enable_xvmc" =
                                                    xno -a \<br>
                                                             
                                                    "x$enable_vdpau" =
                                                    xno -a \<br>
                                                             
                                                    "x$enable_omx_bellagio"
                                                    = xno -a \<br>
                                                    +       
                                                    "x$enable_omx_tizonia"
                                                    = xno -a \<br>
                                                             
                                                    "x$enable_va" = xno
                                                    -a \<br>
                                                             
                                                    "x$enable_opencl" =
                                                    xno; then<br>
                                                         
                                                    AC_MSG_ERROR([at
                                                    least one API should
                                                    be enabled])<br>
                                                    @@ -2121,6 +2128,10
                                                    @@ if test -n
                                                    "$with_gallium_drivers"
                                                    -a
                                                    "x$with_gallium_drivers"
                                                    != xswrast; then<br>
                                                             
                                                    PKG_CHECK_EXISTS([libomxil-bel<wbr>lagio
                                                    >=
                                                    $LIBOMXIL_BELLAGIO_REQUIRED],
[enable_omx_bellagio=yes], [enable_omx_bellagio=no])<br>
                                                          fi<br>
                                                      +    if test
                                                    "x$enable_omx_tizonia"
                                                    = xauto -a
                                                    "x$have_omx_platform"
                                                    = xyes; then<br>
                                                    +     
                                                     PKG_CHECK_EXISTS([libtizonia
                                                    >=
                                                    $LIBOMXIL_TIZONIA_REQUIRED],
[enable_omx_tizonia=yes], [enable_omx_tizonia=no])<br>
                                                    +    fi<br>
                                                    +<br>
                                                          if test
                                                    "x$enable_va" =
                                                    xauto -a
                                                    "x$have_va_platform"
                                                    = xyes; then<br>
                                                             
                                                    PKG_CHECK_EXISTS([libva
                                                    >=
                                                    $LIBVA_REQUIRED],
                                                    [enable_va=yes],
                                                    [enable_va=no])<br>
                                                          fi<br>
                                                    @@ -2130,6 +2141,7
                                                    @@ if test
                                                    "x$enable_dri" =
                                                    xyes -o \<br>
                                                             
                                                    "x$enable_xvmc" =
                                                    xyes -o \<br>
                                                             
                                                    "x$enable_vdpau" =
                                                    xyes -o \<br>
                                                             
                                                    "x$enable_omx_bellagio"
                                                    = xyes -o \<br>
                                                    +       
                                                    "x$enable_omx_tizonia"
                                                    = xyes -o \<br>
                                                             
                                                    "x$enable_va" =
                                                    xyes; then<br>
                                                         
                                                    need_gallium_vl=yes<br>
                                                      fi<br>
                                                    @@ -2138,6 +2150,7
                                                    @@
                                                    AM_CONDITIONAL(NEED_GALLIUM_VL<wbr>,
                                                    test
                                                    "x$need_gallium_vl"
                                                    = xyes)<br>
                                                      if test
                                                    "x$enable_xvmc" =
                                                    xyes -o \<br>
                                                             
                                                    "x$enable_vdpau" =
                                                    xyes -o \<br>
                                                             
                                                    "x$enable_omx_bellagio"
                                                    = xyes -o \<br>
                                                    +       
                                                    "x$enable_omx_tizonia"
                                                    = xyes -o \<br>
                                                             
                                                    "x$enable_va" =
                                                    xyes; then<br>
                                                         
                                                    PKG_CHECK_MODULES([VL],
                                                    [x11-xcb xcb
                                                    xcb-dri2 >=
                                                    $XCBDRI2_REQUIRED])<br>
                                                         
                                                    need_gallium_vl_winsys=yes<br>
                                                    @@ -2172,6 +2185,18
                                                    @@ if test
                                                    "x$enable_omx_bellagio"
                                                    = xyes; then<br>
                                                      fi<br>
                                                     
                                                    AM_CONDITIONAL(HAVE_ST_OMX_BEL<wbr>LAGIO,
                                                    test
                                                    "x$enable_omx_bellagio"
                                                    = xyes)<br>
                                                      +if test
                                                    "x$enable_omx_tizonia"
                                                    = xyes; then<br>
                                                    +    if test
                                                    "x$have_omx_platform"
                                                    != xyes; then<br>
                                                    +       
                                                    AC_MSG_ERROR([OMX
                                                    requires at least
                                                    one of the x11 or
                                                    drm platforms])<br>
                                                    +    fi<br>
                                                    +   
                                                    PKG_CHECK_MODULES([OMX_TIZONIA<wbr>],<br>
                                                    +                   
                                                      [libtizonia >=
$LIBOMXIL_TIZONIA_REQUIRED<br>
                                                    +                   
                                                       tizilheaders
                                                    >=
                                                    $LIBOMXIL_TIZONIA_REQUIRED<br>
                                                    +                   
                                                       libtizplatform
                                                    >=
                                                    $LIBOMXIL_TIZONIA_REQUIRED])<br>
                                                    +   
                                                    gallium_st="$gallium_st
                                                    omx_tizonia"<br>
                                                    +fi<br>
+AM_CONDITIONAL(HAVE_ST_OMX_TI<wbr>ZONIA, test "x$enable_omx_tizonia" =
                                                    xyes)<br>
                                                    +<br>
                                                      if test
                                                    "x$enable_va" =
                                                    xyes; then<br>
                                                          if test
                                                    "x$have_va_platform"
                                                    != xyes; then<br>
                                                             
                                                    AC_MSG_ERROR([VA
                                                    requires at least
                                                    one of the x11 drm
                                                    or wayland
                                                    platforms])<br>
                                                    @@ -2342,6 +2367,15
                                                    @@
                                                    AC_ARG_WITH([omx-bellagio-libd<wbr>ir],<br>
                                                                       
                                                                   
                                                     $PKG_CONFIG
                                                    --define-variable=libdir=\$lib<wbr>dir
--variable=pluginsdir libomxil-bellagio`])<br>
                                                     
                                                    AC_SUBST([OMX_BELLAGIO_LIB_INS<wbr>TALL_DIR])<br>
                                                      +dnl Directory for
                                                    OMX_TIZONIA libs<br>
                                                    +<br>
+AC_ARG_WITH([omx-tizonia-libd<wbr>ir],<br>
                                                    +   
                                                    [AS_HELP_STRING([--with-omx-ti<wbr>zonia-libdir=DIR],<br>
                                                    +        [directory
                                                    for the OMX_TIZONIA
                                                    libraries])],<br>
                                                    +   
                                                    [OMX_TIZONIA_LIB_INSTALL_DIR="<wbr>$withval"],<br>
                                                    +   
                                                    [OMX_TIZONIA_LIB_INSTALL_DIR=`<wbr>$PKG_CONFIG
--define-variable=libdir=\$lib<wbr>dir --variable=pluginsdir
                                                    libtizcore`])<br>
+AC_SUBST([OMX_TIZONIA_LIB_INS<wbr>TALL_DIR])<br>
                                                    +<br>
                                                      dnl Directory for
                                                    VA libs<br>
                                                       
                                                    AC_ARG_WITH([va-libdir],<br>
                                                    @@ -2840,6 +2874,7
                                                    @@
                                                    AC_CONFIG_FILES([Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/gl<wbr>x/xlib/Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/ni<wbr>ne/Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/om<wbr>x_bellagio/Makefile<br>
                                                    +               
                                                     src/gallium/state_trackers/om<wbr>x_tizonia/Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/os<wbr>mesa/Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/va<wbr>/Makefile<br>
                                                                     
                                                     src/gallium/state_trackers/vd<wbr>pau/Makefile<br>
                                                    @@ -2850,6 +2885,7
                                                    @@
                                                    AC_CONFIG_FILES([Makefile<br>
                                                                     
                                                     src/gallium/targets/dri/Makef<wbr>ile<br>
                                                                     
                                                     src/gallium/targets/libgl-xli<wbr>b/Makefile<br>
                                                                     
                                                     src/gallium/targets/omx-bella<wbr>gio/Makefile<br>
                                                    +               
                                                     src/gallium/targets/omx-tizon<wbr>ia/Makefile<br>
                                                                     
                                                     src/gallium/targets/opencl/Ma<wbr>kefile<br>
                                                                     
                                                     src/gallium/targets/opencl/me<wbr>sa.icd<br>
                                                                     
                                                     src/gallium/targets/osmesa/Ma<wbr>kefile<br>
                                                    diff --git
                                                    a/src/gallium/Makefile.am
b/src/gallium/Makefile.am<br>
                                                    index
                                                    2b930ac..e17c679
                                                    100644<br>
                                                    ---
                                                    a/src/gallium/Makefile.am<br>
                                                    +++
                                                    b/src/gallium/Makefile.am<br>
                                                    @@ -153,6 +153,10 @@
                                                    if
                                                    HAVE_ST_OMX_BELLAGIO<br>
                                                      SUBDIRS +=
                                                    state_trackers/omx_bellagio
                                                    targets/omx-bellagio<br>
                                                      endif<br>
                                                      +if
                                                    HAVE_ST_OMX_TIZONIA<br>
                                                    +SUBDIRS +=
                                                    state_trackers/omx_tizonia
                                                    targets/omx-tizonia<br>
                                                    +endif<br>
                                                    +<br>
                                                  </blockquote>
                                                  <br>
                                                </div>
                                              </div>
                                              So now we have 2 OMX ST
                                              under gallium state
                                              trackers<br>
                                              <br>
                                              Is possible that we could
                                              have both tizonia and
                                              bellagio  as sub-directory
                                              under st/omx?, so that
                                              they could share the same
                                              Makefile.am,
                                              Makefile.source, and
                                              entrypoint.[ch] etc.<br>
                                              even maybe same idea for
                                              gallium/target?<br>
                                              <br>
                                              Regards,<br>
                                              Leo
                                              <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-HOEnZb">
                                                <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-h5"><br>
                                                </div>
                                              </div>
                                            </blockquote>
                                          </div>
                                        </div>
                                        <div>The original plan was to
                                          completely replace the
                                          existing OMX IL ST however
                                          porting wasn't that
                                          straightforward since the new
                                          OMX IL version breaks backward
                                          compatibility.</div>
                                        <div>I think Julien will be able
                                          to expain better here.</div>
                                        <div><br>
                                        </div>
                                        <div>So the plan is still to
                                          replace the ST but for now
                                          we've postponed that task
                                          until all the components have
                                          been ported. As it is now
                                          we'll later just remove
                                          omx_bellagio and rename
                                          omx_tizonia to omx.</div>
                                        <div>I think that merging them
                                          both will make the transition
                                          a little bit harder.</div>
                                        <div><br>
                                        </div>
                                        <div>So maybe if you still
                                          insist can we mark it as
                                          good-to-have for later?
                                          Actually Julien gave the same
                                          idea earlier when we started
                                          on the project and decided to
                                          postpone it for later due to
                                          project time restraints.</div>
                                        <div><br>
                                        </div>
                                        <div>Cheers</div>
                                        <div>
                                          <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-h5">
                                            <div><br>
                                            </div>
                                            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                              <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-HOEnZb">
                                                <div class="gmail-m_-6451156942813313588gmail-m_3923089807667968230gmail-m_2961257225691895630gmail-m_-2082200114346312972gmail-m_7810120117750727457gmail-h5"><br>
                                                  <br>
                                                  <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                                      if
                                                    HAVE_GALLIUM_OSMESA<br>
                                                      SUBDIRS +=
                                                    state_trackers/osmesa
                                                    targets/osmesa<br>
                                                      endif<br>
                                                    diff --git
                                                    a/src/gallium/targets/omx-tizo<wbr>nia/Makefile.am
b/src/gallium/targets/omx-tizo<wbr>nia/Makefile.am<br>
                                                    new file mode 100644<br>
                                                    index
                                                    0000000..6baacaa<br>
                                                    --- /dev/null<br>
                                                    +++
                                                    b/src/gallium/targets/omx-tizo<wbr>nia/Makefile.am<br>
                                                    @@ -0,0 +1,77 @@<br>
                                                    +include
                                                    $(top_srcdir)/src/gallium/Auto<wbr>make.inc<br>
                                                    +<br>
                                                    +AM_CFLAGS = \<br>
                                                    +     
                                                     $(GALLIUM_TARGET_CFLAGS)<br>
                                                    +<br>
                                                    +omxdir =
                                                    $(OMX_TIZONIA_LIB_INSTALL_DIR)<br>
                                                    +omx_LTLIBRARIES = <a href="http://libomxtiz_mesa.la" rel="noreferrer" target="_blank">libomxtiz_mesa.la</a><br>
                                                    +<br>
+nodist_EXTRA_libomxtiz_mesa_l<wbr>a_SOURCES = dummy.cpp<br>
+libomxtiz_mesa_la_SOURCES =<br>
                                                    +<br>
+libomxtiz_mesa_la_LDFLAGS = \<br>
                                                    +       -shared \<br>
                                                    +       -module \<br>
                                                    +     
                                                     -no-undefined \<br>
                                                    +     
                                                     -avoid-version \<br>
                                                    +     
                                                     $(GC_SECTIONS) \<br>
                                                    +     
                                                     $(LD_NO_UNDEFINED)<br>
                                                    +<br>
                                                    +if
                                                    HAVE_LD_VERSION_SCRIPT<br>
+libomxtiz_mesa_la_LDFLAGS += \<br>
                                                    +     
                                                     -Wl,--version-script=$(top_sr<wbr>cdir)/src/gallium/targets/omx-<wbr>tizonia/omx.sym<br>
                                                    +endif #
                                                    HAVE_LD_VERSION_SCRIPT<br>
                                                    +<br>
+libomxtiz_mesa_la_LIBADD = \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/s<wbr>tate_trackers/omx_tizonia/<a href="http://libomxtiztracker.la" rel="noreferrer" target="_blank">libo<wbr>mxtiztracker.la</a>
                                                    \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/a<wbr>uxiliary/<a href="http://libgalliumvlwinsys.la" rel="noreferrer" target="_blank">libgalliumvlwinsys.la</a>
                                                    \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/a<wbr>uxiliary/<a href="http://libgalliumvl.la" rel="noreferrer" target="_blank">libgalliumvl.la</a>
                                                    \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/a<wbr>uxiliary/<a href="http://libgallium.la" rel="noreferrer" target="_blank">libgallium.la</a>
                                                    \<br>
                                                    +     
                                                     $(top_builddir)/src/util/<a href="http://libmesautil.la" rel="noreferrer" target="_blank">libm<wbr>esautil.la</a>
                                                    \<br>
                                                    +     
                                                     $(OMX_TIZONIA_LIBS)
                                                    \<br>
                                                    +     
                                                     $(OMX_TIZILHEADERS_LIBS)
                                                    \<br>
                                                    +     
                                                     $(OMX_TIZPLATFORM_LIBS)
                                                    \<br>
                                                    +     
                                                     $(LIBDRM_LIBS) \<br>
                                                    +     
                                                     $(GALLIUM_COMMON_LIB_DEPS)<br>
                                                    +<br>
                                                    +if
                                                    HAVE_PLATFORM_X11<br>
+libomxtiz_mesa_la_LIBADD += \<br>
                                                    +       $(VL_LIBS) \<br>
                                                    +     
                                                     $(XCB_DRI3_LIBS)<br>
                                                    +endif<br>
                                                    +<br>
+EXTRA_libomxtiz_mesa_la_DEPEN<wbr>DENCIES = omx.sym<br>
                                                    +EXTRA_DIST =
                                                    omx.sym<br>
                                                    +<br>
                                                    +if
                                                    HAVE_GALLIUM_STATIC_TARGETS<br>
                                                    +<br>
                                                    +TARGET_DRIVERS =<br>
                                                    +TARGET_CPPFLAGS =<br>
                                                    +TARGET_LIB_DEPS =<br>
                                                    +<br>
                                                    +<br>
                                                    +include
                                                    $(top_srcdir)/src/gallium/driv<wbr>ers/nouveau/Automake.inc<br>
                                                    +<br>
                                                    +include
                                                    $(top_srcdir)/src/gallium/driv<wbr>ers/r600/Automake.inc<br>
                                                    +include
                                                    $(top_srcdir)/src/gallium/driv<wbr>ers/radeonsi/Automake.inc<br>
                                                    +<br>
+libomxtiz_mesa_la_SOURCES += target.c<br>
+libomxtiz_mesa_la_CPPFLAGS = $(TARGET_CPPFLAGS)<br>
+libomxtiz_mesa_la_LIBADD += \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/a<wbr>uxiliary/pipe-loader/<a href="http://libpipe_loader_static.la" rel="noreferrer" target="_blank">libpipe_l<wbr>oader_static.la</a> \<br>
                                                    +     
                                                     $(GALLIUM_PIPE_LOADER_WINSYS_<wbr>LIBS)
                                                    \<br>
                                                    +     
                                                     $(TARGET_LIB_DEPS)
                                                    \<br>
                                                    +     
                                                     $(TARGET_COMPILER_LIB_DEPS)
                                                    \<br>
                                                    +     
                                                     $(TARGET_RADEON_WINSYS)
$(TARGET_RADEON_COMMON)<br>
                                                    +<br>
                                                    +else #
                                                    HAVE_GALLIUM_STATIC_TARGETS<br>
                                                    +<br>
+libomxtiz_mesa_la_LIBADD += \<br>
                                                    +     
                                                     $(top_builddir)/src/gallium/a<wbr>uxiliary/pipe-loader/<a href="http://libpipe_loader_dynamic.la" rel="noreferrer" target="_blank">libpipe_l<wbr>oader_dynamic.la</a><br>
                                                    +<br>
                                                    +endif #
                                                    HAVE_GALLIUM_STATIC_TARGETS<br>
                                                    +<br>
                                                    +if
                                                    HAVE_GALLIUM_LLVM<br>
+libomxtiz_mesa_la_LIBADD += $(LLVM_LIBS)<br>
+libomxtiz_mesa_la_LDFLAGS += $(LLVM_LDFLAGS)<br>
                                                    +endif<br>
                                                    diff --git
                                                    a/src/gallium/targets/omx-tizo<wbr>nia/omx.sym
b/src/gallium/targets/omx-tizo<wbr>nia/omx.sym<br>
                                                    new file mode 100644<br>
                                                    index
                                                    0000000..2aafb29<br>
                                                    --- /dev/null<br>
                                                    +++
                                                    b/src/gallium/targets/omx-tizo<wbr>nia/omx.sym<br>
                                                    @@ -0,0 +1,11 @@<br>
                                                    +{<br>
                                                    +       global:<br>
                                                    +             
                                                     OMX_ComponentInit;<br>
                                                    +<br>
                                                    +               #
                                                    Workaround for an
                                                    LLVM warning with
                                                    -simplifycfg-sink-common<br>
                                                    +               #
                                                    due to LLVM being
                                                    initialized multiple
                                                    times.<br>
                                                    +             
                                                     radeon_drm_winsys_create;<br>
                                                    +             
                                                     amdgpu_winsys_create;<br>
                                                    +       local:<br>
                                                    +               *;<br>
                                                    +};<br>
                                                    diff --git
                                                    a/src/gallium/targets/omx-tizo<wbr>nia/target.c
b/src/gallium/targets/omx-tizo<wbr>nia/target.c<br>
                                                    new file mode 100644<br>
                                                    index
                                                    0000000..308e23b<br>
                                                    --- /dev/null<br>
                                                    +++
                                                    b/src/gallium/targets/omx-tizo<wbr>nia/target.c<br>
                                                    @@ -0,0 +1,2 @@<br>
                                                    +#include
                                                    "target-helpers/drm_helper.h"<br>
                                                    +#include
                                                    "target-helpers/sw_helper.h"<br>
                                                  </blockquote>
                                                  <br>
                                                </div>
                                              </div>
                                            </blockquote>
                                          </div>
                                        </div>
                                      </div>
                                      <br>
                                    </div>
                                  </div>
                                  <br>
                                  ______________________________<wbr>_________________<br>
                                  mesa-dev mailing list<br>
                                  <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">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>
                          </div>
                        </div>
                      </div>
                    </blockquote>
                    <br>
                  </div>
                </div>
              </div>
            </blockquote>
          </div>
          <br>
        </div>
      </div>
    </blockquote>
    <br>
  </div></div></div>

</blockquote></div><br></div></div>