author | Hans de Goede <hdegoede@redhat.com> | |
Sat, 7 Sep 2019 20:32:38 +0000 (22:32 +0200) | ||
committer | Alex Deucher <alexander.deucher@amd.com> | |
Tue, 17 Sep 2019 19:37:27 +0000 (14:37 -0500) | ||
commit | 9dbc88d013b79c62bd845cb9e7c0256e660967c5 | |
tree | 35aae5cc7dc725b8b863531a59374c284c43854a | tree | snapshot (tar.xz tar.gz zip) |
parent | c02d6a161395dfc0c2fdabb9e976a229017288d8 | commit | diff |
drm/radeon: Bail earlier when radeon.cik_/si_support=0 is passed
Bail from the pci_driver probe function instead of from the drm_driver
load function.
This avoid /dev/dri/card0 temporarily getting registered and then
unregistered again, sending unwanted add / remove udev events to
userspace.
Specifically this avoids triggering the (userspace) bug fixed by this
plymouth merge-request:
https://gitlab.freedesktop.org/plymouth/plymouth/merge_requests/59
Note that despite that being an userspace bug, not sending unnecessary
udev events is a good idea in general.
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1490490
Reviewed-by: Michel Dänzer <mdaenzer@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Bail from the pci_driver probe function instead of from the drm_driver
load function.
This avoid /dev/dri/card0 temporarily getting registered and then
unregistered again, sending unwanted add / remove udev events to
userspace.
Specifically this avoids triggering the (userspace) bug fixed by this
plymouth merge-request:
https://gitlab.freedesktop.org/plymouth/plymouth/merge_requests/59
Note that despite that being an userspace bug, not sending unnecessary
udev events is a good idea in general.
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1490490
Reviewed-by: Michel Dänzer <mdaenzer@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/radeon_drv.c | diff | blob | history | |
drivers/gpu/drm/radeon/radeon_kms.c | diff | blob | history |