Re: [PATCH] extcon: max14526: depends on I2C to prevent build warning/errors

[Chanwoo Choi]

Hi,

Applied it. Thanks.

On Fri, May 16, 2025 at 9:26 AM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:
>
> Make EXTCON_MAX14526 depend on I2C to address a kconfig warning and
> subsequent build errors:
>
> WARNING: unmet direct dependencies detected for REGMAP_I2C
>   Depends on [m]: I2C [=m]
>   Selected by [y]:
>   - EXTCON_MAX14526 [=y] && EXTCON [=y]
>
> ld: vmlinux.o: in function `max14526_driver_init':
> extcon-max14526.c:(.init.text+0x77b0e): undefined reference to `i2c_register_driver'
> ld: vmlinux.o: in function `max14526_driver_exit':
> extcon-max14526.c:(.exit.text+0x467c): undefined reference to `i2c_del_driver'
>
> Fixes: c2aeb8647e53 ("extcon: Add basic support for Maxim MAX14526 MUIC")
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Svyatoslav Ryhel <clamor95@xxxxxxxxx>
> Cc: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
> Cc: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>
> ---
>  drivers/extcon/Kconfig |    1 +
>  1 file changed, 1 insertion(+)
>
> --- linux-next-20250515.orig/drivers/extcon/Kconfig
> +++ linux-next-20250515/drivers/extcon/Kconfig
> @@ -136,6 +136,7 @@ config EXTCON_MAX8997
>
>  config EXTCON_MAX14526
>         tristate "Maxim MAX14526 EXTCON Support"
> +       depends on I2C
>         select IRQ_DOMAIN
>         select REGMAP_I2C
>         help
>


-- 
Best Regards,
Chanwoo Choi
Samsung Electronics