All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul()
@ 2013-07-19  6:54 Jingoo Han
  2013-08-23  0:28 ` Jingoo Han
  2013-08-24  4:54 ` Brian Norris
  0 siblings, 2 replies; 4+ messages in thread
From: Jingoo Han @ 2013-07-19  6:54 UTC (permalink / raw)
  To: Artem Bityutskiy
  Cc: linux-mtd, 'Jingoo Han', 'David Woodhouse',
	'Artem Bityutskiy'

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul()
  2013-07-19  6:54 [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul() Jingoo Han
@ 2013-08-23  0:28 ` Jingoo Han
  2013-08-24  4:54 ` Brian Norris
  1 sibling, 0 replies; 4+ messages in thread
From: Jingoo Han @ 2013-08-23  0:28 UTC (permalink / raw)
  To: 'Brian Norris'
  Cc: 'Artem Bityutskiy', 'Jingoo Han',
	'David Woodhouse', linux-mtd, 'Artem Bityutskiy'

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul()
  2013-07-19  6:54 [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul() Jingoo Han
  2013-08-23  0:28 ` Jingoo Han
@ 2013-08-24  4:54 ` Brian Norris
  1 sibling, 0 replies; 4+ messages in thread
From: Brian Norris @ 2013-08-24  4:54 UTC (permalink / raw)
  To: Jingoo Han
  Cc: Artem Bityutskiy, linux-mtd, 'David Woodhouse',
	'Artem Bityutskiy'

On Fri, Jul 19, 2013 at 03:54:11PM +0900, Jingoo Han wrote:
> 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>

Applied to l2-mtd.git. Thanks!

Brian

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul()
@ 2013-06-01  7:17 Jingoo Han
  0 siblings, 0 replies; 4+ messages in thread
From: Jingoo Han @ 2013-06-01  7:17 UTC (permalink / raw)
  To: Artem Bityutskiy
  Cc: linux-mtd, 'Jingoo Han', 'David Woodhouse',
	'Artem Bityutskiy'

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-08-24  4:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-19  6:54 [PATCH] mtd: mtdswap: replace strict_strtoul() with kstrtoul() Jingoo Han
2013-08-23  0:28 ` Jingoo Han
2013-08-24  4:54 ` Brian Norris
  -- strict thread matches above, loose matches on Subject: below --
2013-06-01  7:17 Jingoo Han

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.