mbox series

[v2,00/17] pmdomain: Move Kconfig options/files to the pmdomain subsystem

Message ID 20230915092003.658361-1-ulf.hansson@linaro.org
Headers show
Series pmdomain: Move Kconfig options/files to the pmdomain subsystem | expand

Message

Ulf Hansson Sept. 15, 2023, 9:19 a.m. UTC
Changes in v2:
	- Posted as a proper series, I screwed up last time.
	- Fixed bug in patch1, according to comments from Geert.
	- Moved to use "PM domains" in menus/submenus.
	- Updated MAINTAINERS for starfive, according comments from Conor/Walker.
	- Added some tags.

Recently we added a new subsystem to host pmdomain (genpd) providers and moved
a bunch of code for them in there. However, their corresponding Kconfig
options/files was not moved in that first step, but instead that is taken care
of in this series. Ideally it should be easier to keep Kconfig options closer
to their corresponding implementations.

Note that, I am planning to queue this up via my pmdomain tree [1] as soon as I
have received enough of reviewed/acked-by tags from soc maintainers. If you
have reasons to belive that this can be problem, due to conflicts etc, please
let me know so we can figure out a way forward.

Kind regards
Uffe

[1]
git git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/linux-pm.git next


Ulf Hansson (17):
  pmdomain: Prepare to move Kconfig files into the pmdomain subsystem
  pmdomain: actions: Move Kconfig file to the pmdomain subsystem
  pmdomain: amlogic: Move Kconfig options to the pmdomain subsystem
  pmdomain: apple: Move Kconfig option to the pmdomain subsystem
  pmdomain: bcm: Move Kconfig options to the pmdomain subsystem
  pmdomain: imx: Move Kconfig options to the pmdomain subsystem
  pmdomain: mediatek: Move Kconfig options to the pmdomain subsystem
  pmdomain: qcom: Move Kconfig options to the pmdomain subsystem
  pmdomain: renesas: Move Kconfig options to the pmdomain subsystem
  pmdomain: rockchip: Move Kconfig option to the pmdomain subsystem
  pmdomain: samsung: Move Kconfig option to the pmdomain subsystem
  pmdomain: st: Add a Kconfig option for the ux500 power domain
  pmdomain: starfive: Move Kconfig file to the pmdomain subsystem
  pmdomain: sunxi: Move Kconfig option to the pmdomain subsystem
  pmdomain: tegra: Move Kconfig option to the pmdomain subsystem
  pmdomain: ti: Move and add Kconfig options to the pmdomain subsystem
  pmdomain: xilinx: Move Kconfig option to the pmdomain subsystem

 MAINTAINERS                                |   5 +-
 drivers/Kconfig                            |   2 +
 drivers/firmware/imx/Kconfig               |   6 --
 drivers/pmdomain/Kconfig                   |  21 ++++
 drivers/{soc => pmdomain}/actions/Kconfig  |   0
 drivers/pmdomain/amlogic/Kconfig           |  39 ++++++++
 drivers/pmdomain/apple/Kconfig             |  18 ++++
 drivers/pmdomain/bcm/Kconfig               |  46 +++++++++
 drivers/pmdomain/imx/Kconfig               |  29 ++++++
 drivers/pmdomain/mediatek/Kconfig          |  29 ++++++
 drivers/pmdomain/qcom/Kconfig              |  41 ++++++++
 drivers/pmdomain/renesas/Kconfig           | 109 +++++++++++++++++++++
 drivers/pmdomain/rockchip/Kconfig          |  16 +++
 drivers/pmdomain/samsung/Kconfig           |   8 ++
 drivers/pmdomain/st/Kconfig                |   5 +
 drivers/pmdomain/st/Makefile               |   2 +-
 drivers/{soc => pmdomain}/starfive/Kconfig |   0
 drivers/pmdomain/sunxi/Kconfig             |  10 ++
 drivers/pmdomain/tegra/Kconfig             |   6 ++
 drivers/pmdomain/ti/Kconfig                |  22 +++++
 drivers/pmdomain/ti/Makefile               |   2 +-
 drivers/pmdomain/xilinx/Kconfig            |  10 ++
 drivers/soc/Kconfig                        |   2 -
 drivers/soc/amlogic/Kconfig                |  35 -------
 drivers/soc/apple/Kconfig                  |  13 ---
 drivers/soc/bcm/Kconfig                    |  42 --------
 drivers/soc/imx/Kconfig                    |  19 ----
 drivers/soc/mediatek/Kconfig               |  23 -----
 drivers/soc/qcom/Kconfig                   |  37 -------
 drivers/soc/renesas/Kconfig                | 105 --------------------
 drivers/soc/rockchip/Kconfig               |  12 ---
 drivers/soc/samsung/Kconfig                |   4 -
 drivers/soc/sunxi/Kconfig                  |   9 --
 drivers/soc/tegra/Kconfig                  |   5 -
 drivers/soc/ti/Kconfig                     |  12 ---
 drivers/soc/xilinx/Kconfig                 |   9 --
 36 files changed, 415 insertions(+), 338 deletions(-)
 create mode 100644 drivers/pmdomain/Kconfig
 rename drivers/{soc => pmdomain}/actions/Kconfig (100%)
 create mode 100644 drivers/pmdomain/amlogic/Kconfig
 create mode 100644 drivers/pmdomain/apple/Kconfig
 create mode 100644 drivers/pmdomain/bcm/Kconfig
 create mode 100644 drivers/pmdomain/imx/Kconfig
 create mode 100644 drivers/pmdomain/mediatek/Kconfig
 create mode 100644 drivers/pmdomain/qcom/Kconfig
 create mode 100644 drivers/pmdomain/renesas/Kconfig
 create mode 100644 drivers/pmdomain/rockchip/Kconfig
 create mode 100644 drivers/pmdomain/samsung/Kconfig
 create mode 100644 drivers/pmdomain/st/Kconfig
 rename drivers/{soc => pmdomain}/starfive/Kconfig (100%)
 create mode 100644 drivers/pmdomain/sunxi/Kconfig
 create mode 100644 drivers/pmdomain/tegra/Kconfig
 create mode 100644 drivers/pmdomain/ti/Kconfig
 create mode 100644 drivers/pmdomain/xilinx/Kconfig

