[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