From bebd2b3536bce3bb1d50eff76e51a50310fb29e5 Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Wed, 14 Nov 2001 19:54:59 +0000 Subject: [PATCH] when #size-cells is zero, don't expect a size. --- sim/ppc/ChangeLog | 7 +++++++ sim/ppc/tree.c | 15 ++++++++++----- 2 files changed, 17 insertions(+), 5 deletions(-) diff --git a/sim/ppc/ChangeLog b/sim/ppc/ChangeLog index 3600c79e461..4fae50ec17a 100644 --- a/sim/ppc/ChangeLog +++ b/sim/ppc/ChangeLog @@ -1,3 +1,10 @@ +2001-10-29 Andrew Cagney + + * tree.c (parse_size): Assert #size-cells > 0. + (parse_address): Ditto for #address-cells. + (parse_reg_property): Only parse the size when #size-cells is + non-zero. + 2001-10-25 Andrew Cagney * emul_generic.c (OEA_MEMORY_SIZE): Increase to 4mb. diff --git a/sim/ppc/tree.c b/sim/ppc/tree.c index 360059353ce..c5106f47f3f 100644 --- a/sim/ppc/tree.c +++ b/sim/ppc/tree.c @@ -463,6 +463,7 @@ parse_address(device *current, const char *chp, device_unit *address) { + ASSERT(device_nr_address_cells(bus) > 0); if (device_decode_unit(bus, chp, address) < 0) device_error(current, "invalid unit address in %s", chp); return skip_token(chp); @@ -485,6 +486,7 @@ parse_size(device *current, /* parse the numeric list */ size->nr_cells = device_nr_size_cells(bus); nr = 0; + ASSERT(size->nr_cells > 0); while (1) { char *next; size->cells[nr] = strtoul(curr, &next, 0); @@ -521,9 +523,11 @@ parse_reg_property(device *current, int reg_nr; reg_property_spec *regs; const char *chp; + device *bus = device_parent(current); /* determine the number of reg entries by counting tokens */ - nr_regs = count_entries(current, property_name, property_value, 2); + nr_regs = count_entries(current, property_name, property_value, + 1 + (device_nr_size_cells(bus) > 0)); /* create working space */ regs = zalloc(nr_regs * sizeof(*regs)); @@ -531,10 +535,11 @@ parse_reg_property(device *current, /* fill it in */ chp = property_value; for (reg_nr = 0; reg_nr < nr_regs; reg_nr++) { - chp = parse_address(current, device_parent(current), - chp, ®s[reg_nr].address); - chp = parse_size(current, device_parent(current), - chp, ®s[reg_nr].size); + chp = parse_address(current, bus, chp, ®s[reg_nr].address); + if (device_nr_size_cells(bus) > 0) + chp = parse_size(current, bus, chp, ®s[reg_nr].size); + else + memset(®s[reg_nr].size, sizeof (®s[reg_nr].size), 0); } /* create it */ -- 2.30.2