Re: [PATCH] drm/fbdev: Make skip_vt_switch the default

From: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>,
	Intel Graphics Development <intel-gfx@lists.freedesktop.org>
Cc: "Heiko Stübner" <heiko@sntech.de>,
	"Maxime Ripard" <maxime.ripard@bootlin.com>,
	nouveau@lists.freedesktop.org,
	"DRI Development" <dri-devel@lists.freedesktop.org>,
	"Huang Rui" <ray.huang@amd.com>,
	"Daniel Vetter" <daniel.vetter@intel.com>,
	linux-rockchip@lists.infradead.org,
	"David Airlie" <airlied@linux.ie>,
	"Ben Skeggs" <bskeggs@redhat.com>,
	"Daniel Stone" <daniels@collabora.com>,
	"Michel Dänzer" <michel@daenzer.net>,
	"Rodrigo Vivi" <rodrigo.vivi@intel.com>,
	"Junwei Zhang" <Jerry.Zhang@amd.com>,
	linux-arm-kernel@lists.infradead.org,
	"Michel Dänzer" <michel.daenzer@amd.com>,
	"Sandy Huang" <hjc@rock-chips.com>,
	"Samuel Li" <Samuel.Li@amd.com>, "Shirish S" <shirish.s@amd.com>,
	"Noralf Trønnes" <noralf@tronnes.org>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH] drm/fbdev: Make skip_vt_switch the default
Date: Wed, 28 Nov 2018 08:17:04 +0100	[thread overview]
Message-ID: <3d406677-9d7d-6d8d-85af-1cfb253d1838@linux.intel.com> (raw)
In-Reply-To: <20181127173424.301-1-daniel.vetter@ffwll.ch>

Op 27-11-18 om 18:34 schreef Daniel Vetter:
> KMS drivers really should all be able to restore their display state
> on resume without fbcon helping out. So make this the default.
>
> Since I'm not entirely foolish, make it only a default, which drivers
> can still override. That way when the inevitable regression report
> happens I can fix things up with a one-liner plus FIXME comment that
> someone should fix up the suspend/resume code in that driver.
>
> But at least all new drivers won't be broken by accident as soon as
> you turn off fbcon because "suspend/resume worked when I tested it".
>
> v2: Keep this for radeon because of
>
> commit 18c437caa5b18a235dd65cec224eab54bebcee65
> Author: Alex Deucher <alexander.deucher@amd.com>
> Date:   Tue Nov 14 17:19:29 2017 -0500
>
>     Revert "drm/radeon: dont switch vt on suspend"
>
> Thanks to Michel Dänzer for pointing this one out.

Maybe just reload the gamma lut on resume for radeon, instead of relying on fbcon?

Otherwise patch looks sane, would be nice if radeon was fixed instead of worked around.

Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx