[U-Boot] [PATCH 0/3 v2] ARM: OMAP: replace custom sr32() by standard I/O accessors

Tom Rini trini at ti.com
Fri Apr 18 15:21:44 CEST 2014


On Tue, Mar 25, 2014 at 02:49:47PM +0100, Wolfgang Denk wrote:

> 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 <wd at denx.de>
> Cc: Tom Rini <trini at ti.com>
> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>

Applied to u-boot-ti/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140418/28eab1c1/attachment.pgp>


More information about the U-Boot mailing list