From: Harry Wentland <harry.wentland-5C7GfCeVMHo@public.gmane.org>
To: "Michel Dänzer" <michel-otUistvHUpPR7s880joybQ@public.gmane.org>
Cc: Jordan Lazare <Jordan.Lazare-5C7GfCeVMHo@public.gmane.org>,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH 05/15] drm/amd/display: Fix gamma colour corruption for 10 bit surfaces
Date: Wed, 1 Mar 2017 10:01:08 -0500 [thread overview]
Message-ID: <5b7a6fe3-9ef1-145d-dc09-e238b68be3fc@amd.com> (raw)
In-Reply-To: <440e5b55-ec15-a939-1626-f546208e37df-otUistvHUpPR7s880joybQ@public.gmane.org>
On 2017-02-28 08:17 PM, Michel Dänzer wrote:
> On 01/03/17 09:26 AM, Harry Wentland wrote:
>> From: Jordan Lazare <Jordan.Lazare@amd.com>
>>
>> Gamma LUT shouldn't be used for 10-bit and above. Should instead be
>> using prescale and bypassing input gamma.
>>
>> Change-Id: I55c08c746bb1e1ab1739e47162ddf06db12fe873
>> Signed-off-by: Jordan Lazare <Jordan.Lazare@amd.com>
>> Acked-by: Harry Wentland <Harry.Wentland@amd.com>
>> Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
>> ---
>> .../amd/display/dc/dce110/dce110_hw_sequencer.c | 15 ++++-
>> drivers/gpu/drm/amd/display/dc/dce80/Makefile | 3 +-
>> .../gpu/drm/amd/display/dc/dce80/dce80_ipp_gamma.c | 71 ----------------------
>
> The removal of dce80_ipp_gamma.c and corresponding Makefile change looks
> unrelated and should be a separate change.
>
>
The change is removing dce80_ipp_gamma.c and has to remove it from
Makefile as well, unless there's something I'm not seeing.
Harry
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
next prev parent reply other threads:[~2017-03-01 15:01 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-01 0:26 [PATCH 00/15] DC Patches Feb 28, 2017 Harry Wentland
[not found] ` <20170301002620.24595-1-harry.wentland-5C7GfCeVMHo@public.gmane.org>
2017-03-01 0:26 ` [PATCH 01/15] drm/amd/display: rename bandwidth_calcs.c to dce_calcs.c Harry Wentland
2017-03-01 0:26 ` [PATCH 02/15] drm/amd/display: HDMI deep color mode audio issue Harry Wentland
2017-03-01 0:26 ` [PATCH 03/15] drm/amd/display: 420 clock divided by 2 twice Harry Wentland
2017-03-01 0:26 ` [PATCH 04/15] drm/amd/display: DMCU PSR Refactor Harry Wentland
2017-03-01 0:26 ` [PATCH 05/15] drm/amd/display: Fix gamma colour corruption for 10 bit surfaces Harry Wentland
[not found] ` <20170301002620.24595-6-harry.wentland-5C7GfCeVMHo@public.gmane.org>
2017-03-01 1:17 ` Michel Dänzer
[not found] ` <440e5b55-ec15-a939-1626-f546208e37df-otUistvHUpPR7s880joybQ@public.gmane.org>
2017-03-01 15:01 ` Harry Wentland [this message]
[not found] ` <5b7a6fe3-9ef1-145d-dc09-e238b68be3fc-5C7GfCeVMHo@public.gmane.org>
2017-03-01 15:53 ` Harry Wentland
2017-03-01 0:26 ` [PATCH 06/15] drm/amd/display: add scaler coefficients for 64 phase 5-8 taps Harry Wentland
2017-03-01 0:26 ` [PATCH 07/15] drm/amd/display: Surface Validation Fixes + Audio Mask Harry Wentland
2017-03-01 0:26 ` [PATCH 08/15] drm/amd/display: Fix warnings in amdgpu_dm Harry Wentland
2017-03-01 0:26 ` [PATCH 09/15] drm/amd/display: Fix warnings in DC Harry Wentland
2017-03-01 0:26 ` [PATCH 10/15] drm/amd/display: remove apply_clk_constraints, used validate_bandwidth universally Harry Wentland
2017-03-01 0:26 ` [PATCH 11/15] drm/amd/display: bandwidth update fix Harry Wentland
2017-03-01 0:26 ` [PATCH 12/15] drm/amd/display: move visual confirm recout adjustment to scaler Harry Wentland
2017-03-01 0:26 ` [PATCH 13/15] drm/amd/display: Some more warning fixes Harry Wentland
2017-03-01 0:26 ` [PATCH 14/15] drm/amd/display: Enable warnings as errors Harry Wentland
[not found] ` <20170301002620.24595-15-harry.wentland-5C7GfCeVMHo@public.gmane.org>
2017-03-01 1:19 ` Michel Dänzer
2017-03-01 1:20 ` Emil Velikov
[not found] ` <CACvgo52iOcL5S3=93NEafvoD9yCsvVtryGwKTmVoxwFYZfe1eQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-01 15:04 ` Harry Wentland
2017-03-01 0:26 ` [PATCH 15/15] drm/amd/display: fix dce100_validate_bandwidth return value Harry Wentland
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=5b7a6fe3-9ef1-145d-dc09-e238b68be3fc@amd.com \
--to=harry.wentland-5c7gfcevmho@public.gmane.org \
--cc=Jordan.Lazare-5C7GfCeVMHo@public.gmane.org \
--cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=michel-otUistvHUpPR7s880joybQ@public.gmane.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.