diff options
| author | CQ Xiao <[email protected]> | 2023-12-01 09:53:45 +0800 |
|---|---|---|
| committer | GitHub <[email protected]> | 2023-12-01 09:53:45 +0800 |
| commit | f856e41e7fbc4620e12faba04c97c2da6abc10fc (patch) | |
| tree | 6a81fe0124ce87ad70703292481b1c83b245542e | |
| parent | 6ed7092b772c7363132e3d5332c0d7b0817708cf (diff) | |
Fixed standalone device HID issue and enable all tests by default. (#129)
* Fixed device hid receiver save position issue.
* Enable all tests by default.
| -rw-r--r-- | .github/workflows/regression_test.yml | 6 | ||||
| -rw-r--r-- | common/usbx_device_classes/src/ux_device_class_hid_receiver_tasks_run.c | 9 |
2 files changed, 10 insertions, 5 deletions
diff --git a/.github/workflows/regression_test.yml b/.github/workflows/regression_test.yml index e725c77..57f936b 100644 --- a/.github/workflows/regression_test.yml +++ b/.github/workflows/regression_test.yml @@ -8,7 +8,7 @@ on: tests_to_run: description: 'all, single or multiple of default_build_coverage error_check_build_full_coverage tracex_enable_build device_buffer_owner_build device_zero_copy_build nofx_build_coverage optimized_build standalone_device_build_coverage standalone_device_buffer_owner_build standalone_device_zero_copy_build standalone_host_build_coverage standalone_build_coverage generic_build otg_support_build memory_management_build_coverage msrc_rtos_build msrc_standalone_build' required: false - default: 'default_build_coverage' + default: 'all' skip_coverage: required: false type: boolean @@ -55,7 +55,7 @@ jobs: uses: azure-rtos/threadx/.github/workflows/regression_template.yml@master with: cmake_path: ./test/cmake/usbx - build_script: ./scripts/build.sh default_build_coverage - test_script: ./scripts/test.sh default_build_coverage + build_script: ./scripts/build.sh all + test_script: ./scripts/test.sh all coverage_name: default_build_coverage skip_coverage: false diff --git a/common/usbx_device_classes/src/ux_device_class_hid_receiver_tasks_run.c b/common/usbx_device_classes/src/ux_device_class_hid_receiver_tasks_run.c index a90e3cf..9531efe 100644 --- a/common/usbx_device_classes/src/ux_device_class_hid_receiver_tasks_run.c +++ b/common/usbx_device_classes/src/ux_device_class_hid_receiver_tasks_run.c @@ -35,7 +35,7 @@ /* FUNCTION RELEASE */ /* */ /* _ux_device_class_hid_receiver_tasks_run PORTABLE C */ -/* 6.3.0 */ +/* 6.x */ /* AUTHOR */ /* */ /* Chaoqiong Xiao, Microsoft Corporation */ @@ -70,6 +70,9 @@ /* 10-31-2023 Chaoqiong Xiao Modified comment(s), */ /* added zero copy support, */ /* resulting in version 6.3.0 */ +/* xx-xx-xxxx Chaoqiong Xiao Modified comment(s), */ +/* fixed save position issue, */ +/* resulting in version 6.x */ /* */ /**************************************************************************/ UINT _ux_device_class_hid_receiver_tasks_run(UX_SLAVE_CLASS_HID *hid) @@ -165,6 +168,9 @@ ULONG temp; return(UX_STATE_NEXT); } + /* Get current save position. */ + pos = receiver -> ux_device_class_hid_receiver_event_save_pos; + #if (UX_DEVICE_ENDPOINT_BUFFER_OWNER == 1) && defined(UX_DEVICE_CLASS_HID_ZERO_COPY) /* Save received event length. */ @@ -172,7 +178,6 @@ ULONG temp; #else /* Save received event data and length. */ - pos = receiver -> ux_device_class_hid_receiver_event_save_pos; buffer = (UCHAR *)&pos -> ux_device_class_hid_received_event_data; temp = transfer -> ux_slave_transfer_request_actual_length; _ux_utility_memory_copy(buffer, |