diff mbox series

[06/18] soc: imx: Move power-domain drivers to the genpd dir

Message ID 20230707140434.723349-7-ulf.hansson@linaro.org
State Accepted
Commit e5300b2c3fe0c02ef3bf2cf3fc3c16f021344043
Headers show
Series genpd: Create a genpd directory to host genpd providers | expand

Commit Message

Ulf Hansson July 7, 2023, 2:04 p.m. UTC
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: <kernel@pengutronix.de>
Cc: <linux-imx@nxp.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
---
 drivers/genpd/imx/Makefile                   | 7 +++++++
 drivers/{soc => genpd}/imx/gpc.c             | 0
 drivers/{soc => genpd}/imx/gpcv2.c           | 0
 drivers/{soc => genpd}/imx/imx8m-blk-ctrl.c  | 0
 drivers/{soc => genpd}/imx/imx8mp-blk-ctrl.c | 0
 drivers/{soc => genpd}/imx/imx93-blk-ctrl.c  | 0
 drivers/{soc => genpd}/imx/imx93-pd.c        | 0
 drivers/soc/imx/Makefile                     | 7 +------
 8 files changed, 8 insertions(+), 6 deletions(-)
 create mode 100644 drivers/genpd/imx/Makefile
 rename drivers/{soc => genpd}/imx/gpc.c (100%)
 rename drivers/{soc => genpd}/imx/gpcv2.c (100%)
 rename drivers/{soc => genpd}/imx/imx8m-blk-ctrl.c (100%)
 rename drivers/{soc => genpd}/imx/imx8mp-blk-ctrl.c (100%)
 rename drivers/{soc => genpd}/imx/imx93-blk-ctrl.c (100%)
 rename drivers/{soc => genpd}/imx/imx93-pd.c (100%)

Comments

Ulf Hansson July 7, 2023, 2:33 p.m. UTC | #1
On Fri, 7 Jul 2023 at 16:25, Fabio Estevam <festevam@gmail.com> wrote:
>
> On Fri, Jul 7, 2023 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> >
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: <kernel@pengutronix.de>
> > Cc: <linux-imx@nxp.com>
> > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
>
> A commit log explaining the motivation would be helpful here.

Right. I was a bit lazy and added motivation only as part of the
commit message in patch 1/18 (and in the cover letter).

I deliberately didn't want to spam everyone with this, but you find it
at the generic linux-kernel list.

>
> With that added:
>
> Reviewed-by: Fabio Estevam <festevam@gmail.com>
>
> Thanks

Thanks!

Kind regards
Uffe
Shawn Guo July 18, 2023, 6:54 a.m. UTC | #2
On Fri, Jul 07, 2023 at 04:04:22PM +0200, Ulf Hansson wrote:
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: <kernel@pengutronix.de>
> Cc: <linux-imx@nxp.com>
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> ---
>  drivers/genpd/imx/Makefile                   | 7 +++++++
>  drivers/{soc => genpd}/imx/gpc.c             | 0
>  drivers/{soc => genpd}/imx/gpcv2.c           | 0
>  drivers/{soc => genpd}/imx/imx8m-blk-ctrl.c  | 0
>  drivers/{soc => genpd}/imx/imx8mp-blk-ctrl.c | 0
>  drivers/{soc => genpd}/imx/imx93-blk-ctrl.c  | 0
>  drivers/{soc => genpd}/imx/imx93-pd.c        | 0
>  drivers/soc/imx/Makefile                     | 7 +------
>  8 files changed, 8 insertions(+), 6 deletions(-)
>  create mode 100644 drivers/genpd/imx/Makefile
>  rename drivers/{soc => genpd}/imx/gpc.c (100%)
>  rename drivers/{soc => genpd}/imx/gpcv2.c (100%)
>  rename drivers/{soc => genpd}/imx/imx8m-blk-ctrl.c (100%)
>  rename drivers/{soc => genpd}/imx/imx8mp-blk-ctrl.c (100%)
>  rename drivers/{soc => genpd}/imx/imx93-blk-ctrl.c (100%)
>  rename drivers/{soc => genpd}/imx/imx93-pd.c (100%)

