Skip to content

Commit 2700e35

Browse files
lotheacgregkh
authored andcommitted
drm/amd/display: fix initial backlight brightness calculation
commit 9c28830 upstream. DIV_ROUND_CLOSEST(x, 100) returns either 0 or 1 if 0<x<=100, so the division needs to be performed after the multiplication and not the other way around, to properly scale the value. Fixes: 8b5f3a2 ("drm/amd/display: Fix default DC and AC levels") Signed-off-by: Lauri Tirkkonen <lauri@hacktheplanet.fi> Cc: stable@vger.kernel.org Reviewed-by: Mario Limonciello <mario.limonciello@amd.com> Link: https://lore.kernel.org/r/aH2Q_HJvxKbW74vU@hacktheplanet.fi Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent a3fe590 commit 2700e35

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4952,9 +4952,9 @@ amdgpu_dm_register_backlight_device(struct amdgpu_dm_connector *aconnector)
49524952
caps = &dm->backlight_caps[aconnector->bl_idx];
49534953
if (get_brightness_range(caps, &min, &max)) {
49544954
if (power_supply_is_system_supplied() > 0)
4955-
props.brightness = (max - min) * DIV_ROUND_CLOSEST(caps->ac_level, 100);
4955+
props.brightness = DIV_ROUND_CLOSEST((max - min) * caps->ac_level, 100);
49564956
else
4957-
props.brightness = (max - min) * DIV_ROUND_CLOSEST(caps->dc_level, 100);
4957+
props.brightness = DIV_ROUND_CLOSEST((max - min) * caps->dc_level, 100);
49584958
/* min is zero, so max needs to be adjusted */
49594959
props.max_brightness = max - min;
49604960
drm_dbg(drm, "Backlight caps: min: %d, max: %d, ac %d, dc %d\n", min, max,

0 commit comments

Comments
 (0)