<div dir="ltr"><div>Your v2 has my <br></div><div>Reviewed-by: Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>></div><div><br></div><div>thanks,</div><div>-mario</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 17, 2021 at 11:51 PM Alex Deucher <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>><br>
<br>
Spatial dithering to 10 bpc depth was disabled for all DCE's.<br>
<br>
Testing on DCE-8.3 and DCE-11.2 did not show any obvious ill<br>
effects, but a measureable precision improvement (via colorimeter)<br>
when displaying a fp16 framebuffer to a 10 bpc DP or HDMI connected<br>
HDR-10 monitor.<br>
<br>
v2: enable it for all DCEs (Alex)<br>
<br>
Signed-off-by: Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>><br>
Cc: Alex Deucher <<a href="mailto:alexdeucher@gmail.com" target="_blank">alexdeucher@gmail.com</a>><br>
Signed-off-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com" target="_blank">alexander.deucher@amd.com</a>><br>
---<br>
 drivers/gpu/drm/amd/display/dc/dce/dce_opp.c | 4 +---<br>
 1 file changed, 1 insertion(+), 3 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c b/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c<br>
index 4600231da6cb..895b015b02e8 100644<br>
--- a/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c<br>
+++ b/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c<br>
@@ -216,9 +216,7 @@ static void set_spatial_dither(<br>
        REG_UPDATE(FMT_BIT_DEPTH_CONTROL,<br>
                FMT_TEMPORAL_DITHER_EN, 0);<br>
<br>
-       /* no 10bpc on DCE11*/<br>
-       if (params->flags.SPATIAL_DITHER_ENABLED == 0 ||<br>
-               params->flags.SPATIAL_DITHER_DEPTH == 2)<br>
+       if (params->flags.SPATIAL_DITHER_ENABLED == 0)<br>
                return;<br>
<br>
        /* only use FRAME_COUNTER_MAX if frameRandom == 1*/<br>
-- <br>
2.29.2<br>
<br>
</blockquote></div>