Remove unnecessary HID functionality (#243)

* Include HID report functionality only when the HID feature is enabled
* Define HID_CONTROLS=1 so HID unit tests work correctly
This commit is contained in:
Michael Banther
2022-02-21 13:20:52 +00:00
committed by GitHub
parent 29c9b3ea75
commit 769bfa1a5a
4 changed files with 11 additions and 0 deletions

View File

@@ -1,6 +1,11 @@
lib_xua Change Log lib_xua Change Log
================== ==================
UNRELEASED
----------
* CHANGED: Exclude HID Report functions unless the HID feature is enabled
3.1.0 3.1.0
----- -----

View File

@@ -1,5 +1,7 @@
// Copyright 2021-2022 XMOS LIMITED. // Copyright 2021-2022 XMOS LIMITED.
// This Software is subject to the terms of the XMOS Public Licence: Version 1. // This Software is subject to the terms of the XMOS Public Licence: Version 1.
#if( 0 < HID_CONTROLS )
#include <assert.h> #include <assert.h>
#include <stddef.h> #include <stddef.h>
#include <stdio.h> #include <stdio.h>
@@ -851,3 +853,5 @@ unsigned hidReportValidate( void )
return hidReportValidateInfoStruct( &info ); return hidReportValidateInfoStruct( &info );
} }
} }
#endif // ( 0 < HID_CONTROLS )

View File

@@ -49,6 +49,7 @@ foreach( testsourcefile ${APP_SOURCES} )
"-fxscope" "-fxscope"
"-target=XCORE-AI-EXPLORER" "-target=XCORE-AI-EXPLORER"
"${CMAKE_CURRENT_SOURCE_DIR}/config.xscope" "${CMAKE_CURRENT_SOURCE_DIR}/config.xscope"
"-DHID_CONTROLS=1"
"-DUNITY_SUPPORT_64" "-DUNITY_SUPPORT_64"
"-DUNITY_INCLUDE_DOUBLE" "-DUNITY_INCLUDE_DOUBLE"
) )

View File

@@ -230,6 +230,7 @@ def build(bld):
makefile_opts['XCC_FLAGS'] = ['-O2', makefile_opts['XCC_FLAGS'] = ['-O2',
'-g', '-g',
'-Wall', '-Wall',
'-DHID_CONTROLS=1',
'-DUNITY_SUPPORT_64', '-DUNITY_SUPPORT_64',
'-DUNITY_INCLUDE_DOUBLE', '-DUNITY_INCLUDE_DOUBLE',
'-DXUD_CORE_CLOCK=600', '-DXUD_CORE_CLOCK=600',