[PATCH] omap3: igep0x00: Switch to the I2C driver model

Enric Balletbo i Serra eballetb at redhat.com
Wed Jul 10 08:53:11 CEST 2024


Hi Javier,

Many thanks for still maintaining this.


On Sat, Jun 29, 2024 at 1:17 AM Javier Martinez Canillas
<javier at dowhile0.org> wrote:
>
> From: Javier Martinez Canillas <javierm at redhat.com>
>
> When building with the igep00x0_defconfig, the following warning is shown:
>
>   $ make -j $(nproc)
>   ...
>     LD      spl/u-boot-spl
>     OBJCOPY spl/u-boot-spl-nodtb.bin
>     SYM     spl/u-boot-spl.sym
>     CAT     spl/u-boot-spl-dtb.bin
>     COPY    spl/u-boot-spl.bin
>     MKIMAGE MLO
>   ===================== WARNING ======================
>   This board does not use CONFIG_DM_I2C (Driver Model
>   for I2C drivers). Please update the board to use
>   CONFIG_DM_I2C before the v2022.04 release. Failure to
>   update by the deadline may result in board removal.
>   See doc/develop/driver-model/migration.rst for more info.
>   ====================================================
>
> The only reason why I2C is enabled for the IGEP boards is that the TWL4030
> driver requires it.
>
> But both the TWL4034 and the OMAP I2C controller drivers were converted to
> the driver model by commits daa69ffe3d4d ("drivers: i2c: omap24xx_i2c:
> adopt omap_i2c driver to driver model") and fb1b7712ad3f power: make most
> tps drivers and the twl4030 driver compatible with DM_I2C") respectively.
>
> So there's no reason anymore to keep using the I2C legacy API and instead
> the DM_I2C option could just be enabled.
>
> Signed-off-by: Javier Martinez Canillas <javierm at redhat.com>

The changes looks good to me so:

Reviewed-by: Enric Balletbo i Serra <eballetbo at gmail.com>


> ---
>
>  configs/igep00x0_defconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configs/igep00x0_defconfig b/configs/igep00x0_defconfig
> index 87fd2797eace..473891607b0f 100644
> --- a/configs/igep00x0_defconfig
> +++ b/configs/igep00x0_defconfig
> @@ -68,8 +68,8 @@ CONFIG_SYS_RELOC_GD_ENV_ADDR=y
>  CONFIG_VERSION_VARIABLE=y
>  # CONFIG_NET is not set
>  CONFIG_SPL_DM=y
> -CONFIG_SYS_I2C_LEGACY=y
> -CONFIG_SPL_SYS_I2C_LEGACY=y
> +CONFIG_DM_I2C=y
> +CONFIG_I2C_SET_DEFAULT_BUS_NUM=y
>  CONFIG_MMC_OMAP_HS=y
>  CONFIG_MTD=y
>  CONFIG_SYS_MTDPARTS_RUNTIME=y
> --
> 2.45.2
>



More information about the U-Boot mailing list