1. 08 Aug, 2020 2 commits
  2. 26 Oct, 2019 1 commit
  3. 13 Oct, 2019 3 commits
  4. 29 Jan, 2019 1 commit
  5. 28 Jan, 2019 1 commit
  6. 24 Jan, 2019 2 commits
  7. 22 Jan, 2019 18 commits
  8. 21 Jan, 2019 9 commits
    • Rhys Kidd's avatar
      xv/nv30, nv40: Mark local NV30GetSurfaceFormat and NV40GetSurfaceFormat as static · 764b42ba
      Rhys Kidd authored
      Avoids warnings with gcc 8.2:
      
      nv30_xv_tex.c:138:1: warning: no previous prototype for ‘NV30GetSurfaceFormat’ [-Wmissing-prototypes]
       NV30GetSurfaceFormat(PixmapPtr ppix, int *fmt_ret)
       ^~~~~~~~~~~~~~~~~~~~
      
      nv40_xv_tex.c:138:1: warning: no previous prototype for ‘NV40GetSurfaceFormat’ [-Wmissing-prototypes]
       NV40GetSurfaceFormat(PixmapPtr ppix, int *fmt_ret)
       ^~~~~~~~~~~~~~~~~~~~
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      764b42ba
    • Rhys Kidd's avatar
      xv: Mark local NVSetupTexturedVideo function as static · 0fa6ba42
      Rhys Kidd authored
      Avoids warning with gcc 8.2:
      
      nouveau_xv.c:2136:1: warning: no previous prototype for ‘NVSetupTexturedVideo’ [-Wmissing-prototypes]
       NVSetupTexturedVideo (ScreenPtr pScreen, XF86VideoAdaptorPtr *textureAdaptor)
       ^~~~~~~~~~~~~~~~~~~~
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      0fa6ba42
    • Rhys Kidd's avatar
      dri2: Mark local create/destroy buffer and copy region functions as static · 2905ea76
      Rhys Kidd authored
      Avoids warnings with gcc 8.2:
      
      nouveau_dri2.c:38:1: warning: no previous prototype for ‘nouveau_dri2_create_buffer2’ [-Wmissing-prototypes]
       nouveau_dri2_create_buffer2(ScreenPtr pScreen, DrawablePtr pDraw, unsigned int attachment,
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
      nouveau_dri2.c:109:1: warning: no previous prototype for ‘nouveau_dri2_create_buffer’ [-Wmissing-prototypes]
       nouveau_dri2_create_buffer(DrawablePtr pDraw, unsigned int attachment,
       ^~~~~~~~~~~~~~~~~~~~~~~~~~
      nouveau_dri2.c:117:1: warning: no previous prototype for ‘nouveau_dri2_destroy_buffer2’ [-Wmissing-prototypes]
       nouveau_dri2_destroy_buffer2(ScreenPtr pScreen, DrawablePtr pDraw, DRI2BufferPtr buf)
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      nouveau_dri2.c:131:1: warning: no previous prototype for ‘nouveau_dri2_destroy_buffer’ [-Wmissing-prototypes]
       nouveau_dri2_destroy_buffer(DrawablePtr pDraw, DRI2BufferPtr buf)
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
      nouveau_dri2.c:137:1: warning: no previous prototype for ‘nouveau_dri2_copy_region2’ [-Wmissing-prototypes]
       nouveau_dri2_copy_region2(ScreenPtr pScreen, DrawablePtr pDraw, RegionPtr pRegion,
       ^~~~~~~~~~~~~~~~~~~~~~~~~
      nouveau_dri2.c:215:1: warning: no previous prototype for ‘nouveau_dri2_copy_region’ [-Wmissing-prototypes]
       nouveau_dri2_copy_region(DrawablePtr pDraw, RegionPtr pRegion,
       ^~~~~~~~~~~~~~~~~~~~~~~~
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      2905ea76
    • Rhys Kidd's avatar
      wfb: Remove declaration for undefined function nouveau_wfb_init() · eb93278f
      Rhys Kidd authored
      Fixes: 794a2772 ("nv50: use libwfb for pixmap access when driver pixmaps enabled")
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      eb93278f
    • Rhys Kidd's avatar
      xv: Avoid shadowed declaration of 'int ret' in NVPutImage · 5233f3c8
      Rhys Kidd authored
      Warning reported by gcc 8.2:
      
      nouveau_xv.c: In function ‘NVPutImage’:
      nouveau_xv.c:1369:7: warning: declaration of ‘ret’ shadows a previous local [-Wshadow]
         int ret = BadImplementation;
             ^~~
      nouveau_xv.c:1073:6: note: shadowed declaration is here
        int ret, i;
            ^~~
      
      Fixes: a50bba66 ("xv: misc cleanups")
      Cc: Ben Skeggs <bskeggs@redhat.com>
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      5233f3c8
    • Rhys Kidd's avatar
      xv: Avoid shadowed declaration of 'int i' in NVPutImage · 22dd49f6
      Rhys Kidd authored
      int i is accessed outside immediate scope so leave declaration at
      the highest common scope level:
      
        1073:  int ret, i;
               ...
               // Highest common scope
               ...
        1193:  if (newTTSize <= destination_buffer->size) {
                 ...
                 // Used in this scope
                 ...
        1248:  } else {
                 ...
                 // Used in this scope
                 ...
        1316:  }
               // Not used in this scope
      
      Warning reported by gcc 8.2:
      
      nouveau_xv.c: In function ‘NVPutImage’:
      nouveau_xv.c:1195:7: warning: declaration of ‘i’ shadows a previous local [-Wshadow]
         int i = 0;
             ^
      nouveau_xv.c:1073:11: note: shadowed declaration is here
        int ret, i;
                 ^
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      22dd49f6
    • Rhys Kidd's avatar
      wfb: Fix missing init function decls behind FB_ACCESS_WRAPPER · d7ba24fb
      Rhys Kidd authored
      Fixes warning with xserver 1.20 and gcc 8.2:
      
      nv_driver.c:1443:9: warning: implicit declaration of function ‘wfbScreenInit’; did you mean ‘fbScreenInit’? [-Wimplicit-function-declaration]
         ret = wfbScreenInit(pScreen, FBStart, pScrn->virtualX,
               ^~~~~~~~~~~~~
               fbScreenInit
      
      (See xserver 706e6d9cd074da606016ed4ecff51e9c2a822087)
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      d7ba24fb
    • Rhys Kidd's avatar
      xv: Silence build warning regarding const qualifier · dbba3246
      Rhys Kidd authored
      Silences warning with gcc 8.2:
      
      nouveau_xv.c: In function ‘NVInitVideo’:
      nouveau_xv.c:2247:68: warning: passing argument 2 of ‘vlCreateAdaptorXvMC’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
          adaptorsXvMC[0] = vlCreateAdaptorXvMC(pScreen, textureAdaptor[0]->name);
                                                         ~~~~~~~~~~~~~~~~~^~~~~~
      In file included from nouveau_xv.c:42:
      vl_hwmc.h:6:63: note: expected ‘char *’ but argument is of type ‘const char *’
       XF86MCAdaptorPtr vlCreateAdaptorXvMC(ScreenPtr pScreen, char *xv_adaptor_name);
                                                               ~~~~~~^~~~~~~~~~~~~~~
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      Reviewed-by: Ilia Mirkin's avatarIlia Mirkin <imirkin@alum.mit.edu>
      dbba3246
    • Rhys Kidd's avatar
      Check for xf86CursorResetCursor() · cc284803
      Rhys Kidd authored
      If it's available, xorg-server calls it on each mode configuration change. It
      does what xf86_reload_cursors does (and more), so we don't need to call
      the latter anymore.
      
      Avoids gcc 8.2 warning:
      
      drmmode_display.c: In function ‘drmmode_set_mode_major’:
      drmmode_display.c:525:2: warning: ‘xf86_reload_cursors’ is deprecated [-Wdeprecated-declarations]
        xf86_reload_cursors(crtc->scrn->pScreen);
        ^~~~~~~~~~~~~~~~~~~
      In file included from nv_type.h:10,
                       from nv_include.h:69,
                       from drmmode_display.c:36:
      /usr/include/xorg/xf86Crtc.h:1068:37: note: declared here
       static _X_INLINE _X_DEPRECATED void xf86_reload_cursors(ScreenPtr screen) {}
                                           ^~~~~~~~~~~~~~~~~~~
      
      (Ported from radeon commit d670c5c9851b4eff21c845d26c7d7e4eb5ee0fa9)
      Signed-off-by: Rhys Kidd's avatarRhys Kidd <rhyskidd@gmail.com>
      cc284803
  9. 09 Dec, 2018 3 commits