[U-Boot] [PATCH 05/21] sunxi: soft-i2c: Fix gpio handling to work with the device-model

[Simon Glass]

Hi Hans,

On 24 April 2015 at 07:48, Hans de Goede <hdegoede at redhat.com> wrote:
> i2c_init_board() gets called before the device-model (gpio) code is
> initialized, so move the setup of the soft-i2c pins out of i2c_init_board()
> and into board_init(), at which time the device-model setup has been done.
>
> Also add proper error checking and properly request the gpios as that is
> mandatory with the device-model.
>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
>  board/sunxi/board.c | 44 ++++++++++++++++++++++++++++++++++++++------
>  1 file changed, 38 insertions(+), 6 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

But see i2c-gpio.c which should handle all of this once you move to device tree.

>
> diff --git a/board/sunxi/board.c b/board/sunxi/board.c
> index 171f0bc..6b93f92 100644
> --- a/board/sunxi/board.c
> +++ b/board/sunxi/board.c
> @@ -29,6 +29,7 @@
>  #include <asm/arch/gpio.h>
>  #include <asm/arch/mmc.h>
>  #include <asm/arch/usbc.h>
> +#include <asm/gpio.h>
>  #include <asm/io.h>
>  #include <linux/usb/musb.h>
>  #include <net.h>
> @@ -37,6 +38,41 @@
>  /* So that we can use pin names in Kconfig and sunxi_name_to_gpio() */
>  int soft_i2c_gpio_sda;
>  int soft_i2c_gpio_scl;
> +
> +static int soft_i2c_board_init(void)
> +{
> +       int ret;
> +
> +       soft_i2c_gpio_sda = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_PANEL_I2C_SDA);
> +       if (soft_i2c_gpio_sda < 0) {
> +               printf("Error invalid soft i2c sda pin: '%s', err %d\n",
> +                      CONFIG_VIDEO_LCD_PANEL_I2C_SDA, soft_i2c_gpio_sda);
> +               return soft_i2c_gpio_sda;
> +       }
> +       ret = gpio_request(soft_i2c_gpio_sda, "soft-i2c-sda");
> +       if (ret) {
> +               printf("Error requesting soft i2c sda pin: '%s', err %d\n",
> +                      CONFIG_VIDEO_LCD_PANEL_I2C_SDA, ret);
> +               return ret;
> +       }
> +
> +       soft_i2c_gpio_scl = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_PANEL_I2C_SCL);
> +       if (soft_i2c_gpio_scl < 0) {
> +               printf("Error invalid soft i2c scl pin: '%s', err %d\n",
> +                      CONFIG_VIDEO_LCD_PANEL_I2C_SCL, soft_i2c_gpio_scl);
> +               return soft_i2c_gpio_scl;
> +       }
> +       ret = gpio_request(soft_i2c_gpio_scl, "soft-i2c-scl");
> +       if (ret) {
> +               printf("Error requesting soft i2c scl pin: '%s', err %d\n",
> +                      CONFIG_VIDEO_LCD_PANEL_I2C_SCL, ret);
> +               return ret;
> +       }
> +
> +       return 0;
> +}
> +#else
> +static int soft_i2c_board_init(void) { return 0; }
>  #endif
>
>  DECLARE_GLOBAL_DATA_PTR;
> @@ -57,7 +93,8 @@ int board_init(void)
>                 asm volatile("mcr p15, 0, %0, c14, c0, 0" : : "r"(24000000));
>         }
>
> -       return 0;
> +       /* Uses dm gpio code so do this here and not in i2c_init_board() */
> +       return soft_i2c_board_init();
>  }
>
>  int dram_init(void)
> @@ -351,11 +388,6 @@ void i2c_init_board(void)
>         clock_twi_onoff(4, 1);
>  #endif
>  #endif
> -
> -#if defined CONFIG_VIDEO_LCD_PANEL_I2C && !(defined CONFIG_SPL_BUILD)
> -       soft_i2c_gpio_sda = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_PANEL_I2C_SDA);
> -       soft_i2c_gpio_scl = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_PANEL_I2C_SCL);
> -#endif
>  }
>
>  #ifdef CONFIG_SPL_BUILD
> --
> 2.3.5
>

Regards,
Simon