On Wed, 21 Mar 2007, Zhang Rui wrote:
> On Tue, 2007-03-20 at 12:31 -0300, Henrique de Moraes Holschuh wrote:
> > On Tue, 20 Mar 2007, Zhang Rui wrote:
> > > Add sysfs interface for ACPI devices.
> > 
> > While doing the ibm-acpi sysfs work (not submitted yet), I have found I need
> > something like this to properly parse simple ulongs from sysfs:
> > 
> > static int parse_strtoul(const char *buf,
> >             unsigned long max, unsigned long *value)
> > {
> >     char *endp;
> > 
> >     *value = simple_strtoul(buf, &endp, 0);
> >     while (*endp && isspace(*endp))
> >             endp++;
> >     if (*endp || *value > max)
> >             return -EINVAL;
> > 
> >     return 0;
> > }
> > 
> Sounds nice.
> But I think it can work better without the parameter "max".

Well, it is a generic parser for unsigned longs as I needed them in
ibm-acpi, where the minimum value is always zero, and there is always a
maximum value.  If you don't need "max", just remove it :-)

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to