[PATCH] libfdt: fdt_address_cells() and fdt_size_cells()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



Make the default cells compatible with Linux
(OF_ROOT_NODE_ADDR_CELLS_DEFAULT and OF_ROOT_NODE_SIZE_CELLS_DEFAULT)
which is one cell (except on SPARC, here the default address cells are
two).

In case the current node has no cells property, then query the parent,
and so on.  This is in line with the Linux implementation of
of_n_addr_cells() and of_n_size_cells().

Signed-off-by: Sebastian Huber <sebastian.huber@xxxxxxxxxxxxxxxxxx>
---
 libfdt/fdt_addresses.c   | 59 +++++++++++++++++++++++-------------------
 tests/.gitignore         |  1 +
 tests/Makefile.tests     |  1 +
 tests/addr_size_cells.c  |  5 ++++
 tests/addr_size_cells2.c | 67 ++++++++++++++++++++++++++++++++++++++++++++++++
 tests/addresses.dts      | 23 +++++++++++++++++
 tests/run_tests.sh       |  2 ++
 7 files changed, 131 insertions(+), 27 deletions(-)
 create mode 100644 tests/addr_size_cells2.c

diff --git a/libfdt/fdt_addresses.c b/libfdt/fdt_addresses.c
index eff4dbc..5eb735a 100644
--- a/libfdt/fdt_addresses.c
+++ b/libfdt/fdt_addresses.c
@@ -55,42 +55,47 @@
 
 #include "libfdt_internal.h"
 
-int fdt_address_cells(const void *fdt, int nodeoffset)
+static int fdt_cells(const void *fdt, int nodeoffset, const char *name,
+		     int default_cells)
 {
-	const fdt32_t *ac;
+	const fdt32_t *c;
 	int val;
 	int len;
 
-	ac = fdt_getprop(fdt, nodeoffset, "#address-cells", &len);
-	if (!ac)
-		return 2;
-
-	if (len != sizeof(*ac))
-		return -FDT_ERR_BADNCELLS;
+	do {
+		c = fdt_getprop(fdt, nodeoffset, name, &len);
 
-	val = fdt32_to_cpu(*ac);
-	if ((val <= 0) || (val > FDT_MAX_NCELLS))
-		return -FDT_ERR_BADNCELLS;
+		if (c) {
+			if (len != sizeof(*c))
+				return -FDT_ERR_BADNCELLS;
 
-	return val;
-}
+			val = fdt32_to_cpu(*c);
+			if ((val <= 0) || (val > FDT_MAX_NCELLS))
+				return -FDT_ERR_BADNCELLS;
 
-int fdt_size_cells(const void *fdt, int nodeoffset)
-{
-	const fdt32_t *sc;
-	int val;
-	int len;
+			return val;
+		}
 
-	sc = fdt_getprop(fdt, nodeoffset, "#size-cells", &len);
-	if (!sc)
-		return 2;
+		nodeoffset = fdt_parent_offset(fdt, nodeoffset);
+	} while (nodeoffset >= 0);
 
-	if (len != sizeof(*sc))
-		return -FDT_ERR_BADNCELLS;
+	return default_cells;
+}
 
-	val = fdt32_to_cpu(*sc);
-	if ((val < 0) || (val > FDT_MAX_NCELLS))
-		return -FDT_ERR_BADNCELLS;
+int fdt_address_cells(const void *fdt, int nodeoffset)
+{
+	/* Use default value of Linux kernel (OF_ROOT_NODE_ADDR_CELLS_DEFAULT) */
+	int default_cells;
+#ifdef __sparc__
+	default_cells = 2;
+#else
+	default_cells = 1;
+#endif
+	return fdt_cells(fdt, nodeoffset, "#address-cells", default_cells);
+}
 
-	return val;
+int fdt_size_cells(const void *fdt, int nodeoffset)
+{
+	/* Use default value of Linux kernel (OF_ROOT_NODE_SIZE_CELLS_DEFAULT) */
+	return fdt_cells(fdt, nodeoffset, "#size-cells", 1);
 }
diff --git a/tests/.gitignore b/tests/.gitignore
index d423570..7a99f5a 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -4,6 +4,7 @@
 tmp.*
 /add_subnode_with_nops
 /addr_size_cells
+/addr_size_cells2
 /appendprop[12]
 /asm_tree_dump
 /boot-cpuid
diff --git a/tests/Makefile.tests b/tests/Makefile.tests
index 2c2c4fd..e7cf6bc 100644
--- a/tests/Makefile.tests
+++ b/tests/Makefile.tests
@@ -9,6 +9,7 @@ LIB_TESTS_L = get_mem_rsv \
 	sized_cells \
 	notfound \
 	addr_size_cells \
