Changes in typedefs.checkpatch

f6ec3329a9a8eb7fc77a0408dc31d9d6e1e813f6 - Add STAILQ_HEAD() and STAILQ_ENTRY() to typedefs.checkpatch

Mon, 03 Oct 2022 08:45:38 +0000

Add STAILQ_HEAD() and STAILQ_ENTRY() to typedefs.checkpatchWhen using BSD queues from , a singly linked tail queueentry is declared with a macro:STAILQ_ENTRY(type) var;This makes checkpatch.pl unhappy because the type is unknown:WARNING: Missing a blank line after declarations+ struct file *file;+ STAILQ_ENTRY(load_seg) link;Checkpatch has the same problem with the macro STAILQ_HEAD().This patch adds a regular expression to typedefs.checkpatch thatmatches the macro part, thus fixing the warning.Signed-off-by: Jorge Ramirez-Ortiz Reviewed-by: Jerome Forissier Reviewed-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

9d224046e71083e0fe3e8c26c22f729399c2c950 - ldelf, libutee: add minimal Thread Local Storage support

Tue, 21 Jul 2020 08:55:09 +0000

ldelf, libutee: add minimal Thread Local Storage supportPreparing for C++ support in TAs.Adds enough runtime Thread Local Storage (TLS) support for the GNU C++compilers (arm-linux-gnueabihf-g++, aarch64-linux-gnu-g++) to work withOP-TEE. That is: - A Thread Control Block, - The __tls_get_addr() and dl_iterate_phdr() functions.Note that __tls_get_addr() is an ABI helper so it has no prototype in auser-accessible header file. dl_iterate_phdr() however is defined in and may be used in a TA. The file lib/libutee/include/link.h isborrowed from Android's Bionic [1] with minor changes (added therequired #include statement and named the function parameters). Asimilar header is provided by other C libraries such as GNUlibc, musl and FreeBSD/NetBSD/OpenBSD.Link: [1] https://android.googlesource.com/platform/bionic/+/master/libc/include/link.hSigned-off-by: Jerome Forissier Acked-by: Etienne Carriere Reviewed-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

44bdcee6e23f6e8a27188a55cba3e883d7f3a883 - Add SLIST_HEAD() and SLIST_ENTRY() to typedefs.checkpatch

Mon, 03 Jun 2019 11:00:11 +0000

Add SLIST_HEAD() and SLIST_ENTRY() to typedefs.checkpatchWhen using BSD queues from , a simple list entry is declaredwith a macro:SLIST_ENTRY(type) var;This makes checkpatch.pl unhappy because the type is unknown:WARNING: Missing a blank line after declarations+ struct file *file;+ SLIST_ENTRY(load_seg) link;Checkpatch has the same problem with the macro SLIST_HEAD().This patch adds a regular expression to typedefs.chackpatch thatmatches the macro part, thus fixing the warning.Acked-by: Etienne Carriere Signed-off-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

7decbaae19146fafc0a6284463d2292932d29920 - Add TAILQ_ENTRY() to typedefs.checkpatch

Fri, 26 Apr 2019 18:03:00 +0000

Add TAILQ_ENTRY() to typedefs.checkpatchWhen using BSD queues from , a queue entry is declaredwith a macro: TAILQ_ENTRY(type) var;This makes checkpatch.pl unhappy because the type is unknown: WARNING: Missing a blank line after declarations #52: FILE: core/arch/arm/include/mm/tee_pager.h:32: + struct pgt *pgt; + TAILQ_ENTRY(tee_pager_area) link;This patch adds a regular expression to typedefs.chackpatch thatmatches the macro part, thus fixing the warning.Signed-off-by: Jerome Forissier Reviewed-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

b37d3240bdab60a8833dabf3b920a849af057c37 - Add mbed TLS types to typedefs.checkpatch

Fri, 15 Mar 2019 08:35:45 +0000

Add mbed TLS types to typedefs.checkpatchAdds a couple of mbed TLS types to typedefs.checkpatch to avoid thefollowing warning: WARNING: Missing a blank line after declarations #100: FILE: lib/libutee/tee_api_arith_mpi.c:105: + const struct bigint_hdr *hdr = (struct bigint_hdr *)bigInt; + const mbedtls_mpi_uint *p = (const mbedtls_mpi_uint *)(hdr + 1);Signed-off-by: Jerome Forissier Reviewed-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

94db01ef448d1e552161c2d861d57a5f8bda0cc0 - Update typedefs.checkpatch

Mon, 03 Dec 2018 12:36:32 +0000

Update typedefs.checkpatchAdds missing typedefs from lib/libutee/include/tee_api_types.h totypedefs.checkpatch. This fixes checkpatch errors such as: ERROR: need consistent spacing around '*' (ctx:WxV) #807: FILE: lib/libutee/tee_api_arith_mpi.c:773: +void TEE_BigIntInitFMMContext(TEE_BigIntFMMContext *context __unused, ^The file is sorted in reverse alphabetical order, otherwise checkpatchwould ignore some entries that have a common radix (such as TEE_BigInt/TEE_BigIntFMM/TEE_BigIntFMMContext). Not sure if it is expected or if itwill be fixed in upstream checkpatch at some point.Signed-off-by: Jerome Forissier Acked-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch

1472c996cca6291b6cbfca2115e9d307759fb43b - Take some GP types into account when running patch check tool

Wed, 10 May 2017 08:41:48 +0000

Take some GP types into account when running patch check toolWhen running checkpatch on the OP-TEE code, for instance during theTravis checks, it happens that we get false warnings and errors causedby the use of GlobalPlatform typedefs. In the following example, thepatch introduces pointers to functions returning TEE_Result, butcheckpatch won't accept it as a valid type: $ ./scripts/checkpatch.sh d776721 | tail -1 total: 1 errors, 5 warnings, 555 lines checkedTo address such issues, declare our custom types in a local file(typedefs.checkpatch) and use the new --typedefsfile option provided bycheckpatch.pl.Signed-off-by: Jerome Forissier Reviewed-by: Jens Wiklander List of files: /optee_os/typedefs.checkpatch