[PATCH 0/1] Enable stm32_pwr regulator driver for stm32mp157

[PATCH 0/1] Enable stm32_pwr regulator driver for stm32mp157

[Pascal Paillet]

Enable the STM32 PWR regulator driver for stm32mp157 machine.

Pascal Paillet (1):
  regulator: stm32_pwr: Enable driver for stm32mp157

 drivers/regulator/Kconfig | 1 +
 1 file changed, 1 insertion(+)

-- 
2.17.1

[PATCH 1/1] regulator: stm32_pwr: Enable driver for stm32mp157

[Pascal Paillet]

Enable stm32 pwr regulators for stm32mp157 machine.

Signed-off-by: Pascal Paillet <p.paillet@st.com>
---
 drivers/regulator/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 3ee63531f6d5..ed10f4f46fdf 100644
--- a/drivers/regulator/Kconfig
+++ b/drivers/regulator/Kconfig
@@ -875,6 +875,7 @@ config REGULATOR_STM32_VREFBUF
 config REGULATOR_STM32_PWR
 	bool "STMicroelectronics STM32 PWR"
 	depends on ARCH_STM32 || COMPILE_TEST
+	default MACH_STM32MP157
 	help
 	  This driver supports internal regulators (1V1, 1V8, 3V3) in the
 	  STMicroelectronics STM32 chips.
-- 
2.17.1

Re: [PATCH 1/1] regulator: stm32_pwr: Enable driver for stm32mp157

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]

On Thu, Oct 24, 2019 at 05:41:21PM +0200, Pascal Paillet wrote:

> @@ -875,6 +875,7 @@ config REGULATOR_STM32_VREFBUF
>  config REGULATOR_STM32_PWR
>  	bool "STMicroelectronics STM32 PWR"
>  	depends on ARCH_STM32 || COMPILE_TEST
> +	default MACH_STM32MP157
>  	help

This isn't the sort of stuff we usually put in Kconfig, we usually just
leave it in the defconfig - why do things differently here?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH 0/1] Enable stm32_pwr regulator driver for stm32mp157

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 438 bytes --]

On Thu, Oct 24, 2019 at 05:41:20PM +0200, Pascal Paillet wrote:

> Enable the STM32 PWR regulator driver for stm32mp157 machine.

Please don't send cover letters for single patches, if there is anything
that needs saying put it in the changelog of the patch or after the ---
if it's administrative stuff.  This reduces mail volume and ensures that 
any important information is recorded in the changelog rather than being
lost. 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]