This patch series replaces the custom bit manipulation function sr32()
by standard I/O accessors.  A major motivation for this cleanup was
the fact that a number of calls of that function resulted in 32 bit
wide shift operations on u32 data, which according to the C-ISO
IEC-9899-Standard provokes undefined behaviour:

        6.5.7 Bitwise shift operators
        ...
            If the value of the right operand is negative or is
            greater than or equal to the width of the promoted left
            operand, the behavior is undefined.

The code was built-tested for all TI boards (using "./MAKEALL -v ti")
without errors or warnings.

Signed-off-by: Wolfgang Denk <w...@denx.de>
Cc: Tom Rini <tr...@ti.com>
Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>

Wolfgang Denk (3):
  ARM: OMAP: remove sr32() from OMAP board code
  ARM: OMAP: hide custom bit manipulation function sr32()
  ARM: OMAP: replace custom sr32() by standard I/O accessors

 arch/arm/cpu/armv7/omap3/board.c            |   4 +-
 arch/arm/cpu/armv7/omap3/clock.c            | 354 ++++++++++++++++------------
 arch/arm/cpu/armv7/syslib.c                 |  13 -
 arch/arm/include/asm/arch-omap3/sys_proto.h |   1 -
 arch/arm/include/asm/arch-omap4/sys_proto.h |   1 -
 arch/arm/include/asm/arch-omap5/sys_proto.h |   1 -
 board/ti/panda/panda.c                      |   2 +-
 7 files changed, 207 insertions(+), 169 deletions(-)

-- 
1.8.5.3

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to