git: 211ca9061b5d - main - dev/fdt: Remove old fdt_get memory functions

From: Andrew Turner <andrew_at_FreeBSD.org>
Date: Tue, 08 Apr 2025 10:49:13 UTC
The branch main has been updated by andrew:

URL: https://cgit.FreeBSD.org/src/commit/?id=211ca9061b5d561405edeaf1df1f7fb296506d2c

commit 211ca9061b5d561405edeaf1df1f7fb296506d2c
Author:     Andrew Turner <andrew@FreeBSD.org>
AuthorDate: 2025-04-08 10:30:46 +0000
Commit:     Andrew Turner <andrew@FreeBSD.org>
CommitDate: 2025-04-08 10:48:28 +0000

    dev/fdt: Remove old fdt_get memory functions
    
    These have been replaces with fdt_foreach equivalents so are unused.
    
    Reviewed by:    imp
    Sponsored by:   Arm Ltd
    Differential Revision:  https://reviews.freebsd.org/D49702
---
 sys/dev/fdt/fdt_common.c | 163 -----------------------------------------------
 sys/dev/fdt/fdt_common.h |   3 -
 2 files changed, 166 deletions(-)

diff --git a/sys/dev/fdt/fdt_common.c b/sys/dev/fdt/fdt_common.c
index ba323ea56d96..8a0e414595f2 100644
--- a/sys/dev/fdt/fdt_common.c
+++ b/sys/dev/fdt/fdt_common.c
@@ -491,63 +491,6 @@ fdt_foreach_reserved_region(fdt_mem_region_cb cb, void *arg)
 	return (0);
 }
 
-int
-fdt_get_reserved_regions(struct mem_region *mr, int *mrcnt)
-{
-	pcell_t reserve[FDT_REG_CELLS * FDT_MEM_REGIONS];
-	pcell_t *reservep;
-	phandle_t memory, root;
-	int addr_cells, size_cells;
-	int i, res_len, rv, tuple_size, tuples;
-
-	root = OF_finddevice("/");
-	memory = OF_finddevice("/memory");
-	if (memory == -1) {
-		rv = ENXIO;
-		goto out;
-	}
-
-	if ((rv = fdt_addrsize_cells(OF_parent(memory), &addr_cells,
-	    &size_cells)) != 0)
-		goto out;
-
-	if (addr_cells > 2) {
-		rv = ERANGE;
-		goto out;
-	}
-
-	tuple_size = sizeof(pcell_t) * (addr_cells + size_cells);
-
-	res_len = OF_getproplen(root, "memreserve");
-	if (res_len <= 0 || res_len > sizeof(reserve)) {
-		rv = ERANGE;
-		goto out;
-	}
-
-	if (OF_getprop(root, "memreserve", reserve, res_len) <= 0) {
-		rv = ENXIO;
-		goto out;
-	}
-
-	tuples = res_len / tuple_size;
-	reservep = (pcell_t *)&reserve;
-	for (i = 0; i < tuples; i++) {
-
-		rv = fdt_data_to_res(reservep, addr_cells, size_cells,
-			(u_long *)&mr[i].mr_start, (u_long *)&mr[i].mr_size);
-
-		if (rv != 0)
-			goto out;
-
-		reservep += addr_cells + size_cells;
-	}
-
-	*mrcnt = i;
-	rv = 0;
-out:
-	return (rv);
-}
-
 int
 fdt_foreach_reserved_mem(fdt_mem_region_cb cb, void *arg)
 {
@@ -586,47 +529,6 @@ fdt_foreach_reserved_mem(fdt_mem_region_cb cb, void *arg)
 	return (0);
 }
 
