espressif/esp-idf
1
0
Fork 1
mirror of https://github.com/espressif/esp-idf.git synced 2025-07-30 18:57:19 +02:00
Code Issues Packages Projects Releases Wiki Activity

usb: Remove -Wno-format compile option

Browse Source
This commit is contained in:
Tomas Rezucha
2022-08-09 11:58:59 +02:00
parent f30eab33a1
commit c8585267ab
4 changed files with 5 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
components/usb/CMakeLists.txt
View File

@ -23,7 +23,3 @@ idf_component_register(SRCS ${srcs}
PRIV_INCLUDE_DIRS ${priv_include} PRIV_INCLUDE_DIRS ${priv_include}
PRIV_REQUIRES ${priv_require} PRIV_REQUIRES ${priv_require}
) )
if(CONFIG_USB_OTG_SUPPORTED)
target_compile_options(${COMPONENT_LIB} PRIVATE "-Wno-format")
endif()

1
components/usb/test/CMakeLists.txt
View File

@ -10,4 +10,3 @@ idf_component_register(
PRIV_INCLUDE_DIRS "../private_include" "common" "hcd" "usb_host" PRIV_INCLUDE_DIRS "../private_include" "common" "hcd" "usb_host"
PRIV_REQUIRES cmock usb test_utils PRIV_REQUIRES cmock usb test_utils
) )
target_compile_options(${COMPONENT_LIB} PRIVATE "-Wno-format")

8
components/usb/test/common/test_usb_mock_classes.c
View File

@ -4,7 +4,7 @@
* SPDX-License-Identifier: Apache-2.0 * SPDX-License-Identifier: Apache-2.0
*/ */
#include <stdint.h> #include <inttypes.h>
#include <stdbool.h> #include <stdbool.h>
#include <stdio.h> #include <stdio.h>
#include "usb/usb_types_ch9.h" #include "usb/usb_types_ch9.h"
@ -83,15 +83,15 @@ bool mock_msc_scsi_check_csw(mock_msc_bulk_csw_t *csw, uint32_t tag_expect)
bool no_issues = true; bool no_issues = true;
if (csw->dCSWSignature != 0x53425355) { if (csw->dCSWSignature != 0x53425355) {
no_issues = false; no_issues = false;
printf("Warning: csw signature corrupt (0x%X)\n", csw->dCSWSignature); printf("Warning: csw signature corrupt (0x%"PRIX32")\n", csw->dCSWSignature);
} }
if (csw->dCSWTag != tag_expect) { if (csw->dCSWTag != tag_expect) {
no_issues = false; no_issues = false;
printf("Warning: csw tag unexpected! Expected %d got %d\n", tag_expect, csw->dCSWTag); printf("Warning: csw tag unexpected! Expected %"PRIu32" got %"PRIu32"\n", tag_expect, csw->dCSWTag);
} }
if (csw->dCSWDataResidue) { if (csw->dCSWDataResidue) {
no_issues = false; no_issues = false;
printf("Warning: csw indicates data residue of %d bytes!\n", csw->dCSWDataResidue); printf("Warning: csw indicates data residue of %"PRIu32" bytes!\n", csw->dCSWDataResidue);
} }
if (csw->bCSWStatus) { if (csw->bCSWStatus) {
no_issues = false; no_issues = false;

2
components/usb/usbh.c
View File

@ -407,7 +407,7 @@ esp_err_t usbh_process(void)
Exit critical section to handle device action flags in their listed order Exit critical section to handle device action flags in their listed order
--------------------------------------------------------------------- */ --------------------------------------------------------------------- */
USBH_EXIT_CRITICAL(); USBH_EXIT_CRITICAL();
ESP_LOGD(USBH_TAG, "Processing actions 0x%x", action_flags); ESP_LOGD(USBH_TAG, "Processing actions 0x%"PRIx32"", action_flags);
//Sanity check. If the device is being freed, there must not be any other action flags set //Sanity check. If the device is being freed, there must not be any other action flags set
assert(!(action_flags & DEV_FLAG_ACTION_FREE) || action_flags == DEV_FLAG_ACTION_FREE); assert(!(action_flags & DEV_FLAG_ACTION_FREE) || action_flags == DEV_FLAG_ACTION_FREE);