Messages in this thread | | | From | David Brownell <> | Subject | Re: PATCH/RFC: driver model/pmcore wakeup hooks (1/4) | Date | Tue, 5 Oct 2004 13:09:02 -0700 |
| |
On Tuesday 05 October 2004 12:37 pm, Pavel Machek wrote: > Hi! > > > This lets drivers standardize how they present their ability to issue > > wakeups, and how they manage whether that ability should be used. > > Why do you assign "enabled" to variable instead of using it directly?
So there's exactly one copy of that string in use, agreeing with itself. Also, so strncmp() can be used. It won't matter if the sysadmin goes
echo -n enabled > wakeup echo enabled > wakeup
I'd personally rather use "on" and "off", but there seems to be a convention in /proc/acpi/wakeup in favor of polysyllabicism.
> And perhaps you should print "not supported" instead of empty string...
Except that's two words, not one, which will make shell script bugs happen more readily. I thought about "(none)" which has the same issue, and "-". But I figured that if it were very important, a good solution would appear ... ;)
- Dave - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |