svn commit: r278032 - head/sys/boot/fdt

[Ian Lepore]

Author: ian
Date: Sun Feb  1 18:21:16 2015
New Revision: 278032
URL: https://svnweb.freebsd.org/changeset/base/278032

Log:
  Do not skip setting the memory 'reg' property if the fdt data already
  contains one.  Published dts source often includes a minimal default
  memory definition and expects it to be overridden by the bootloader after
  determining the actual physical memory in the system.

Modified:
  head/sys/boot/fdt/fdt_loader_cmd.c

Modified: head/sys/boot/fdt/fdt_loader_cmd.c
==============================================================================
--- head/sys/boot/fdt/fdt_loader_cmd.c	Sun Feb  1 17:19:57 2015	(r278031)
+++ head/sys/boot/fdt/fdt_loader_cmd.c	Sun Feb  1 18:21:16 2015	(r278032)
@@ -566,17 +566,6 @@ fdt_fixup_memory(struct fdt_mem_region *
 		return;
 	}
 
-	if ((reg = (uint32_t *)fdt_getprop(fdtp, memory, "reg",
-	    &len)) != NULL) {
-
-		if (fdt_reg_valid(reg, len, addr_cells, size_cells) == 0)
-			/*
-			 * Do not apply fixup if existing 'reg' property
-			 * seems to be valid.
-			 */
-			return;
-	}
-
 	len = (addr_cells + size_cells) * realmrno * sizeof(uint32_t);
 	sb = buf = (uint8_t *)malloc(len);
 	if (!buf)