-int
-fdt_get_reserved_mem(struct mem_region *reserved, int *mreserved)
-{
-	pcell_t reg[FDT_REG_CELLS];
-	phandle_t child, root;
-	int addr_cells, size_cells;
-	int i, rv;
-
-	root = OF_finddevice("/reserved-memory");
-	if (root == -1) {
-		return (ENXIO);
-	}
-
-	if ((rv = fdt_addrsize_cells(root, &addr_cells, &size_cells)) != 0)
-		return (rv);
-
-	if (addr_cells + size_cells > FDT_REG_CELLS)
-		panic("Too many address and size cells %d %d", addr_cells,
-		    size_cells);
-
-	i = 0;
-	for (child = OF_child(root); child != 0; child = OF_peer(child)) {
-		if (!OF_hasprop(child, "no-map"))
-			continue;
-
-		rv = OF_getprop(child, "reg", reg, sizeof(reg));
-		if (rv <= 0)
-			/* XXX: Does a no-map of a dynamic range make sense? */
-			continue;
-
-		fdt_data_to_res(reg, addr_cells, size_cells,
-		    (u_long *)&reserved[i].mr_start,
-		    (u_long *)&reserved[i].mr_size);
-		i++;
-	}
-
-	*mreserved = i;
-
-	return (0);
-}
-
 int
 fdt_foreach_mem_region(fdt_mem_region_cb cb, void *arg)
 {
@@ -674,71 +576,6 @@ fdt_foreach_mem_region(fdt_mem_region_cb cb, void *arg)
 	return (0);
 }
 
-int
-fdt_get_mem_regions(struct mem_region *mr, int *mrcnt, uint64_t *memsize)
-{
-	pcell_t reg[FDT_REG_CELLS * FDT_MEM_REGIONS];
-	pcell_t *regp;
-	phandle_t memory;
-	uint64_t memory_size;
-	int addr_cells, size_cells;
-	int i, reg_len, rv, tuple_size, tuples;
-
-	memory = OF_finddevice("/memory");
-	if (memory == -1) {
-		rv = ENXIO;
-		goto out;
-	}
-
-	if ((rv = fdt_addrsize_cells(OF_parent(memory), &addr_cells,
-	    &size_cells)) != 0)
-		goto out;
-
-	if (addr_cells > 2) {
-		rv = ERANGE;
-		goto out;
-	}
-
-	tuple_size = sizeof(pcell_t) * (addr_cells + size_cells);
-	reg_len = OF_getproplen(memory, "reg");
-	if (reg_len <= 0 || reg_len > sizeof(reg)) {
-		rv = ERANGE;
-		goto out;
-	}
-
-	if (OF_getprop(memory, "reg", reg, reg_len) <= 0) {
-		rv = ENXIO;
-		goto out;
-	}
-
-	memory_size = 0;
-	tuples = reg_len / tuple_size;
-	regp = (pcell_t *)&reg;
-	for (i = 0; i < tuples; i++) {
-
-		rv = fdt_data_to_res(regp, addr_cells, size_cells,
-			(u_long *)&mr[i].mr_start, (u_long *)&mr[i].mr_size);
-
-		if (rv != 0)
-			goto out;
-
-		regp += addr_cells + size_cells;
-		memory_size += mr[i].mr_size;
-	}
-
-	if (memory_size == 0) {
-		rv = ERANGE;
-		goto out;
-	}
-
-	*mrcnt = i;
-	if (memsize != NULL)
-		*memsize = memory_size;
-	rv = 0;
-out:
-	return (rv);
-}
-
 int
 fdt_get_chosen_bootargs(char *bootargs, size_t max_size)
 {
diff --git a/sys/dev/fdt/fdt_common.h b/sys/dev/fdt/fdt_common.h
index 22842d0e6ae7..dbf809625da3 100644
--- a/sys/dev/fdt/fdt_common.h
+++ b/sys/dev/fdt/fdt_common.h
@@ -84,11 +84,8 @@ int fdt_data_to_res(pcell_t *, int, int, u_long *, u_long *);
 phandle_t fdt_find_compatible(phandle_t, const char *, int);
 phandle_t fdt_depth_search_compatible(phandle_t, const char *, int);
 int fdt_foreach_mem_region(fdt_mem_region_cb, void *);
-int fdt_get_mem_regions(struct mem_region *, int *, uint64_t *);
 int fdt_foreach_reserved_mem(fdt_mem_region_cb, void *);
-int fdt_get_reserved_mem(struct mem_region *, int *);
 int fdt_foreach_reserved_region(fdt_mem_region_cb, void *);
-int fdt_get_reserved_regions(struct mem_region *, int *);
 int fdt_get_phyaddr(phandle_t, device_t, int *, void **);
 int fdt_get_range(phandle_t, int, u_long *, u_long *);
 int fdt_immr_addr(vm_offset_t);