On 2021-03-04 12:41 p.m., Alex Deucher wrote:
It just spams the logs.
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
This series in general looks reasonable to me:
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>
---
drivers/gpu/drm/amd/display/dc/core/dc_link.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
index fa9a62dc174b..974b70f21837 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
@@ -2614,7 +2614,6 @@ bool dc_link_set_backlight_level(const struct dc_link
*link,
if (pipe_ctx->plane_state == NULL)
frame_ramp = 0;
} else {
- ASSERT(false);
Just a comment on what's actually going on here with this warning:
Technically we can't apply the backlight level without a plane_state in
the context but the panel is also off anyway.
I think there might be a bug here when the panel turns on and we're not
applying values set when it was off but I don't think anyone's reported
this as an issue.
I'm not entirely sure if the value gets cached and reapplied with the
correct value later, but it's something to keep in mind.
Regards,
Nicholas Kazlauskas
return false;
}
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
