* [PATCH 1/2] drm/i915: Remove unnecessary () used with WARN()
@ 2015-06-04 15:42 Damien Lespiau
2015-06-04 15:42 ` [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading Damien Lespiau
0 siblings, 1 reply; 6+ messages in thread
From: Damien Lespiau @ 2015-06-04 15:42 UTC (permalink / raw)
To: intel-gfx; +Cc: Suketu Shah
In Linux, macros are usually well done and protect their arguments
properly, even avoiding multiple evaluations of the parameters. Extra ()
are really not needed.
Cc: Suketu Shah <suketu.j.shah@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
drivers/gpu/drm/i915/intel_csr.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
index 5cb8cc1..aae0652 100644
--- a/drivers/gpu/drm/i915/intel_csr.c
+++ b/drivers/gpu/drm/i915/intel_csr.c
@@ -459,7 +459,8 @@ void intel_csr_ucode_fini(struct drm_device *dev)
void assert_csr_loaded(struct drm_i915_private *dev_priv)
{
- WARN((intel_csr_load_status_get(dev_priv) != FW_LOADED), "CSR is not loaded.\n");
+ WARN(intel_csr_load_status_get(dev_priv) != FW_LOADED,
+ "CSR is not loaded.\n");
WARN(!I915_READ(CSR_PROGRAM_BASE),
"CSR program storage start is NULL\n");
WARN(!I915_READ(CSR_SSP_BASE), "CSR SSP Base Not fine\n");
--
2.1.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading
2015-06-04 15:42 [PATCH 1/2] drm/i915: Remove unnecessary () used with WARN() Damien Lespiau
@ 2015-06-04 15:42 ` Damien Lespiau
2015-06-04 15:47 ` Chris Wilson
2015-06-05 13:01 ` shuang.he
0 siblings, 2 replies; 6+ messages in thread
From: Damien Lespiau @ 2015-06-04 15:42 UTC (permalink / raw)
To: intel-gfx
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@intel.com>
---
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);
out:
if (fw_loaded)
intel_runtime_pm_put(dev_priv);
@@ -422,6 +423,8 @@ void intel_csr_ucode_init(struct drm_device *dev)
return;
}
+ DRM_DEBUG_KMS("Loading %s\n", csr->fw_path);
+
/*
* Obtain a runtime pm reference, until CSR is loaded,
* to avoid entering runtime-suspend.
--
2.1.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading
2015-06-04 15:42 ` [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading Damien Lespiau
@ 2015-06-04 15:47 ` Chris Wilson
2015-06-04 17:10 ` Damien Lespiau
2015-06-05 13:01 ` shuang.he
1 sibling, 1 reply; 6+ messages in thread
From: Chris Wilson @ 2015-06-04 15:47 UTC (permalink / raw)
To: Damien Lespiau; +Cc: intel-gfx
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@intel.com>
> ---
> 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.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading
2015-06-04 15:47 ` Chris Wilson
@ 2015-06-04 17:10 ` Damien Lespiau
2015-06-15 12:31 ` Daniel Vetter
0 siblings, 1 reply; 6+ messages in thread
From: Damien Lespiau @ 2015-06-04 17:10 UTC (permalink / raw)
To: Chris Wilson, intel-gfx
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@intel.com>
> > ---
> > 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.
--
Damien
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading
2015-06-04 15:42 ` [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading Damien Lespiau
2015-06-04 15:47 ` Chris Wilson
@ 2015-06-05 13:01 ` shuang.he
1 sibling, 0 replies; 6+ messages in thread
From: shuang.he @ 2015-06-05 13:01 UTC (permalink / raw)
To: shuang.he, lei.a.liu, intel-gfx, damien.lespiau
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com)
Task id: 6537
-------------------------------------Summary-------------------------------------
Platform Delta drm-intel-nightly Series Applied
PNV 270/270 270/270
ILK 303/303 303/303
SNB 312/312 312/312
IVB 343/343 343/343
BYT 287/287 287/287
BDW 318/318 318/318
-------------------------------------Detailed-------------------------------------
Platform Test drm-intel-nightly Series Applied
Note: You need to pay more attention to line start with '*'
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading
2015-06-04 17:10 ` Damien Lespiau
@ 2015-06-15 12:31 ` Daniel Vetter
0 siblings, 0 replies; 6+ messages in thread
From: Daniel Vetter @ 2015-06-15 12:31 UTC (permalink / raw)
To: Damien Lespiau; +Cc: intel-gfx
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@intel.com>
> > > ---
> > > 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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-06-15 12:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-04 15:42 [PATCH 1/2] drm/i915: Remove unnecessary () used with WARN() Damien Lespiau
2015-06-04 15:42 ` [PATCH 2/2] drm/i915/skl: Add debug messages at the start/end of DMC firmware loading Damien Lespiau
2015-06-04 15:47 ` Chris Wilson
2015-06-04 17:10 ` Damien Lespiau
2015-06-15 12:31 ` Daniel Vetter
2015-06-05 13:01 ` shuang.he
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.