On Wed, Apr 20, 2005 at 05:59:56PM +0200, Pavel Machek wrote: -->snip > > We already have int system_state... How is this different? > -->snip > > Having "unsigned int state" is asking for trouble. > -->snip > > Are they same type as system_power_state.state? This should be clear, > and type-checked by sparse. Otherwise people will get it wrong. > > Hi Pavel, I appreciate the suggestions. I'm currently making some revisions. I'll clarify things further once they're ready. Thanks, Adam