From 71b6c19f1d4307b45e0bc17a7437daa3b4ec49c1 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Wed, 14 Oct 2015 08:36:32 +1000 Subject: [PATCH] hidpp10: convert 0x01 Enable Individual Features into a proper bitmask The caller doesn't need to worry about registers or something here, let's just expose one single bitmask, provided by an enum. This also adds s couple of missing bits from the previous implementation. Signed-off-by: Peter Hutterer --- src/hidpp10.c | 23 +++++----- src/hidpp10.h | 119 ++++++++++++++++++++++++++++++++++++++++++++------ 2 files changed, 119 insertions(+), 23 deletions(-) diff --git a/src/hidpp10.c b/src/hidpp10.c index be074a8b..fe309820 100644 --- a/src/hidpp10.c +++ b/src/hidpp10.c @@ -275,8 +275,7 @@ hidpp10_set_hidpp_notifications(struct hidpp10_device *dev, int hidpp10_get_individual_features(struct hidpp10_device *dev, - uint8_t *feature_bit_r0, - uint8_t *feature_bit_r2) + uint32_t *feature_mask) { unsigned idx = dev->index; union hidpp10_message features = CMD_ENABLE_INDIVIDUAL_FEATURES(idx, GET_REGISTER_REQ); @@ -288,16 +287,18 @@ hidpp10_get_individual_features(struct hidpp10_device *dev, if (res) return res; - *feature_bit_r0 = features.msg.parameters[0]; - *feature_bit_r2 = features.msg.parameters[2]; + *feature_mask = features.msg.parameters[0]; + /* bits 0 and 4-7 are reserved */ + *feature_mask |= (features.msg.parameters[1] & 0x0E) << 8; + /* bits 6-7 are reserved */ + *feature_mask |= (features.msg.parameters[2] & 0x3F) << 16; return 0; } int -hidpp10_set_individual_feature(struct hidpp10_device *dev, - uint8_t feature_bit_r0, - uint8_t feature_bit_r2) +hidpp10_set_individual_features(struct hidpp10_device *dev, + uint32_t feature_mask) { unsigned idx = RECEIVER_IDX; union hidpp10_message mode = CMD_ENABLE_INDIVIDUAL_FEATURES(idx, SET_REGISTER_REQ); @@ -307,8 +308,9 @@ hidpp10_set_individual_feature(struct hidpp10_device *dev, mode.msg.device_idx = dev->index; - mode.msg.parameters[0] = feature_bit_r0; - mode.msg.parameters[1] = feature_bit_r2; + mode.msg.parameters[0] = feature_mask & 0xFF; + mode.msg.parameters[1] = (feature_mask >> 8) & 0x0E; + mode.msg.parameters[2] = (feature_mask >> 16) & 0x3F; res = hidpp10_request_command(dev, &mode); @@ -1240,11 +1242,12 @@ void hidpp10_list_devices(struct ratbag_device *device) { static int hidpp10_get_device_info(struct hidpp10_device *dev) { + uint32_t feature_mask; uint8_t f1, f2; uint8_t reflect; int i; - hidpp10_get_individual_features(dev, &f1, &f2); + hidpp10_get_individual_features(dev, &feature_mask); hidpp10_get_hidpp_notifications(dev, &f1, &f2); hidpp10_get_current_resolution(dev, &dev->xres, &dev->yres); diff --git a/src/hidpp10.h b/src/hidpp10.h index 239c1cd0..2db14288 100644 --- a/src/hidpp10.h +++ b/src/hidpp10.h @@ -89,24 +89,117 @@ hidpp10_set_hidpp_notifications(struct hidpp10_device *dev, /* -------------------------------------------------------------------------- */ /* 0x01: Enable Individual Features */ /* -------------------------------------------------------------------------- */ -#define FEATURE_BIT_R0_SPECIAL_BUTTON_FUNCTION 1 -#define FEATURE_BIT_R0_ENHANCED_KEY_USAGE 2 -#define FEATURE_BIT_R0_FAST_FORWARD_REWIND 3 -#define FEATURE_BIT_R0_SCROLLING_ACCELERATION 6 -#define FEATURE_BIT_R0_BUTTONS_CONTROL_THE_RESOLUTION 7 -#define FEATURE_BIT_R2_INHIBIT_LOCK_KEY_SOUND 0 -#define FEATURE_BIT_R2_3D_ENGINE 2 -#define FEATURE_BIT_R2_HOST_SW_CONTROLS_LEDS 3 + +enum hidpp10_individual_features { + HIDPP10_FEATURE_BIT_MOUSE_SENSOR_RESOLUTION = (1 << 0), + /** + * disabled: buttons send button codes + * enabled: buttons have special functions (default) + * @note Do not use, use 0x63 instead + */ + HIDPP10_FEATURE_BIT_SPECIAL_BUTTON_FUNCTION = (1 << 1), + /** + * disabled: normal key usage (default) + * enabled: enhanced key usage + */ + HIDPP10_FEATURE_BIT_ENHANCED_KEY_USAGE = (1 << 2), + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_FAST_FORWARD_REWIND = (1 << 3), + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_SEND_CALCULATOR_RESULT = (1 << 4), + /** + * disabled: + * enabled: (default) + */ + HIDPP10_FEATURE_BIT_MOTION_WAKEUP = (1 << 5), + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_FAST_SCROLLING = (1 << 6), + /** + * disabled: work as buttons + * enabled: control the resolution (default) + */ + HIDPP10_FEATURE_BIT_BUTTONS_CONTROL_RESOLUTION = (1 << 7), + + /* 1 << 8 is reserved */ + + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_RECEIVER_MULTIPLE_RF_LOCK = (1 << 9), + + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_RECEIVER_DISABLE_RFSCAN_IN_SUSPEND = (1 << 10), + + /** + * disabled: (default) + * enabled: + * + * When enabled,removes all compatibility checks. + */ + HIDPP10_FEATURE_BIT_RECEIVER_ACCEPT_ALL_DEVICES_IN_PAIRING = (1 << 11), + + /* 1 << 12 is reserved */ + /* 1 << 13 is reserved */ + /* 1 << 14 is reserved */ + /* 1 << 15 is reserved */ + + /** + * disabled: (default) + * enabled: no sound + */ + HIDPP10_FEATURE_BIT_INHIBIT_LOCK_KEY_SOUND = (1 << 16), + + /** + * disabled: (default) + * enabled: + */ + HIDPP10_FEATURE_BIT_INHIBIT_TOUCHPAD = (1 << 17), + + /** + * disabled: + * enabled: (default) + */ + HIDPP10_FEATURE_BIT_3D_ENGINE = (1 << 18), + + /** + * disabled: (disabled) + * enabled: + */ + HIDPP10_FEATURE_BIT_SW_CONTROLS_LEDS = (1 << 19), + + /** + * disabled: (disabled) + * enabled: + */ + HIDPP10_FEATURE_BIT_NO_NUMLOCK_TOGGLE = (1 << 20), + + /** + * disabled: (disabled) + * enabled: + */ + HIDPP10_FEATURE_BIT_INHIBIT_PRESENCE_DETECTION = (1 << 21), +}; int hidpp10_get_individual_features(struct hidpp10_device *dev, - uint8_t *feature_bit_r0, - uint8_t *feature_bit_r2); + uint32_t *feature_mask); int -hidpp10_set_individual_feature(struct hidpp10_device *dev, - uint8_t feature_bit_r0, - uint8_t feature_bit_r2); +hidpp10_set_individual_features(struct hidpp10_device *dev, + uint32_t feature_mask); /* -------------------------------------------------------------------------- */ /* 0x07: Battery Status */