drm/nouveau: Don't set allow_fb_modifiers explicitly
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 27 Apr 2021 09:20:16 +0000 (11:20 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 6 May 2021 09:43:58 +0000 (11:43 +0200)
Since

commit 890880ddfdbe256083170866e49c87618b706ac7
Author: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Date:   Fri Jan 4 09:56:10 2019 +0100

    drm: Auto-set allow_fb_modifiers when given modifiers at plane init

this is done automatically as part of plane init, if drivers set the
modifier list correctly. Which is the case here.

Note that this fixes an inconsistency: We've set the cap everywhere,
but only nv50+ supports modifiers. Hence cc stable, but not further
back then the patch from Paul.

Reviewed-by: Lyude Paul <lyude@redhat.com>
Cc: stable@vger.kernel.org # v5.1 +
Cc: Pekka Paalanen <pekka.paalanen@collabora.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Cc: Ben Skeggs <bskeggs@redhat.com>
Cc: nouveau@lists.freedesktop.org
Link: https://patchwork.freedesktop.org/patch/msgid/20210427092018.832258-6-daniel.vetter@ffwll.ch
drivers/gpu/drm/nouveau/nouveau_display.c

index 14101bd..929de41 100644 (file)
@@ -697,7 +697,6 @@ nouveau_display_create(struct drm_device *dev)
 
        dev->mode_config.preferred_depth = 24;
        dev->mode_config.prefer_shadow = 1;
-       dev->mode_config.allow_fb_modifiers = true;
 
        if (drm->client.device.info.chipset < 0x11)
                dev->mode_config.async_page_flip = false;