additional patch for your 5.13.y branch

Open
#4,534 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
35/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Tech stack
c, linux

Research direction

Open drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c and inspect amdgpu_dm_atomic_commit_tail around the reported line. Verify the missing declaration and compare the suggested replacement with the surrounding kernel APIs, then compile the 5.13.y branch to confirm the error is resolved.

Written by the indexing model from the issue text.

Description

I am compiling your 5.13.y kernel, but get a compile error:

/home/flaroche/data/kernel/linux-5/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In function ‘amdgpu_dm_atomic_commit_tail’:
/home/flaroche/data/kernel/linux-5/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8854:4: error: implicit declaration of function ‘is_hdr_metadata_different’ [-Werror=implicit-function-declaration]
8854 | is_hdr_metadata_different(old_con_state, new_con_state);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
make[7]: *** [/home/flaroche/data/kernel/linux-5/scripts/Makefile.build:278: drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.o] Fehler 1
make[6]: *** [/home/flaroche/data/kernel/linux-5/scripts/Makefile.build:521: drivers/gpu/drm/amd/amdgpu] Fehler 2

I haven't looked into this in more detail, but a similar patch is part of your raspberry pi patches and
thus the following change looks suitable:

--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -8851,7 +8851,7 @@
                              dm_old_crtc_state->abm_level;
 
                hdr_changed =
-                       is_hdr_metadata_different(old_con_state, new_con_state);
+                       !drm_connector_atomic_hdr_metadata_equal(old_con_state, new_con_state);
 
                if (!scaling_changed && !abm_changed && !hdr_changed)
                        continue;
Dominant language
C
Stars
13.2k
Forks
5.5k
Avg merge
2d 21h
Merged PRs (30d)
21

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from raspberrypi/linux

All issues in raspberrypi/linux

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.