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