[OpenBIOS] [commit] r1286 - trunk/openbios-devel/libopenbios

repository service svn at openbios.org
Fri Apr 4 11:46:25 CEST 2014


Author: mcayland
Date: Fri Apr  4 11:46:24 2014
New Revision: 1286
URL: http://tracker.coreboot.org/trac/openbios/changeset/1286

Log:
OFMEM: remove redundant minimum alignment check from find_area()

This is now handled by the section of code preceding it.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland at ilande.co.uk>

Modified:
   trunk/openbios-devel/libopenbios/ofmem_common.c

Modified: trunk/openbios-devel/libopenbios/ofmem_common.c
==============================================================================
--- trunk/openbios-devel/libopenbios/ofmem_common.c	Fri Apr  4 11:46:21 2014	(r1285)
+++ trunk/openbios-devel/libopenbios/ofmem_common.c	Fri Apr  4 11:46:24 2014	(r1286)
@@ -442,15 +442,11 @@
 	if( (align & (align-1)) ) {
 	
 		/* As per IEEE1275 specification, round up to the nearest power of 2 */
-		if (old_align <= PAGE_SIZE) {
-			align = PAGE_SIZE;
-		} else {
-			align--;
-			for (i = 1; i < sizeof(ucell) * 8; i<<=1) {
-				align |= align >> i;
-			}
-			align++;
+		align--;
+		for (i = 1; i < sizeof(ucell) * 8; i<<=1) {
+			align |= align >> i;
 		}
+		align++;
 		
 		OFMEM_TRACE("warning: bad alignment " FMT_ucellx " rounded up to " FMT_ucellx "\n", old_align, align);
 	}



More information about the OpenBIOS mailing list