[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