On Mon, Oct 14 2013, Andrzej Pietrasiewicz wrote: > strtobool is more flexible for the user and is more appropriate in the > context. > > Signed-off-by: Andrzej Pietrasiewicz <andrze...@samsung.com> > Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
Acked-by: Michal Nazarewicz <min...@mina86.com>
> diff --git a/drivers/usb/gadget/storage_common.c
> b/drivers/usb/gadget/storage_common.c
> index c7b78a1..8bd5f2d 100644
> --- a/drivers/usb/gadget/storage_common.c
> +++ b/drivers/usb/gadget/storage_common.c
> @@ -375,9 +375,9 @@ ssize_t fsg_store_ro(struct fsg_lun *curlun, struct
> rw_semaphore *filesem,
> const char *buf, size_t count)
> {
> ssize_t rc;
> - unsigned ro;
> + bool ro;
>
> - rc = kstrtouint(buf, 2, &ro);
> + rc = strtobool(buf, &ro);
> if (rc)
> return rc;
>
> @@ -402,10 +402,10 @@ EXPORT_SYMBOL(fsg_store_ro);
>
> ssize_t fsg_store_nofua(struct fsg_lun *curlun, const char *buf, size_t
> count)
> {
> - unsigned nofua;
> + bool nofua;
> int ret;
>
> - ret = kstrtouint(buf, 2, &nofua);
> + ret = strtobool(buf, &nofua);
> if (ret)
> return ret;
>
> @@ -452,10 +452,10 @@ EXPORT_SYMBOL(fsg_store_file);
>
> ssize_t fsg_store_cdrom(struct fsg_lun *curlun, const char *buf, size_t
> count)
> {
> - unsigned cdrom;
> + bool cdrom;
> int ret;
>
> - ret = kstrtouint(buf, 2, &cdrom);
> + ret = strtobool(buf, &cdrom);
> if (ret)
> return ret;
>
> @@ -468,10 +468,10 @@ EXPORT_SYMBOL(fsg_store_cdrom);
> ssize_t fsg_store_removable(struct fsg_lun *curlun, const char *buf,
> size_t count)
> {
> - unsigned removable;
> + bool removable;
> int ret;
>
> - ret = kstrtouint(buf, 2, &removable);
> + ret = strtobool(buf, &removable);
> if (ret)
> return ret;
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michał “mina86” Nazarewicz (o o)
ooo +--<m...@google.com>--<xmpp:min...@jabber.org>--ooO--(_)--Ooo--
signature.asc
Description: PGP signature
