Quoting Jani Nikula (2018-03-21 14:07:54) > Fix the last two direct ->gen usages. > > Cc: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com> > Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> > Signed-off-by: Jani Nikula <jani.nik...@intel.com> > --- > drivers/gpu/drm/i915/intel_device_info.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_device_info.c > b/drivers/gpu/drm/i915/intel_device_info.c > index a504281e2afa..af538c4a3aeb 100644 > --- a/drivers/gpu/drm/i915/intel_device_info.c > +++ b/drivers/gpu/drm/i915/intel_device_info.c > @@ -119,7 +119,7 @@ void intel_device_info_dump(const struct > intel_device_info *info, > INTEL_DEVID(dev_priv), > INTEL_REVID(dev_priv), > intel_platform_name(info->platform), > - info->gen); > + INTEL_GEN(dev_priv));
This is wrong, this should be from the local intel_device_info *info. (Passed in from the caller, so dev_priv is very wrong here, and yes the INTEL_foo() above are on my hit list :) -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx