From: Marek Szyprowski <m.szyprowski@samsung.com>
To: "'Laurent Pinchart'" <laurent.pinchart@ideasonboard.com>,
        linux-kernel@vger.kernel.org
References: <1350483442-13180-1-git-send-email-laurent.pinchart@ideasonboard.com>
In-reply-to: <1350483442-13180-1-git-send-email-laurent.pinchart@ideasonboard.com>
Subject: RE: [PATCH] drivers: cma: Fix wrong CMA selected region size default
 value
Date: Thu, 18 Oct 2012 15:14:11 +0200
Organization: SPRC
Message-id: <00c501cdad32$78729e40$6957dac0$%szyprowski@samsung.com>
MIME-version: 1.0
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7bit
Thread-index: Ac2scgaQRjrdq8+lTpqTMg4KlI51WAAvVnAw
Content-language: pl
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1199
Lines: 41

Hi Laurent,

On Wednesday, October 17, 2012 4:17 PM Laurent Pinchart wrote:

> Kconfig lists CMA_SIZE_SEL_ABSOLUTE as the default value fo the CMA
> selected region size, but that option isn't available in the defined
> choices. Set the default to CMA_SIZE_SEL_MBYTES instead.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  drivers/base/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/base/Kconfig b/drivers/base/Kconfig
> index 08b4c52..b34b5cd 100644
> --- a/drivers/base/Kconfig
> +++ b/drivers/base/Kconfig
> @@ -236,7 +236,7 @@ config CMA_SIZE_PERCENTAGE
> 
>  choice
>  	prompt "Selected region size"
> -	default CMA_SIZE_SEL_ABSOLUTE
> +	default CMA_SIZE_SEL_MBYTES
> 
>  config CMA_SIZE_SEL_MBYTES
>  	bool "Use mega bytes value only"
> --

Right, thanks for the fix, applied to my dma-mapping tree.

Best regards
-- 
Marek Szyprowski
Samsung Poland R&D Center


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/