All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matt Roper <matthew.d.roper@intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: [PATCH v5 0/3] CRTC background color
Date: Thu, 31 Jan 2019 15:55:05 -0800	[thread overview]
Message-ID: <20190131235508.6970-1-matthew.d.roper@intel.com> (raw)

Previous patch series/discussion was here:
  https://lists.freedesktop.org/archives/dri-devel/2019-January/205504.html

Reviewed userspace (chromeos) is here:
  https://chromium-review.googlesource.com/c/chromium/src/+/1278858
  https://chromium-review.googlesource.com/c/chromiumos/platform/drm-tests/+/1241436

i-g-t test is here:
  https://lists.freedesktop.org/archives/igt-dev/2019-January/008751.html


Just minor changes in this revision:
 - Patch 1:
    * drm_crtc_add_bgcolor_property() also explicitly sets the crtc
      state field to solid black in case the driver doesn't already do
      this.
    * Added kerneldoc for drm_crtc_add_bgcolor_property()

 - Patch 2:
    * s/uint64_t/u64/

 - Patch 3:
    * New i915 patch to add bgcolor to hardware readout & state
      verification as suggested by Ville


I think patches #1 and 2 have the necessary reviews and ABI prereqs to
merge at this point.  Dave/Daniel, any worries if we merge this through
the intel tree rather than drm-misc?  I think merging through the Intel
tree may help avoid some minor conflicts with an upcoming series from
Ville.


Matt Roper (3):
  drm: Add CRTC background color property (v5)
  drm/i915/gen9+: Add support for pipe background color (v5)
  drm/i915: Add background color hardware readout and state check

 drivers/gpu/drm/drm_atomic_uapi.c    |  4 ++
 drivers/gpu/drm/drm_blend.c          | 41 +++++++++++++++++--
 drivers/gpu/drm/drm_mode_config.c    |  6 +++
 drivers/gpu/drm/i915/i915_debugfs.c  |  9 +++++
 drivers/gpu/drm/i915/intel_display.c | 78 +++++++++++++++++++++++++++++++-----
 include/drm/drm_blend.h              |  1 +
 include/drm/drm_crtc.h               | 12 ++++++
 include/drm/drm_mode_config.h        |  5 +++
 include/uapi/drm/drm_mode.h          | 28 +++++++++++++
 9 files changed, 170 insertions(+), 14 deletions(-)

-- 
2.14.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

             reply	other threads:[~2019-01-31 23:56 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-31 23:55 Matt Roper [this message]
2019-01-31 23:55 ` [PATCH v5 1/3] drm: Add CRTC background color property (v5) Matt Roper
2019-01-31 23:55 ` [PATCH v5 2/3] drm/i915/gen9+: Add support for pipe background color (v5) Matt Roper
2019-01-31 23:55 ` [PATCH v5 3/3] drm/i915: Add background color hardware readout and state check Matt Roper
2019-02-05 20:09   ` Ville Syrjälä
2019-02-01  0:29 ` ✗ Fi.CI.CHECKPATCH: warning for CRTC background color (rev6) Patchwork
2019-02-01  0:31 ` ✗ Fi.CI.SPARSE: " Patchwork
2019-02-01  1:06 ` ✓ Fi.CI.BAT: success " Patchwork
2019-02-01  4:36 ` ✓ Fi.CI.IGT: " 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=20190131235508.6970-1-matthew.d.roper@intel.com \
    --to=matthew.d.roper@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.