Hi Chris, > -----Original Message----- > From: Chris Wilson <[email protected]> > Sent: 25 February 2020 19:32 > To: David Airlie <[email protected]>; Joonas Lahtinen > <[email protected]>; Laxminarayan Bharadiya, Pankaj > <[email protected]>; Vivi, Rodrigo > <[email protected]>; [email protected]; [email protected]; > [email protected]; [email protected] > Cc: Laxminarayan Bharadiya, Pankaj > <[email protected]> > Subject: Re: [Intel-gfx][PATCH 01/10] drm/i915: Add i915 device based > MISSING_CASE macro > > Quoting Pankaj Bharadiya (2020-02-25 13:47:00) > > Now that we have struct drm_device based drm_WARN, introduce struct > > drm_i915_private based i915_MISSING_CASE macro which uses > drm_WARN so > > that device specific information will also get printed in backtrace. > > > > i915_MISSING_CASE macro should be preferred over MISSING_CASE, > > wherever possible. > > Whatever for? MISSING_CASE() itself should be a complete picture for the > forgotten code.
Are you saying, no need to have a new device specific macro? We want convert all the calls of WARN* with device specific drm_WARN* in i915, hence I introduced new i915_MISSING_CASE macro. Jani, Will you please share your opinion on this? Thanks, Pankaj > -Chris _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
