[rtems commit] bsp/leon3: Avoid implicit integer conversions

Sebastian Huber sebh at rtems.org
Tue Oct 18 20:21:13 UTC 2016


Module:    rtems
Branch:    master
Commit:    93614125c559d3cc7db8c46f13c14b9e1d220352
Changeset: http://git.rtems.org/rtems/commit/?id=93614125c559d3cc7db8c46f13c14b9e1d220352

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Wed Oct 19 00:21:21 2016 +0200

bsp/leon3: Avoid implicit integer conversions

---

 c/src/lib/libbsp/sparc/leon3/include/leon.h | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/c/src/lib/libbsp/sparc/leon3/include/leon.h b/c/src/lib/libbsp/sparc/leon3/include/leon.h
index b751163..36f9495 100644
--- a/c/src/lib/libbsp/sparc/leon3/include/leon.h
+++ b/c/src/lib/libbsp/sparc/leon3/include/leon.h
@@ -186,12 +186,12 @@ extern rtems_interrupt_lock LEON3_IrqCtrl_Lock;
 
 #define LEON_Clear_interrupt( _source ) \
   do { \
-    LEON3_IrqCtrl_Regs->iclear = (1 << (_source)); \
+    LEON3_IrqCtrl_Regs->iclear = (1U << (_source)); \
   } while (0)
 
 #define LEON_Force_interrupt( _source ) \
   do { \
-    LEON3_IrqCtrl_Regs->iforce = (1 << (_source)); \
+    LEON3_IrqCtrl_Regs->iforce = (1U << (_source)); \
   } while (0)
 
 #define LEON_Enable_interrupt_broadcast( _source ) \
@@ -213,16 +213,16 @@ extern rtems_interrupt_lock LEON3_IrqCtrl_Lock;
   } while (0)
 
 #define LEON_Is_interrupt_pending( _source ) \
-  (LEON3_IrqCtrl_Regs->ipend & (1 << (_source)))
+  (LEON3_IrqCtrl_Regs->ipend & (1U << (_source)))
 
 #define LEON_Cpu_Is_interrupt_masked( _source, _cpu ) \
-     (!(LEON3_IrqCtrl_Regs->mask[_cpu] & (1 << (_source))))
+     (!(LEON3_IrqCtrl_Regs->mask[_cpu] & (1U << (_source))))
 
 #define LEON_Cpu_Mask_interrupt( _source, _cpu ) \
   do { \
     rtems_interrupt_lock_context _lock_context; \
     LEON3_IRQCTRL_ACQUIRE( &_lock_context ); \
-     LEON3_IrqCtrl_Regs->mask[_cpu]  &= ~(1 << (_source)); \
+     LEON3_IrqCtrl_Regs->mask[_cpu]  &= ~(1U << (_source)); \
     LEON3_IRQCTRL_RELEASE( &_lock_context ); \
   } while (0)
 
@@ -230,14 +230,14 @@ extern rtems_interrupt_lock LEON3_IrqCtrl_Lock;
   do { \
     rtems_interrupt_lock_context _lock_context; \
     LEON3_IRQCTRL_ACQUIRE( &_lock_context ); \
-    LEON3_IrqCtrl_Regs->mask[_cpu]  |= (1 << (_source)); \
+    LEON3_IrqCtrl_Regs->mask[_cpu]  |= (1U << (_source)); \
     LEON3_IRQCTRL_RELEASE( &_lock_context ); \
   } while (0)
 
 #define LEON_Cpu_Disable_interrupt( _source, _previous, _cpu ) \
   do { \
     rtems_interrupt_lock_context _lock_context; \
-    uint32_t _mask = 1 << (_source); \
+    uint32_t _mask = 1U << (_source); \
     LEON3_IRQCTRL_ACQUIRE( &_lock_context ); \
      (_previous) = LEON3_IrqCtrl_Regs->mask[_cpu]; \
      LEON3_IrqCtrl_Regs->mask[_cpu] = _previous & ~_mask; \
@@ -248,7 +248,7 @@ extern rtems_interrupt_lock LEON3_IrqCtrl_Lock;
 #define LEON_Cpu_Restore_interrupt( _source, _previous, _cpu ) \
   do { \
     rtems_interrupt_lock_context _lock_context; \
-    uint32_t _mask = 1 << (_source); \
+    uint32_t _mask = 1U << (_source); \
     LEON3_IRQCTRL_ACQUIRE( &_lock_context ); \
       LEON3_IrqCtrl_Regs->mask[_cpu] = \
         (LEON3_IrqCtrl_Regs->mask[_cpu] & ~_mask) | (_previous); \



More information about the vc mailing list