[PATCH xf86-video-mga 0/9] Silence compile warnings and remove redundant defines

Nicolas Kaiser nikai at nikai.net
Wed Jun 8 01:57:22 PDT 2011


These patches silence compile warnings, and remove some
unused or duplicated defines.
Tested on a Matrox G550 AGP.

[PATCH 1/9] include unistd.h for usleep()
[PATCH 2/9] fix formats in xf86DrvMsg()
[PATCH 3/9] hide conditionally used variable in mga_read_and_process_bios
[PATCH 4/9] remove unused variables in MGAG200EHPIXPLLSET
[PATCH 5/9] remove unused variables in CopyMungedScanlineFilterDown_AXP
[PATCH 6/9] hide conditionally used variable in MGACopyILOAD
[PATCH 7/9] maven: remove duplicated defines
[PATCH 8/9] storm: remove unused define PSZ
[PATCH 9/9] include xf86Modes.h for xf86ModeBandwidth()

Patch 1 and 9 additionally silence Gentoo QA warnings when
compiling as Gentoo ebuild:

* QA Notice: Package has poor programming practices which may compile
*            fine but exhibit random runtime failures.
(..)
* Please do not file a Gentoo bug and instead report the above QA
* issues directly to the upstream developers of this software.
* Homepage: http://xorg.freedesktop.org/


 src/mga_bios.c   |    2 +-
 src/mga_dacG.c   |    6 +-----
 src/mga_driver.c |    1 +
 src/mga_exa.c    |    2 +-
 src/mga_maven.h  |    3 ---
 src/mga_storm.c  |    2 --
 src/mga_vga.c    |    2 ++
 src/mga_video.c  |    6 ++++--
 8 files changed, 10 insertions(+), 14 deletions(-)


Best regards,
Nicolas Kaiser


More information about the xorg-devel mailing list