OSDN Git Service

drm/amd/display/amdgpu_dm: Refactor register_backlight_device()
authorHans de Goede <hdegoede@redhat.com>
Sun, 12 Mar 2023 19:17:47 +0000 (20:17 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 22 Mar 2023 04:47:59 +0000 (00:47 -0400)
commitceb4a5619c734ea679bf5a2446e6869056e353d4
tree6f21d422e95f64acad9cf704767d0be4c47655b4
parent4db231d7dd80163fd251635f81280f1c93a393ab
drm/amd/display/amdgpu_dm: Refactor register_backlight_device()

Refactor register_backlight_device():

1) Turn the connector-type + signal check into an early exit
condition to avoid the indentation level of the rest of the code

2) Add an array bounds check for the arrays indexed by dm->num_of_edps

3) register_backlight_device() always increases dm->num_of_edps if
amdgpu_dm_register_backlight_device() has assigned a backlight_dev to
the current dm->backlight_link[dm->num_of_edps] slot.

So on its next call dm->backlight_dev[dm->num_of_edps] always point to
the next empty slot and the "if (!dm->backlight_dev[dm->num_of_edps])"
check will thus always succeed and can be removed.

4) Add a bl_idx local variable to use as array index, rather then
using dm->num_of_edps to improve the code readability.

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c