Re: [PATCH 1/8] drivers/staging/line6/control.c: Changestrict_strtoul to kstrtoul

From: Johannes Thumshirn
Date: Sun May 06 2012 - 04:27:35 EST


On Sun, 2012-05-06 at 09:34 +0300, Dan Carpenter wrote:
> On Sat, May 05, 2012 at 04:31:45PM +0200, Johannes Thumshirn wrote:
> > Changed strict_strtoul() call to kstrtoul to make checkpatch.pl happy
> >
>
> This would probably be better as kstrtou8(). You'd have to change
> the type of value, and the function parameters which take value as
> well. Also you would have to change the name of pod_set_param_int()
> because it says "int".
>
>
> regards,
> dan carpenter
>

Ok I'll look over these patches again, but I've got another question
regarding these, should I re-diff against the current linux-next or
against these patches?

regards,
Johannes Thumshirn

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