summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--tests/test/host/hid/test_hidh_keyboard.c3
-rw-r--r--tests/test/host/hid/test_hidh_mouse.c31
-rw-r--r--tests/test/host/usbh/test_enum_task.c12
-rw-r--r--tinyusb/class/hid.h4
-rw-r--r--tinyusb/class/hid_host.c34
5 files changed, 74 insertions, 10 deletions
diff --git a/tests/test/host/hid/test_hidh_keyboard.c b/tests/test/host/hid/test_hidh_keyboard.c
index a3ef34542..56ba9dfbd 100644
--- a/tests/test/host/hid/test_hidh_keyboard.c
+++ b/tests/test/host/hid/test_hidh_keyboard.c
@@ -60,9 +60,10 @@ tusb_keyboard_report_t sample_key[2] =
};
uint8_t dev_addr;
-tusb_keyboard_report_t report;
hidh_interface_info_t *p_hidh_kbd;
+tusb_keyboard_report_t report;
+
tusb_descriptor_interface_t const *p_kbd_interface_desc = &desc_configuration.keyboard_interface;
tusb_descriptor_endpoint_t const *p_kdb_endpoint_desc = &desc_configuration.keyboard_endpoint;
diff --git a/tests/test/host/hid/test_hidh_mouse.c b/tests/test/host/hid/test_hidh_mouse.c
index e1e0d8fa0..b2086b9ad 100644
--- a/tests/test/host/hid/test_hidh_mouse.c
+++ b/tests/test/host/hid/test_hidh_mouse.c
@@ -46,9 +46,23 @@
#include "descriptor_test.h"
extern hidh_interface_info_t mouse_data[TUSB_CFG_HOST_DEVICE_MAX];
+hidh_interface_info_t *p_hidh_mouse;
+uint8_t dev_addr;
void setUp(void)
{
+ hidh_init();
+
+ dev_addr = RANDOM(TUSB_CFG_HOST_DEVICE_MAX)+1;
+
+ p_hidh_mouse = &mouse_data[dev_addr-1];
+
+ p_hidh_mouse->report_size = sizeof(tusb_mouse_report_t);
+ p_hidh_mouse->pipe_hdl = (pipe_handle_t) {
+ .dev_addr = dev_addr,
+ .xfer_type = TUSB_XFER_INTERRUPT,
+ .index = 1
+ };
}
void tearDown(void)
@@ -63,7 +77,24 @@ void test_mouse_init(void)
TEST_ASSERT_MEM_ZERO(mouse_data, sizeof(hidh_interface_info_t)*TUSB_CFG_HOST_DEVICE_MAX);
}
+void test_mouse_is_supported_fail_unplug(void)
+{
+ tusbh_device_get_state_IgnoreAndReturn(TUSB_DEVICE_STATE_UNPLUG);
+ TEST_ASSERT_FALSE( tusbh_hid_mouse_is_supported(dev_addr) );
+}
+
+void test_mouse_is_supported_fail_not_opened(void)
+{
+ hidh_init();
+ tusbh_device_get_state_IgnoreAndReturn(TUSB_DEVICE_STATE_CONFIGURED);
+ TEST_ASSERT_FALSE( tusbh_hid_mouse_is_supported(dev_addr) );
+}
+void test_mouse_is_supported_ok(void)
+{
+ tusbh_device_get_state_IgnoreAndReturn(TUSB_DEVICE_STATE_CONFIGURED);
+ TEST_ASSERT_TRUE( tusbh_hid_mouse_is_supported(dev_addr) );
+}
diff --git a/tests/test/host/usbh/test_enum_task.c b/tests/test/host/usbh/test_enum_task.c
index f942af74c..d0b757ad7 100644
--- a/tests/test/host/usbh/test_enum_task.c
+++ b/tests/test/host/usbh/test_enum_task.c
@@ -179,7 +179,7 @@ void test_addr0_failed_dev_desc(void)
void test_addr0_failed_set_address(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(1));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
tusbh_device_mount_failed_cb_Expect(TUSB_ERROR_USBH_MOUNT_DEVICE_NOT_RESPOND, NULL);
usbh_enumeration_task();
@@ -191,7 +191,7 @@ void test_addr0_failed_set_address(void)
void test_enum_failed_get_full_dev_desc(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(2));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
hcd_pipe_control_close_ExpectAndReturn(0, TUSB_ERROR_NONE);
osal_semaphore_reset_Expect( usbh_devices[0].control.sem_hdl );
@@ -211,7 +211,7 @@ void test_enum_failed_get_full_dev_desc(void)
void test_enum_failed_get_9byte_config_desc(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(3));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
hcd_pipe_control_close_ExpectAndReturn(0, TUSB_ERROR_NONE);
osal_semaphore_reset_Expect( usbh_devices[0].control.sem_hdl );
hcd_pipe_control_open_ExpectAndReturn(1, desc_device.bMaxPacketSize0, TUSB_ERROR_NONE);
@@ -228,7 +228,7 @@ void test_enum_failed_get_9byte_config_desc(void)
void test_enum_failed_get_full_config_desc(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(4));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
hcd_pipe_control_close_ExpectAndReturn(0, TUSB_ERROR_NONE);
osal_semaphore_reset_Expect( usbh_devices[0].control.sem_hdl );
hcd_pipe_control_open_ExpectAndReturn(1, desc_device.bMaxPacketSize0, TUSB_ERROR_NONE);
@@ -246,7 +246,7 @@ void class_install_expect(void)
void test_enum_parse_config_desc(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(5));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
hcd_pipe_control_close_ExpectAndReturn(0, TUSB_ERROR_NONE);
osal_semaphore_reset_Expect( usbh_devices[0].control.sem_hdl );
hcd_pipe_control_open_ExpectAndReturn(1, desc_device.bMaxPacketSize0, TUSB_ERROR_NONE);
@@ -262,7 +262,7 @@ void test_enum_parse_config_desc(void)
void test_enum_set_configure(void)
{
osal_semaphore_wait_StubWithCallback(semaphore_wait_timeout_stub(6));
- hcd_port_reset_Expect( usbh_devices[0].core_id );
+// hcd_port_reset_Expect( usbh_devices[0].core_id );
hcd_pipe_control_close_ExpectAndReturn(0, TUSB_ERROR_NONE);
osal_semaphore_reset_Expect( usbh_devices[0].control.sem_hdl );
hcd_pipe_control_open_ExpectAndReturn(1, desc_device.bMaxPacketSize0, TUSB_ERROR_NONE);
diff --git a/tinyusb/class/hid.h b/tinyusb/class/hid.h
index 11018b422..cfc203034 100644
--- a/tinyusb/class/hid.h
+++ b/tinyusb/class/hid.h
@@ -90,12 +90,12 @@ typedef ATTR_PREPACKED struct ATTR_PACKED {
*
* Type define for a standard Boot Protocol Mouse report
*/
-typedef ATTR_PREPACKED struct
+typedef ATTR_PACKED_STRUCT(struct)
{
uint8_t buttons; /**< buttons mask for currently pressed buttons in the mouse. */
int8_t x; /**< Current delta x movement of the mouse. */
int8_t y; /**< Current delta y movement on the mouse. */
-} ATTR_PACKED tusb_mouse_report_t;
+} tusb_mouse_report_t;
/**
* \brief Standard HID Boot Protocol Keyboard Report.
diff --git a/tinyusb/class/hid_host.c b/tinyusb/class/hid_host.c
index 61e184db6..e9e12c71b 100644
--- a/tinyusb/class/hid_host.c
+++ b/tinyusb/class/hid_host.c
@@ -94,7 +94,15 @@ tusb_error_t tusbh_hid_keyboard_get_report(uint8_t dev_addr, uint8_t instance_nu
tusb_interface_status_t tusbh_hid_keyboard_status(uint8_t dev_addr, uint8_t instance_num)
{
- return tusbh_device_get_state(dev_addr) ? keyboard_data[dev_addr-1].status : TUSB_INTERFACE_STATUS_INVALID_PARA;
+ switch( tusbh_device_get_state(dev_addr) )
+ {
+ case TUSB_DEVICE_STATE_UNPLUG:
+ case TUSB_DEVICE_STATE_INVALID_PARAMETER:
+ return TUSB_INTERFACE_STATUS_INVALID_PARA;
+
+ default:
+ return keyboard_data[dev_addr-1].status;
+ }
}
//------------- Internal API -------------//
static inline tusb_error_t hidh_keyboard_open(uint8_t dev_addr, tusb_descriptor_endpoint_t const *p_endpoint_desc) ATTR_ALWAYS_INLINE;
@@ -130,6 +138,30 @@ static inline void hidh_keyboard_close(uint8_t dev_addr)
STATIC_ hidh_interface_info_t mouse_data[TUSB_CFG_HOST_DEVICE_MAX]; // does not have addr0, index = dev_address-1
+//------------- Public API -------------//
+bool tusbh_hid_mouse_is_supported(uint8_t dev_addr)
+{
+ return tusbh_device_is_configured(dev_addr) && pipehandle_is_valid(mouse_data[dev_addr-1].pipe_hdl);
+}
+
+tusb_error_t tusbh_hid_mouse_get_report(uint8_t dev_addr, uint8_t instance_num, uint8_t * const report)
+{
+ return TUSB_ERROR_NONE;
+}
+
+tusb_interface_status_t tusbh_hid_mouse_status(uint8_t dev_addr, uint8_t instance_num)
+{
+ /* switch( tusbh_device_get_state(dev_addr) )
+ {
+ case TUSB_DEVICE_STATE_UNPLUG:
+ case TUSB_DEVICE_STATE_INVALID_PARAMETER:
+ return TUSB_INTERFACE_STATUS_INVALID_PARA;
+
+ default:
+ return mouse_data[dev_addr-1].status;
+ }*/
+}
+
//------------- Internal API -------------//
static inline tusb_error_t hidh_mouse_open(uint8_t dev_addr, tusb_descriptor_endpoint_t const *p_endpoint_desc) ATTR_ALWAYS_INLINE;
static inline tusb_error_t hidh_mouse_open(uint8_t dev_addr, tusb_descriptor_endpoint_t const *p_endpoint_desc)
generated by cgit v1.3.1 (git 2.54.0) at 2026-08-18 18:50:02 +0000