aboutsummaryrefslogtreecommitdiff
path: root/platform/include
diff options
context:
space:
mode:
authorTTornblom <thomas.tornblom@iar.com>2020-03-04 17:56:27 +0100
committerTTornblom <thomas.tornblom@iar.com>2020-04-23 13:06:20 +0200
commitfaf74f575a7edf7410ade35ceb79cd7a63b3dc02 (patch)
tree102cd919fdd3e90712b47486a68dfe72004309c7 /platform/include
parentb9e5ed000321d757b7ef2e59d2fabc8af60f0a53 (diff)
downloadtrusted-firmware-m-faf74f575a7edf7410ade35ceb79cd7a63b3dc02.tar.gz
Build: Major warnings cleanup
This is a major type-mismatch cleanup and warnings elimination. The IAR toolchain runs with more warnings enabled than ARMCLANG or GNUARM, which has resulted in this effort. This should make it easier to enable "pedantic" mode also for GNUARM. There are still a few warnings about jumping past variable initialization and falling off the end of non-void functions, but there are comments in some of these parts that implies that this is done to catch programming errors, like having case labels for all possible enum values. Reordered initializer in tfm_attestation_req_mngr.c to eliminate CI warning. Disabled "Pe940" warnings in CommonConfig.cmake to suppress warnings about inline assembly functions declared to return value that doesn't declare a return value in the __ASM() statement. Disabled "Pe546" warnings to supress warnings about jumping over unused initializers. Fixed what appears to be a copy/paste bug in tfm_ss_core_test.c. Removed unused variable "ret" in Driver_PPC.c for AN519 and AN521, to make it similar to AN524. Signed-off-by: TTornblom <thomas.tornblom@iar.com> Change-Id: I2b729c73e4b004cff6b0530cc1350fcf900e4272
Diffstat (limited to 'platform/include')
-rw-r--r--platform/include/tfm_spm_hal.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/platform/include/tfm_spm_hal.h b/platform/include/tfm_spm_hal.h
index 3607753630..caed249148 100644
--- a/platform/include/tfm_spm_hal.h
+++ b/platform/include/tfm_spm_hal.h
@@ -221,7 +221,7 @@ uint32_t tfm_spm_hal_get_ns_entry_point(void);
*
* \return Returns values as specified by the \ref tfm_plat_err_t
*/
-enum tfm_plat_err_t tfm_spm_hal_set_secure_irq_priority(int32_t irq_line,
+enum tfm_plat_err_t tfm_spm_hal_set_secure_irq_priority(IRQn_Type irq_line,
uint32_t priority);
/**
@@ -229,21 +229,21 @@ enum tfm_plat_err_t tfm_spm_hal_set_secure_irq_priority(int32_t irq_line,
*
* \param[in] irq_line The IRQ to clear pending for.
*/
-void tfm_spm_hal_clear_pending_irq(int32_t irq_line);
+void tfm_spm_hal_clear_pending_irq(IRQn_Type irq_line);
/**
* \brief Enables an IRQ
*
* \param[in] irq_line The IRQ to be enabled.
*/
-void tfm_spm_hal_enable_irq(int32_t irq_line);
+void tfm_spm_hal_enable_irq(IRQn_Type irq_line);
/**
* \brief Disables an IRQ
*
* \param[in] irq_line The IRQ to be disabled
*/
-void tfm_spm_hal_disable_irq(int32_t irq_line);
+void tfm_spm_hal_disable_irq(IRQn_Type irq_line);
/**
* \brief Set the target state of an IRQ
@@ -257,7 +257,7 @@ void tfm_spm_hal_disable_irq(int32_t irq_line);
* assigned to Non-Secure
*/
enum irq_target_state_t tfm_spm_hal_set_irq_target_state(
- int32_t irq_line,
+ IRQn_Type irq_line,
enum irq_target_state_t target_state);
#ifdef TFM_MULTI_CORE_TOPOLOGY