Commit 354305c3 authored by Antonio Nino Diaz's avatar Antonio Nino Diaz
Browse files

Fix MISRA defects in SP805 driver


Fix violations of MISRA C-2012 Rules 10.1, 10.3 and 10.4.

Change-Id: I13c6acda798c1666892f630f097a23e68748f9e4
Signed-off-by: default avatarAntonio Nino Diaz <antonio.ninodiaz@arm.com>
parent e7b9886c
Showing with 23 additions and 21 deletions
+23 -21
/* /*
* Copyright (c) 2015, ARM Limited and Contributors. All rights reserved. * Copyright (c) 2015-2018, ARM Limited and Contributors. All rights reserved.
* *
* SPDX-License-Identifier: BSD-3-Clause * SPDX-License-Identifier: BSD-3-Clause
*/ */
...@@ -10,17 +10,17 @@ ...@@ -10,17 +10,17 @@
/* Inline register access functions */ /* Inline register access functions */
static inline void sp805_write_wdog_load(uintptr_t base, unsigned long value) static inline void sp805_write_wdog_load(uintptr_t base, uint32_t value)
{ {
mmio_write_32(base + SP805_WDOG_LOAD_OFF, value); mmio_write_32(base + SP805_WDOG_LOAD_OFF, value);
} }
static inline void sp805_write_wdog_ctrl(uintptr_t base, unsigned long value) static inline void sp805_write_wdog_ctrl(uintptr_t base, uint32_t value)
{ {
mmio_write_32(base + SP805_WDOG_CTR_OFF, value); mmio_write_32(base + SP805_WDOG_CTR_OFF, value);
} }
static inline void sp805_write_wdog_lock(uintptr_t base, unsigned long value) static inline void sp805_write_wdog_lock(uintptr_t base, uint32_t value)
{ {
mmio_write_32(base + SP805_WDOG_LOCK_OFF, value); mmio_write_32(base + SP805_WDOG_LOCK_OFF, value);
} }
...@@ -28,23 +28,23 @@ static inline void sp805_write_wdog_lock(uintptr_t base, unsigned long value) ...@@ -28,23 +28,23 @@ static inline void sp805_write_wdog_lock(uintptr_t base, unsigned long value)
/* Public API implementation */ /* Public API implementation */
void sp805_start(uintptr_t base, unsigned long ticks) void sp805_start(uintptr_t base, unsigned int ticks)
{ {
sp805_write_wdog_load(base, ticks); sp805_write_wdog_load(base, ticks);
sp805_write_wdog_ctrl(base, SP805_CTR_RESEN | SP805_CTR_INTEN); sp805_write_wdog_ctrl(base, SP805_CTR_RESEN | SP805_CTR_INTEN);
/* Lock registers access */ /* Lock registers access */
sp805_write_wdog_lock(base, 0); sp805_write_wdog_lock(base, 0U);
} }
void sp805_stop(uintptr_t base) void sp805_stop(uintptr_t base)
{ {
sp805_write_wdog_lock(base, WDOG_UNLOCK_KEY); sp805_write_wdog_lock(base, WDOG_UNLOCK_KEY);
sp805_write_wdog_ctrl(base, 0); sp805_write_wdog_ctrl(base, 0U);
} }
void sp805_refresh(uintptr_t base, unsigned long ticks) void sp805_refresh(uintptr_t base, unsigned int ticks)
{ {
sp805_write_wdog_lock(base, WDOG_UNLOCK_KEY); sp805_write_wdog_lock(base, WDOG_UNLOCK_KEY);
sp805_write_wdog_load(base, ticks); sp805_write_wdog_load(base, ticks);
sp805_write_wdog_lock(base, 0); sp805_write_wdog_lock(base, 0U);
} }
/* /*
* Copyright (c) 2015, ARM Limited and Contributors. All rights reserved. * Copyright (c) 2015-2018, ARM Limited and Contributors. All rights reserved.
* *
* SPDX-License-Identifier: BSD-3-Clause * SPDX-License-Identifier: BSD-3-Clause
*/ */
#ifndef __SP805_H__ #ifndef SP805_H
#define __SP805_H__ #define SP805_H
#include <utils_def.h>
/* SP805 register offset */ /* SP805 register offset */
#define SP805_WDOG_LOAD_OFF 0x000 #define SP805_WDOG_LOAD_OFF UL(0x000)
#define SP805_WDOG_CTR_OFF 0x008 #define SP805_WDOG_CTR_OFF UL(0x008)
#define SP805_WDOG_LOCK_OFF 0xc00 #define SP805_WDOG_LOCK_OFF UL(0xc00)
/* Magic word to unlock the wd registers */ /* Magic word to unlock the wd registers */
#define WDOG_UNLOCK_KEY 0x1ACCE551 #define WDOG_UNLOCK_KEY U(0x1ACCE551)
/* Register field definitions */ /* Register field definitions */
#define SP805_CTR_RESEN (1 << 1) #define SP805_CTR_RESEN (U(1) << 1)
#define SP805_CTR_INTEN (1 << 0) #define SP805_CTR_INTEN (U(1) << 0)
#ifndef __ASSEMBLY__ #ifndef __ASSEMBLY__
...@@ -25,10 +27,10 @@ ...@@ -25,10 +27,10 @@
/* Public high level API */ /* Public high level API */
void sp805_start(uintptr_t base, unsigned long ticks); void sp805_start(uintptr_t base, unsigned int ticks);
void sp805_stop(uintptr_t base); void sp805_stop(uintptr_t base);
void sp805_refresh(uintptr_t base, unsigned long ticks); void sp805_refresh(uintptr_t base, unsigned int ticks);
#endif /* __ASSEMBLY__ */ #endif /* __ASSEMBLY__ */
#endif /* __SP805_H__ */ #endif /* SP805_H */
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment