On 19-Sep-19 6:01 PM, Jani Nikula wrote:
On Wed, 18 Sep 2019, "Sharma, Swati2" <swati2.sharma@xxxxxxxxx> wrote:
On 18-Sep-19 3:31 PM, Jani Nikula wrote:
On Tue, 17 Sep 2019, Swati Sharma <swati2.sharma@xxxxxxxxx> wrote:
For icl+, have hw read out to create hw blob of gamma
lut values. icl+ platforms supports multi segmented gamma
mode, add hw lut creation for this mode.
This will be used to validate gamma programming using dsb
(display state buffer) which is a tgl feature.
v2: -readout code for multisegmented gamma has to come
up with some intermediate entries that aren't preserved
in hardware (Jani N)
-linear interpolation (Ville)
-moved common code to check gamma_enable to specific funcs,
since icl doesn't support that
Signed-off-by: Swati Sharma <swati2.sharma@xxxxxxxxx>
---
drivers/gpu/drm/i915/display/intel_color.c | 243 ++++++++++++++++++++++++++---
drivers/gpu/drm/i915/i915_reg.h | 7 +
2 files changed, 230 insertions(+), 20 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm/i915/display/intel_color.c
index b1f0f7e..0008011 100644
--- a/drivers/gpu/drm/i915/display/intel_color.c
+++ b/drivers/gpu/drm/i915/display/intel_color.c
@@ -1370,6 +1370,9 @@ static int icl_color_check(struct intel_crtc_state *crtc_state)
static int i9xx_gamma_precision(const struct intel_crtc_state *crtc_state)
{
+ if (!crtc_state->gamma_enable)
+ return 0; >> +
Why are you moving these checks back to the individual functions?
As stated in commit message, moved common code to check gamma_enable to
specific funcs, since icl doesn't support gamma_enable and code will
return 0. If i need to make it generic, i need to make gamma_enable true
in icl_color_check() func. Is it fine? ICL enables gamma through
gamma_mode unlike other platforms.
Argh. Right. Okay, let's go with what you have in this patch. We can
clean this stuff up later.
Please write the main part of the commit message such that it is
independent of the changelog. The changelog is good, but the actual
changes need to be evident from the message part.
sure.
BR,
Jani.
--
~Swati Sharma
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
