[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