Comments

Conor Dooley Sept. 15, 2023, 9:29 a.m. UTC | #1
On Fri, Sep 15, 2023 at 11:19:59AM +0200, Ulf Hansson wrote:
> The Kconfig belongs closer to the corresponding implementation, hence let's
> move it from the soc subsystem to the pmdomain subsystem.

Thanks for updating the MAINTAINERS bits.

> Cc: Walker Chen <walker.chen@starfivetech.com>
> Cc: Conor Dooley <conor@kernel.org>
> Acked-by: Conor Dooley <conor@kernel.org>

Unless my macro changed without me noticing, I don't think that's
the email address I used for the ack. Just to be sure:
Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> ---
>  MAINTAINERS                                | 3 +--
>  drivers/pmdomain/Kconfig                   | 1 +
>  drivers/{soc => pmdomain}/starfive/Kconfig | 0
>  drivers/soc/Kconfig                        | 1 -
>  4 files changed, 2 insertions(+), 3 deletions(-)
>  rename drivers/{soc => pmdomain}/starfive/Kconfig (100%)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6b491ebcf790..40744fefed3d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20515,7 +20515,7 @@ M:	Walker Chen <walker.chen@starfivetech.com>
>  M:	Changhuang Liang <changhuang.liang@starfivetech.com>
>  S:	Supported
>  F:	Documentation/devicetree/bindings/power/starfive*
> -F:	drivers/pmdomain/starfive/jh71xx-pmu.c
> +F:	drivers/pmdomain/starfive/
>  F:	include/dt-bindings/power/starfive,jh7110-pmu.h
>  
>  STARFIVE SOC DRIVERS
> @@ -20523,7 +20523,6 @@ M:	Conor Dooley <conor@kernel.org>
>  S:	Maintained
>  T:	git https://git.kernel.org/pub/scm/linux/kernel/git/conor/linux.git/
>  F:	Documentation/devicetree/bindings/soc/starfive/
> -F:	drivers/soc/starfive/
>  
>  STARFIVE TRNG DRIVER
>  M:	Jia Jie Ho <jiajie.ho@starfivetech.com>
> diff --git a/drivers/pmdomain/Kconfig b/drivers/pmdomain/Kconfig
> index 08f8a3aa9805..2286c36076db 100644
> --- a/drivers/pmdomain/Kconfig
> +++ b/drivers/pmdomain/Kconfig
> @@ -12,5 +12,6 @@ source "drivers/pmdomain/renesas/Kconfig"
>  source "drivers/pmdomain/rockchip/Kconfig"
>  source "drivers/pmdomain/samsung/Kconfig"
>  source "drivers/pmdomain/st/Kconfig"
> +source "drivers/pmdomain/starfive/Kconfig"
>  
>  endmenu
> diff --git a/drivers/soc/starfive/Kconfig b/drivers/pmdomain/starfive/Kconfig
> similarity index 100%
> rename from drivers/soc/starfive/Kconfig
> rename to drivers/pmdomain/starfive/Kconfig
> diff --git a/drivers/soc/Kconfig b/drivers/soc/Kconfig
> index 8b46da40f107..10a9ff84ff41 100644
> --- a/drivers/soc/Kconfig
> +++ b/drivers/soc/Kconfig
> @@ -23,7 +23,6 @@ source "drivers/soc/renesas/Kconfig"
>  source "drivers/soc/rockchip/Kconfig"
>  source "drivers/soc/samsung/Kconfig"
>  source "drivers/soc/sifive/Kconfig"
> -source "drivers/soc/starfive/Kconfig"
>  source "drivers/soc/sunxi/Kconfig"
>  source "drivers/soc/tegra/Kconfig"
>  source "drivers/soc/ti/Kconfig"
> -- 
> 2.34.1
>
Geert Uytterhoeven Sept. 15, 2023, 10:07 a.m. UTC | #2
On Fri, Sep 15, 2023 at 11:21 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> Rather than having the various Kconfig files for the genpd providers
> sprinkled across subsystems, let's prepare to move them into the pmdomain
> subsystem along with the implementations.
>
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert