diff options
| author | Yann Gautier <yann.gautier@st.com> | 2026-05-06 17:29:12 +0200 |
|---|---|---|
| committer | Yann Gautier <yann.gautier@st.com> | 2026-06-23 18:26:38 +0200 |
| commit | c02a89b2d330b93d7df1b8b0afb5cd73c39bd285 (patch) | |
| tree | c5751e432f04dd4eea807a64b418235b0616a9f9 | |
| parent | c69734c36f503f99b3b8ba7453891f26404debad (diff) | |
| download | arm-trusted-firmware-c02a89b2d330b93d7df1b8b0afb5cd73c39bd285.tar.gz arm-trusted-firmware-c02a89b2d330b93d7df1b8b0afb5cd73c39bd285.zip | |
fix(io): use simple braces for uuid_null definition
When enabling -Wmissing-field-initializers macro (with W=1), there is
a warning:
drivers/io/io_fip.c:312:40: warning: missing field 'time_mid'
initializer [-Wmissing-field-initializers]
312 | static const uuid_t uuid_null = { {0} }; /* Double
braces for clang */
| ^
To avoid that, use simple braces. The compilation with clang is still
working with this change (tested with LLVM-21.1.5).
While at it, as all fields of uuid structure are unsigned, use 0U.
Change-Id: Ib46fe057c308b796bff6ef4dbfcf9854b9703fc1
Signed-off-by: Yann Gautier <yann.gautier@st.com>
| -rw-r--r-- | drivers/io/io_fip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/io/io_fip.c b/drivers/io/io_fip.c index dd7800507..f06e98a2f 100644 --- a/drivers/io/io_fip.c +++ b/drivers/io/io_fip.c @@ -309,7 +309,7 @@ static int fip_file_open(io_dev_info_t *dev_info, const uintptr_t spec, int result; uintptr_t backend_handle; const io_uuid_spec_t *uuid_spec = (io_uuid_spec_t *)spec; - static const uuid_t uuid_null = { {0} }; /* Double braces for clang */ + static const uuid_t uuid_null = { 0U }; size_t bytes_read; size_t toc_offset; size_t fip_size; |
