[PATCH v2 3/4] powerpc/memhotplug: Make lmb size 64bit

[Aneesh Kumar K.V]

Similar to commit 89c140bbaeee ("pseries: Fix 64 bit logical memory block 
panic")
make sure different variables tracking lmb_size are updated to be 64 bit.

This was found by code audit.

Cc: sta...@vger.kernel.org
Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.ibm.com>
---
 .../platforms/pseries/hotplug-memory.c        | 37 +++++++++++--------
 1 file changed, 22 insertions(+), 15 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/hotplug-memory.c 
b/arch/powerpc/platforms/pseries/hotplug-memory.c
index 5d545b78111f..1fe3204c843a 100644
--- a/arch/powerpc/platforms/pseries/hotplug-memory.c
+++ b/arch/powerpc/platforms/pseries/hotplug-memory.c
@@ -277,7 +277,7 @@ static int dlpar_offline_lmb(struct drmem_lmb *lmb)
        return dlpar_change_lmb_state(lmb, false);
 }
 
-static int pseries_remove_memblock(unsigned long base, unsigned int 
memblock_size)
+static int pseries_remove_memblock(unsigned long base, unsigned long 
memblock_size)
 {
        unsigned long block_sz, start_pfn;
        int sections_per_block;
@@ -308,9 +308,9 @@ static int pseries_remove_memblock(unsigned long base, 
unsigned int memblock_siz
 
 static int pseries_remove_mem_node(struct device_node *np)
 {
-       const __be32 *regs;
+       const __be32 *prop;
        unsigned long base;
-       unsigned int lmb_size;
+       unsigned long lmb_size;
        int ret = -EINVAL;
 
        /*
@@ -322,12 +322,16 @@ static int pseries_remove_mem_node(struct device_node *np)
        /*
         * Find the base address and size of the memblock
         */
-       regs = of_get_property(np, "reg", NULL);
-       if (!regs)
+       prop = of_get_property(np, "reg", NULL);
+       if (!prop)
                return ret;
 
-       base = be64_to_cpu(*(unsigned long *)regs);
-       lmb_size = be32_to_cpu(regs[3]);
+       /*
+        * "reg" property represents (addr,size) tuple.
+        */
+       base = of_read_number(prop, mem_addr_cells);
+       prop += mem_addr_cells;
+       lmb_size = of_read_number(prop, mem_size_cells);
 
        pseries_remove_memblock(base, lmb_size);
        return 0;
@@ -557,7 +561,7 @@ static int dlpar_memory_remove_by_ic(u32 lmbs_to_remove, 
u32 drc_index)
 
 #else
 static inline int pseries_remove_memblock(unsigned long base,
-                                         unsigned int memblock_size)
+                                         unsigned long memblock_size)
 {
        return -EOPNOTSUPP;
 }
@@ -878,9 +882,9 @@ int dlpar_memory(struct pseries_hp_errorlog *hp_elog)
 
 static int pseries_add_mem_node(struct device_node *np)
 {
-       const __be32 *regs;
+       const __be32 *prop;
        unsigned long base;
-       unsigned int lmb_size;
+       unsigned long lmb_size;
        int ret = -EINVAL;
 
        /*
@@ -892,12 +896,15 @@ static int pseries_add_mem_node(struct device_node *np)
        /*
         * Find the base and size of the memblock
         */
-       regs = of_get_property(np, "reg", NULL);
-       if (!regs)
+       prop = of_get_property(np, "reg", NULL);
+       if (!prop)
                return ret;
-
-       base = be64_to_cpu(*(unsigned long *)regs);
-       lmb_size = be32_to_cpu(regs[3]);
+       /*
+        * "reg" property represents (addr,size) tuple.
+        */
+       base = of_read_number(prop, mem_addr_cells);
+       prop += mem_addr_cells;
+       lmb_size = of_read_number(prop, mem_size_cells);
 
        /*
         * Update memory region to represent the memory add
-- 
2.26.2