gpio: mpc8xxx: don't modify gpdat when setting gpio as input

[platform/kernel/u-boot.git] / drivers / gpio / mpc8xxx_gpio.c

diff --git a/drivers/gpio/mpc8xxx_gpio.c b/drivers/gpio/mpc8xxx_gpio.c
index 0aa72ec..d4f3092 100644 (file)
--- a/drivers/gpio/mpc8xxx_gpio.c
+++ b/drivers/gpio/mpc8xxx_gpio.c
@@ -1,12 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2016
- * Mario Six, Guntermann & Drunck GmbH, six@gdsys.de
+ * Mario Six, Guntermann & Drunck GmbH, mario.six@gdsys.cc
  *
  * based on arch/powerpc/include/asm/mpc85xx_gpio.h, which is
  *
  * Copyright 2010 eXMeritus, A Boeing Company
- *
- * SPDX-License-Identifier:    GPL-2.0+
  */
 
 #include <common.h>
@@ -14,8 +13,6 @@
 #include <mapmem.h>
 #include <asm/gpio.h>
 
-DECLARE_GLOBAL_DATA_PTR;
-
 struct ccsr_gpio {
        u32     gpdir;
        u32     gpodr;
@@ -60,13 +57,6 @@ static inline u32 mpc8xxx_gpio_get_dir(struct ccsr_gpio *base, u32 mask)
        return in_be32(&base->gpdir) & mask;
 }
 
-static inline void mpc8xxx_gpio_set_in(struct ccsr_gpio *base, u32 gpios)
-{
-       clrbits_be32(&base->gpdat, gpios);
-       /* GPDIR register 0 -> input */
-       clrbits_be32(&base->gpdir, gpios);
-}
-
 static inline void mpc8xxx_gpio_set_low(struct ccsr_gpio *base, u32 gpios)
 {
        clrbits_be32(&base->gpdat, gpios);
@@ -103,8 +93,11 @@ static inline void mpc8xxx_gpio_open_drain_off(struct ccsr_gpio *base,
 static int mpc8xxx_gpio_direction_input(struct udevice *dev, uint gpio)
 {
        struct mpc8xxx_gpio_data *data = dev_get_priv(dev);
+       u32 mask = gpio_mask(gpio);
+
+       /* GPDIR register 0 -> input */
+       clrbits_be32(&data->base->gpdir, mask);
 
-       mpc8xxx_gpio_set_in(data->base, gpio_mask(gpio));
        return 0;
 }
 
@@ -181,15 +174,14 @@ static int mpc8xxx_gpio_ofdata_to_platdata(struct udevice *dev)
 {
        struct mpc8xxx_gpio_plat *plat = dev_get_platdata(dev);
        fdt_addr_t addr;
-       fdt_size_t size;
+       u32 reg[2];
+
+       dev_read_u32_array(dev, "reg", reg, 2);
+       addr = dev_translate_address(dev, reg);
 
-       addr = fdtdec_get_addr_size_auto_noparent(gd->fdt_blob,
-                                                 dev_of_offset(dev),
-                                                 "reg", 0, &size, false);
        plat->addr = addr;
-       plat->size = size;
-       plat->ngpios = fdtdec_get_int(gd->fdt_blob, dev_of_offset(dev),
-                                     "ngpios", 32);
+       plat->size = reg[1];
+       plat->ngpios = dev_read_u32_default(dev, "ngpios", 32);
 
        return 0;
 }
@@ -206,7 +198,7 @@ static int mpc8xxx_gpio_platdata_to_priv(struct udevice *dev)
                size = 0x100;
 
        priv->addr = plat->addr;
-       priv->base = map_sysmem(CONFIG_SYS_IMMR + plat->addr, size);
+       priv->base = map_sysmem(plat->addr, size);
 
        if (!priv->base)
                return -ENOMEM;