Conflict with "fallthrough" define

Hi,

We are using OP-TEE together with a library that provides its own "fallthrough" define, similar to what OP-TEE does in lib/libutils/ext/include/compiler.h:

#define fallthrough __attribute__((__fallthrough__))

However, this other library checks for fallthrough attribute support by first doing:

#if __has_cpp_attribute(fallthrough) ... #endif

which fails to compile due to the OP-TEE define.

Would OP-TEE accept a patch to rename the "fallthrough" define to something like "optee_fallthrough", "FALLTHROUGH", <other suggestion> ?

Or should we upstream a patch to the library to do:

#if __has_cpp_attribute(__fallthrough__) ... #endif

(which is what OP-TEE currently does)

Regards Jacob