From: Josip Pavic <josip.pa...@amd.com>

[Why]
Current configuration 0 is just a placeholder, and final parameters needed.
Also, configuration 1 is expected to emulate ABM 2.1 but is too aggressive.

[How]
Redefine configuration 0 with the finalized parameters, and increase the
contrast gain of configuration 1 so that it properly emulates ABM 2.1.

Signed-off-by: Josip Pavic <josip.pa...@amd.com>
Reviewed-by: Anthony Koo <anthony....@amd.com>
Acked-by: Leo Li <sunpeng...@amd.com>
---
 .../amd/display/modules/power/power_helpers.c    | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/modules/power/power_helpers.c 
b/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
index 05e2be856037..cc6b794821d9 100644
--- a/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
+++ b/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
@@ -80,18 +80,18 @@ struct abm_parameters {
 
 static const struct abm_parameters abm_settings_config0[abm_defines_max_level] 
= {
 //  min_red  max_red  bright_pos  dark_pos  brightness_gain  contrast  
deviation  min_knee  max_knee
-       {0xff,   0xbf,    0x20,       0x00,     0xff,            0x99,     
0xb3,      0x40,     0xE0},
-       {0xff,   0x85,    0x20,       0x00,     0xff,            0x90,     
0xa8,      0x40,     0xE0},
-       {0xff,   0x40,    0x20,       0x00,     0xff,            0x90,     
0x68,      0x40,     0xE0},
-       {0x82,   0x4d,    0x20,       0x00,     0x00,            0x90,     
0xb3,      0x70,     0x70},
+       {0xff,   0xbf,    0x20,       0x00,     0xff,            0x99,     
0xb3,      0x40,     0xe0},
+       {0xde,   0x85,    0x20,       0x00,     0xff,            0x90,     
0xa8,      0x40,     0xdf},
+       {0xb0,   0x50,    0x20,       0x00,     0xc0,            0x88,     
0x78,      0x70,     0xa0},
+       {0x82,   0x40,    0x20,       0x00,     0x00,            0xff,     
0xb3,      0x70,     0x70},
 };
 
 static const struct abm_parameters abm_settings_config1[abm_defines_max_level] 
= {
 //  min_red  max_red  bright_pos  dark_pos  brightness_gain  contrast  
deviation  min_knee  max_knee
-       {0xf0,   0xd9,    0x20,       0x00,     0x00,            0xa8,     
0xb3,      0x70,     0x70},
-       {0xcd,   0xa5,    0x20,       0x00,     0x00,            0xa8,     
0xb3,      0x70,     0x70},
-       {0x99,   0x65,    0x20,       0x00,     0x00,            0xa8,     
0xb3,      0x70,     0x70},
-       {0x82,   0x4d,    0x20,       0x00,     0x00,            0xa8,     
0xb3,      0x70,     0x70},
+       {0xf0,   0xd9,    0x20,       0x00,     0x00,            0xff,     
0xb3,      0x70,     0x70},
+       {0xcd,   0xa5,    0x20,       0x00,     0x00,            0xff,     
0xb3,      0x70,     0x70},
+       {0x99,   0x65,    0x20,       0x00,     0x00,            0xff,     
0xb3,      0x70,     0x70},
+       {0x82,   0x4d,    0x20,       0x00,     0x00,            0xff,     
0xb3,      0x70,     0x70},
 };
 
 static const struct abm_parameters * const abm_settings[] = {
-- 
2.22.0

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

Reply via email to