[Intel-gfx] [PATCH] drm/i915: Provide a modparam to disable firmware loading

[Ben Widawsky]

On Wed, Apr 13, 2016 at 06:45:32PM +0100, Dave Gordon wrote:
> On 13/04/16 17:57, Ben Widawsky wrote:
> > For debug and development purposes only.
> > 
> > Cc: Mika Kuoppala <mika.kuoppala at intel.com>
> > Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
> > ---
> >   drivers/gpu/drm/i915/i915_debugfs.c     | 13 +++++++++++++
> >   drivers/gpu/drm/i915/i915_gpu_error.c   |  3 +++
> >   drivers/gpu/drm/i915/i915_params.c      |  6 ++++++
> >   drivers/gpu/drm/i915/i915_params.h      |  1 +
> >   drivers/gpu/drm/i915/intel_csr.c        | 11 +++++++++++
> >   drivers/gpu/drm/i915/intel_guc_loader.c | 11 +++++++++++
> >   6 files changed, 45 insertions(+)
> 
> You can already disable GuC loading in the existing driver by turning off
> GuC submission,

I saw this. I believe that behavior is incorrect but I didn't bother to fix it.
GuC submission and GuC loading are separate.

> or if you have the previously-posted "add enable_guc_loading" patch, then you
> can separately control the loading of the GuC firmware and the use of the GuC
> for batch submission.

Sounds to me like that patch should be merged, and we could roll that into this
and give me what I want.

> 
> .Dave.