* [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls
@ 2012-05-20 12:55 Johannes Thumshirn
2012-06-02 11:36 ` morbid-rsa
0 siblings, 1 reply; 4+ messages in thread
From: Johannes Thumshirn @ 2012-05-20 12:55 UTC (permalink / raw)
To: gregkh, devel, linux-kernel
Cc: grabner, andor.daam, thomas, jj, stefanha, Johannes Thumshirn
Changed strict_strtoul() calls which do not result in a need to change
interfaces of called functions.
Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Johannes Thumshirn <morbidrsa@googlemail.com>
---
drivers/staging/line6/variax.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/line6/variax.c b/drivers/staging/line6/variax.c
index d366222..efa496d 100644
--- a/drivers/staging/line6/variax.c
+++ b/drivers/staging/line6/variax.c
@@ -353,10 +353,10 @@ static ssize_t variax_set_model(struct device *dev,
{
struct usb_line6_variax *variax =
usb_get_intfdata(to_usb_interface(dev));
- unsigned long value;
+ u8 value;
int ret;
- ret = strict_strtoul(buf, 10, &value);
+ ret = kstrtou8(buf, 10, &value);
if (ret)
return ret;
@@ -387,10 +387,10 @@ static ssize_t variax_set_active(struct device *dev,
{
struct usb_line6_variax *variax =
usb_get_intfdata(to_usb_interface(dev));
- unsigned long value;
+ u8 value;
int ret;
- ret = strict_strtoul(buf, 10, &value);
+ ret = kstrtou8(buf, 10, &value);
if (ret)
return ret;
--
1.7.7.6
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls
2012-05-20 12:55 [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls Johannes Thumshirn
@ 2012-06-02 11:36 ` morbid-rsa
2012-06-02 11:56 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: morbid-rsa @ 2012-06-02 11:36 UTC (permalink / raw)
To: gregkh, devel, linux-kernel
Cc: grabner, andor.daam, thomas, jj, stefanha, Johannes Thumshirn
Just wanted to ask about the status of this patch.
On Sun, May 20, 2012 at 2:55 PM, Johannes Thumshirn
<morbidrsa@googlemail.com> wrote:
> Changed strict_strtoul() calls which do not result in a need to change
> interfaces of called functions.
>
> Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
> Signed-off-by: Johannes Thumshirn <morbidrsa@googlemail.com>
> ---
> drivers/staging/line6/variax.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/line6/variax.c b/drivers/staging/line6/variax.c
> index d366222..efa496d 100644
> --- a/drivers/staging/line6/variax.c
> +++ b/drivers/staging/line6/variax.c
> @@ -353,10 +353,10 @@ static ssize_t variax_set_model(struct device *dev,
> {
> struct usb_line6_variax *variax =
> usb_get_intfdata(to_usb_interface(dev));
> - unsigned long value;
> + u8 value;
> int ret;
>
> - ret = strict_strtoul(buf, 10, &value);
> + ret = kstrtou8(buf, 10, &value);
> if (ret)
> return ret;
>
> @@ -387,10 +387,10 @@ static ssize_t variax_set_active(struct device *dev,
> {
> struct usb_line6_variax *variax =
> usb_get_intfdata(to_usb_interface(dev));
> - unsigned long value;
> + u8 value;
> int ret;
>
> - ret = strict_strtoul(buf, 10, &value);
> + ret = kstrtou8(buf, 10, &value);
> if (ret)
> return ret;
>
> --
> 1.7.7.6
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls
2012-06-02 11:36 ` morbid-rsa
@ 2012-06-02 11:56 ` Greg KH
2012-06-02 12:13 ` morbid-rsa
0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2012-06-02 11:56 UTC (permalink / raw)
To: morbid-rsa; +Cc: devel, linux-kernel, stefanha, andor.daam, grabner, jj, thomas
On Sat, Jun 02, 2012 at 01:36:10PM +0200, morbid-rsa wrote:
> Just wanted to ask about the status of this patch.
>
> On Sun, May 20, 2012 at 2:55 PM, Johannes Thumshirn
> <morbidrsa@googlemail.com> wrote:
> > Changed strict_strtoul() calls which do not result in a need to change
> > interfaces of called functions.
> >
> > Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
> > Signed-off-by: Johannes Thumshirn <morbidrsa@googlemail.com>
> > ---
> > drivers/staging/line6/variax.c | 8 ++++----
> > 1 files changed, 4 insertions(+), 4 deletions(-)
As this came in after the merge window closed for patches, I have to
hold on to it until 3.5-rc1 comes out. It's in my queue and not lost,
don't worry.
thanks for your patience,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls
2012-06-02 11:56 ` Greg KH
@ 2012-06-02 12:13 ` morbid-rsa
0 siblings, 0 replies; 4+ messages in thread
From: morbid-rsa @ 2012-06-02 12:13 UTC (permalink / raw)
To: Greg KH; +Cc: devel, linux-kernel, stefanha, andor.daam, grabner, jj, thomas
On Sat, Jun 2, 2012 at 1:56 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Sat, Jun 02, 2012 at 01:36:10PM +0200, morbid-rsa wrote:
>> Just wanted to ask about the status of this patch.
>>
>> On Sun, May 20, 2012 at 2:55 PM, Johannes Thumshirn
>> <morbidrsa@googlemail.com> wrote:
>> > Changed strict_strtoul() calls which do not result in a need to change
>> > interfaces of called functions.
>> >
>> > Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
>> > Signed-off-by: Johannes Thumshirn <morbidrsa@googlemail.com>
>> > ---
>> > drivers/staging/line6/variax.c | 8 ++++----
>> > 1 files changed, 4 insertions(+), 4 deletions(-)
>
> As this came in after the merge window closed for patches, I have to
> hold on to it until 3.5-rc1 comes out. It's in my queue and not lost,
> don't worry.
>
> thanks for your patience,
>
> greg k-h
Oh ok, thanks for the fast answer.
Johannes
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-06-02 12:13 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-20 12:55 [PATCH] drivers/staging/line6/variax.c: Change some strict_strtoul() calls Johannes Thumshirn
2012-06-02 11:36 ` morbid-rsa
2012-06-02 11:56 ` Greg KH
2012-06-02 12:13 ` morbid-rsa
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).