<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Priority</th>
          <td>medium
          </td>
        </tr>

        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - xf86-video-openchrome doesn't work with xorg 1.14"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=62112">62112</a>
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>openchrome-devel@lists.freedesktop.org
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>xf86-video-openchrome doesn't work with xorg 1.14
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>normal
          </td>
        </tr>

        <tr>
          <th>Classification</th>
          <td>Unclassified
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>bugs.freedesktop.org@hardfalcon.net
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>Other
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>unspecified
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Driver/openchrome
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>xorg
          </td>
        </tr></table>
      <p>
        <div>
        <pre>xf86-video-openchrome doesn't work with xorg 1.14
because of the formerly deprecated function miInitializeBackingStore() being
used  in line 1876 of src/via_driver.c :

<a href="http://cgit.freedesktop.org/openchrome/xf86-video-openchrome/tree/src/via_driver.c?id=a925749eb21ba9e2e58845bc91554a09f20322ca">http://cgit.freedesktop.org/openchrome/xf86-video-openchrome/tree/src/via_driver.c?id=a925749eb21ba9e2e58845bc91554a09f20322ca</a>

miInitializeBackingStore() has been an empty function anyhow since november
2010, so the line in question can safely be removed:

<a href="http://cgit.freedesktop.org/xorg/xserver/commit/mi?id=c4c4676e6874b42c2371eee96faa2c2dbb59a704">http://cgit.freedesktop.org/xorg/xserver/commit/mi?id=c4c4676e6874b42c2371eee96faa2c2dbb59a704</a>

I therefore propose the following patch:

diff -rup a/src/via_driver.c b/src/via_driver.c
--- a/src/via_driver.c    2013-03-10 16:11:57.614059200 +0100
+++ b/src/via_driver.c    2013-03-10 16:12:46.374062244 +0100
@@ -1873,7 +1873,6 @@ VIAScreenInit(SCREEN_INIT_ARGS_DECL)
     if (!pVia->NoAccel && !viaInitExa(pScreen))
         return FALSE;

-    miInitializeBackingStore(pScreen);
     xf86SetBackingStore(pScreen);
 #if 0
     xf86SetSilkenMouse(pScreen);</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>