[PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Sascha Hauer
On Wed, Jun 27, 2012 at 03:20:27PM +0200, Laurent Pinchart wrote: > Hi Sascha, > > Thanks for the patch. > > On Wednesday 27 June 2012 15:00:05 Sascha Hauer wrote: > > Many embedded drm devices do not have a IOMMU and no dedicated > > memory for graphics. These devices use CMA (Contiguous Memory

[PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Laurent Pinchart
Hi Sascha, Thanks for the patch. On Wednesday 27 June 2012 15:00:05 Sascha Hauer wrote: > Many embedded drm devices do not have a IOMMU and no dedicated > memory for graphics. These devices use CMA (Contiguous Memory > Allocator) backed graphics memory. This patch provides helper > functions to b

[PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Sascha Hauer
Many embedded drm devices do not have a IOMMU and no dedicated memory for graphics. These devices use CMA (Contiguous Memory Allocator) backed graphics memory. This patch provides helper functions to be able to share the code. The code technically does not depend on CMA as the backend allocator, th

Re: [PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Sascha Hauer
On Wed, Jun 27, 2012 at 03:20:27PM +0200, Laurent Pinchart wrote: > Hi Sascha, > > Thanks for the patch. > > On Wednesday 27 June 2012 15:00:05 Sascha Hauer wrote: > > Many embedded drm devices do not have a IOMMU and no dedicated > > memory for graphics. These devices use CMA (Contiguous Memory

Re: [PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Laurent Pinchart
Hi Sascha, Thanks for the patch. On Wednesday 27 June 2012 15:00:05 Sascha Hauer wrote: > Many embedded drm devices do not have a IOMMU and no dedicated > memory for graphics. These devices use CMA (Contiguous Memory > Allocator) backed graphics memory. This patch provides helper > functions to b

[PATCH] DRM: add drm gem CMA helper

2012-06-27 Thread Sascha Hauer
Many embedded drm devices do not have a IOMMU and no dedicated memory for graphics. These devices use CMA (Contiguous Memory Allocator) backed graphics memory. This patch provides helper functions to be able to share the code. The code technically does not depend on CMA as the backend allocator, th

[PATCH] DRM: add drm gem CMA helper

2012-05-31 Thread Laurent Pinchart
Hi Sascha, On Wednesday 30 May 2012 18:28:12 Sascha Hauer wrote: > On Wed, May 30, 2012 at 05:40:13PM +0200, Laurent Pinchart wrote: > > Hi Sascha, > > > > Thank you for the patch. I've successfully tested the helper with the new > > SH Mobile DRM driver. Just a couple of comments below in additi

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-31 Thread Laurent Pinchart
Hi Sascha, On Wednesday 30 May 2012 18:28:12 Sascha Hauer wrote: > On Wed, May 30, 2012 at 05:40:13PM +0200, Laurent Pinchart wrote: > > Hi Sascha, > > > > Thank you for the patch. I've successfully tested the helper with the new > > SH Mobile DRM driver. Just a couple of comments below in additi

[PATCH] DRM: add drm gem CMA helper

2012-05-30 Thread Sascha Hauer
On Wed, May 30, 2012 at 05:40:13PM +0200, Laurent Pinchart wrote: > Hi Sascha, > > Thank you for the patch. I've successfully tested the helper with the new SH > Mobile DRM driver. Just a couple of comments below in addition to Lars' > comments (this is not a full review, just details that caugh

[PATCH] DRM: add drm gem CMA helper

2012-05-30 Thread Laurent Pinchart
Hi Sascha, Thank you for the patch. I've successfully tested the helper with the new SH Mobile DRM driver. Just a couple of comments below in addition to Lars' comments (this is not a full review, just details that caught my attention when comparing the code with my implementation, and trying t

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-30 Thread Sascha Hauer
On Wed, May 30, 2012 at 05:40:13PM +0200, Laurent Pinchart wrote: > Hi Sascha, > > Thank you for the patch. I've successfully tested the helper with the new SH > Mobile DRM driver. Just a couple of comments below in addition to Lars' > comments (this is not a full review, just details that caugh

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-30 Thread Laurent Pinchart
Hi Sascha, Thank you for the patch. I've successfully tested the helper with the new SH Mobile DRM driver. Just a couple of comments below in addition to Lars' comments (this is not a full review, just details that caught my attention when comparing the code with my implementation, and trying t

[PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Sascha Hauer
Hi Lars, Thanks for your quick comments. On Tue, May 29, 2012 at 04:46:36PM +0200, Lars-Peter Clausen wrote: > On 05/29/2012 04:10 PM, Sascha Hauer wrote: > > > > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > > index e354bc0..f62717e 100644 > > --- a/drivers/gpu/drm/Kconfig >

[PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Lars-Peter Clausen
On 05/29/2012 04:46 PM, Lars-Peter Clausen wrote: > On 05/29/2012 04:10 PM, Sascha Hauer wrote: >> Many embedded drm devices do not have a IOMMU and no dedicated >> memory for graphics. These devices use CMA (Contiguous Memory >> Allocator) backed graphics memory. This patch provides helper >> func

[PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Lars-Peter Clausen
On 05/29/2012 04:10 PM, Sascha Hauer wrote: > Many embedded drm devices do not have a IOMMU and no dedicated > memory for graphics. These devices use CMA (Contiguous Memory > Allocator) backed graphics memory. This patch provides helper > functions to be able to share the code. > > Signed-off-by:

[PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Sascha Hauer
Many embedded drm devices do not have a IOMMU and no dedicated memory for graphics. These devices use CMA (Contiguous Memory Allocator) backed graphics memory. This patch provides helper functions to be able to share the code. Signed-off-by: Sascha Hauer --- Lars-Peter, please let me know if thi

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Sascha Hauer
Hi Lars, Thanks for your quick comments. On Tue, May 29, 2012 at 04:46:36PM +0200, Lars-Peter Clausen wrote: > On 05/29/2012 04:10 PM, Sascha Hauer wrote: > > > > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > > index e354bc0..f62717e 100644 > > --- a/drivers/gpu/drm/Kconfig >

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Lars-Peter Clausen
On 05/29/2012 04:46 PM, Lars-Peter Clausen wrote: > On 05/29/2012 04:10 PM, Sascha Hauer wrote: >> Many embedded drm devices do not have a IOMMU and no dedicated >> memory for graphics. These devices use CMA (Contiguous Memory >> Allocator) backed graphics memory. This patch provides helper >> func

Re: [PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Lars-Peter Clausen
On 05/29/2012 04:10 PM, Sascha Hauer wrote: > Many embedded drm devices do not have a IOMMU and no dedicated > memory for graphics. These devices use CMA (Contiguous Memory > Allocator) backed graphics memory. This patch provides helper > functions to be able to share the code. > > Signed-off-by:

[PATCH] DRM: add drm gem CMA helper

2012-05-29 Thread Sascha Hauer
Many embedded drm devices do not have a IOMMU and no dedicated memory for graphics. These devices use CMA (Contiguous Memory Allocator) backed graphics memory. This patch provides helper functions to be able to share the code. Signed-off-by: Sascha Hauer --- Lars-Peter, please let me know if thi