+	addr_size_cells2 \
 	stringlist \
 	setprop_inplace nop_property nop_node \
 	sw_tree1 sw_states \
diff --git a/tests/addr_size_cells.c b/tests/addr_size_cells.c
index 6090d93..2a487e6 100644
--- a/tests/addr_size_cells.c
+++ b/tests/addr_size_cells.c
@@ -2,6 +2,7 @@
  * libfdt - Flat Device Tree manipulation
  *	Testcase for #address-cells and #size-cells handling
  * Copyright (C) 2014 David Gibson, <david@xxxxxxxxxxxxxxxxxxxxx>
+ * Copyright (C) 2018 embedded brains GmbH
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public License
@@ -60,5 +61,9 @@ int main(int argc, char *argv[])
 	check_node(fdt, "/", 2, 2);
 	check_node(fdt, "/identity-bus@0", 2, 2);
 	check_node(fdt, "/simple-bus@1000000", 2, 1);
+	check_node(fdt, "/c0", -FDT_ERR_BADNCELLS, -FDT_ERR_BADNCELLS);
+	check_node(fdt, "/c1", -FDT_ERR_BADNCELLS, -FDT_ERR_BADNCELLS);
+	check_node(fdt, "/c2", -FDT_ERR_BADNCELLS, -FDT_ERR_BADNCELLS);
+	check_node(fdt, "/c3/c4", 3, 4);
 	PASS();
 }
diff --git a/tests/addr_size_cells2.c b/tests/addr_size_cells2.c
new file mode 100644
index 0000000..c250866
--- /dev/null
+++ b/tests/addr_size_cells2.c
@@ -0,0 +1,67 @@
+/*
+ * libfdt - Flat Device Tree manipulation
+ *	Testcase for #address-cells and #size-cells handling
+ * Copyright (C) 2014 David Gibson, <david@xxxxxxxxxxxxxxxxxxxxx>
+ * Copyright (C) 2018 embedded brains GmbH
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public License
+ * as published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+#include <stdlib.h>
+#include <stdio.h>
+#include <string.h>
+#include <stdint.h>
+
+#include <libfdt.h>
+
+#include "tests.h"
+#include "testdata.h"
+
+static void check_node(const void *fdt, const char *path, int ac, int sc)
+{
+	int offset;
+	int xac, xsc;
+
+	offset = fdt_path_offset(fdt, path);
+	if (offset < 0)
+		FAIL("Couldn't find path %s", path);
+
+	xac = fdt_address_cells(fdt, offset);
+	xsc = fdt_size_cells(fdt, offset);
+
+	if (xac != ac)
+		FAIL("Address cells for %s is %d instead of %d\n",
+		     path, xac, ac);
+	if (xsc != sc)
+		FAIL("Size cells for %s is %d instead of %d\n",
+		     path, xsc, sc);
+}
+
+int main(int argc, char *argv[])
+{
+	void *fdt;
+
+	if (argc != 2)
+		CONFIG("Usage: %s <dtb file>\n", argv[0]);
+
+	test_init(argc, argv);
+	fdt = load_blob(argv[1]);
+
+#ifdef __sparc__
+	check_node(fdt, "/", 2, 1);
+#else
+	check_node(fdt, "/", 1, 1);
+#endif
+	PASS();
+}
diff --git a/tests/addresses.dts b/tests/addresses.dts
index a2faaf5..fb6d389 100644
--- a/tests/addresses.dts
+++ b/tests/addresses.dts
@@ -12,4 +12,27 @@
 		#address-cells = <2>;
 		#size-cells = <1>;
 	};
+
+	c0 {
+		#address-cells = <1 1>;
+		#size-cells = <2 2>;
+	};
+
+	c1 {
+		#address-cells = <0x80000000>;
+		#size-cells = <0x80000001>;
+	};
+
+	c2 {
+		#address-cells = <5>;
+		#size-cells = <6>;
+	};
+
+	c3 {
+		#address-cells = <3>;
+		#size-cells = <4>;
+
+		c4 {
+		};
+	};
 };
diff --git a/tests/run_tests.sh b/tests/run_tests.sh
index cf87066..7348c9c 100755
--- a/tests/run_tests.sh
+++ b/tests/run_tests.sh
@@ -336,6 +336,8 @@ libfdt_tests () {
 
     run_dtc_test -I dts -O dtb -o addresses.test.dtb addresses.dts
     run_test addr_size_cells addresses.test.dtb
+    run_dtc_test -I dts -O dtb -o addresses2.test.dtb empty.dts
+    run_test addr_size_cells2 addresses2.test.dtb
 
     run_dtc_test -I dts -O dtb -o stringlist.test.dtb stringlist.dts
     run_test stringlist stringlist.test.dtb
-- 
2.13.7

--
To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux