[PATCH 1/9] drm/amdgpu: Use backlight power constants

Thomas Zimmermann tzimmermann at suse.de
Tue Aug 6 07:06:51 UTC 2024


Hi

Am 05.08.24 um 21:00 schrieb Deucher, Alexander:
> [Public]
>
>> -----Original Message-----
>> From: Thomas Zimmermann <tzimmermann at suse.de>
>> Sent: Wednesday, July 31, 2024 8:17 AM
>> To: maarten.lankhorst at linux.intel.com; mripard at kernel.org;
>> airlied at gmail.com; daniel at ffwll.ch
>> Cc: amd-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org; intel-
>> gfx at lists.freedesktop.org; intel-xe at lists.freedesktop.org; Thomas
>> Zimmermann <tzimmermann at suse.de>; Deucher, Alexander
>> <Alexander.Deucher at amd.com>; Koenig, Christian
>> <Christian.Koenig at amd.com>; Pan, Xinhui <Xinhui.Pan at amd.com>
>> Subject: [PATCH 1/9] drm/amdgpu: Use backlight power constants
>>
>> Replace FB_BLANK_ constants with their counterparts from the backlight
>> subsystem. The values are identical, so there's no change in functionality or
>> semantics.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
>> Cc: Alex Deucher <alexander.deucher at amd.com>
>> Cc: "Christian König" <christian.koenig at amd.com>
>> Cc: Xinhui Pan <Xinhui.Pan at amd.com>
> This patch and the radeon patch are:
> Acked-by: Alex Deucher <alexander.deucher at amd.com>
>
> Feel free to take them via whatever tree makes sense if you are trying to keep the patches together, or let me know if you want me to pick them up.

Thank you for the reviews. Please pick up both patches into the rsp AMD 
trees. The i915 patch already went into Intel trees and the rest of the 
series will go into drm-misc.

Best regards
Thomas

>
> Thanks,
>
> Alex
>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/atombios_encoders.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/atombios_encoders.c
>> b/drivers/gpu/drm/amd/amdgpu/atombios_encoders.c
>> index 25feab188dfe..650ec95bb40a 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/atombios_encoders.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/atombios_encoders.c
>> @@ -215,7 +215,7 @@ void
>> amdgpu_atombios_encoder_init_backlight(struct amdgpu_encoder
>> *amdgpu_encode
>>        dig->bl_dev = bd;
>>
>>        bd->props.brightness =
>> amdgpu_atombios_encoder_get_backlight_brightness(bd);
>> -     bd->props.power = FB_BLANK_UNBLANK;
>> +     bd->props.power = BACKLIGHT_POWER_ON;
>>        backlight_update_status(bd);
>>
>>        DRM_INFO("amdgpu atom DIG backlight initialized\n");
>> --
>> 2.45.2

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)



More information about the Intel-gfx mailing list