Date: Tue, 11 Apr 2023 13:26:35 +0200
An immediate return is performed by this function after a null pointer
was detected for the member “dc_link” in the data
structure “amdgpu_dm_connector”.
This check was repeated within one if branch.
Thus omit such a redundant statement.
Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
index a37d23a13d7b..4805a482dc49 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
@@ -1278,9 +1278,6 @@ static ssize_t trigger_hotplug(struct file *f, const char __user *buf,
drm_kms_helper_connector_hotplug_event(connector);
} else if (param[0] == 0) {
- if (!aconnector->dc_link)
- goto unlock;
-
link = aconnector->dc_link;
if (link->local_sink) {
--
2.40.0
Am 11.04.23 um 15:36 schrieb Markus Elfring:
> Date: Tue, 11 Apr 2023 14:36:36 +0200
>
> Some update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (5)
> amdgpu: Move a variable assignment behind a null pointer check in amdgpu_ras_interrupt_dispatch()
> display: Move three variable assignments behind condition checks in trigger_hotplug()
> display: Delete three unnecessary variable initialisations in trigger_hotplug()
> display: Delete a redundant statement in trigger_hotplug()
> display: Move an expression into a return statement in dcn201_link_encoder_create()
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 3 ++-
> .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 19 ++++++++++---------
> .../amd/display/dc/dcn201/dcn201_resource.c | 4 +---
> 3 files changed, 13 insertions(+), 13 deletions(-)
>
