staging: ks7010: replace defines with enum types
Header has multiple constants defined using preprocessor directive. In the cases where these are an integer progression an enumeration type can be used. Doing so adds documentation to the code and makes the usage explicit. Maintain original constant value, this value is returned by the device. Replace (integer progression) preprocessor constants with enumeration type. Signed-off-by: Tobin C. Harding <me@tobin.cc> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
bfee6a2319
commit
68711cebcb
1 changed files with 9 additions and 5 deletions
|
@ -62,14 +62,18 @@ enum reg_status_type {
|
||||||
|
|
||||||
/* General Communication Register A */
|
/* General Communication Register A */
|
||||||
#define GCR_A 0x000028
|
#define GCR_A 0x000028
|
||||||
#define GCR_A_INIT 0
|
enum gen_com_reg_a {
|
||||||
#define GCR_A_REMAP 1
|
GCR_A_INIT,
|
||||||
#define GCR_A_RUN 2
|
GCR_A_REMAP,
|
||||||
|
GCR_A_RUN
|
||||||
|
};
|
||||||
|
|
||||||
/* General Communication Register B */
|
/* General Communication Register B */
|
||||||
#define GCR_B 0x00002C
|
#define GCR_B 0x00002C
|
||||||
#define GCR_B_ACTIVE 0
|
enum gen_com_reg_b {
|
||||||
#define GCR_B_DOZE 1
|
GCR_B_ACTIVE,
|
||||||
|
GCR_B_DOZE
|
||||||
|
};
|
||||||
|
|
||||||
/* Wakeup Register */
|
/* Wakeup Register */
|
||||||
/* #define WAKEUP 0x008104 */
|
/* #define WAKEUP 0x008104 */
|
||||||
|
|
Loading…
Reference in a new issue