diff options
| author | Simon Ser <[email protected]> | 2020-12-04 16:49:03 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2020-12-09 04:02:24 +0000 |
| commit | 048faf2783a273e162fad5d09089a75c6e2cc10e (patch) | |
| tree | 96c3426f7a81e66da3ad923c58a8d6d12527066b | |
| parent | drm/amd: use drm_dbg_kms to log addfb2 failures (diff) | |
| download | kernel-048faf2783a273e162fad5d09089a75c6e2cc10e.tar.gz kernel-048faf2783a273e162fad5d09089a75c6e2cc10e.zip | |
drm/amd: print error on convert_tiling_flags_to_modifier failure
If this function fails, it means the tiling flags didn't make sense.
This likely indicates a user-space bug. Log the error alongside with the
provided tiling flags to make debugging easier.
Signed-off-by: Simon Ser <[email protected]>
Cc: Alex Deucher <[email protected]>
Cc: Harry Wentland <[email protected]>
Cc: Nicholas Kazlauskas <[email protected]>
Cc: Bas Nieuwenhuizen <[email protected]>
Cc: Michel Dänzer <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c index 73b85da2ccbf..e90be2e8eb2a 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c @@ -901,8 +901,11 @@ int amdgpu_display_framebuffer_init(struct drm_device *dev, if (dev->mode_config.allow_fb_modifiers && !(rfb->base.flags & DRM_MODE_FB_MODIFIERS)) { ret = convert_tiling_flags_to_modifier(rfb); - if (ret) + if (ret) { + drm_dbg_kms(dev, "Failed to convert tiling flags 0x%llX to a modifier", + rfb->tiling_flags); goto fail; + } } for (i = 1; i < rfb->base.format->num_planes; ++i) { |
