diff options
| author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2018-06-23 12:18:09 +0530 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-09-13 15:19:36 +1000 |
| commit | dd2dacf8ee0604107e93354cc2968e8f3bdf5cf0 (patch) | |
| tree | 3b94c58d33d498cdcd4635ed35c19ff75bb7aa49 /hdata | |
| parent | df5da053c459369b4fc4a831dd09ce1859efe436 (diff) | |
| download | blackbird-skiboot-dd2dacf8ee0604107e93354cc2968e8f3bdf5cf0.tar.gz blackbird-skiboot-dd2dacf8ee0604107e93354cc2968e8f3bdf5cf0.zip | |
hdata: Fix dtc warnings
Fix dtc warnings related to mcbist node.
Warning (reg_format): "reg" property in /xscom@623fc00000000/mcbist@1 has invalid length (4 bytes) (#address-cells == 1, #size-cells == 1)
Warning (reg_format): "reg" property in /xscom@623fc00000000/mcbist@2 has invalid length (4 bytes) (#address-cells == 1, #size-cells == 1)
Warning (reg_format): "reg" property in /xscom@603fc00000000/mcbist@1 has invalid length (4 bytes) (#address-cells == 1, #size-cells == 1)
Warning (reg_format): "reg" property in /xscom@603fc00000000/mcbist@2 has invalid length (4 bytes) (#address-cells == 1, #size-cells == 1)
Ideally we should add proper xscom range here... but we are not getting that
information in HDAT today. Lets fix warning until we get proper data in HDAT.
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hdata')
| -rw-r--r-- | hdata/memory.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/hdata/memory.c b/hdata/memory.c index ecce881d..72a72e11 100644 --- a/hdata/memory.c +++ b/hdata/memory.c @@ -448,7 +448,9 @@ static void add_memory_controller(const struct HDIF_common_hdr *msarea, if (!mcbist) { mcbist = dt_new_addr(xscom, "mcbist", mcbist_id); assert(mcbist); - dt_add_mem_reg_property(mcbist, mcbist_id); + dt_add_property_cells(mcbist, "#address-cells", 1); + dt_add_property_cells(mcbist, "#size-cells", 0); + dt_add_property_cells(mcbist, "reg", mcbist_id, 0); } mcs_id = MS_CONTROLLER_MCS_ID(controller_id); |
