[PATCH 03/11] power: pmic: Consistently depend on DM_PMIC

Igor Opaniuk igor.opaniuk at foundries.io
Wed Sep 15 18:08:20 CEST 2021


Hi Samuel,

On Sun, Aug 22, 2021 at 2:06 AM Samuel Holland <samuel at sholland.org> wrote:
>
> Kconfig symbols for two PMIC drivers (PMIC_AS3722 and DM_PMIC_MC34708)
> were missing a dependency on DM_PMIC. To fix this inconsistency, and to
> keep it from happening again, wrap the driver section with "if DM_PMIC"
> instead of using a "depends on DM_PMIC" clause for each driver.
>
> Signed-off-by: Samuel Holland <samuel at sholland.org>
> ---
>
>  drivers/power/pmic/Kconfig | 43 +++++++-------------------------------
>  1 file changed, 8 insertions(+), 35 deletions(-)
>
> diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
> index fd6648b313e..56c0ef70312 100644
> --- a/drivers/power/pmic/Kconfig
> +++ b/drivers/power/pmic/Kconfig
> @@ -10,9 +10,10 @@ config DM_PMIC
>         - 'drivers/power/pmic/pmic-uclass.c'
>         - 'include/power/pmic.h'
>
> +if DM_PMIC
> +
>  config PMIC_CHILDREN
>         bool "Allow child devices for PMICs"
> -       depends on DM_PMIC
>         default y
>         ---help---
>         This allows PMICs to support child devices (such as regulators) in
> @@ -22,7 +23,6 @@ config PMIC_CHILDREN
>
>  config SPL_PMIC_CHILDREN
>         bool "Allow child devices for PMICs in SPL"
> -       depends on DM_PMIC
>         default y
>         ---help---
>         This allows PMICs to support child devices (such as regulators) in
> @@ -33,7 +33,6 @@ config SPL_PMIC_CHILDREN
>
>  config PMIC_AB8500
>         bool "Enable driver for ST-Ericsson AB8500 PMIC via PRCMU"
> -       depends on DM_PMIC
>         select REGMAP
>         select SYSCON
>         help
> @@ -43,7 +42,7 @@ config PMIC_AB8500
>
>  config PMIC_ACT8846
>         bool "Enable support for the active-semi 8846 PMIC"
> -       depends on DM_PMIC && DM_I2C
> +       depends on DM_I2C
>         ---help---
>         This PMIC includes 4 DC/DC step-down buck regulators and 8 low-dropout
>         regulators (LDOs). It also provides some GPIO, reset and battery
> @@ -52,14 +51,13 @@ config PMIC_ACT8846
>
>  config DM_PMIC_DA9063
>         bool "Enable Driver Model for the Dialog DA9063 PMIC"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC DA9063. The driver implements read/write operations.
>
>  config SPL_DM_PMIC_DA9063
>         bool "Enable Driver Model for the Dialog DA9063 PMIC in SPL"
> -       depends on DM_PMIC && SPL
> +       depends on SPL
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC DA9063. The driver implements read/write operations.
> @@ -74,14 +72,12 @@ config PMIC_AS3722
>
>  config DM_PMIC_BD71837
>         bool "Enable Driver Model for PMIC BD71837"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC BD71837. The driver implements read/write operations.
>
>  config SPL_DM_PMIC_BD71837
>         bool "Enable Driver Model for PMIC BD71837 in SPL stage"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass
>           features for PMIC BD71837. The driver implements read/write
> @@ -89,7 +85,7 @@ config SPL_DM_PMIC_BD71837
>
>  config DM_PMIC_FAN53555
>         bool "Enable support for OnSemi FAN53555"
> -       depends on DM_PMIC && DM_REGULATOR && DM_I2C
> +       depends on DM_REGULATOR && DM_I2C
>         select DM_REGULATOR_FAN53555
>         help
>           This config enables implementation of driver-model PMIC
> @@ -103,14 +99,12 @@ config DM_PMIC_FAN53555
>
>  config DM_PMIC_MP5416
>         bool "Enable Driver Model for PMIC MP5416"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC MP5416. The driver implements read/write operations.
>
>  config SPL_DM_PMIC_MP5416
>         bool "Enable Driver Model for PMIC MP5416 in SPL stage"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass
>           features for PMIC MP5416. The driver implements read/write
> @@ -118,56 +112,48 @@ config SPL_DM_PMIC_MP5416
>
>  config DM_PMIC_PCA9450
>         bool "Enable Driver Model for PMIC PCA9450"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC PCA9450. The driver implements read/write operations.
>
>  config SPL_DM_PMIC_PCA9450
>         bool "Enable Driver Model for PMIC PCA9450"
> -       depends on DM_PMIC
>         help
>           This config enables implementation of driver-model pmic uclass features
>           for PMIC PCA9450 in SPL. The driver implements read/write operations.
>
>  config DM_PMIC_PFUZE100
>         bool "Enable Driver Model for PMIC PFUZE100"
> -       depends on DM_PMIC
>         ---help---
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC PFUZE100. The driver implements read/write operations.
>
>  config SPL_DM_PMIC_PFUZE100
>         bool "Enable Driver Model for PMIC PFUZE100 in SPL"
> -       depends on DM_PMIC
>         ---help---
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC PFUZE100 in SPL. The driver implements read/write operations.
>
>  config DM_PMIC_MAX77686
>         bool "Enable Driver Model for PMIC MAX77686"
> -       depends on DM_PMIC
>         ---help---
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC MAX77686. The driver implements read/write operations.
>
>  config DM_PMIC_MAX8998
>         bool "Enable Driver Model for PMIC MAX8998"
> -       depends on DM_PMIC
>         ---help---
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC MAX8998. The driver implements read/write operations.
>
>  config DM_PMIC_MC34708
>         bool "Enable Driver Model for PMIC MC34708"
> -       depends on DM_PMIC
>         help
>          This config enables implementation of driver-model pmic uclass features
>          for PMIC MC34708. The driver implements read/write operations.
>
>  config PMIC_MAX8997
>         bool "Enable Driver Model for PMIC MAX8997"
> -       depends on DM_PMIC
>         ---help---
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC MAX8997. The driver implements read/write operations.
> @@ -182,7 +168,6 @@ config PMIC_MAX8997
>
>  config PMIC_PM8916
>         bool "Enable Driver Model for Qualcomm PM8916 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The PM8916 is a PMIC connected to one (or several) processors
>         with SPMI bus. It has 2 slaves with several peripherals:
> @@ -198,7 +183,6 @@ config PMIC_PM8916
>
>  config PMIC_RK8XX
>         bool "Enable support for Rockchip PMIC RK8XX"
> -       depends on DM_PMIC
>         ---help---
>         The Rockchip RK808 PMIC provides four buck DC-DC convertors, 8 LDOs,
>         an RTC and two low Rds (resistance (drain to source)) switches. It is
> @@ -207,7 +191,6 @@ config PMIC_RK8XX
>
>  config PMIC_S2MPS11
>         bool "Enable Driver Model for PMIC Samsung S2MPS11"
> -       depends on DM_PMIC
>         ---help---
>         The Samsung S2MPS11 PMIC provides:
>          - 38 adjustable LDO regulators
> @@ -221,7 +204,6 @@ config PMIC_S2MPS11
>
>  config DM_PMIC_SANDBOX
>         bool "Enable Driver Model for emulated Sandbox PMIC"
> -       depends on DM_PMIC
>         ---help---
>         Enable the driver for Sandbox PMIC emulation. The emulated PMIC device
>         depends on two drivers:
> @@ -246,7 +228,6 @@ config DM_PMIC_SANDBOX
>
>  config PMIC_S5M8767
>         bool "Enable Driver Model for the Samsung S5M8767 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The S5M8767 PMIC provides a large array of LDOs and BUCKs for use
>         as a SoC power controller. It also provides 32KHz clock outputs. This
> @@ -255,7 +236,6 @@ config PMIC_S5M8767
>
>  config PMIC_RN5T567
>         bool "Enable driver for Ricoh RN5T567 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The RN5T567 is a PMIC with 4 step-down DC/DC converters, 5 LDO
>         regulators Real-Time Clock and 4 GPIOs. This driver provides
> @@ -263,7 +243,6 @@ config PMIC_RN5T567
>
>  config PMIC_TPS65090
>         bool "Enable driver for Texas Instruments TPS65090 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The TPS65090 is a PMIC containing several LDOs, DC to DC convertors,
>         FETs and a battery charger. This driver provides register access
> @@ -272,21 +251,18 @@ config PMIC_TPS65090
>
>  config PMIC_PALMAS
>         bool "Enable driver for Texas Instruments PALMAS PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The PALMAS is a PMIC containing several LDOs, SMPS.
>         This driver binds the pmic children.
>
>  config PMIC_LP873X
>         bool "Enable driver for Texas Instruments LP873X PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The LP873X is a PMIC containing couple of LDOs and couple of SMPS.
>         This driver binds the pmic children.
>
>  config PMIC_LP87565
>         bool "Enable driver for Texas Instruments LP87565 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The LP87565 is a PMIC containing a bunch of SMPS.
>         This driver binds the pmic children.
> @@ -300,7 +276,6 @@ config POWER_MC34VR500
>
>  config DM_PMIC_TPS65910
>         bool "Enable driver for Texas Instruments TPS65910 PMIC"
> -       depends on DM_PMIC
>         ---help---
>         The TPS65910 is a PMIC containing 3 buck DC-DC converters, one boost
>         DC-DC converter, 8 LDOs and a RTC. This driver binds the SMPS and LDO
> @@ -308,7 +283,7 @@ config DM_PMIC_TPS65910
>
>  config PMIC_STPMIC1
>         bool "Enable support for STMicroelectronics STPMIC1 PMIC"
> -       depends on DM_PMIC && DM_I2C
> +       depends on DM_I2C
>         select SYSRESET_CMD_POWEROFF if CMD_POWEROFF && !ARM_PSCI_FW
>         ---help---
>         The STPMIC1 PMIC provides 4 BUCKs, 6 LDOs, 1 VREF and 2 power switches.
> @@ -317,28 +292,26 @@ config PMIC_STPMIC1
>
>  config SPL_PMIC_PALMAS
>         bool "Enable driver for Texas Instruments PALMAS PMIC"
> -       depends on DM_PMIC
>         help
>         The PALMAS is a PMIC containing several LDOs, SMPS.
>         This driver binds the pmic children in SPL.
>
>  config SPL_PMIC_LP873X
>         bool "Enable driver for Texas Instruments LP873X PMIC"
> -       depends on DM_PMIC
>         help
>         The LP873X is a PMIC containing couple of LDOs and couple of SMPS.
>         This driver binds the pmic children in SPL.
>
>  config SPL_PMIC_LP87565
>         bool "Enable driver for Texas Instruments LP87565 PMIC"
> -       depends on DM_PMIC
>         help
>         The LP87565 is a PMIC containing a bunch of SMPS.
>         This driver binds the pmic children in SPL.
>
>  config PMIC_TPS65941
>         bool "Enable driver for Texas Instruments TPS65941 PMIC"
> -       depends on DM_PMIC
>         help
>         The TPS65941 is a PMIC containing a bunch of SMPS & LDOs.
>         This driver binds the pmic children.
> +
> +endif
> --
> 2.31.1
>

Reviewed-by: Igor Opaniuk <igor.opaniuk at foundries.io>

-- 
Best regards - Freundliche Grüsse - Meilleures salutations

Igor Opaniuk
Embedded Software Engineer
T:  +380 938364067
E: igor.opaniuk at foundries.io
W: www.foundries.io


More information about the U-Boot mailing list