Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
On 27 May 2014 11:52, Stephen Rothwell wrote: > Hi Dave, > > On Tue, 27 May 2014 15:52:46 +1000 Dave Airlie wrote: >> >> Then we should cc him. > > He was cc'd on my email that you replied to ... > > But I guess the extra prod won't hurt :-) :) thanks for the prod Stephen! All, I'll merge it in today's for-next for me. > -- > Cheers, > Stephen Rothwells...@canb.auug.org.au -- Thanks and regards, Sumit Semwal Graphics Engineer - Graphics working group Linaro.org │ Open source software for ARM SoCs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
Hi Dave, On Tue, 27 May 2014 15:52:46 +1000 Dave Airlie wrote: > > Then we should cc him. He was cc'd on my email that you replied to ... But I guess the extra prod won't hurt :-) -- Cheers, Stephen Rothwells...@canb.auug.org.au signature.asc Description: PGP signature
Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
On 26 May 2014 09:06, Stephen Rothwell wrote: > Hi all, > > On Sun, 25 May 2014 23:54:35 +0100 Russell King - ARM Linux > wrote: >> >> On Sun, May 25, 2014 at 02:08:48PM +0200, David Herrmann wrote: >> > >> > On Sat, May 24, 2014 at 11:05 PM, Vincent Stehlé >> > wrote: >> > > The dma_buf_export function was updated in commit 4bcec44ffaf9 'dma-buf: >> > > use >> > > reservation objects' to take a reservation object parameter; update >> > > Armada >> > > export method accordingly. >> > > >> > > This fixes the following compilation error: >> > > >> > > drivers/gpu/drm/armada/armada_gem.c: In function >> > > ‘armada_gem_prime_export’: >> > > drivers/gpu/drm/armada/armada_gem.c:544:16: error: macro >> > > "dma_buf_export" requires 5 arguments, but only 4 given >> > > >> > > Signed-off-by: Vincent Stehlé >> > > Cc: Russell King >> > > Cc: David Airlie >> > > Cc: Maarten Lankhorst >> > > Cc: Sumit Semwal >> > >> > Reviewed-by: David Herrmann >> >> Acked-by: Russell King >> >> Airlied, can you merge this please? Thanks. > > Except that commit 4bcec44ffaf9 'dma-buf: use reservation objects' > comes from the dma-buf tree, so Sumit needs to add this commit to his > tree. > Then we should cc him. Dave. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
Hi all,
On Sun, 25 May 2014 23:54:35 +0100 Russell King - ARM Linux
wrote:
>
> On Sun, May 25, 2014 at 02:08:48PM +0200, David Herrmann wrote:
> >
> > On Sat, May 24, 2014 at 11:05 PM, Vincent Stehlé
> > wrote:
> > > The dma_buf_export function was updated in commit 4bcec44ffaf9 'dma-buf:
> > > use
> > > reservation objects' to take a reservation object parameter; update Armada
> > > export method accordingly.
> > >
> > > This fixes the following compilation error:
> > >
> > > drivers/gpu/drm/armada/armada_gem.c: In function
> > > ‘armada_gem_prime_export’:
> > > drivers/gpu/drm/armada/armada_gem.c:544:16: error: macro
> > > "dma_buf_export" requires 5 arguments, but only 4 given
> > >
> > > Signed-off-by: Vincent Stehlé
> > > Cc: Russell King
> > > Cc: David Airlie
> > > Cc: Maarten Lankhorst
> > > Cc: Sumit Semwal
> >
> > Reviewed-by: David Herrmann
>
> Acked-by: Russell King
>
> Airlied, can you merge this please? Thanks.
Except that commit 4bcec44ffaf9 'dma-buf: use reservation objects'
comes from the dma-buf tree, so Sumit needs to add this commit to his
tree.
> > > This can be seen with e.g. linux next-20140523 and arm allmodconfig.
> > >
> > > Best regards,
> > >
> > > V.
> > >
> > > drivers/gpu/drm/armada/armada_gem.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/gpu/drm/armada/armada_gem.c
> > > b/drivers/gpu/drm/armada/armada_gem.c
> > > index 887816f..7adb0c3 100644
> > > --- a/drivers/gpu/drm/armada/armada_gem.c
> > > +++ b/drivers/gpu/drm/armada/armada_gem.c
> > > @@ -541,7 +541,7 @@ armada_gem_prime_export(struct drm_device *dev,
> > > struct drm_gem_object *obj,
> > > int flags)
> > > {
> > > return dma_buf_export(obj, &armada_gem_prime_dmabuf_ops,
> > > obj->size,
> > > - O_RDWR);
> > > + O_RDWR, NULL);
> > > }
> > >
> > > struct drm_gem_object *
> > > --
> > > 2.0.0.rc2
--
Cheers,
Stephen Rothwells...@canb.auug.org.au
signature.asc
Description: PGP signature
Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
On Sun, May 25, 2014 at 02:08:48PM +0200, David Herrmann wrote:
> Hi
>
> On Sat, May 24, 2014 at 11:05 PM, Vincent Stehlé
> wrote:
> > The dma_buf_export function was updated in commit 4bcec44ffaf9 'dma-buf: use
> > reservation objects' to take a reservation object parameter; update Armada
> > export method accordingly.
> >
> > This fixes the following compilation error:
> >
> > drivers/gpu/drm/armada/armada_gem.c: In function
> > ‘armada_gem_prime_export’:
> > drivers/gpu/drm/armada/armada_gem.c:544:16: error: macro "dma_buf_export"
> > requires 5 arguments, but only 4 given
> >
> > Signed-off-by: Vincent Stehlé
> > Cc: Russell King
> > Cc: David Airlie
> > Cc: Maarten Lankhorst
> > Cc: Sumit Semwal
>
> Reviewed-by: David Herrmann
Acked-by: Russell King
Airlied, can you merge this please? Thanks.
> Thanks
> David
>
> > ---
> >
> > Hi,
> >
> > This can be seen with e.g. linux next-20140523 and arm allmodconfig.
> >
> > Best regards,
> >
> > V.
> >
> > drivers/gpu/drm/armada/armada_gem.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/armada/armada_gem.c
> > b/drivers/gpu/drm/armada/armada_gem.c
> > index 887816f..7adb0c3 100644
> > --- a/drivers/gpu/drm/armada/armada_gem.c
> > +++ b/drivers/gpu/drm/armada/armada_gem.c
> > @@ -541,7 +541,7 @@ armada_gem_prime_export(struct drm_device *dev, struct
> > drm_gem_object *obj,
> > int flags)
> > {
> > return dma_buf_export(obj, &armada_gem_prime_dmabuf_ops, obj->size,
> > - O_RDWR);
> > + O_RDWR, NULL);
> > }
> >
> > struct drm_gem_object *
> > --
> > 2.0.0.rc2
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majord...@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH linux-next] DRM: Armada: update dma_buf_export use
Hi
On Sat, May 24, 2014 at 11:05 PM, Vincent Stehlé
wrote:
> The dma_buf_export function was updated in commit 4bcec44ffaf9 'dma-buf: use
> reservation objects' to take a reservation object parameter; update Armada
> export method accordingly.
>
> This fixes the following compilation error:
>
> drivers/gpu/drm/armada/armada_gem.c: In function ‘armada_gem_prime_export’:
> drivers/gpu/drm/armada/armada_gem.c:544:16: error: macro "dma_buf_export"
> requires 5 arguments, but only 4 given
>
> Signed-off-by: Vincent Stehlé
> Cc: Russell King
> Cc: David Airlie
> Cc: Maarten Lankhorst
> Cc: Sumit Semwal
Reviewed-by: David Herrmann
Thanks
David
> ---
>
> Hi,
>
> This can be seen with e.g. linux next-20140523 and arm allmodconfig.
>
> Best regards,
>
> V.
>
> drivers/gpu/drm/armada/armada_gem.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/armada/armada_gem.c
> b/drivers/gpu/drm/armada/armada_gem.c
> index 887816f..7adb0c3 100644
> --- a/drivers/gpu/drm/armada/armada_gem.c
> +++ b/drivers/gpu/drm/armada/armada_gem.c
> @@ -541,7 +541,7 @@ armada_gem_prime_export(struct drm_device *dev, struct
> drm_gem_object *obj,
> int flags)
> {
> return dma_buf_export(obj, &armada_gem_prime_dmabuf_ops, obj->size,
> - O_RDWR);
> + O_RDWR, NULL);
> }
>
> struct drm_gem_object *
> --
> 2.0.0.rc2
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
