On Tue, Jul 05, 2016 at 11:44:07AM +0100, Matthew Auld wrote: > > + > > +#include "i915_drv.h" > > + > > +void intel_device_info_dump(struct drm_i915_private *dev_priv) > > +{ > > + const struct intel_device_info *info = &dev_priv->info; > mkwrite_device_info(dev_priv)? Here we just want the const pointer, so no need for the casts behind C's back. > On a similar note there is one other in intel_lrc.c, maybe fix in a > follow up patch :) Yup. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx