Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 00/13] drm/i915: drm_dbg_kms() conversions and cleanups
Date: Fri, 9 Feb 2024 14:54:05 +0200	[thread overview]
Message-ID: <ZcYgbcy_t5jMpZri@intel.com> (raw)
In-Reply-To: <87o7crar7d.fsf@intel.com>

On Thu, Feb 08, 2024 at 05:52:38PM +0200, Jani Nikula wrote:
> On Thu, 08 Feb 2024, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > Convert a bunch of lingering DRM_DEBUG_KMS() & co. to the
> > per-device drm_dbg_kms() variants. The stuff remining after this
> > are DVO encoder drivers, DMI match functions, and ACPI/DSM stuff.
> >
> > And also include a bit of other house cleaning I spotted.
> >
> > Ville Syrjälä (13):
> >   drm/i915: Correct for_each_old_global_obj_in_state() arguments
> >   drm/i915/sdvo: Convert to per-device debugs
> >   drm/i915/sdvo: Fix up code alignment
> >   drm/i915/color: Use per-device debugs
> >   drm/i915/fb: Use per-device debugs
> >   drm/i915/bios: Switch to kms debugs
> >   drm/i915/bios: Use per-device debugs for VBT related stuff
> >   drm/i915/hdcp: Use per-device debugs
> >   drm/i915/wm: Pass the whole i916 to intel_get_cxsr_latency()
> 
> Off by one in the subject there. ;)

Doh. I did spot that myself but then forgot to fix it
anyway.

> 
> Other than that, the series is
> 
> Reviewed-by: Jani Nikula <jani.nikula@intel.com>

Thanks. Pushed the lot.

> 
> 
> >   drm/i915/wm: Use per-device debugs in pre-ilk wm code
> >   drm/i915/wm: Use per-device debugs ilk wm code
> >   drm/i915/dvo/ns2501: Nuke pointless casts
> >   drm/i915/dvo: Use sizeof(*variable) instead of sizeof(type)
> >
> >  drivers/gpu/drm/i915/display/dvo_ch7017.c     |   2 +-
> >  drivers/gpu/drm/i915/display/dvo_ch7xxx.c     |   2 +-
> >  drivers/gpu/drm/i915/display/dvo_ivch.c       |   2 +-
> >  drivers/gpu/drm/i915/display/dvo_ns2501.c     |   6 +-
> >  drivers/gpu/drm/i915/display/dvo_sil164.c     |   2 +-
> >  drivers/gpu/drm/i915/display/dvo_tfp410.c     |   2 +-
> >  drivers/gpu/drm/i915/display/i9xx_wm.c        |  81 +++----
> >  drivers/gpu/drm/i915/display/intel_bios.c     |  73 +++---
> >  drivers/gpu/drm/i915/display/intel_bios.h     |   3 +-
> >  drivers/gpu/drm/i915/display/intel_color.c    |  11 +-
> >  drivers/gpu/drm/i915/display/intel_dp_hdcp.c  |  12 +-
> >  drivers/gpu/drm/i915/display/intel_fb.c       |   7 +-
> >  .../gpu/drm/i915/display/intel_global_state.h |   4 +-
> >  drivers/gpu/drm/i915/display/intel_opregion.c |   6 +-
> >  drivers/gpu/drm/i915/display/intel_sdvo.c     | 217 ++++++++++--------
> >  15 files changed, 233 insertions(+), 197 deletions(-)
> 
> -- 
> Jani Nikula, Intel

-- 
Ville Syrjälä
Intel

  reply	other threads:[~2024-02-09 12:54 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-08 15:17 [PATCH 00/13] drm/i915: drm_dbg_kms() conversions and cleanups Ville Syrjala
2024-02-08 15:17 ` [PATCH 01/13] drm/i915: Correct for_each_old_global_obj_in_state() arguments Ville Syrjala
2024-02-08 15:17 ` [PATCH 02/13] drm/i915/sdvo: Convert to per-device debugs Ville Syrjala
2024-02-08 15:17 ` [PATCH 03/13] drm/i915/sdvo: Fix up code alignment Ville Syrjala
2024-02-08 15:17 ` [PATCH 04/13] drm/i915/color: Use per-device debugs Ville Syrjala
2024-02-08 15:17 ` [PATCH 05/13] drm/i915/fb: " Ville Syrjala
2024-02-08 15:17 ` [PATCH 06/13] drm/i915/bios: Switch to kms debugs Ville Syrjala
2024-02-08 15:17 ` [PATCH 07/13] drm/i915/bios: Use per-device debugs for VBT related stuff Ville Syrjala
2024-02-08 15:17 ` [PATCH 08/13] drm/i915/hdcp: Use per-device debugs Ville Syrjala
2024-02-08 15:17 ` [PATCH 09/13] drm/i915/wm: Pass the whole i916 to intel_get_cxsr_latency() Ville Syrjala
2024-02-08 15:17 ` [PATCH 10/13] drm/i915/wm: Use per-device debugs in pre-ilk wm code Ville Syrjala
2024-02-08 15:17 ` [PATCH 11/13] drm/i915/wm: Use per-device debugs ilk " Ville Syrjala
2024-02-08 15:17 ` [PATCH 12/13] drm/i915/dvo/ns2501: Nuke pointless casts Ville Syrjala
2024-02-08 15:17 ` [PATCH 13/13] drm/i915/dvo: Use sizeof(*variable) instead of sizeof(type) Ville Syrjala
2024-02-08 15:52 ` [PATCH 00/13] drm/i915: drm_dbg_kms() conversions and cleanups Jani Nikula
2024-02-09 12:54   ` Ville Syrjälä [this message]
2024-02-08 21:43 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2024-02-08 21:43 ` ✗ Fi.CI.SPARSE: " Patchwork
2024-02-08 22:02 ` ✓ Fi.CI.BAT: success " Patchwork
2024-02-09  3:35 ` ✗ Fi.CI.IGT: failure " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZcYgbcy_t5jMpZri@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox