As the address read from device tree is being cast to a pointer, it's
better to use dev_read_addr_ptr() API for getting that address. The more
detailed explanation can be found in commit a12a73b66476 ("drivers: use
dev_read_addr_ptr when cast to pointer").

Signed-off-by: Sam Protsenko <semen.protse...@linaro.org>
---
 drivers/serial/serial_s5p.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
index c24d9bca84c9..7d04dcff54fc 100644
--- a/drivers/serial/serial_s5p.c
+++ b/drivers/serial/serial_s5p.c
@@ -221,13 +221,11 @@ static int s5p_serial_of_to_plat(struct udevice *dev)
 {
        struct s5p_serial_plat *plat = dev_get_plat(dev);
        const ulong port_type = dev_get_driver_data(dev);
-       fdt_addr_t addr;
 
-       addr = dev_read_addr(dev);
-       if (addr == FDT_ADDR_T_NONE)
+       plat->reg = dev_read_addr_ptr(dev);
+       if (!plat->reg)
                return -EINVAL;
 
-       plat->reg = (struct s5p_uart *)addr;
        plat->reg_width = dev_read_u32_default(dev, "reg-io-width", 1);
        plat->port_id = dev_read_u8_default(dev, "id", dev_seq(dev));
 
-- 
2.39.2

Reply via email to