Re: [PATCH 05/16] mm/drivers: use vm_flags_t for vma flags

From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Konstantin Khlebnikov <khlebnikov@openvz.org>
Cc: "Andrew Morton" <akpm@linux-foundation.org>,
	devel@driverdev.osuosl.org,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	linux-kernel@vger.kernel.org,
	"Mauro Carvalho Chehab" <mchehab@infradead.org>,
	linux-mm@kvack.org, "Arve Hjønnevåg" <arve@android.com>,
	"John Stultz" <john.stultz@linaro.org>,
	linux-media@vger.kernel.org
Subject: Re: [PATCH 05/16] mm/drivers: use vm_flags_t for vma flags
Date: Wed, 21 Mar 2012 11:34:56 +0100	[thread overview]
Message-ID: <3319224.gEMkjEgmG9@avalon> (raw)
In-Reply-To: <20120321065633.13852.11903.stgit@zurg>

Hi Konstantin,

Thanks for the patch.

On Wednesday 21 March 2012 10:56:33 Konstantin Khlebnikov wrote:
> Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org>
> Cc: linux-media@vger.kernel.org
> Cc: devel@driverdev.osuosl.org
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: John Stultz <john.stultz@linaro.org>
> Cc: "Arve Hjønnevåg" <arve@android.com>
> ---
>  drivers/media/video/omap3isp/ispqueue.h |    2 +-

For the OMAP3 ISP driver,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

>  drivers/staging/android/ashmem.c        |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/media/video/omap3isp/ispqueue.h
> b/drivers/media/video/omap3isp/ispqueue.h index 92c5a12..908dfd7 100644
> --- a/drivers/media/video/omap3isp/ispqueue.h
> +++ b/drivers/media/video/omap3isp/ispqueue.h
> @@ -90,7 +90,7 @@ struct isp_video_buffer {
>  	void *vaddr;
> 
>  	/* For userspace buffers. */
> -	unsigned long vm_flags;
> +	vm_flags_t vm_flags;
>  	unsigned long offset;
>  	unsigned int npages;
>  	struct page **pages;
> diff --git a/drivers/staging/android/ashmem.c
> b/drivers/staging/android/ashmem.c index 9f1f27e..4511420 100644
> --- a/drivers/staging/android/ashmem.c
> +++ b/drivers/staging/android/ashmem.c
> @@ -269,7 +269,7 @@ out:
>  	return ret;
>  }
> 
> -static inline unsigned long calc_vm_may_flags(unsigned long prot)
> +static inline vm_flags_t calc_vm_may_flags(unsigned long prot)
>  {
>  	return _calc_vm_trans(prot, PROT_READ,  VM_MAYREAD) |
>  	       _calc_vm_trans(prot, PROT_WRITE, VM_MAYWRITE) |
> 

-- 
Regards,

Laurent Pinchart

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>