On Wed, 21 Mar 2018, Chris Wilson <[email protected]> wrote: > Quoting Jani Nikula (2018-03-21 14:07:54) >> Fix the last two direct ->gen usages. >> >> Cc: Tvrtko Ursulin <[email protected]> >> Cc: Mika Kuoppala <[email protected]> >> Signed-off-by: Jani Nikula <[email protected]> >> --- >> 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 :)
Contemplated that. I guess you'll need to remove them for turning intel_device_info into a pointer rather than a copy in dev_priv? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
