From 8325a395ea9b82be7b53ae93e9e05b5ce5b1706e Mon Sep 17 00:00:00 2001 From: Patrik Flykt Date: Thu, 29 Nov 2018 11:16:15 -0800 Subject: [PATCH] soc: Add 'U' to unsigned variable assignments Add 'U' to a value when assigning it to an unsigned variable. MISRA-C rule 7.2 Signed-off-by: Patrik Flykt --- soc/arm/nxp_imx/mcimx7_m4/soc_clk_freq.c | 2 +- soc/x86/intel_quark/quark_se/power.c | 2 +- soc/x86/intel_quark/quark_se/soc.c | 2 +- soc/xtensa/intel_s1000/soc.c | 6 +++--- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/soc/arm/nxp_imx/mcimx7_m4/soc_clk_freq.c b/soc/arm/nxp_imx/mcimx7_m4/soc_clk_freq.c index fb01f0d02cd..18d9cfb9776 100644 --- a/soc/arm/nxp_imx/mcimx7_m4/soc_clk_freq.c +++ b/soc/arm/nxp_imx/mcimx7_m4/soc_clk_freq.c @@ -38,7 +38,7 @@ u32_t get_pwm_clock_freq(PWM_Type *base) switch (root) { case ccmRootmuxPwmOsc24m: - hz = 24000000; + hz = 24000000U; break; case ccmRootmuxPwmSysPllDiv4: hz = CCM_ANALOG_GetSysPllFreq(CCM_ANALOG) >> 2; diff --git a/soc/x86/intel_quark/quark_se/power.c b/soc/x86/intel_quark/quark_se/power.c index 5a4673c598a..016d85f5233 100644 --- a/soc/x86/intel_quark/quark_se/power.c +++ b/soc/x86/intel_quark/quark_se/power.c @@ -84,7 +84,7 @@ void _sys_soc_power_state_post_ops(enum power_states state) { switch (state) { case SYS_POWER_STATE_CPU_LPS_2: - *_REG_TIMER_ICR = 1; + *_REG_TIMER_ICR = 1U; case SYS_POWER_STATE_CPU_LPS_1: case SYS_POWER_STATE_CPU_LPS: __asm__ volatile("sti"); diff --git a/soc/x86/intel_quark/quark_se/soc.c b/soc/x86/intel_quark/quark_se/soc.c index 85d452ca55b..4efaa3854a5 100644 --- a/soc/x86/intel_quark/quark_se/soc.c +++ b/soc/x86/intel_quark/quark_se/soc.c @@ -80,7 +80,7 @@ int _arc_init(struct device *arg) reset_vector = (u32_t *)RESET_VECTOR; LOG_DBG("Reset vector address: %x", *reset_vector); shared_data->arc_start = *reset_vector; - shared_data->flags = 0; + shared_data->flags = 0U; if (!shared_data->arc_start) { /* Reset vector points to NULL => skip ARC init. */ LOG_DBG("Reset vector is NULL, skipping ARC init."); diff --git a/soc/xtensa/intel_s1000/soc.c b/soc/xtensa/intel_s1000/soc.c index f852e6d4d47..e88ee07f95d 100644 --- a/soc/xtensa/intel_s1000/soc.c +++ b/soc/xtensa/intel_s1000/soc.c @@ -205,14 +205,14 @@ static void soc_read_bootstraps(void) switch (bootstrap) { case SOC_S1000_STRAP_REF_CLK_19P2: - ref_clk_freq = 19200000; + ref_clk_freq = 19200000U; break; case SOC_S1000_STRAP_REF_CLK_24P576: - ref_clk_freq = 24576000; + ref_clk_freq = 24576000U; break; case SOC_S1000_STRAP_REF_CLK_38P4: default: - ref_clk_freq = 38400000; + ref_clk_freq = 38400000U; break; } }