[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-22 Thread Inki Dae
; kyungmin.park at samsung.com; linux-arm- > kernel at lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > > > > > > + DRM_ERROR("desired size is bigger then real > > size.\n"); > > > >

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Marek Szyprowski
Hello, On Wednesday, September 14, 2011 11:53 PM Rob Clark wrote: > On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom wrote: (snipped) > > Yes, that is true. A root client may be assumed to have AUTH permissions, > > but the inverse does not hold, meaning that an AUTH client may *not* be > > a

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Inki Dae
samsung.com; linux-arm- > ker...@lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > > > > > > + DRM_ERROR("desired size is bigger then real > > size.\n"); > > > >

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Konrad Rzeszutek Wilk
> > > > > + DRM_ERROR("desired size is bigger then real > size.\n"); > > > > > > > > So .. you can't continue by just using the real size instead? > > > > > > > > > > I am afraid I don't understand what you mean but I think that condition > > is > > > fine. size is a vm area to user-des

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Marek Szyprowski
Hello, On Wednesday, September 14, 2011 11:53 PM Rob Clark wrote: > On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom > wrote: (snipped) > > Yes, that is true. A root client may be assumed to have AUTH permissions, > > but the inverse does not hold, meaning that an AUTH client may *not* be >

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Konrad Rzeszutek Wilk
> > > > > + DRM_ERROR("desired size is bigger then real > size.\n"); > > > > > > > > So .. you can't continue by just using the real size instead? > > > > > > > > > > I am afraid I don't understand what you mean but I think that condition > > is > > > fine. size is a vm area to user-des

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Rob Clark
On Wed, Sep 21, 2011 at 7:41 AM, Marek Szyprowski wrote: >> I'm not entirely sure what will happen w/ dma_alloc_coherent, etc, if >> the global CMA pool is exhausted. >> >> Marek? ?I guess you know what would happen? > > The allocation will simply fail and dma_alloc_coherent will return NULL. > O

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-21 Thread Rob Clark
On Wed, Sep 21, 2011 at 7:41 AM, Marek Szyprowski wrote: >> I'm not entirely sure what will happen w/ dma_alloc_coherent, etc, if >> the global CMA pool is exhausted. >> >> Marek?  I guess you know what would happen? > > The allocation will simply fail and dma_alloc_coherent will return NULL. > O

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-19 Thread Inki Dae
; kyungmin.park at samsung.com; linux-arm- > kernel at lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On Thu, Sep 15, 2011 at 06:37:39PM +0900, Inki Dae wrote: > > Hello, Konrad Rzeszutek Wilk. > > > > Your review and

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Inki Dae
ied at linux.ie; dri-devel at lists.freedesktop.org; > sw0312.kim at samsung.com; kyungmin.park at samsung.com; linux-arm- > kernel at lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On Fri, Sep 09, 2011 at 08:38:53PM +0900, Inki Dae

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Inki Dae
sts.infradead.org; dri-devel at lists.freedesktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/14/2011 07:55 AM, Inki Dae wrote: > > > > > >> -Original Message- > >> From: Rob Clark [mailto:robdclark at gmail.co

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Thomas Hellstrom
in.park at samsung.com; sw0312.kim at samsung.com; >> linux- >> arm-kernel at lists.infradead.org; dri-devel at lists.freedesktop.org >> Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. >> >> On 09/14/2011 07:55 AM, Inki Dae wrote: >> >

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Konrad Rzeszutek Wilk
On Thu, Sep 15, 2011 at 06:37:39PM +0900, Inki Dae wrote: > Hello, Konrad Rzeszutek Wilk. > > Your review and comments are so very detail. it was very helpful. thank you > again. .. snip .. > > > + /* FIXME: error check */ > > > > You might as well do it now - before you do the next post

[Linaro-mm-sig] [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Kyungmin Park
On Thu, Sep 15, 2011 at 6:53 AM, Rob Clark wrote: > On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom > wrote: >> >> +static struct drm_ioctl_desc samsung_ioctls[] = { >> + ? ? ? DRM_IOCTL_DEF_DRV(SAMSUNG_GEM_CREATE, >> samsung_drm_gem_create_ioctl, >> + ?

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-15 Thread Thomas Hellstrom
x- arm-ker...@lists.infradead.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. On 09/14/2011 07:55 AM, Inki Dae wrote: -Original Message- From: Rob Clark [mailto:robdcl...@gmail.com] Sent: Wednesday, September 14, 2011 11:26 AM To: I

Re: [Linaro-mm-sig] [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Kyungmin Park
On Thu, Sep 15, 2011 at 6:53 AM, Rob Clark wrote: > On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom wrote: >> >> +static struct drm_ioctl_desc samsung_ioctls[] = { >> +       DRM_IOCTL_DEF_DRV(SAMSUNG_GEM_CREATE, >> samsung_drm_gem_create_ioctl, >> +    

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Inki Dae
sts.infradead.org; dri-devel@lists.freedesktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/14/2011 07:55 AM, Inki Dae wrote: > > > > > >> -Original Message- > >> From: Rob Clark [mailto:robdcl...@gmail.com] > >> Se

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Konrad Rzeszutek Wilk
On Fri, Sep 09, 2011 at 08:38:53PM +0900, Inki Dae wrote: > This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables > only FIMD yet but we will add HDMI support also in the future. > > from now on, I will remove RFC prefix because I think we have got comments > enough. > > this patc

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Rob Clark
On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom wrote: > > +static struct drm_ioctl_desc samsung_ioctls[] = { > + ? ? ? DRM_IOCTL_DEF_DRV(SAMSUNG_GEM_CREATE, > samsung_drm_gem_create_ioctl, > + ? ? ? ? ? ? ? ? ? ? ? DRM_UNLOCKED | DRM_AUTH), > >>>

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Inki Dae
.org; dri-devel at lists.freedesktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On Tue, Sep 13, 2011 at 9:03 PM, Inki Dae wrote: > > Hi Thomas. > > > >> -Original Message- > >> From: Thomas Hellstrom [mailto:thomas at shipmail.o

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Rob Clark
On Wed, Sep 14, 2011 at 2:57 AM, Thomas Hellstrom wrote: > > +static struct drm_ioctl_desc samsung_ioctls[] = { > +       DRM_IOCTL_DEF_DRV(SAMSUNG_GEM_CREATE, > samsung_drm_gem_create_ioctl, > +                       DRM_UNLOCKED | DRM_AUTH), >

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Inki Dae
.org; dri-devel at lists.freedesktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: > > On 09/10/2011 07:31 PM, Rob Clark wrote: > >> On Sat, Sep 10, 2011 at 9:04 AM, Thomas > >> Hellstrom

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Inki Dae
; kyungmin.park at samsung.com; linux-arm- > kernel at lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/09/2011 01:38 PM, Inki Dae wrote: > > This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables > > only FIMD ye

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Thomas Hellstrom
>> samsung.com; >> linux-arm-kernel at lists.infradead.org; dri-devel at lists.freedesktop.org >> Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. >> >> On Tue, Sep 13, 2011 at 9:03 PM, Inki Dae wrote: >> >>> Hi Th

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-14 Thread Thomas Hellstrom
@lists.freedesktop.org Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. On Tue, Sep 13, 2011 at 9:03 PM, Inki Dae wrote: Hi Thomas. -Original Message- From: Thomas Hellstrom [mailto:tho...@shipmail.org] Sent: Monday, September 12, 2011 3:32 PM To

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-13 Thread Inki Dae
ktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On Tue, Sep 13, 2011 at 9:03 PM, Inki Dae wrote: > > Hi Thomas. > > > >> -Original Message- > >> From: Thomas Hellstrom [mailto:tho...@shipmail.org] > >> S

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-13 Thread Rob Clark
om; sw0312.kim at samsung.com; linux- >> arm-kernel at lists.infradead.org; dri-devel at lists.freedesktop.org >> Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. >> >> On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: >> > On 09/10/2011 07:

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-13 Thread Rob Clark
2@samsung.com; linux- >> arm-ker...@lists.infradead.org; dri-devel@lists.freedesktop.org >> Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. >> >> On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: >> > On 09/10/2011 07:31 PM, Rob Clark wrote:

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-13 Thread Inki Dae
ts.freedesktop.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: > > On 09/10/2011 07:31 PM, Rob Clark wrote: > >> On Sat, Sep 10, 2011 at 9:04 AM, Thomas > >> Hellstrom wrote: > >>

RE: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-13 Thread Inki Dae
ung.com; linux-arm- > ker...@lists.infradead.org > Subject: Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210. > > On 09/09/2011 01:38 PM, Inki Dae wrote: > > This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables > > only FIMD yet but w

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-12 Thread Thomas Hellstrom
On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: > On 09/10/2011 07:31 PM, Rob Clark wrote: >> On Sat, Sep 10, 2011 at 9:04 AM, Thomas >> Hellstrom wrote: >>> On 09/09/2011 01:38 PM, Inki Dae wrote: This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables only FIMD yet but w

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-11 Thread Thomas Hellstrom
On 09/11/2011 11:26 PM, Thomas Hellstrom wrote: On 09/10/2011 07:31 PM, Rob Clark wrote: On Sat, Sep 10, 2011 at 9:04 AM, Thomas Hellstrom wrote: On 09/09/2011 01:38 PM, Inki Dae wrote: This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables only FIMD yet but we will add HDMI su

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-11 Thread Thomas Hellstrom
On 09/10/2011 07:31 PM, Rob Clark wrote: > On Sat, Sep 10, 2011 at 9:04 AM, Thomas Hellstrom > wrote: > >> On 09/09/2011 01:38 PM, Inki Dae wrote: >> >>> This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables >>> only FIMD yet but we will add HDMI support also in the futu

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-11 Thread Thomas Hellstrom
On 09/10/2011 07:31 PM, Rob Clark wrote: On Sat, Sep 10, 2011 at 9:04 AM, Thomas Hellstrom wrote: On 09/09/2011 01:38 PM, Inki Dae wrote: This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables only FIMD yet but we will add HDMI support also in the future. from now on,

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-10 Thread Thomas Hellstrom
On 09/09/2011 01:38 PM, Inki Dae wrote: > This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables > only FIMD yet but we will add HDMI support also in the future. > > from now on, I will remove RFC prefix because I think we have got comments > enough. > > this patch is based on git re

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-10 Thread Rob Clark
On Sat, Sep 10, 2011 at 9:04 AM, Thomas Hellstrom wrote: > On 09/09/2011 01:38 PM, Inki Dae wrote: >> >> This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables >> only FIMD yet but we will add HDMI support also in the future. >> >> from now on, I will remove RFC prefix because I th

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-10 Thread Rob Clark
On Sat, Sep 10, 2011 at 9:04 AM, Thomas Hellstrom wrote: > On 09/09/2011 01:38 PM, Inki Dae wrote: >> >> This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables >> only FIMD yet but we will add HDMI support also in the future. >> >> from now on, I will remove RFC prefix because I thi

Re: [PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-10 Thread Thomas Hellstrom
On 09/09/2011 01:38 PM, Inki Dae wrote: This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables only FIMD yet but we will add HDMI support also in the future. from now on, I will remove RFC prefix because I think we have got comments enough. this patch is based on git repository b

[PATCH v4] DRM: add DRM Driver for Samsung SoC EXYNOS4210.

2011-09-09 Thread Inki Dae
This patch is a DRM Driver for Samsung SoC Exynos4210 and now enables only FIMD yet but we will add HDMI support also in the future. from now on, I will remove RFC prefix because I think we have got comments enough. this patch is based on git repository below: git://git.kernel.org/pub/scm/linux/k