From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailout1.samsung.com ([203.254.224.24])
 by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux))
 id 1VCfEu-0005vt-Rn
 for linux-mtd@lists.infradead.org; Fri, 23 Aug 2013 00:28:35 +0000
Received: from epcpsbgr2.samsung.com
 (u142.gpu120.samsung.co.kr [203.254.230.142])
 by mailout1.samsung.com (Oracle Communications Messaging Server 7u4-24.01
 (7.0.4.24.0) 64bit (built Nov 17 2011))
 with ESMTP id <0MRY00KDRJYT7CD0@mailout1.samsung.com> for
 linux-mtd@lists.infradead.org; Fri, 23 Aug 2013 09:28:05 +0900 (KST)
From: Jingoo Han <jg1.han@samsung.com>
To: 'Brian Norris' <computersforpeace@gmail.com>
References: <000701ce844c$c63f4820$52bdd860$@samsung.com>
In-reply-to: <000701ce844c$c63f4820$52bdd860$@samsung.com>
Subject: Re: [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul()
Date: Fri, 23 Aug 2013 09:28:05 +0900
Message-id: <001c01ce9f97$a249b2a0$e6dd17e0$%han@samsung.com>
MIME-version: 1.0
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7bit
Content-language: ko
Cc: 'Artem Bityutskiy' <artem.bityutskiy@linux.intel.com>,
 'Jingoo Han' <jg1.han@samsung.com>, 'David Woodhouse' <dwmw2@infradead.org>,
 linux-mtd@lists.infradead.org, 'Artem Bityutskiy' <dedekind1@gmail.com>
List-Id: Linux MTD discussion mailing list <linux-mtd.lists.infradead.org>
List-Unsubscribe: <http://lists.infradead.org/mailman/options/linux-mtd>,
 <mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe>
List-Archive: <http://lists.infradead.org/pipermail/linux-mtd/>
List-Post: <mailto:linux-mtd@lists.infradead.org>
List-Help: <mailto:linux-mtd-request@lists.infradead.org?subject=help>
List-Subscribe: <http://lists.infradead.org/mailman/listinfo/linux-mtd>,
 <mailto:linux-mtd-request@lists.infradead.org?subject=subscribe>

On Friday, July 19, 2013 3:54 PM, Jingoo Han wrote:

Hi Brian,

Would you apply this patch to l2-mtd.git?
I hope that it is not late. :)
Thank you.

Best regards,
Jingoo Han

> 
> The usage of strict_strtoul() is not preferred, because
> strict_strtoul() is obsolete. Thus, kstrtoul() should be
> used.
> 
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
>  drivers/mtd/mtdswap.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mtd/mtdswap.c b/drivers/mtd/mtdswap.c
> index c92f0f6..8b33b26 100644
> --- a/drivers/mtd/mtdswap.c
> +++ b/drivers/mtd/mtdswap.c
> @@ -1425,7 +1425,7 @@ static void mtdswap_add_mtd(struct mtd_blktrans_ops *tr, struct mtd_info *mtd)
>  		return;
> 
>  	while ((this_opt = strsep(&parts, ",")) != NULL) {
> -		if (strict_strtoul(this_opt, 0, &part) < 0)
> +		if (kstrtoul(this_opt, 0, &part) < 0)
>  			return;
> 
>  		if (mtd->index == part)
> --
> 1.7.10.4