Re: [PATCH v4 6/6] hwmon: (spd5118) Add configuration option for auto-detection

From: Guenter Roeck
Date: Tue Jun 04 2024 - 10:04:40 EST


On 6/4/24 00:44, Wolfram Sang wrote:
+#ifdef CONFIG_SENSORS_SPD5118_DETECT
.detect = spd5118_detect,
+#endif
.address_list = normal_i2c,
};

So, I think we have too many Kconfig symbols already. What about using
!DMI here?


That would be nice, but i2c_register_spd() is only called from i2c-i801.c
and, with the recent patch, from i2c-piix4.c. DMI is also supported on
arm, arm64, loongarch, and mips. i2c_register_spd() will not be called
on those systems or, more generally, if the adapter used to connect to
the DIMMs is not one of those two.

On top of that, it may be desirable to disable it for a system with DMI
support if that system boots from devicetree. I don't know if such a
system exists, but I would not want to bet that it doesn't.

Thanks,
Guenter