[PATCH] platform/mellanox: MLXREG_DPU depends on I2C

Randy Dunlap posted 1 patch 9 months, 1 week ago
drivers/platform/mellanox/Kconfig |    1 +
1 file changed, 1 insertion(+)
[PATCH] platform/mellanox: MLXREG_DPU depends on I2C
Posted by Randy Dunlap 9 months, 1 week ago
WARNING: unmet direct dependencies detected for REGMAP_I2C
  Depends on [m]: I2C [=m]
  Selected by [y]:
  - MLXREG_DPU [=y] && MELLANOX_PLATFORM [=y]

Without the dependency, REGMAP_I2C=y since MLXREG_DPU=y.
Since CONFIG_I2C=m, this causes many (> 230) unmet dependency warnings.
E.g.:
  Selected by [m]:
  - EEPROM_AT24 [=m] && I2C [=m] && SYSFS [=y]
  - SERIAL_MAX310X [=m] && TTY [=y] && HAS_IOMEM [=y] && SPI_MASTER [=y] && I2C [=m]
  - PINCTRL_AW9523 [=m] && PINCTRL [=y] && OF [=y] && I2C [=m]
  - PINCTRL_CY8C95X0 [=m] && PINCTRL [=y] && I2C [=m]
  - GPIO_FXL6408 [=m] && GPIOLIB [=y] && I2C [=m]
  - GPIO_DS4520 [=m] && GPIOLIB [=y] && I2C [=m]
  - GPIO_PCA953X [=m] && GPIOLIB [=y] && I2C [=m]
  - IP5XXX_POWER [=m] && POWER_SUPPLY [=y] && I2C [=m]
  - BATTERY_CW2015 [=m] && POWER_SUPPLY [=y] && I2C [=m]
  - CHARGER_SBS [=m] && POWER_SUPPLY [=y] && I2C [=m]

Fixes: 3e75f2954116 ("platform/mellanox: mlxreg-dpu: Add initial support for Nvidia DPU")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Vadim Pasternak <vadimp@nvidia.com>
Cc: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: platform-driver-x86@vger.kernel.org
---
 drivers/platform/mellanox/Kconfig |    1 +
 1 file changed, 1 insertion(+)

--- linux-next-20250501.orig/drivers/platform/mellanox/Kconfig
+++ linux-next-20250501/drivers/platform/mellanox/Kconfig
@@ -29,6 +29,7 @@ config MLX_PLATFORM
 
 config MLXREG_DPU
 	tristate "Nvidia Data Processor Unit platform driver support"
+	depends on I2C
 	select REGMAP_I2C
 	help
 	  This driver provides support for the Nvidia BF3 Data Processor Units,
Re: [PATCH] platform/mellanox: MLXREG_DPU depends on I2C
Posted by Ilpo Järvinen 9 months, 1 week ago
On Thu, 1 May 2025, Randy Dunlap wrote:

> WARNING: unmet direct dependencies detected for REGMAP_I2C
>   Depends on [m]: I2C [=m]
>   Selected by [y]:
>   - MLXREG_DPU [=y] && MELLANOX_PLATFORM [=y]
> 
> Without the dependency, REGMAP_I2C=y since MLXREG_DPU=y.
> Since CONFIG_I2C=m, this causes many (> 230) unmet dependency warnings.
> E.g.:
>   Selected by [m]:
>   - EEPROM_AT24 [=m] && I2C [=m] && SYSFS [=y]
>   - SERIAL_MAX310X [=m] && TTY [=y] && HAS_IOMEM [=y] && SPI_MASTER [=y] && I2C [=m]
>   - PINCTRL_AW9523 [=m] && PINCTRL [=y] && OF [=y] && I2C [=m]
>   - PINCTRL_CY8C95X0 [=m] && PINCTRL [=y] && I2C [=m]
>   - GPIO_FXL6408 [=m] && GPIOLIB [=y] && I2C [=m]
>   - GPIO_DS4520 [=m] && GPIOLIB [=y] && I2C [=m]
>   - GPIO_PCA953X [=m] && GPIOLIB [=y] && I2C [=m]
>   - IP5XXX_POWER [=m] && POWER_SUPPLY [=y] && I2C [=m]
>   - BATTERY_CW2015 [=m] && POWER_SUPPLY [=y] && I2C [=m]
>   - CHARGER_SBS [=m] && POWER_SUPPLY [=y] && I2C [=m]
> 
> Fixes: 3e75f2954116 ("platform/mellanox: mlxreg-dpu: Add initial support for Nvidia DPU")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Vadim Pasternak <vadimp@nvidia.com>
> Cc: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> Cc: Hans de Goede <hdegoede@redhat.com>
> Cc: platform-driver-x86@vger.kernel.org
> ---
>  drivers/platform/mellanox/Kconfig |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- linux-next-20250501.orig/drivers/platform/mellanox/Kconfig
> +++ linux-next-20250501/drivers/platform/mellanox/Kconfig
> @@ -29,6 +29,7 @@ config MLX_PLATFORM
>  
>  config MLXREG_DPU
>  	tristate "Nvidia Data Processor Unit platform driver support"
> +	depends on I2C
>  	select REGMAP_I2C
>  	help
>  	  This driver provides support for the Nvidia BF3 Data Processor Units,
> 

Thanks for the patch but the review-ilpo-next branch already has this 
fixed, just waiting for lkp to build test it until it propagates into 
for-next.

-- 
 i.