]> git.ipfire.org Git - thirdparty/plymouth.git/commitdiff
ply-device-manager: Update verify_drm_device () comment
authorHans de Goede <hdegoede@redhat.com>
Thu, 6 Mar 2025 16:25:12 +0000 (17:25 +0100)
committerHans de Goede <hdegoede@redhat.com>
Mon, 10 Mar 2025 10:47:04 +0000 (11:47 +0100)
The comment about why SimpleDRM devices should be skipped is no longer
accurate, the kernel does provide rotation info now; and plymouth now
has heuristics to guess the device-scale.

If SimpleDRM devices should be used or not now mostly is a user
preference.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
src/libply-splash-core/ply-device-manager.c

index fd54d3a9dbd7dde8bc6578b67b308c1e50e363f0..7406d5f5c523b071844515d0badf53ca7984a42a 100644 (file)
@@ -345,18 +345,11 @@ syspath_is_simpledrm (const char *syspath)
         return ply_string_has_suffix (syspath, "simple-framebuffer.0/drm/card0");
 }
 
+/* Only use SimpleDRM devices if requested to do so */
 static bool
 verify_drm_device (ply_device_manager_t *manager,
                    struct udev_device   *device)
 {
-        /*
-         * Simple-framebuffer devices driven by simpledrm lack information
-         * like panel-rotation info and physical size, causing the splash
-         * to briefly render on its side / without HiDPI scaling, switching
-         * to the correct rendering when the native driver loads.
-         * To avoid this treat simpledrm devices as fbdev devices and only
-         * use them after the timeout.
-         */
         if (!syspath_is_simpledrm (udev_device_get_syspath (device)))
                 return true; /* Not a SimpleDRM device */