Re: [PATCH 12/18] soc: starfive: Move the power-domain driver to the genpd dir

On Tue, Jul 11, 2023 at 03:57:49PM +0200, Ulf Hansson wrote:

On Fri, 7 Jul 2023 at 17:51, Conor Dooley [off-list ref] wrote:

quoted

On Fri, Jul 07, 2023 at 04:04:28PM +0200, Ulf Hansson wrote:

quoted

Cc: Walker Chen <walker.chen@starfivetech.com>
Cc: Conor Dooley <conor@kernel.org>
Signed-off-by: Ulf Hansson <redacted>

quoted

If you are not going to write a commit message with an explanation,

I will, sorry for being lazy.

quoted

the least you could do is CC me on the cover letter.

Yes, I will soon post a new version and add you to the cover letter and patch 1.

Thanks. Obviously I was able to pull the message-id & look at lore etc,
but what goes for the goose, goes for the gander and all that.

quoted

quoted

---
 MAINTAINERS                                                    | 2 +-
 drivers/genpd/Makefile                                         | 1 +
 drivers/genpd/starfive/Makefile                                | 2 ++
 .../{soc/starfive/jh71xx_pmu.c => genpd/starfive/jh71xx-pmu.c} | 0
 drivers/soc/starfive/Makefile                                  | 3 ---

Why not also move the Kconfig entry while you are at it?

I just replied to Geert [1], who had similar questions. Hope that
makes sense to you?

In general, it does explain why you did not do it.
I think that the Kconfig stuff should move too & the whole
soc/starfive directory should vanish, but that could be left for a
follow-up activity.

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.