[libdrm][PATCH 2/2] Fix gcc -Wextra warnings

Ian Romanick idr at freedesktop.org
Mon Feb 9 15:12:25 PST 2015


This patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 02/09/2015 01:39 PM, Jan Vesely wrote:
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> ---
>  tests/vbltest/vbltest.c | 3 ++-
>  xf86drm.c               | 4 ++--
>  xf86drmMode.c           | 2 +-
>  3 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
> index cdc1ef6..6e13c57 100644
> --- a/tests/vbltest/vbltest.c
> +++ b/tests/vbltest/vbltest.c
> @@ -104,7 +104,8 @@ static void usage(char *name)
>  
>  int main(int argc, char **argv)
>  {
> -	int i, c, fd, ret;
> +	unsigned i;
> +	int c, fd, ret;
>  	char *modules[] = { "i915", "radeon", "nouveau", "vmwgfx", "exynos", "omapdrm", "tilcdc", "msm", "tegra" };
>  	drmVBlank vbl;
>  	drmEventContext evctx;
> diff --git a/xf86drm.c b/xf86drm.c
> index 345325a..fb673b5 100644
> --- a/xf86drm.c
> +++ b/xf86drm.c
> @@ -303,7 +303,7 @@ static int chown_check_return(const char *path, uid_t owner, gid_t group)
>   * special file node with the major and minor numbers specified by \p dev and
>   * parent directory if necessary and was called by root.
>   */
> -static int drmOpenDevice(long dev, int minor, int type)
> +static int drmOpenDevice(dev_t dev, int minor, int type)
>  {
>      stat_t          st;
>      const char      *dev_name;
> @@ -2213,7 +2213,7 @@ int drmGetClient(int fd, int idx, int *auth, int *pid, int *uid,
>  int drmGetStats(int fd, drmStatsT *stats)
>  {
>      drm_stats_t s;
> -    int         i;
> +    unsigned    i;
>  
>      if (drmIoctl(fd, DRM_IOCTL_GET_STATS, &s))
>  	return -errno;
> diff --git a/xf86drmMode.c b/xf86drmMode.c
> index 60ce369..e3e599b 100644
> --- a/xf86drmMode.c
> +++ b/xf86drmMode.c
> @@ -854,7 +854,7 @@ int drmHandleEvent(int fd, drmEventContextPtr evctx)
>  	len = read(fd, buffer, sizeof buffer);
>  	if (len == 0)
>  		return 0;
> -	if (len < sizeof *e)
> +	if (len < (int)sizeof *e)
>  		return -1;
>  
>  	i = 0;
> 



More information about the dri-devel mailing list