Skip to contentSkip to conversation

linux-next: build failure after merge of the amdgpu tree

DISCUSSION Stephen Rothwell 3 participants 6 messages May 4, 2021 – May 10, 2021 Linux-next tree

lore
Share & tools
lore
#0 lore raw

Hi all,

After merging the amdgpu tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In function 'amdgpu_dm_initialize_drm_device':
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:3728:7: error: implicit declaration of function 'register_outbox_irq_handlers'; did you mean 'register_hpd_handlers'? [-Werror=implicit-function-declaration]

 3728 |   if (register_outbox_irq_handlers(dm->adev)) {
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |       register_hpd_handlers

Caused by commit

77a49c458931 ("drm/amd/display: Support for DMUB AUX")

I have used the amdgpu tree from next-20210504 for today.

Signature · 2 lines
Cheers,
Stephen Rothwell
#1 lore raw

Hi all,

Quoted 10 lines · On Wed, 5 May 2021 09:34:58 +1000 Stephen Rothwell <[email protected]> wrote:

After merging the amdgpu tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In function 'amdgpu_dm_initialize_drm_device':
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:3728:7: error: implicit declaration of function 'register_outbox_irq_handlers'; did you mean 'register_hpd_handlers'? [-Werror=implicit-function-declaration]

 3728 |   if (register_outbox_irq_handlers(dm->adev)) {
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |       register_hpd_handlers
Quoted 5 lines

Caused by commit

77a49c458931 ("drm/amd/display: Support for DMUB AUX")

I have used the amdgpu tree from next-20210504 for today.

This has now been broken for 6 days ... please fix or revert.

Signature · 2 lines
Cheers,
Stephen Rothwell
#2 lore raw
Quoted 14 lines · Stephen Rothwell

Hi all,

Quoted 10 lines · On Wed, 5 May 2021 09:34:58 +1000 Stephen Rothwell <[email protected]> wrote:

After merging the amdgpu tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In function 'amdgpu_dm_initialize_drm_device':
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:3728:7: error: implicit declaration of function 'register_outbox_irq_handlers'; did you mean 'register_hpd_handlers'? [-Werror=implicit-function-declaration]

 3728 |   if (register_outbox_irq_handlers(dm->adev)) {
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |       register_hpd_handlers
Quoted 7 lines
Quoted 5 lines

Caused by commit

77a49c458931 ("drm/amd/display: Support for DMUB AUX")

I have used the amdgpu tree from next-20210504 for today.

This has now been broken for 6 days ... please fix or revert.

Fixed in this patch set:
https://patchwork.freedesktop.org/series/89890/

Alex

Quoted 4 lines

--
Cheers,
Stephen Rothwell

#4 lore raw
Quoted 11 lines · Stephen Rothwell

Hi Alex,

Quoted 3 lines · On Mon, 10 May 2021 10:24:58 -0400 Alex Deucher <[email protected]> wrote:

Fixed in this patch set:
https://patchwork.freedesktop.org/series/89890/

Sure, but that is not in the amdgpu tree that linux-next includes
(https://gitlab.freedesktop.org/agd5f/linux#drm-next). Did you forget to push it out, or is it still being reviewed/tested?

Was still being reviewed. I just pushed out the tree with it included.

Alex

Quoted 4 lines

--
Cheers,
Stephen Rothwell

#5 lore raw

Hi Alex,

Quoted 5 lines · On Mon, 10 May 2021 18:23:35 -0400 Alex Deucher <[email protected]> wrote:
Quoted 3 lines · Stephen Rothwell
Quoted 1 line · On Mon, 10 May 2021 10:24:58 -0400 Alex Deucher <[email protected]> wrote:
Quoted 8 lines
Quoted 6 lines
Quoted 2 lines

Fixed in this patch set:
https://patchwork.freedesktop.org/series/89890/

Sure, but that is not in the amdgpu tree that linux-next includes
(https://gitlab.freedesktop.org/agd5f/linux#drm-next). Did you forget to push it out, or is it still being reviewed/tested?

Was still being reviewed. I just pushed out the tree with it included.

Excellent, thanks.

Signature · 2 lines
Cheers,
Stephen Rothwell