Commit e87b2c42 authored by Jesse Barnes's avatar Jesse Barnes Committed by Dave Airlie
Browse files

drm: fix drm_fb_helper handling of kernel crtcs 



The drm_fb_helper shouldn't mess with CRTCs that aren't enabled or in
its initial config. Ideally it shouldn't even include CRTCs in its
initial config if they're not in use, but my old fix for that no longer
works.  At any rate, this fixes a real bug I was seeing where after a
console blank, both pipes would come back on, even though only one had
been enabled before that.  Since the other pipe had a bogus config,
this led to some screen corruption.

Signed-off-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent 8dfaa8a7

drivers/gpu/drm/drm_fb_helper.c

+43 −37
Original line number Diff line number Diff line
@@ -116,13 +116,15 @@ static void drm_fb_helper_on(struct fb_info *info) 
	 * For each CRTC in this fb, turn the crtc on then,

	 * find all associated encoders and turn them on.

	 */

	for (i = 0; i < fb_helper->crtc_count; i++) {

		list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {

		struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;

			struct drm_crtc_helper_funcs *crtc_funcs =

				crtc->helper_private;



		for (i = 0; i < fb_helper->crtc_count; i++) {

			if (crtc->base.id == fb_helper->crtc_info[i].crtc_id)

				break;

		}

			/* Only mess with CRTCs in this fb */

			if (crtc->base.id != fb_helper->crtc_info[i].crtc_id ||

			    !crtc->enabled)

				continue;



			mutex_lock(&dev->mode_config.mutex);

			crtc_funcs->dpms(crtc, DRM_MODE_DPMS_ON);
@@ -141,6 +143,7 @@ static void drm_fb_helper_on(struct fb_info *info) 
			}

		}

	}

}



static void drm_fb_helper_off(struct fb_info *info, int dpms_mode)

{
@@ -154,13 +157,15 @@ static void drm_fb_helper_off(struct fb_info *info, int dpms_mode) 
	 * For each CRTC in this fb, find all associated encoders

	 * and turn them off, then turn off the CRTC.

	 */

	for (i = 0; i < fb_helper->crtc_count; i++) {

		list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {

		struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;

			struct drm_crtc_helper_funcs *crtc_funcs =

				crtc->helper_private;



		for (i = 0; i < fb_helper->crtc_count; i++) {

			if (crtc->base.id == fb_helper->crtc_info[i].crtc_id)

				break;

		}

			/* Only mess with CRTCs in this fb */

			if (crtc->base.id != fb_helper->crtc_info[i].crtc_id ||

			    !crtc->enabled)

				continue;



			/* Found a CRTC on this fb, now find encoders */

			list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
@@ -180,6 +185,7 @@ static void drm_fb_helper_off(struct fb_info *info, int dpms_mode) 
			}

		}

	}

}



int drm_fb_helper_blank(int blank, struct fb_info *info)

{

CRA Git | Maintained and supported by SUSTech CRA and CCSE