On Mon, Mar 19, 2018 at 04:20:02PM +0000, Michal Wajdeczko wrote: > Usually we use shift/mask macros for bit field definitions. > Union guc_log_control was not following that pattern. > > Additional bonus: > > add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-25 (-25) > Function old new delta > intel_guc_log_level_set 388 363 -25 > > v2: prevent out-of-range verbosity (MichalWi) > > Signed-off-by: Michal Wajdeczko <[email protected]> > Cc: Michal Winiarski <[email protected]> > Cc: Sagar Arun Kamble <[email protected]>
Reviewed-by: Michał Winiarski <[email protected]> -Michał > --- > drivers/gpu/drm/i915/intel_guc_fwif.h | 16 +++++----------- > drivers/gpu/drm/i915/intel_guc_log.c | 13 +++++-------- > 2 files changed, 10 insertions(+), 19 deletions(-) _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
