Use dev_read_addr_size to get size of the controller's register area.

Signed-off-by: Dario Binacchi <dario...@libero.it>
Reviewed-by: Pratyush Yadav <p.ya...@ti.com>

---

(no changes since v3)

Changes in v3:
- Added Pratyush Yadav review tag.

Changes in v2:
- Check dev_read_addr_size return value.

 drivers/pinctrl/pinctrl-single.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index cec00e289c..d5656de8e8 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -182,17 +182,19 @@ static int single_set_state(struct udevice *dev,
 static int single_of_to_plat(struct udevice *dev)
 {
        fdt_addr_t addr;
-       u32 of_reg[2];
-       int res;
+       fdt_size_t size;
        struct single_pdata *pdata = dev_get_plat(dev);
 
        pdata->width =
                dev_read_u32_default(dev, "pinctrl-single,register-width", 0);
 
-       res = dev_read_u32_array(dev, "reg", of_reg, 2);
-       if (res)
-               return res;
-       pdata->offset = of_reg[1] - pdata->width / 8;
+       addr = dev_read_addr_size(dev, "reg", &size);
+       if (addr == FDT_ADDR_T_NONE) {
+               dev_err(dev, "failed to get base register size\n");
+               return -EINVAL;
+       }
+
+       pdata->offset = size - pdata->width / BITS_PER_BYTE;
 
        addr = dev_read_addr(dev);
        if (addr == FDT_ADDR_T_NONE) {
-- 
2.17.1

Reply via email to