From: Ben Skeggs <bskeggs@redhat.com>
- output info will be used later to determine MST support
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Acked-by: Danilo Krummrich <me@dakr.org>
Signed-off-by: Lyude Paul <lyude@redhat.com>
---
drivers/gpu/drm/nouveau/dispnv50/disp.c | 64 ++++++++++++-------------
1 file changed, 32 insertions(+), 32 deletions(-)
On Tue, 2023-09-19 at 17:56 -0400, Lyude Paul wrote:
> + /*
> + * FIXME: This is a hack to workaround the
> following
> + * issues:
> + *
> + *
> https://gitlab.gnome.org/GNOME/mutter/issues/759
> + *
> https://gitlab.freedesktop.org/xorg/xserver/merge_requests/277
> + *
> + * Once these issues are closed, this should be
> + * removed
> + */
> + head->msto->encoder.possible_crtcs = disp->disp-
> >head_mask;
This bug is 4 years old, and it looks like a fix was merged in 3 years ago.
Shouldn't the bug be closed by now?
@@ -2774,38 +2774,6 @@ nv50_display_create(struct drm_device *dev)
dev->mode_config.cursor_height = 64;
}
- /* create crtc objects to represent the hw heads */
- for_each_set_bit(i, &disp->disp->head_mask, sizeof(disp->disp->head_mask) * 8) {
- struct nv50_head *head;
-
- head = nv50_head_create(dev, i);
- if (IS_ERR(head)) {
- ret = PTR_ERR(head);
- goto out;
- }
-
- if (has_mst) {
- head->msto = nv50_msto_new(dev, head, i);
- if (IS_ERR(head->msto)) {
- ret = PTR_ERR(head->msto);
- head->msto = NULL;
- goto out;
- }
-
- /*
- * FIXME: This is a hack to workaround the following
- * issues:
- *
- * https://gitlab.gnome.org/GNOME/mutter/issues/759
- * https://gitlab.freedesktop.org/xorg/xserver/merge_requests/277
- *
- * Once these issues are closed, this should be
- * removed
- */
- head->msto->encoder.possible_crtcs = disp->disp->head_mask;
- }
- }
-
/* create encoder/connector objects based on VBIOS DCB table */
for (i = 0, dcbe = &dcb->entry[0]; i < dcb->entries; i++, dcbe++) {
struct nouveau_encoder *outp;
@@ -2868,6 +2836,38 @@ nv50_display_create(struct drm_device *dev)
connector->funcs->destroy(connector);
}
+ /* create crtc objects to represent the hw heads */
+ for_each_set_bit(i, &disp->disp->head_mask, sizeof(disp->disp->head_mask) * 8) {
+ struct nv50_head *head;
+
+ head = nv50_head_create(dev, i);
+ if (IS_ERR(head)) {
+ ret = PTR_ERR(head);
+ goto out;
+ }
+
+ if (has_mst) {
+ head->msto = nv50_msto_new(dev, head, i);
+ if (IS_ERR(head->msto)) {
+ ret = PTR_ERR(head->msto);
+ head->msto = NULL;
+ goto out;
+ }
+
+ /*
+ * FIXME: This is a hack to workaround the following
+ * issues:
+ *
+ * https://gitlab.gnome.org/GNOME/mutter/issues/759
+ * https://gitlab.freedesktop.org/xorg/xserver/merge_requests/277
+ *
+ * Once these issues are closed, this should be
+ * removed
+ */
+ head->msto->encoder.possible_crtcs = disp->disp->head_mask;
+ }
+ }
+
/* Disable vblank irqs aggressively for power-saving, safe on nv50+ */
dev->vblank_disable_immediate = true;