> Linux aesthetics disaproves of no-value-added definitions like 
> +struct apm_get_power_state_inparms { 
> +       unsigned short dev_id; 
> +}; 

This is included for reasons of symmetry---it is
one of the components of a union.

> and in the code you present, rtnval seems redundant,
> since user-level can look at errno to see whether the
> ioctl succeeded.

rtnval is the APM BIOS return code whereas errno
is the ioctl return code.

Thanks

Thomas Hood
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
Please read the FAQ at http://www.tux.org/lkml/

Reply via email to