ACK: [PATCH 0/4][SRU][M] The screen brightness is unable to adjust on BOE panel DPN#R6FD8

[Tim Gardner]

On 3/12/24 03:06, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao at canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/2057430
> 
> [Impact]
> The screen brightness is unable to adjust on BOE panel DPN#R6FD8
> 
> [Fix]
> AMD provides a patch which is included in v6.8-rc7 to fix this issue
> b7cdccc6a849 drm/amd/display: Add monitor patch for specific eDP
> 
> To avoid conflicts, pull in 3 other small patches
> c4e532f75336 drm/amd/display: Re-add aux intercept disable delay generically for 2+ LTTPRs
> 923bbfe6c888 drm/amd/display: Clear dpcd_sink_ext_caps if not set
> 3d71a8726e05 drm/amd/display: Add monitor patch for specific eDP
> 
> [Test Case]
> 1. Boot to OS
> 2. Adjust the screen brightness via the hotkeys on the keyboard or brightness bar in the setting.
> 3. The screen brightness should change
> 
> [Where problems could occur]
> The first commit introduces no functional changes, it merely relocates the delay flag to dc_debug_options and makes some consequent modifications.
> The second commit introduce a new flag, but we don't use it.
> The third and forth commits add 4 panel IDs which should be pretty safe to include them.
> 
> Ivan Lipski (1):
>    drm/amd/display: Add monitor patch for specific eDP
> 
> Michael Strauss (1):
>    drm/amd/display: Re-add aux intercept disable delay generically for 2+
>      LTTPRs
> 
> Paul Hsieh (1):
>    drm/amd/display: Clear dpcd_sink_ext_caps if not set
> 
> Ryan Lin (1):
>    drm/amd/display: Add monitor patch for specific eDP
> 
>   .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 12 +++++++--
>   drivers/gpu/drm/amd/display/dc/dc.h           |  1 +
>   drivers/gpu/drm/amd/display/dc/dc_types.h     |  2 +-
>   .../drm/amd/display/dc/link/link_detection.c  |  3 +++
>   .../link_dp_training_fixed_vs_pe_retimer.c    | 25 +++++++++----------
>   5 files changed, 27 insertions(+), 16 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>
-- 
-----------
Tim Gardner
Canonical, Inc