Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-06 Thread Daniel Vetter
On Wed, May 06, 2020 at 10:06:15AM +0200, Thomas Zimmermann wrote: > Hi > > Am 04.05.20 um 17:31 schrieb Daniel Vetter: > > On Mon, May 04, 2020 at 02:07:22PM +0200, Thomas Zimmermann wrote: > >> Hi Emil > >> > >> Am 01.05.20 um 15:20 schrieb Emil Velikov: > >>> Hi Thomas, > >>> > >>> Couple of

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-06 Thread Thomas Zimmermann
Hi Am 04.05.20 um 17:31 schrieb Daniel Vetter: > On Mon, May 04, 2020 at 02:07:22PM +0200, Thomas Zimmermann wrote: >> Hi Emil >> >> Am 01.05.20 um 15:20 schrieb Emil Velikov: >>> Hi Thomas, >>> >>> Couple of fly-by ideas/suggestions. >>> >>> On Thu, 30 Apr 2020 at 10:13, Thomas Zimmermann

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-05 Thread Emil Velikov
On Mon, 4 May 2020 at 13:07, Thomas Zimmermann wrote: > > Hi Emil > > Am 01.05.20 um 15:20 schrieb Emil Velikov: > > Hi Thomas, > > > > Couple of fly-by ideas/suggestions. > > > > On Thu, 30 Apr 2020 at 10:13, Thomas Zimmermann wrote: > >> > >> Suspending failed because there's no mode if the

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-04 Thread Daniel Vetter
On Mon, May 04, 2020 at 02:07:22PM +0200, Thomas Zimmermann wrote: > Hi Emil > > Am 01.05.20 um 15:20 schrieb Emil Velikov: > > Hi Thomas, > > > > Couple of fly-by ideas/suggestions. > > > > On Thu, 30 Apr 2020 at 10:13, Thomas Zimmermann wrote: > >> > >> Suspending failed because there's no

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-04 Thread Thomas Zimmermann
Hi Emil Am 01.05.20 um 15:20 schrieb Emil Velikov: > Hi Thomas, > > Couple of fly-by ideas/suggestions. > > On Thu, 30 Apr 2020 at 10:13, Thomas Zimmermann wrote: >> >> Suspending failed because there's no mode if the CRTC is being >> disabled. Early-out in this case. This fixes runtime PM for

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-05-01 Thread Emil Velikov
Hi Thomas, Couple of fly-by ideas/suggestions. On Thu, 30 Apr 2020 at 10:13, Thomas Zimmermann wrote: > > Suspending failed because there's no mode if the CRTC is being > disabled. Early-out in this case. This fixes runtime PM for ast. > > Signed-off-by: Thomas Zimmermann > --- >

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-04-30 Thread Thomas Zimmermann
Hi Am 30.04.20 um 11:22 schrieb Sam Ravnborg: > Hi Thomas. > > On Thu, Apr 30, 2020 at 11:13:30AM +0200, Thomas Zimmermann wrote: >> Suspending failed because there's no mode if the CRTC is being >> disabled. Early-out in this case. This fixes runtime PM for ast. >> >> Signed-off-by: Thomas

Re: [PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-04-30 Thread Sam Ravnborg
Hi Thomas. On Thu, Apr 30, 2020 at 11:13:30AM +0200, Thomas Zimmermann wrote: > Suspending failed because there's no mode if the CRTC is being > disabled. Early-out in this case. This fixes runtime PM for ast. > > Signed-off-by: Thomas Zimmermann Don't you miss: Reported-by: Tested-by: Fixes:

[PATCH] drm/ast: Don't check new mode if CRTC is being disabled

2020-04-30 Thread Thomas Zimmermann
Suspending failed because there's no mode if the CRTC is being disabled. Early-out in this case. This fixes runtime PM for ast. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/ast/ast_mode.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/ast/ast_mode.c