Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-27 Thread Daniel Vetter
On Fri, Nov 27, 2020 at 03:49:31PM +0100, Christian König wrote: > Am 27.11.20 um 15:46 schrieb Daniel Vetter: > > On Fri, Nov 27, 2020 at 3:10 PM Christian König > > wrote: > > > Am 27.11.20 um 09:31 schrieb Dave Airlie: > > > > Oops sorry for delay LGTM > > > > > > > > Reviewed-by: Dave Airlie

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-27 Thread Christian König
Am 27.11.20 um 15:46 schrieb Daniel Vetter: On Fri, Nov 27, 2020 at 3:10 PM Christian König wrote: Am 27.11.20 um 09:31 schrieb Dave Airlie: Oops sorry for delay LGTM Reviewed-by: Dave Airlie Thanks. On Fri, 27 Nov 2020 at 02:34, Daniel Vetter wrote: On Wed, Nov 25, 2020 at 3:34 PM Chri

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-27 Thread Daniel Vetter
On Fri, Nov 27, 2020 at 3:10 PM Christian König wrote: > > Am 27.11.20 um 09:31 schrieb Dave Airlie: > > Oops sorry for delay LGTM > > > > Reviewed-by: Dave Airlie > > Thanks. > > > > > On Fri, 27 Nov 2020 at 02:34, Daniel Vetter wrote: > >> On Wed, Nov 25, 2020 at 3:34 PM Christian König > >>

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-27 Thread Christian König
Am 27.11.20 um 09:31 schrieb Dave Airlie: Oops sorry for delay LGTM Reviewed-by: Dave Airlie Thanks. On Fri, 27 Nov 2020 at 02:34, Daniel Vetter wrote: On Wed, Nov 25, 2020 at 3:34 PM Christian König wrote: Reorder the code to fix checking if blitting is available. Might be good to ex

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-27 Thread Dave Airlie
Oops sorry for delay LGTM Reviewed-by: Dave Airlie On Fri, 27 Nov 2020 at 02:34, Daniel Vetter wrote: > > On Wed, Nov 25, 2020 at 3:34 PM Christian König > wrote: > > > > Reorder the code to fix checking if blitting is available. > > Might be good to explain why blitting might not be available

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-26 Thread Daniel Vetter
On Wed, Nov 25, 2020 at 3:34 PM Christian König wrote: > > Reorder the code to fix checking if blitting is available. Might be good to explain why blitting might not be available, e.g. suspend/resume and or chip death and stuff like that. > Signed-off-by: Christian König Needs Fixes: 28a68f828

Re: [PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-26 Thread Christian König
Ping, Dave this is another fix for the Multihop patch set. Without it radeon is completely broken on drm-misc-next. Thanks, Christian. Am 25.11.20 um 15:34 schrieb Christian König: Reorder the code to fix checking if blitting is available. Signed-off-by: Christian König --- drivers/gpu/drm

[PATCH] drm/radeon: fix check order in radeon_bo_move

2020-11-25 Thread Christian König
Reorder the code to fix checking if blitting is available. Signed-off-by: Christian König --- drivers/gpu/drm/radeon/radeon_ttm.c | 54 + 1 file changed, 24 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/rade