backend-drm: do not print head info twice

Before this patch, when a new head is found its information is printed
first as "updated" and then as "found" in the log.

The reason is that drm_head_create() calls drm_head_update_info() which
printed the head as "changed". Then drm_head_create() itself prints it
as "found".

This fixes it to print only once as "found".

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.com>
This commit is contained in:
Pekka Paalanen 2022-10-14 14:54:13 +03:00
parent 771bd8acff
commit f3bed68100

View File

@ -2215,9 +2215,6 @@ drm_head_update_info(struct drm_head *head, drmModeConnector *conn)
weston_head_set_content_protection_status(&head->base,
drm_head_get_current_protection(head));
if (head->base.device_changed)
drm_head_log_info(head, "updated");
return ret;
}
@ -2534,12 +2531,15 @@ drm_backend_update_connectors(struct drm_device *device,
* one of the searches must fail. */
assert(head == NULL || writeback == NULL);
if (head)
if (head) {
ret = drm_head_update_info(head, conn);
else if (writeback)
if (head->base.device_changed)
drm_head_log_info(head, "updated");
} else if (writeback) {
ret = drm_writeback_update_info(writeback, conn);
else
} else {
ret = drm_backend_add_connector(b->drm, conn, drm_device);
}
if (ret < 0)
drmModeFreeConnector(conn);