On Tue, Jun 11, 2013 at 08:39:16AM +0100, Chris Wilson wrote: > On Tue, Jun 11, 2013 at 10:03:06AM +0300, Ville Syrjälä wrote: > > On Tue, Jun 11, 2013 at 08:55:03AM +1000, Dave Airlie wrote: > > > CC [M] drivers/gpu/drm/i915/i915_gem.o > > > /ssd/git/drm-next/drivers/gpu/drm/i915/i915_gem.c: In function > > > ‘i915_gem_object_bind_to_gtt’: > > > /ssd/git/drm-next/drivers/gpu/drm/i915/i915_gem.c:3000:3: warning: > > > format ‘%ld’ expects argument of type ‘long int’, but argument 5 has > > > type ‘size_t’ [-Wformat] > > > CC [M] drivers/gpu/drm/i915/i915_gem_context.o > > > > I think Jani had a fix for this one: > > https://patchwork.kernel.org/patch/2686801/ > > It is fixing the wrong thing, imo. The issue is that we have a random mix > of types that we need to sort out. (Here is a prime example, the > conditional switching between gtt.total and gtt.mappable_end, both > describing limits in the GTT have size_t and unsigned long respectively.) Agreed, but since I've bitched around about that on irc I haven't seen patches float around yet to fix it. So duct-tape looks like a decent solution for now ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
