Hello Tom,
On 01.07.26 18:18, Tom Rini wrote:
On Wed, Jul 01, 2026 at 05:55:14PM +0200, Julien Stephan wrote:
CMD_I2C depends on the DM I2C API but currently does not enforce
DM_I2C in Kconfig.
This can lead to link errors caused by unresolved i2c_* symbols when
CMD_I2C is enabled without DM_I2C.
Add the missing dependency to prevent unsupported configurations.
Signed-off-by: Julien Stephan <[email protected]>
---
cmd/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/cmd/Kconfig b/cmd/Kconfig
index 032e55e8127..636891ed493 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1392,6 +1392,7 @@ config CMD_IOTRACE
config CMD_I2C
bool "i2c"
+ depends on DM_I2C
help
I2C support.
This removes the command from the handful of still legacy I2C boards:
$ ./tools/qconfig.py -f CMD_I2C ~DM_I2C
15 matches
MPC837XERDB cl-som-imx7 d2net_v2 db-88f6720 db-88f6820-gp inetspace_v2
ls2080aqds_nand ls2080aqds_qspi ls2080ardb_nand net2big_v2 netspace_lite_v2
netspace_max_v2 netspace_mini_v2 netspace_v2 work_92105
So the depends on needs an "|| ..." in there, for the I2C but not DM_I2C
case. Thanks.
Thanks for checking this!
bye,
Heiko
--
Nabla Software Engineering
HRB 40522 Augsburg
Phone: +49 821 45592596
E-Mail: [email protected]
Geschäftsführer : Stefano Babic