[Skiboot] [PATCH 4/7] hdata: split idata accessor function

Nicholas Piggin npiggin at gmail.com
Wed Sep 6 20:10:14 AEST 2023


Add HDIF_idata that returns the idata pointer structure, not a
pointer to the data itself.

Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
---
 hdata/hdif.c | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/hdata/hdif.c b/hdata/hdif.c
index aba7f3660..98bd98d3c 100644
--- a/hdata/hdif.c
+++ b/hdata/hdif.c
@@ -4,10 +4,9 @@
 #include "hdif.h"
 #include <stack.h>
 
-const void *HDIF_get_idata(const struct HDIF_common_hdr *hdif, unsigned int di,
-			   unsigned int *size)
+static const struct HDIF_idata_ptr *
+HDIF_idata(const struct HDIF_common_hdr *hdif, unsigned int idx)
 {
-	const struct HDIF_common_hdr *hdr = hdif;
 	const struct HDIF_idata_ptr *iptr;
 
 	if (!HDIF_check(hdif, NULL)) {
@@ -16,14 +15,25 @@ const void *HDIF_get_idata(const struct HDIF_common_hdr *hdif, unsigned int di,
 		return NULL;
 	}
 
-	if (di >= be16_to_cpu(hdr->idptr_count)) {
+	if (idx >= be16_to_cpu(hdif->idptr_count)) {
 		prlog(PR_DEBUG, "HDIF: idata %d out of range for %.6s!\n",
-			di, hdr->id);
+			idx, hdif->id);
 		return NULL;
 	}
 
-	iptr = (void *)hdif + be32_to_cpu(hdr->idptr_off)
-		+ di * sizeof(struct HDIF_idata_ptr);
+	iptr = (void *)hdif + be32_to_cpu(hdif->idptr_off);
+
+	return &iptr[idx];
+}
+
+const void *HDIF_get_idata(const struct HDIF_common_hdr *hdif, unsigned int di,
+			   unsigned int *size)
+{
+	const struct HDIF_idata_ptr *iptr;
+
+	iptr = HDIF_idata(hdif, di);
+	if (!iptr)
+		return NULL;
 
 	if (size)
 		*size = be32_to_cpu(iptr->size);
-- 
2.40.1



More information about the Skiboot mailing list