[Openchrome-devel] [PATCH] Treat all build warnings as errors

Bartosz gang65 at wp.pl
Fri Aug 5 02:40:12 UTC 2016


Hi
In attachment you have all warnings which was produced by using "-Wall"

We could always silent some types of warnings if we think it is not needed.
 For example to silent all [-Wunused-variable] warnings:

 -Werror -Wall -Wno-unused-variable

Best Regards
Bartosz

2016-08-05 4:28 GMT+02:00 Bartosz <gang65 at wp.pl>:

> Thanks Xavier.
>
> It will be great to also enable "-Wall" parameter.
> http://www.rapidtables.com/code/linux/gcc/gcc-wall.htm
> https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html
>
> -Wall turns on the following warning flags:
>
>           -Waddress
>           -Warray-bounds=1 (only with -O2)
>           -Wbool-compare
>           -Wc++11-compat  -Wc++14-compat
>           -Wchar-subscripts
>           -Wcomment
>           -Wduplicate-decl-specifier (C and Objective-C only)
>           -Wenum-compare (in C/ObjC; this is on by default in C++)
>           -Wformat
>           -Wimplicit (C and Objective-C only)
>           -Wimplicit-int (C and Objective-C only)
>           -Wimplicit-function-declaration (C and Objective-C only)
>           -Winit-self (only for C++)
>           -Wlogical-not-parentheses
>           -Wmain (only for C/ObjC and unless -ffreestanding)
>           -Wmaybe-uninitialized
>           -Wmemset-elt-size
>           -Wmemset-transposed-args
>           -Wmisleading-indentation (only for C/C++)
>           -Wmissing-braces (only for C/ObjC)
>           -Wnarrowing (only for C++)
>           -Wnonnull
>           -Wnonnull-compare
>           -Wopenmp-simd
>           -Wparentheses
>           -Wpointer-sign
>           -Wreorder
>           -Wreturn-type
>           -Wsequence-point
>           -Wsign-compare (only in C++)
>           -Wsizeof-pointer-memaccess
>           -Wstrict-aliasing
>           -Wstrict-overflow=1
>           -Wswitch
>           -Wtautological-compare
>           -Wtrigraphs
>           -Wuninitialized
>           -Wunknown-pragmas
>           -Wunused-function
>           -Wunused-label
>           -Wunused-value
>           -Wunused-variable
>           -Wvolatile-register-var
>
>
> It will keep code cleaner, and will give you a message if something
> suspicious will happen.
>
> Also there will be a lot of warnings which will need to be resolved.
> Fortunately fixing warnings will be a good task for novice developer.
>
> What do you think about enabling "-Wall"?
>
> Best Regards
> Bartosz
>
>
> 2016-08-04 14:33 GMT+02:00 Xavier Bachelot <xavier at bachelot.org>:
>
>> Signed-off-by: Xavier Bachelot <xavier at bachelot.org>
>> ---
>> I don't know my way through autotools very well, so not sure this is the
>> best way, but it works...
>>
>>  src/Makefile.am | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/Makefile.am b/src/Makefile.am
>> index 82224d4..6ff7e31 100644
>> --- a/src/Makefile.am
>> +++ b/src/Makefile.am
>> @@ -29,7 +29,7 @@ SUBDIRS = xvmc
>>  BUILT_SOURCES = version.h
>>  EXTRA_DIST = version.h
>>  CONFIG_CLEAN_FILES= version.h
>> -AM_CFLAGS = @XORG_CFLAGS@ @LIBUDEV_CFLAGS@ @DRI_CFLAGS@
>> +AM_CFLAGS = @XORG_CFLAGS@ @LIBUDEV_CFLAGS@ @DRI_CFLAGS@ -Werror
>>  openchrome_drv_la_LTLIBRARIES = openchrome_drv.la
>>  openchrome_drv_la_LDFLAGS = -module -avoid-version @LIBUDEV_LIBS@
>>  openchrome_drv_ladir = @moduledir@/drivers
>> --
>> 2.7.4
>>
>> _______________________________________________
>> Openchrome-devel mailing list
>> Openchrome-devel at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/openchrome-devel
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/openchrome-devel/attachments/20160805/563dcecf/attachment-0001.html>
-------------- next part --------------
via_3d.c: In function ‘via3DEmitQuad’:
via_3d.c:331:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx1)));
     ^
