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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
