From c4334e0048260657c810312e89f84a099f365337 Mon Sep 17 00:00:00 2001 From: notaz Date: Thu, 27 Oct 2011 23:22:55 +0000 Subject: [PATCH] input: make it more modular git-svn-id: file:///home/notaz/opt/svn/PicoDrive/platform@934 be3aeb3a-fb24-0410-a615-afba39da0efa --- common/input.c | 127 +++++++++++++++++++++++-------------------------- common/input.h | 13 ++--- 2 files changed, 62 insertions(+), 78 deletions(-) diff --git a/common/input.c b/common/input.c index a91de8d..bdeebbe 100644 --- a/common/input.c +++ b/common/input.c @@ -1,5 +1,5 @@ /* - * (C) Gražvydas "notaz" Ignotas, 2008-2010 + * (C) Gražvydas "notaz" Ignotas, 2008-2011 * * This work is licensed under the terms of any of these licenses * (at your option): @@ -16,13 +16,12 @@ #include "plat.h" #include "lprintf.h" -#ifdef IN_EVDEV -#include "../linux/in_evdev.h" -#endif #ifdef IN_GP2X +#error needs update: in_gp2x_init in_gp2x_update #include "../gp2x/in_gp2x.h" #endif #ifdef IN_VK +#error needs update: in_vk_init in_vk_update #include "../win32/in_vk.h" #endif @@ -39,14 +38,16 @@ typedef struct unsigned int does_combos:1; } in_dev_t; -static in_drv_t in_drivers[IN_DRVID_COUNT]; +static in_drv_t *in_drivers; static in_dev_t in_devices[IN_MAX_DEVS]; +static int in_driver_count = 0; static int in_dev_count = 0; /* probed + bind devices */ static int in_have_async_devs = 0; +static int in_probe_dev_id; static int menu_key_state = 0; static int menu_last_used_dev = 0; -#define DRV(id) in_drivers[(unsigned)(id) < IN_DRVID_COUNT ? (id) : 0] +#define DRV(id) in_drivers[id] static int *in_alloc_binds(int drv_id, int key_count) @@ -83,7 +84,7 @@ static void in_free(in_dev_t *dev) /* to be called by drivers * async devices must set drv_fd_hnd to -1 */ -void in_register(const char *nname, int drv_id, int drv_fd_hnd, void *drv_data, +void in_register(const char *nname, int drv_fd_hnd, void *drv_data, int key_count, const char * const *key_names, int combos) { int i, ret, dupe_count = 0, *binds; @@ -124,7 +125,7 @@ void in_register(const char *nname, int drv_id, int drv_fd_hnd, void *drv_data, if (tmp == NULL) return; - binds = in_alloc_binds(drv_id, key_count); + binds = in_alloc_binds(in_probe_dev_id, key_count); if (binds == NULL) { free(tmp); return; @@ -140,13 +141,13 @@ void in_register(const char *nname, int drv_id, int drv_fd_hnd, void *drv_data, update: in_devices[i].probed = 1; in_devices[i].does_combos = combos; - in_devices[i].drv_id = drv_id; + in_devices[i].drv_id = in_probe_dev_id; in_devices[i].drv_fd_hnd = drv_fd_hnd; in_devices[i].key_names = key_names; in_devices[i].drv_data = drv_data; if (in_devices[i].binds != NULL) { - ret = DRV(drv_id).clean_binds(drv_data, in_devices[i].binds, + ret = DRV(in_probe_dev_id).clean_binds(drv_data, in_devices[i].binds, in_devices[i].binds + key_count * IN_BINDTYPE_COUNT); if (ret == 0) { /* no useable binds */ @@ -234,8 +235,10 @@ void in_probe(void) for (i = 0; i < in_dev_count; i++) in_unprobe(&in_devices[i]); - for (i = 1; i < IN_DRVID_COUNT; i++) + for (i = 0; i < in_driver_count; i++) { + in_probe_dev_id = i; in_drivers[i].probe(); + } /* get rid of devs without binds and probes */ for (i = 0; i < in_dev_count; i++) { @@ -267,26 +270,8 @@ int in_update(int *result) for (i = 0; i < in_dev_count; i++) { in_dev_t *dev = &in_devices[i]; - if (dev->probed && dev->binds != NULL) { - // FIXME: this is stupid, make it indirect - switch (dev->drv_id) { -#ifdef IN_EVDEV - case IN_DRVID_EVDEV: - ret |= in_evdev_update(dev->drv_data, dev->binds, result); - break; -#endif -#ifdef IN_GP2X - case IN_DRVID_GP2X: - ret |= in_gp2x_update(dev->drv_data, dev->binds, result); - break; -#endif -#ifdef IN_VK - case IN_DRVID_VK: - ret |= in_vk_update(dev->drv_data, dev->binds, result); - break; -#endif - } - } + if (dev->probed && dev->binds != NULL) + ret |= DRV(dev->drv_id).update(dev->drv_data, dev->binds, result); } return ret; @@ -602,6 +587,7 @@ const char *in_get_key_name(int dev_id, int keycode) { const char *name = NULL; static char xname[16]; + in_drv_t *drv; in_dev_t *dev; if (dev_id < 0) /* want last used dev? */ @@ -611,15 +597,17 @@ const char *in_get_key_name(int dev_id, int keycode) return "Unkn0"; dev = &in_devices[dev_id]; + drv = &DRV(dev->drv_id); if (keycode < 0) /* want name for menu key? */ - keycode = DRV(dev->drv_id).menu_translate(dev->drv_data, keycode); + keycode = drv->menu_translate(dev->drv_data, keycode); if (dev->key_names != NULL && 0 <= keycode && keycode < dev->key_count) name = dev->key_names[keycode]; if (name != NULL) return name; - name = DRV(dev->drv_id).get_key_name(keycode); + if (drv->get_key_name != NULL) + name = drv->get_key_name(keycode); if (name != NULL) return name; @@ -731,7 +719,7 @@ int in_config_parse_dev(const char *name) { int drv_id = -1, i; - for (i = 0; i < IN_DRVID_COUNT; i++) { + for (i = 0; i < in_driver_count; i++) { int len = strlen(in_drivers[i].prefix); if (strncmp(name, in_drivers[i].prefix, len) == 0) { drv_id = i; @@ -877,53 +865,56 @@ void in_debug_dump(void) } } -/* handlers for unknown/not_preset drivers */ +/* stubs for drivers that choose not to implement something */ -static void in_def_probe(void) {} static void in_def_free(void *drv_data) {} -static const char * const * - in_def_get_key_names(int *count) { return NULL; } static void in_def_get_def_binds(int *binds) {} -static int in_def_clean_binds(void *drv_data, int *b, int *db) { return 0; } +static int in_def_clean_binds(void *drv_data, int *b, int *db) { return 1; } static int in_def_get_config(void *drv_data, int what, int *val) { return -1; } static int in_def_set_config(void *drv_data, int what, int val) { return -1; } static int in_def_update_keycode(void *drv_data, int *is_down) { return 0; } -static int in_def_menu_translate(void *drv_data, int keycode) { return keycode; } +static int in_def_menu_translate(void *drv_data, int keycode) { return 0; } static int in_def_get_key_code(const char *key_name) { return -1; } static const char *in_def_get_key_name(int keycode) { return NULL; } -void in_init(void) -{ - int i; +#define CHECK_ADD_STUB(d, f) \ + if (d.f == NULL) d.f = in_def_##f - memset(in_drivers, 0, sizeof(in_drivers)); - memset(in_devices, 0, sizeof(in_devices)); - in_dev_count = 0; +/* to be called by drivers */ +int in_register_driver(const in_drv_t *drv) +{ + int count_new = in_driver_count + 1; + in_drv_t *new_drivers; - for (i = 0; i < IN_DRVID_COUNT; i++) { - in_drivers[i].prefix = "none:"; - in_drivers[i].probe = in_def_probe; - in_drivers[i].free = in_def_free; - in_drivers[i].get_key_names = in_def_get_key_names; - in_drivers[i].get_def_binds = in_def_get_def_binds; - in_drivers[i].clean_binds = in_def_clean_binds; - in_drivers[i].get_config = in_def_get_config; - in_drivers[i].set_config = in_def_set_config; - in_drivers[i].update_keycode = in_def_update_keycode; - in_drivers[i].menu_translate = in_def_menu_translate; - in_drivers[i].get_key_code = in_def_get_key_code; - in_drivers[i].get_key_name = in_def_get_key_name; + new_drivers = realloc(in_drivers, count_new * sizeof(in_drivers[0])); + if (new_drivers == NULL) { + lprintf("input: in_register_driver OOM\n"); + return -1; } -#ifdef IN_GP2X - in_gp2x_init(&in_drivers[IN_DRVID_GP2X]); -#endif -#ifdef IN_EVDEV - in_evdev_init(&in_drivers[IN_DRVID_EVDEV]); -#endif -#ifdef IN_VK - in_vk_init(&in_drivers[IN_DRVID_VK]); -#endif + memcpy(&new_drivers[in_driver_count], drv, sizeof(new_drivers[0])); + + CHECK_ADD_STUB(new_drivers[in_driver_count], free); + CHECK_ADD_STUB(new_drivers[in_driver_count], get_def_binds); + CHECK_ADD_STUB(new_drivers[in_driver_count], clean_binds); + CHECK_ADD_STUB(new_drivers[in_driver_count], get_config); + CHECK_ADD_STUB(new_drivers[in_driver_count], set_config); + CHECK_ADD_STUB(new_drivers[in_driver_count], update_keycode); + CHECK_ADD_STUB(new_drivers[in_driver_count], menu_translate); + CHECK_ADD_STUB(new_drivers[in_driver_count], get_key_code); + CHECK_ADD_STUB(new_drivers[in_driver_count], get_key_name); + in_drivers = new_drivers; + in_driver_count = count_new; + + return 0; +} + +void in_init(void) +{ + in_drivers = NULL; + memset(in_devices, 0, sizeof(in_devices)); + in_driver_count = 0; + in_dev_count = 0; } #if 0 diff --git a/common/input.h b/common/input.h index 6ea14bd..65e088f 100644 --- a/common/input.h +++ b/common/input.h @@ -49,14 +49,6 @@ #define PEV_MASK 0x7ff80000 -enum { - IN_DRVID_UNKNOWN = 0, - IN_DRVID_GP2X, - IN_DRVID_EVDEV, - IN_DRVID_VK, - IN_DRVID_COUNT, -}; - enum { IN_CFG_BIND_COUNT = 0, IN_CFG_DOES_COMBOS, @@ -79,13 +71,13 @@ typedef struct { const char *prefix; void (*probe)(void); void (*free)(void *drv_data); - int (*get_bind_count)(void); const char * const * (*get_key_names)(int *count); void (*get_def_binds)(int *binds); int (*clean_binds)(void *drv_data, int *binds, int *def_finds); int (*get_config)(void *drv_data, int what, int *val); int (*set_config)(void *drv_data, int what, int val); + int (*update)(void *drv_data, const int *binds, int *result); /* return -1 on no event, -2 on error */ int (*update_keycode)(void *drv_data, int *is_down); int (*menu_translate)(void *drv_data, int keycode); @@ -100,7 +92,8 @@ struct in_default_bind { }; /* to be called by drivers */ -void in_register(const char *nname, int drv_id, int drv_fd_hnd, void *drv_data, +int in_register_driver(const in_drv_t *drv); +void in_register(const char *nname, int drv_fd_hnd, void *drv_data, int key_count, const char * const *key_names, int combos); void in_combos_find(const int *binds, int last_key, int *combo_keys, int *combo_acts); int in_combos_do(int keys, const int *binds, int last_key, int combo_keys, int combo_acts); -- 2.39.5