There are various literal constants used in the GuC module-parameter processing code; this sequence of patches replaces them with symbolic names for greater clarity. And then it re-enables GuC submission by default :) v3: Original patch broken into two (1/4 + 2/4) Name for GuC log level NONE added (3/4 Re-enable GuC loading & submission (4/4) Added cover letter v4: Additional final patch (5/5) to change treatment of unrecognised option values (ignore them rather than force them into range). Dave Gordon (5): drm/i915/guc: symbolic names for GuC submission preferences drm/i915/guc: symbolic names for GuC firmare loading preferences drm/i915/guc: symbolic name for GuC log-level none drm/i915/guc: use symbolic names in setting defaults for module drm/i915/guc: ignore unrecognised loading & submission options drivers/gpu/drm/i915/i915_guc_submission.c | 4 +-- drivers/gpu/drm/i915/i915_params.c | 10 +++--- drivers/gpu/drm/i915/intel_guc.h | 20 +++++++++++ drivers/gpu/drm/i915/intel_guc_fwif.h | 1 + drivers/gpu/drm/i915/intel_guc_loader.c | 54 ++++++++++++++++++++++-------- drivers/gpu/drm/i915/intel_lrc.c | 4 +-- 6 files changed, 70 insertions(+), 23 deletions(-) -- 1.9.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx