[PATCH 3/7] drm/radeon: implement unpin function, v2

2013-05-01 Thread Maarten Lankhorst
Changes since v1: - Fixup compiler warning in unpin function. Signed-off-by: Maarten Lankhorst Reviewed-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ drivers/gpu/drm/radeon/radeon_prime.c | 18 +- 2 files changed, 15 insertions(+), 5 deletions(-) diff --g

[PATCH 3/7] drm/radeon: implement unpin function, v2

2013-05-01 Thread Maarten Lankhorst
Changes since v1: - Fixup compiler warning in unpin function. Signed-off-by: Maarten Lankhorst Reviewed-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ drivers/gpu/drm/radeon/radeon_prime.c | 18 +- 2 files changed, 15 insertions(+), 5 deletions(-) diff --g

[PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Daniel Vetter
On Tue, Apr 9, 2013 at 4:26 PM, Maarten Lankhorst wrote: > Op 09-04-13 16:16, Jerome Glisse schreef: >> On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst >> wrote: >> >>> Signed-off-by: Maarten Lankhorst >>> >> Can userspace pin directly ? If so then that sounds as a bad idea. >> >> Reviewed-by:

[PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Maarten Lankhorst
Op 09-04-13 16:16, Jerome Glisse schreef: > On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst > wrote: > >> Signed-off-by: Maarten Lankhorst >> > Can userspace pin directly ? If so then that sounds as a bad idea. > > Reviewed-by: Jerome Glisse > It's slightly better than before, it used to pin as

[PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ drivers/gpu/drm/radeon/radeon_prime.c | 18 ++ 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c index 66a7f

[PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Jerome Glisse
On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst wrote: > Signed-off-by: Maarten Lankhorst > Can userspace pin directly ? If so then that sounds as a bad idea. Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ > drivers/gpu/drm/radeon/radeon_prime.c | 18 +++

Re: [PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Daniel Vetter
On Tue, Apr 9, 2013 at 4:26 PM, Maarten Lankhorst wrote: > Op 09-04-13 16:16, Jerome Glisse schreef: >> On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst >> wrote: >> >>> Signed-off-by: Maarten Lankhorst >>> >> Can userspace pin directly ? If so then that sounds as a bad idea. >> >> Reviewed-by:

Re: [PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Maarten Lankhorst
Op 09-04-13 16:16, Jerome Glisse schreef: > On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst > wrote: > >> Signed-off-by: Maarten Lankhorst >> > Can userspace pin directly ? If so then that sounds as a bad idea. > > Reviewed-by: Jerome Glisse > It's slightly better than before, it used to pin as

Re: [PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Jerome Glisse
On Tue, Apr 9, 2013 at 7:37 AM, Maarten Lankhorst wrote: > Signed-off-by: Maarten Lankhorst > Can userspace pin directly ? If so then that sounds as a bad idea. Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ > drivers/gpu/drm/radeon/radeon_prime.c | 18 +++

[PATCH 3/7] drm/radeon: implement unpin function

2013-04-09 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ drivers/gpu/drm/radeon/radeon_prime.c | 18 ++ 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c index 66a7f