On Thu, Jun 04, 2015 at 06:10:04PM +0100, Damien Lespiau wrote:
> On Thu, Jun 04, 2015 at 04:47:00PM +0100, Chris Wilson wrote:
> > On Thu, Jun 04, 2015 at 04:42:16PM +0100, Damien Lespiau wrote:
> > > It's handy to have debug message for the "big" events and this one
> > > qualifies IMHO. Also helpful to see what's happening while we're loading
> > > the firwmare and how much time it takes.
> > >
> > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx>
> > > ---
> > > drivers/gpu/drm/i915/intel_csr.c | 3 +++
> > > 1 file changed, 3 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> > > index aae0652..2cf25ee 100644
> > > --- a/drivers/gpu/drm/i915/intel_csr.c
> > > +++ b/drivers/gpu/drm/i915/intel_csr.c
> > > @@ -389,6 +389,7 @@ static void finish_csr_load(const struct firmware *fw, void *context)
> > > intel_csr_load_program(dev);
> > > fw_loaded = true;
> > >
> > > + DRM_DEBUG_KMS("Finished loading %s\n", dev_priv->csr.fw_path);
> >
> > KMS? DRM_DEBUG_DRIVER() as it is off wider scope.
>
> Is it? AFAIK the DMC is all about saving/restoring display registers
> when we toggle the power wells between frame updates for eDP + PSR
> panels.
>
> But then I guess the loading itself is a generic driver function.
Well I merged both patches. Someone still owes me the conversion to
refcount away from the hand-rolled completion though ;-)
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
