I get lots of warnings about type mismatch for enums.

I have fixed some of them where it seems natural, but I need to discuss the approach for fixing these: --- [ 37%] Building C object test/CMakeFiles/tfm_secure_tests.dir/suites/invert/secure/invert_s_interface_testsuite.o

"Invert with valid buffer", {0} },                                     ^ "C:\Users\thomasto\Projects\tf-m4\trusted-firmware-m\test\suites\invert\secure\invert_s_interface_testsuite.c",20 Warning[Pe188]:           enumerated type mixed with another type

"Invert with invalid buffer", {0} },                                       ^ "C:\Users\thomasto\Projects\tf-m4\trusted-firmware-m\test\suites\invert\secure\invert_s_interface_testsuite.c",22 Warning[Pe188]:           enumerated type mixed with another type ---

The issue is that the "{0}" is the initializer for the enum test_status_t "val" below: --- enum test_status_t {     TEST_PASSED = 0,  /*!< Test has passed */     TEST_FAILED = 1,  /*!< Test has failed */ };

struct test_result_t {     enum test_status_t val;  /*!< Test result \ref test_status_t */     const char *info_msg;    /*!< Information message to show in case of                               *   failure                               */     const char *filename;    /*!< Filename where the failure has occured */     uint32_t    line;        /*!< Line where the failure has occured */ }; ---

I can eliminate the warnings by either casting the "0" to enum test_status_t, or replace the 0 with "TEST_PASSED". Personally I would prefer to avoid having a type cast, but initializing the value to "TEST_PASSED" may seem to indicate that the test has passed even before having been run.

Comments?

/Thomas