From 6625cda9f3eea899c41d1ee5b14913e691075e8a Mon Sep 17 00:00:00 2001 From: Amir Hammad Date: Wed, 8 Apr 2015 08:04:27 +0200 Subject: Device drivers: Refactor: Move driver declaration to the end of file forward declarations are not needed Signed-off-by: Amir Hammad --- src/usbh_driver_gp_xbox.c | 43 +++++++++++++++++++------------------------ src/usbh_driver_hid_mouse.c | 43 +++++++++++++++++++------------------------ src/usbh_driver_hub.c | 43 +++++++++++++++++++------------------------ 3 files changed, 57 insertions(+), 72 deletions(-) diff --git a/src/usbh_driver_gp_xbox.c b/src/usbh_driver_gp_xbox.c index 9115f0b..f742246 100644 --- a/src/usbh_driver_gp_xbox.c +++ b/src/usbh_driver_gp_xbox.c @@ -28,30 +28,6 @@ #include #include -static void *gp_xbox_init(void *usbh_dev); -static bool gp_xbox_analyze_descriptor(void *drvdata, void *descriptor); -static void gp_xbox_poll(void *drvdata, uint32_t time_curr_us); -static void gp_xbox_remove(void *drvdata); - -static const usbh_dev_driver_info_t usbh_gp_xbox_driver_info = { - .deviceClass = 0xff, - .deviceSubClass = 0xff, - .deviceProtocol = 0xff, - .idVendor = 0x045e, - .idProduct = 0x028e, - .ifaceClass = 0xff, - .ifaceSubClass = 93, - .ifaceProtocol = 0x01 -}; - -const usbh_dev_driver_t usbh_gp_xbox_driver = { - .init = gp_xbox_init, - .analyze_descriptor = gp_xbox_analyze_descriptor, - .poll = gp_xbox_poll, - .remove = gp_xbox_remove, - .info = &usbh_gp_xbox_driver_info -}; - enum STATES { STATE_INACTIVE, STATE_READING_COMPLETE, @@ -419,3 +395,22 @@ static void gp_xbox_remove(void *drvdata) gp_xbox->state_next = STATE_INACTIVE; gp_xbox->endpoint_in_address = 0; } + +static const usbh_dev_driver_info_t usbh_gp_xbox_driver_info = { + .deviceClass = 0xff, + .deviceSubClass = 0xff, + .deviceProtocol = 0xff, + .idVendor = 0x045e, + .idProduct = 0x028e, + .ifaceClass = 0xff, + .ifaceSubClass = 93, + .ifaceProtocol = 0x01 +}; + +const usbh_dev_driver_t usbh_gp_xbox_driver = { + .init = gp_xbox_init, + .analyze_descriptor = gp_xbox_analyze_descriptor, + .poll = gp_xbox_poll, + .remove = gp_xbox_remove, + .info = &usbh_gp_xbox_driver_info +}; diff --git a/src/usbh_driver_hid_mouse.c b/src/usbh_driver_hid_mouse.c index 782cac5..0435138 100644 --- a/src/usbh_driver_hid_mouse.c +++ b/src/usbh_driver_hid_mouse.c @@ -27,30 +27,6 @@ #include -static void *mouse_init(void *usbh_dev); -static bool mouse_analyze_descriptor(void *drvdata, void *descriptor); -static void mouse_poll(void *drvdata, uint32_t time_curr_us); -static void mouse_remove(void *drvdata); - -static const usbh_dev_driver_info_t usbh_hid_mouse_driver_info = { - .deviceClass = -1, - .deviceSubClass = -1, - .deviceProtocol = -1, - .idVendor = -1, - .idProduct = -1, - .ifaceClass = 0x03, - .ifaceSubClass = -1, - .ifaceProtocol = 0x02 -}; - -const usbh_dev_driver_t usbh_hid_mouse_driver = { - .init = mouse_init, - .analyze_descriptor = mouse_analyze_descriptor, - .poll = mouse_poll, - .remove = mouse_remove, - .info = &usbh_hid_mouse_driver_info -}; - enum STATES { STATE_INACTIVE, STATE_READING_COMPLETE, @@ -292,3 +268,22 @@ static void mouse_remove(void *drvdata) mouse->state_next = STATE_INACTIVE; mouse->endpoint_in_address = 0; } + +static const usbh_dev_driver_info_t usbh_hid_mouse_driver_info = { + .deviceClass = -1, + .deviceSubClass = -1, + .deviceProtocol = -1, + .idVendor = -1, + .idProduct = -1, + .ifaceClass = 0x03, + .ifaceSubClass = -1, + .ifaceProtocol = 0x02 +}; + +const usbh_dev_driver_t usbh_hid_mouse_driver = { + .init = mouse_init, + .analyze_descriptor = mouse_analyze_descriptor, + .poll = mouse_poll, + .remove = mouse_remove, + .info = &usbh_hid_mouse_driver_info +}; diff --git a/src/usbh_driver_hub.c b/src/usbh_driver_hub.c index c8c7ee9..fab946d 100644 --- a/src/usbh_driver_hub.c +++ b/src/usbh_driver_hub.c @@ -29,31 +29,7 @@ static hub_device_t hub_device[USBH_MAX_HUBS]; - -static void *hub_init(void *usbh_dev); -static bool hub_analyze_descriptor(void *drvdata, void *descriptor); -static void hub_poll(void *drvdata, uint32_t time_curr_us); static void event(usbh_device_t *dev, usbh_packet_callback_data_t cb_data); -static void hub_remove(void *drvdata); - -static const usbh_dev_driver_info_t usbh_hub_driver_info = { - .deviceClass = 0x09, - .deviceSubClass = -1, - .deviceProtocol = -1, - .idVendor = -1, - .idProduct = -1, - .ifaceClass = 0x09, - .ifaceSubClass = -1, - .ifaceProtocol = -1 -}; - -const usbh_dev_driver_t usbh_hub_driver = { - .init = hub_init, - .analyze_descriptor = hub_analyze_descriptor, - .poll = hub_poll, - .remove = hub_remove, - .info = &usbh_hub_driver_info -}; @@ -863,3 +839,22 @@ static void hub_remove(void *drvdata) } } + +static const usbh_dev_driver_info_t usbh_hub_driver_info = { + .deviceClass = 0x09, + .deviceSubClass = -1, + .deviceProtocol = -1, + .idVendor = -1, + .idProduct = -1, + .ifaceClass = 0x09, + .ifaceSubClass = -1, + .ifaceProtocol = -1 +}; + +const usbh_dev_driver_t usbh_hub_driver = { + .init = hub_init, + .analyze_descriptor = hub_analyze_descriptor, + .poll = hub_poll, + .remove = hub_remove, + .info = &usbh_hub_driver_info +}; -- cgit