Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 1 Feb 2015 18:21:17 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r278032 - head/sys/boot/fdt
Message-ID:  <201502011821.t11ILHCo023283@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
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)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201502011821.t11ILHCo023283>