All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ville Syrjala <ville.syrjala@linux.intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: [PATCH 00/11] drm/i915/cdclk: Declutter CDCLK code
Date: Tue, 29 Oct 2024 23:52:06 +0200	[thread overview]
Message-ID: <20241029215217.3697-1-ville.syrjala@linux.intel.com> (raw)

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Declutter the CDCLK code a bit by moving a all the
foo_min_cdclk() things out into their respective files.

And I also reworked the "pixels per clock" and
guardband handling so that we don't have to repeat
essentially the same platform if-ladder two times.

TODO: the dsi and audio min_cdclk stuff could perhaps
      be more cleanly abstracted into some kind of
      encoder->min_cdlk() thing, but haven't really
      given this more than a cursory thought...

Ville Syrjälä (11):
  drm/i915: Introduce HAS_DOUBLE_WIDE()
  drm/i915/cdclk: Extract intel_cdclk_guardband() and intel_cdclk_ppc()
  drm/i915/cdclk: Extract hsw_ips_min_cdclk()
  drm/i915/cdclk: Extract intel_audio_min_cdclk()
  drm/i915/cdclk: Factor out has_audio check in intel_audio_min_cdclk()
  drm/i915/cdclk: Extract vlv_dsi_min_cdclk()
  drm/i915/cdclk: Factor out INTEL_OUTPUT_DSI check in
    vlv_dsi_min_cdclk()
  drm/i915/cdclk: Suck the compression_enable check into
    intel_vdsc_min_cdclk()
  drm/i915/cdclk: Drop pointles max_t() usage in intel_vdsc_min_cdclk()
  drm/i915/cdclk: Relocate intel_vdsc_min_cdclk()
  drm/i915/cdclk: Unify cdclk max() parameter order

 drivers/gpu/drm/i915/display/hsw_ips.c        |  16 +-
 drivers/gpu/drm/i915/display/hsw_ips.h        |   6 +-
 drivers/gpu/drm/i915/display/intel_audio.c    |  47 +++++
 drivers/gpu/drm/i915/display/intel_audio.h    |   1 +
 drivers/gpu/drm/i915/display/intel_bw.c       |   2 +-
 drivers/gpu/drm/i915/display/intel_cdclk.c    | 175 ++++--------------
 drivers/gpu/drm/i915/display/intel_display.c  |   4 +-
 .../drm/i915/display/intel_display_device.h   |   1 +
 drivers/gpu/drm/i915/display/intel_vdsc.c     |  46 +++++
 drivers/gpu/drm/i915/display/intel_vdsc.h     |   1 +
 drivers/gpu/drm/i915/display/vlv_dsi.c        |  25 +++
 drivers/gpu/drm/i915/display/vlv_dsi.h        |   8 +
 12 files changed, 187 insertions(+), 145 deletions(-)

-- 
2.45.2


             reply	other threads:[~2024-10-29 21:52 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-29 21:52 Ville Syrjala [this message]
2024-10-29 21:52 ` [PATCH 01/11] drm/i915: Introduce HAS_DOUBLE_WIDE() Ville Syrjala
2024-10-30 11:23   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 02/11] drm/i915/cdclk: Extract intel_cdclk_guardband() and intel_cdclk_ppc() Ville Syrjala
2024-10-30 11:23   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 03/11] drm/i915/cdclk: Extract hsw_ips_min_cdclk() Ville Syrjala
2024-10-30 11:26   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 04/11] drm/i915/cdclk: Extract intel_audio_min_cdclk() Ville Syrjala
2024-10-30 11:30   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 05/11] drm/i915/cdclk: Factor out has_audio check in intel_audio_min_cdclk() Ville Syrjala
2024-10-30 11:30   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 06/11] drm/i915/cdclk: Extract vlv_dsi_min_cdclk() Ville Syrjala
2024-10-30 11:34   ` Jani Nikula
2024-10-30 13:22     ` Ville Syrjälä
2024-10-30 17:55       ` Jani Nikula
2024-10-29 21:52 ` [PATCH 07/11] drm/i915/cdclk: Factor out INTEL_OUTPUT_DSI check in vlv_dsi_min_cdclk() Ville Syrjala
2024-10-30 11:35   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 08/11] drm/i915/cdclk: Suck the compression_enable check into intel_vdsc_min_cdclk() Ville Syrjala
2024-10-30 11:37   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 09/11] drm/i915/cdclk: Drop pointles max_t() usage in intel_vdsc_min_cdclk() Ville Syrjala
2024-10-30 11:39   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 10/11] drm/i915/cdclk: Relocate intel_vdsc_min_cdclk() Ville Syrjala
2024-10-30 11:40   ` Jani Nikula
2024-10-29 21:52 ` [PATCH 11/11] drm/i915/cdclk: Unify cdclk max() parameter order Ville Syrjala
2024-10-30 11:41   ` Jani Nikula
2024-10-30  1:30 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/cdclk: Declutter CDCLK code Patchwork
2024-10-30  1:30 ` ✗ Fi.CI.SPARSE: " Patchwork
2024-10-30  1:46 ` ✗ Fi.CI.BAT: failure " Patchwork
2024-10-31 13:20 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/cdclk: Declutter CDCLK code (rev2) Patchwork
2024-10-31 13:20 ` ✗ Fi.CI.SPARSE: " Patchwork
2024-10-31 14:16 ` ✓ Fi.CI.BAT: success " Patchwork
2024-10-31 21:02 ` ✗ 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=20241029215217.3697-1-ville.syrjala@linux.intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /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 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.