All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Matt Roper <matthew.d.roper@intel.com>
Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/4] Pipe level color management
Date: Tue, 22 Mar 2016 11:57:17 +0100	[thread overview]
Message-ID: <20160322105717.GV28483@phenom.ffwll.local> (raw)
In-Reply-To: <20160321185534.GB20526@intel.com>

On Mon, Mar 21, 2016 at 11:55:34AM -0700, Matt Roper wrote:
> On Wed, Mar 16, 2016 at 10:57:13AM +0000, Lionel Landwerlin wrote:
> > This series introduces pipe level color management for Intel platforms
> > on CherryView and Broadwell on.
> > 
> > This series is based of a previous set of patches by Shashank Sharma.
> > 
> > Cheers,
> > 
> > Lionel
> > 
> > v9: Rebase on nightly
> > 
> > v10: Mask GAMMA_MODE register value (Matt Roper)
> >      Cleanup legacy LUT helper (Emil Velikov)
> > 
> > v11: Rebase on drm-nightly
> 
> The only issues raised by CI were a sporadic error that we already have
> a bug (and fix) for at
> https://bugs.freedesktop.org/show_bug.cgi?id=94349
> 
> Everything looks good, so merged these to dinq and merged the
> corresponding igt tests as well.  Thanks for the patches.

Can we please, please fast-track this bugfix instead of piling ever more
features on? That ilk wm warning has been seriously annoying everyone for
a long time ...

And imo it's totally fine if the bugfix isn't perfect, as long as it's not
clearly a bad idea and a reasonable (first/minor/whatever) step towards
the real solution.
-Daniel

> 
> 
> Matt
> 
> > 
> > Lionel Landwerlin (4):
> >   drm/i915: Extract out gamma table and CSC to their own file
> >   drm/i915: Do not read GAMMA_MODE register
> >   drm/i915: Implement color management on bdw/skl/bxt/kbl
> >   drm/i915: Implement color management on chv
> > 
> >  Documentation/DocBook/gpu.tmpl       |   6 +-
> >  drivers/gpu/drm/i915/Makefile        |   1 +
> >  drivers/gpu/drm/i915/i915_drv.c      |  27 +-
> >  drivers/gpu/drm/i915/i915_drv.h      |   8 +
> >  drivers/gpu/drm/i915/i915_reg.h      |  53 ++++
> >  drivers/gpu/drm/i915/intel_color.c   | 556 +++++++++++++++++++++++++++++++++++
> >  drivers/gpu/drm/i915/intel_display.c | 184 +++---------
> >  drivers/gpu/drm/i915/intel_drv.h     |  12 +
> >  drivers/gpu/drm/i915/intel_fbdev.c   |   8 +
> >  9 files changed, 695 insertions(+), 160 deletions(-)
> >  create mode 100644 drivers/gpu/drm/i915/intel_color.c
> > 
> > --
> > 2.7.0
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Matt Roper
> Graphics Software Engineer
> IoTG Platform Enabling & Development
> Intel Corporation
> (916) 356-2795

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

      reply	other threads:[~2016-03-22 10:56 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-16 10:57 [PATCH 0/4] Pipe level color management Lionel Landwerlin
2016-03-16 10:57 ` [PATCH 1/4] drm/i915: Extract out gamma table and CSC to their own file Lionel Landwerlin
2016-03-16 10:57 ` [PATCH 2/4] drm/i915: Do not read GAMMA_MODE register Lionel Landwerlin
2016-03-16 10:57 ` [PATCH 3/4] drm/i915: Implement color management on bdw/skl/bxt/kbl Lionel Landwerlin
2016-03-16 10:57 ` [PATCH 4/4] drm/i915: Implement color management on chv Lionel Landwerlin
2016-03-16 13:30 ` ✗ Fi.CI.BAT: failure for Pipe level color management (rev11) Patchwork
2016-03-21 18:55 ` [PATCH 0/4] Pipe level color management Matt Roper
2016-03-22 10:57   ` Daniel Vetter [this message]

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=20160322105717.GV28483@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=daniel.vetter@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=matthew.d.roper@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 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.