2023-09-14 14:39:45

by Ulf Hansson

Subject: [PATCH 12/17] pmdomain: st: Add a Kconfig option for the ux500 power domain

We shouldn't really use the CONFIG_ARCH_U8500 option directly, but rather
have our own dedicated Kconfig option, so let's add that.

Cc: Linus Walleij <[email protected]>
Signed-off-by: Ulf Hansson <[email protected]>
---
drivers/pmdomain/Kconfig | 1 +
drivers/pmdomain/st/Kconfig | 5 +++++
drivers/pmdomain/st/Makefile | 2 +-
3 files changed, 7 insertions(+), 1 deletion(-)
create mode 100644 drivers/pmdomain/st/Kconfig

diff --git a/drivers/pmdomain/Kconfig b/drivers/pmdomain/Kconfig
index 1e2c5f531638..cfe9ea754062 100644
--- a/drivers/pmdomain/Kconfig
+++ b/drivers/pmdomain/Kconfig
@@ -11,5 +11,6 @@ source "drivers/pmdomain/qcom/Kconfig"
source "drivers/pmdomain/renesas/Kconfig"
source "drivers/pmdomain/rockchip/Kconfig"
source "drivers/pmdomain/samsung/Kconfig"
+source "drivers/pmdomain/st/Kconfig"

endmenu
diff --git a/drivers/pmdomain/st/Kconfig b/drivers/pmdomain/st/Kconfig
new file mode 100644
index 000000000000..a77a70211f61
--- /dev/null
+++ b/drivers/pmdomain/st/Kconfig
@@ -0,0 +1,5 @@
+# SPDX-License-Identifier: GPL-2.0-only
+config UX500_PM_DOMAIN
+ bool "ST-Ericsson ux500 Power Domain"
+ depends on ARCH_U8500 || COMPILE_TEST
+ default ARCH_U8500
diff --git a/drivers/pmdomain/st/Makefile b/drivers/pmdomain/st/Makefile
index 8fa5f9855460..6d8b617eb834 100644
--- a/drivers/pmdomain/st/Makefile
+++ b/drivers/pmdomain/st/Makefile
@@ -1,2 +1,2 @@
# SPDX-License-Identifier: GPL-2.0-only
-obj-$(CONFIG_ARCH_U8500) += ste-ux500-pm-domain.o
+obj-$(CONFIG_UX500_PM_DOMAIN) += ste-ux500-pm-domain.o
--
2.34.1

2023-09-14 18:46:49

by Linus Walleij

Subject: Re: [PATCH 12/17] pmdomain: st: Add a Kconfig option for the ux500 power domain

On Thu, Sep 14, 2023 at 1:18 PM Ulf Hansson <[email protected]> wrote:

> We shouldn't really use the CONFIG_ARCH_U8500 option directly, but rather
> have our own dedicated Kconfig option, so let's add that.
>
> Cc: Linus Walleij <[email protected]>
> Signed-off-by: Ulf Hansson <[email protected]>

This looks fine!
Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij