Skip to contentSkip to conversation

drm/radeon: dont switch vt on suspend

PATCH In review Maarten Lankhorst 2 participants 4 messages Jan 13, 2015

Subsystems (2)
DRM drivers Radeon and amdgpu DRM drivers
lore
Share & tools
lore

Maarten Lankhorst

author
#0 lore raw

PATCH 1/1 drm/radeon: dont switch vt on suspend

Signed-off-by Maarten Lankhorst <[email protected]>

---

drivers/gpu/drm/radeon/radeon_fb.c +1 -0
index 29b9220ec399..3d201e937571 100644
@@ -257,6 +257,7 @@ static int radeonfb_create(struct drm_fb_helper *helper,
 	}
 
 	info->par = rfbdev;
+	info->skip_vt_switch = true;
 
 	ret = radeon_framebuffer_init(rdev->ddev, &rfbdev->rfb, &mode_cmd, gobj);
 	if (ret) {

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

#1 lore raw
Quoted 1 line · Maarten Lankhorst
Signed-off-by Maarten Lankhorst <[email protected]>

Does this fix anything or is it purely cosmetic?

Alex

drivers/gpu/drm/radeon/radeon_fb.c static int radeonfb_create(struct drm_fb_helper *helper, · line 263

Quoted 13 lines

---

drivers/gpu/drm/radeon/radeon_fb.c +1 -0
index 29b9220ec399..3d201e937571 100644
@@ -257,6 +257,7 @@ static int radeonfb_create(struct drm_fb_helper *helper,
        }
 
        info->par = rfbdev;
+       info->skip_vt_switch = true;
 
        ret = radeon_framebuffer_init(rdev->ddev, &rfbdev->rfb, &mode_cmd, gobj);
        if (ret) {

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

author
#2 lore raw

Op 13-01-15 om 15:51 schreef Alex Deucher:

Quoted 4 lines
Quoted 1 line · Maarten Lankhorst
Signed-off-by Maarten Lankhorst <[email protected]>

Does this fix anything or is it purely cosmetic?

It gets rid of unnecessarily switching to the suspend console, resulting in no longer having a xorg vt switch on suspend.

So probably just cosmetic. In nouveau it exposed a bug with LUT restore and cursor save/restore. :-)

~Maarten
_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by
#3 lore raw
Quoted 8 lines · Maarten Lankhorst

Op 13-01-15 om 15:51 schreef Alex Deucher:

Quoted 4 lines
Quoted 1 line · Maarten Lankhorst
Signed-off-by Maarten Lankhorst <[email protected]>

Does this fix anything or is it purely cosmetic?

It gets rid of unnecessarily switching to the suspend console, resulting in no longer having a xorg vt switch on suspend.

So probably just cosmetic. In nouveau it exposed a bug with LUT restore and cursor save/restore. :-)

Assuming it works ok,

Acked-by Alex Deucher <[email protected]>

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel