There doesn't seem to be a particularly strong reason to create
and entirely new type for dvt, which is just an unsigned int.
In fact, I think it is a lot clearer just to use unigned directly.
Signed-off-by: Simon Horman <[EMAIL PROTECTED]>
Index: kexec-tools-testing/kexec/arch/ppc64/fs2dt.c
===================================================================
--- kexec-tools-testing.orig/kexec/arch/ppc64/fs2dt.c 2006-12-08
12:40:08.000000000 +0900
+++ kexec-tools-testing/kexec/arch/ppc64/fs2dt.c 2006-12-08
12:41:05.000000000 +0900
@@ -57,16 +57,15 @@
exit(rc);
}
-typedef unsigned dvt;
char pathname[MAXPATH], *pathstart;
char propnames[NAMESPACE];
-dvt dtstruct[TREEWORDS], *dt;
+unsigned dtstruct[TREEWORDS], *dt;
unsigned long long mem_rsrv[2*MEMRESERVE];
static int initrd_found = 0;
static int crash_param = 0;
char local_cmdline[COMMAND_LINE_SIZE] = { "" };
-dvt *dt_len; /* changed len of modified cmdline in flat device-tree */
+unsigned *dt_len; /* changed len of modified cmdline in flat device-tree */
extern mem_rgns_t usablemem_rgns;
struct bootblock bb[1];
@@ -84,7 +83,7 @@
}
/* look for properties we need to reserve memory space for */
-void checkprop(char *name, dvt *data)
+void checkprop(char *name, unsigned *data)
{
static unsigned long long base, size, end;
@@ -114,9 +113,9 @@
* return the property index for a property name, creating a new one
* if needed.
*/
-dvt propnum(const char *name)
+unsigned propnum(const char *name)
{
- dvt offset = 0;
+ unsigned offset = 0;
while(propnames[offset])
if (strcmp(name, propnames+offset))
@@ -438,7 +437,7 @@
bb->off_dt_struct = bb->off_mem_rsvmap + len;
len = dt - dtstruct;
- len *= sizeof(dvt);
+ len *= sizeof(unsigned);
bb->off_dt_strings = bb->off_dt_struct + len;
len = propnum("");
--
--
Horms
H: http://www.vergenet.net/~horms/
W: http://www.valinux.co.jp/en/
_______________________________________________
fastboot mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/fastboot