via_3d.c:332:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy1)));
     ^
via_3d.c:333:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_3d.c:339:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx2)));
     ^
via_3d.c:340:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy1)));
     ^
via_3d.c:341:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_3d.c:347:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx1)));
     ^
via_3d.c:348:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy2)));
     ^
via_3d.c:349:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_3d.c:355:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx1)));
     ^
via_3d.c:356:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy2)));
     ^
via_3d.c:357:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_3d.c:363:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx2)));
     ^
via_3d.c:364:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy1)));
     ^
via_3d.c:365:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_3d.c:371:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dx2)));
     ^
via_3d.c:372:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&dy2)));
     ^
via_3d.c:373:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
     OUT_RING(*((CARD32 *) (&wf)));
     ^
via_analog.c: In function ‘via_analog_mode_set’:
via_analog.c:258:14: warning: unused variable ‘hwp’ [-Wunused-variable]
     vgaHWPtr hwp = VGAHWPTR(pScrn);
              ^
In file included from via_analog.c:40:0:
via_analog.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_analog.c:45:1: warning: ‘ViaPrintMode’ defined but not used [-Wunused-function]
 ViaPrintMode(ScrnInfoPtr pScrn, DisplayModePtr mode)
 ^
In file included from via_exa.c:42:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_exa.c:45:0:
via_rop.h:302:3: warning: ‘VIAROP’ defined but not used [-Wunused-variable]
 } VIAROP[] = {
   ^
In file included from via_driver.h:76:0,
                 from via_exa_h2.c:39:
via_exa_h2.c: In function ‘viaExaPrepareSolid_H2’:
via_dmabuffer.h:87:21: warning: unused variable ‘cb’ [-Wunused-variable]
   ViaCommandBuffer *cb = &pVia->cb
                     ^
via_exa_h2.c:173:5: note: in expansion of macro ‘RING_VARS’
     RING_VARS;
     ^
via_exa_h2.c: In function ‘viaExaPrepareCopy_H2’:
via_dmabuffer.h:87:21: warning: unused variable ‘cb’ [-Wunused-variable]
   ViaCommandBuffer *cb = &pVia->cb
                     ^
via_exa_h2.c:230:5: note: in expansion of macro ‘RING_VARS’
     RING_VARS;
     ^
In file included from via_exa_h2.c:39:0:
via_exa_h2.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_exa_h2.c:49:1: warning: ‘viaAccelClippingHelper_H2’ defined but not used [-Wunused-function]
 viaAccelClippingHelper_H2(VIAPtr pVia, int refY)
 ^
In file included from via_driver.h:76:0,
                 from via_exa_h6.c:39:
via_exa_h6.c: In function ‘viaExaPrepareSolid_H6’:
via_dmabuffer.h:87:21: warning: unused variable ‘cb’ [-Wunused-variable]
   ViaCommandBuffer *cb = &pVia->cb
                     ^
via_exa_h6.c:174:5: note: in expansion of macro ‘RING_VARS’
     RING_VARS;
     ^
via_exa_h6.c: In function ‘viaExaPrepareCopy_H6’:
via_dmabuffer.h:87:21: warning: unused variable ‘cb’ [-Wunused-variable]
   ViaCommandBuffer *cb = &pVia->cb
                     ^
via_exa_h6.c:231:5: note: in expansion of macro ‘RING_VARS’
     RING_VARS;
     ^
In file included from via_exa_h6.c:39:0:
via_exa_h6.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_exa_h6.c:49:1: warning: ‘viaAccelClippingHelper_H6’ defined but not used [-Wunused-function]
 viaAccelClippingHelper_H6(VIAPtr pVia, int refY)
 ^
In file included from via_bandwidth.c:32:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_ch7xxx.c:32:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_display.c:32:0:
via_display.c: In function ‘viaIGA1SetFBStartingAddress’:
via_display.c:833:25: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32 {aka unsigned int}’ [-Wformat=]
                         "Base Address: 0x%lx\n",
                         ^
via_driver.h:33:18: note: in definition of macro ‘DEBUG’
 #define DEBUG(x) x
                  ^
via_display.c:837:17: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32 {aka unsigned int}’ [-Wformat=]
                 "DRI Base Address: 0x%lx\n",
                 ^
In file included from via_display.c:32:0:
via_display.c: In function ‘viaIGA2SetFBStartingAddress’:
via_display.c:2074:25: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32 {aka unsigned int}’ [-Wformat=]
                         "Base Address: 0x%lx\n",
                         ^
via_driver.h:33:18: note: in definition of macro ‘DEBUG’
 #define DEBUG(x) x
                  ^
via_display.c:2078:17: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32 {aka unsigned int}’ [-Wformat=]
                 "DRI Base Address: 0x%lx\n",
                 ^
via_display.c: In function ‘iga1_crtc_dpms’:
via_display.c:2786:20: warning: unused variable ‘pBIOSInfo’ [-Wunused-variable]
     VIABIOSInfoPtr pBIOSInfo = pVia->pBIOSInfo;
                    ^
via_display.c: In function ‘iga1_crtc_save’:
via_display.c:2824:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_display.c: In function ‘iga1_crtc_restore’:
via_display.c:2845:11: warning: unused variable ‘tmp’ [-Wunused-variable]
     CARD8 tmp;
           ^
via_display.c:2844:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_display.c: In function ‘iga1_crtc_set_origin’:
via_display.c:2925:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_display.c: In function ‘iga1_crtc_gamma_set’:
via_display.c:3005:27: warning: variable ‘CR6A’ set but not used [-Wunused-but-set-variable]
     int SR1A, SR1B, CR67, CR6A;
                           ^
via_display.c:3005:21: warning: variable ‘CR67’ set but not used [-Wunused-but-set-variable]
     int SR1A, SR1B, CR67, CR6A;
                     ^
via_display.c:3005:15: warning: variable ‘SR1B’ set but not used [-Wunused-but-set-variable]
     int SR1A, SR1B, CR67, CR6A;
               ^
via_display.c:3005:9: warning: variable ‘SR1A’ set but not used [-Wunused-but-set-variable]
     int SR1A, SR1B, CR67, CR6A;
         ^
via_display.c: In function ‘iga2_crtc_dpms’:
via_display.c:3259:20: warning: unused variable ‘pBIOSInfo’ [-Wunused-variable]
     VIABIOSInfoPtr pBIOSInfo = pVia->pBIOSInfo;
                    ^
via_display.c: In function ‘iga2_crtc_save’:
via_display.c:3291:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_display.c:3290:14: warning: unused variable ‘hwp’ [-Wunused-variable]
     vgaHWPtr hwp = VGAHWPTR(pScrn);
              ^
via_display.c: In function ‘iga2_crtc_restore’:
via_display.c:3308:11: warning: unused variable ‘tmp’ [-Wunused-variable]
     CARD8 tmp;
           ^
via_display.c:3307:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_display.c:3306:14: warning: unused variable ‘hwp’ [-Wunused-variable]
     vgaHWPtr hwp = VGAHWPTR(pScrn);
              ^
via_display.c: In function ‘iga2_crtc_set_origin’:
via_display.c:3375:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
In file included from via_display.c:32:0:
via_display.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_display.c: In function ‘viaIGA1SetFBStartingAddress’:
via_display.c:865:15: warning: ‘cr48’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         DEBUG(xf86DrvMsg(pScrn->scrnIndex, X_INFO,
               ^
via_driver.c: In function ‘via_pci_probe’:
via_driver.c:458:19: warning: variable ‘entity’ set but not used [-Wunused-but-set-variable]
     EntityInfoPtr entity;
                   ^
In file included from via_i2c.c:34:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_id.c:34:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_id.c:37:1: warning: ‘ViaDoubleCheckCLE266Revision’ defined but not used [-Wunused-function]
 ViaDoubleCheckCLE266Revision(ScrnInfoPtr pScrn)
 ^
via_lvds.c: In function ‘via_lvds_mode_valid’:
via_lvds.c:581:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_lvds.c: In function ‘via_lvds_mode_set’:
via_lvds.c:824:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_lvds.c: In function ‘via_lvds_init’:
via_lvds.c:1039:17: warning: unused variable ‘s’ [-Wunused-variable]
     const char *s = NULL;
                 ^
In file included from via_lvds.c:35:0:
via_lvds.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_lvds.c:36:0:
via_mode.h:57:3: warning: ‘ViaDotClocks’ defined but not used [-Wunused-variable]
 } ViaDotClocks[] = {
   ^
via_mode.h:174:3: warning: ‘ViaVesaModes’ defined but not used [-Wunused-variable]
 } ViaVesaModes[] = {
   ^
via_lvds.c:472:1: warning: ‘ViaPanelGetSizeFromDDCv1’ defined but not used [-Wunused-function]
 ViaPanelGetSizeFromDDCv1(xf86OutputPtr output, int *width, int *height)
 ^
via_lvds.c:531:1: warning: ‘ViaGetResolutionIndex’ defined but not used [-Wunused-function]
 ViaGetResolutionIndex(ScrnInfoPtr pScrn, ViaPanelInfoPtr Panel,
 ^
In file included from via_memcpy.c:28:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_memmgr.c:38:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_xv_overlay.h:37:0,
                 from via_driver.h:79,
                 from via_memmgr.c:38:
via_memmgr.c: In function ‘drm_bo_alloc_surface’:
via_priv.h:38:38: warning: ‘pitch’ may be used uninitialized in this function [-Wmaybe-uninitialized]
 #define ALIGN_TO(f, alignment) (((f) + ((alignment)-1)) & ~((alignment)-1))
                                      ^
via_memmgr.c:78:9: note: ‘pitch’ was declared here
     int pitch;
         ^
via_outputs.c: In function ‘ViaComputeProDotClock’:
via_outputs.c:542:24: warning: variable ‘fref’ set but not used [-Wunused-but-set-variable]
     double fvco, fout, fref, err, minErr;
                        ^
In file included from via_outputs.c:40:0:
via_outputs.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_outputs.c:47:0:
via_mode.h:144:3: warning: ‘ViaResolutionTable’ defined but not used [-Wunused-variable]
 } ViaResolutionTable[] = {
   ^
via_mode.h:174:3: warning: ‘ViaVesaModes’ defined but not used [-Wunused-variable]
 } ViaVesaModes[] = {
   ^
via_outputs.c:50:1: warning: ‘ViaPrintMode’ defined but not used [-Wunused-function]
 ViaPrintMode(ScrnInfoPtr pScrn, DisplayModePtr mode)
 ^
via_outputs.c:361:1: warning: ‘VIAGetActiveDisplay’ defined but not used [-Wunused-function]
 VIAGetActiveDisplay(ScrnInfoPtr pScrn)
 ^
via_outputs.c:501:1: warning: ‘ViaSetECKDotclock’ defined but not used [-Wunused-function]
 ViaSetECKDotclock(ScrnInfoPtr pScrn, CARD32 clock)
 ^
via_outputs.c: In function ‘viaProbePinStrapping’:
via_outputs.c:273:9: warning: ‘sr5a’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         ViaSeqMask(hwp, 0x5A, sr5a | 0x01, 0x01);
         ^
via_tmds.c: In function ‘via_dvi_dpms’:
via_tmds.c:249:17: warning: unused variable ‘pScrn’ [-Wunused-variable]
     ScrnInfoPtr pScrn = output->scrn;
                 ^
via_tmds.c: In function ‘via_dvi_mode_set’:
via_tmds.c:305:14: warning: unused variable ‘hwp’ [-Wunused-variable]
     vgaHWPtr hwp = VGAHWPTR(pScrn);
              ^
via_tmds.c: In function ‘via_dvi_detect’:
via_tmds.c:321:14: warning: implicit declaration of function ‘via_vt1632_detect’ [-Wimplicit-function-declaration]
     status = via_vt1632_detect(output);
              ^
via_tmds.c:315:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_tmds.c: In function ‘via_tmds_detect’:
via_tmds.c:521:19: warning: unused variable ‘pVIATMDSRec’ [-Wunused-variable]
     VIATMDSRecPtr pVIATMDSRec = output->driver_private;
                   ^
via_tmds.c: In function ‘viaTMDSInit’:
via_tmds.c:647:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
via_tmds.c: In function ‘via_dvi_init’:
via_tmds.c:736:18: warning: unused variable ‘addr’ [-Wunused-variable]
     I2CSlaveAddr addr = 0x10;
                  ^
via_tmds.c:735:15: warning: unused variable ‘pDev’ [-Wunused-variable]
     I2CDevPtr pDev = NULL;
               ^
via_tmds.c:734:15: warning: unused variable ‘pBus’ [-Wunused-variable]
     I2CBusPtr pBus = NULL;
               ^
via_tmds.c:733:18: warning: unused variable ‘private_data’ [-Wunused-variable]
     ViaVT1632Ptr private_data = NULL;
                  ^
via_tmds.c:732:19: warning: unused variable ‘output’ [-Wunused-variable]
     xf86OutputPtr output = NULL;
                   ^
In file included from via_tmds.c:41:0:
via_tmds.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_tv.c: In function ‘via_tv_mode_valid’:
via_tv.c:179:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
In file included from via_tv.c:40:0:
via_tv.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_tv.c:225:1: warning: ‘ViaDisplayDisableDVO’ defined but not used [-Wunused-function]
 ViaDisplayDisableDVO(ScrnInfoPtr pScrn, int port)
 ^
via_tv.c: In function ‘via_tv_mode_set’:
via_tv.c:266:9: warning: ‘regNum’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         ViaCrtcMask(hwp, regNum, 0x00, 0x10);
         ^
via_tv.c:245:9: note: ‘regNum’ was declared here
     int regNum;
         ^
via_xv_overlay.c: In function ‘AddHQVSurface’:
via_xv_overlay.c:1129:21: warning: variable ‘width’ set but not used [-Wunused-but-set-variable]
     unsigned int i, width, height, pitch, fbsize, addr;
                     ^
In file included from via_xv_overlay.c:38:0:
via_xv_overlay.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_ums.c: In function ‘viaMapMMIO’:
via_ums.c:188:16: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 5 has type ‘long unsigned int’ [-Wformat=]
                "Mapping the frame buffer at address 0x%lX with "
                ^
In file included from via_vgahw.c:35:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_xv.c: In function ‘viaDmaBlitImage’:
via_xv.c:1020:24: warning: variable ‘lumaSync’ set but not used [-Wunused-but-set-variable]
     drm_via_blitsync_t lumaSync;
                        ^
In file included from via_xv.c:43:0:
via_xv.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_xv.c:963:1: warning: ‘planar420cp’ defined but not used [-Wunused-function]
 planar420cp(unsigned char *dst, const unsigned char *src, int dstPitch,
 ^
In file included from via_vt162x.c:30:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_vt1632.c: In function ‘viaVT1632Init’:
via_vt1632.c:207:12: warning: unused variable ‘pVia’ [-Wunused-variable]
     VIAPtr pVia = VIAPTR(pScrn);
            ^
In file included from via_vt1632.c:31:0:
via_vt1632.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_dri.c:41:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
In file included from via_kms.c:43:0:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^
via_xvmc.c: In function ‘ViaXvMCDestroyContext’:
via_xvmc.c:686:32: warning: variable ‘sAPriv’ set but not used [-Wunused-but-set-variable]
     volatile ViaXvMCSAreaPriv *sAPriv;
                                ^
In file included from via_xvmc.c:44:0:
via_xvmc.c: At top level:
via_driver.h:127:12: warning: ‘gVIAEntityIndex’ defined but not used [-Wunused-variable]
 static int gVIAEntityIndex = -1;
            ^


More information about the Openchrome-devel mailing list