Acked-by: Shawn Guo <shawnguo@kernel.org>
diff mbox series

Patch

diff --git a/drivers/genpd/imx/Makefile b/drivers/genpd/imx/Makefile
new file mode 100644
index 000000000000..5f012717a666
--- /dev/null
+++ b/drivers/genpd/imx/Makefile
@@ -0,0 +1,7 @@ 
+# SPDX-License-Identifier: GPL-2.0-only
+obj-$(CONFIG_HAVE_IMX_GPC) += gpc.o
+obj-$(CONFIG_IMX_GPCV2_PM_DOMAINS) += gpcv2.o
+obj-$(CONFIG_IMX8M_BLK_CTRL) += imx8m-blk-ctrl.o
+obj-$(CONFIG_IMX8M_BLK_CTRL) += imx8mp-blk-ctrl.o
+obj-$(CONFIG_SOC_IMX9) += imx93-pd.o
+obj-$(CONFIG_IMX9_BLK_CTRL) += imx93-blk-ctrl.o
diff --git a/drivers/soc/imx/gpc.c b/drivers/genpd/imx/gpc.c
similarity index 100%
rename from drivers/soc/imx/gpc.c
rename to drivers/genpd/imx/gpc.c
diff --git a/drivers/soc/imx/gpcv2.c b/drivers/genpd/imx/gpcv2.c
similarity index 100%
rename from drivers/soc/imx/gpcv2.c
rename to drivers/genpd/imx/gpcv2.c
diff --git a/drivers/soc/imx/imx8m-blk-ctrl.c b/drivers/genpd/imx/imx8m-blk-ctrl.c
similarity index 100%
rename from drivers/soc/imx/imx8m-blk-ctrl.c
rename to drivers/genpd/imx/imx8m-blk-ctrl.c
diff --git a/drivers/soc/imx/imx8mp-blk-ctrl.c b/drivers/genpd/imx/imx8mp-blk-ctrl.c
similarity index 100%
rename from drivers/soc/imx/imx8mp-blk-ctrl.c
rename to drivers/genpd/imx/imx8mp-blk-ctrl.c
diff --git a/drivers/soc/imx/imx93-blk-ctrl.c b/drivers/genpd/imx/imx93-blk-ctrl.c
similarity index 100%
rename from drivers/soc/imx/imx93-blk-ctrl.c
rename to drivers/genpd/imx/imx93-blk-ctrl.c
diff --git a/drivers/soc/imx/imx93-pd.c b/drivers/genpd/imx/imx93-pd.c
similarity index 100%
rename from drivers/soc/imx/imx93-pd.c
rename to drivers/genpd/imx/imx93-pd.c
diff --git a/drivers/soc/imx/Makefile b/drivers/soc/imx/Makefile
index a28c44a1f16a..3ad321ca608a 100644
--- a/drivers/soc/imx/Makefile
+++ b/drivers/soc/imx/Makefile
@@ -2,10 +2,5 @@ 
 ifeq ($(CONFIG_ARM),y)
 obj-$(CONFIG_ARCH_MXC) += soc-imx.o
 endif
-obj-$(CONFIG_HAVE_IMX_GPC) += gpc.o
-obj-$(CONFIG_IMX_GPCV2_PM_DOMAINS) += gpcv2.o
 obj-$(CONFIG_SOC_IMX8M) += soc-imx8m.o
-obj-$(CONFIG_IMX8M_BLK_CTRL) += imx8m-blk-ctrl.o
-obj-$(CONFIG_IMX8M_BLK_CTRL) += imx8mp-blk-ctrl.o
-obj-$(CONFIG_SOC_IMX9) += imx93-src.o imx93-pd.o
-obj-$(CONFIG_IMX9_BLK_CTRL) += imx93-blk-ctrl.o
+obj-$(CONFIG_SOC_IMX9) += imx93-src.o