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

[Conor Dooley <conor@kernel.org>]

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

On Tue, Jul 11, 2023 at 03:57:49PM +0200, Ulf Hansson wrote:
> On Fri, 7 Jul 2023 at 17:51, Conor Dooley <conor@kernel.org> wrote:
> >
> > On Fri, Jul 07, 2023 at 04:04:28PM +0200, Ulf Hansson wrote:
> > > Cc: Walker Chen <walker.chen@starfivetech.com>
> > > Cc: Conor Dooley <conor@kernel.org>
> > > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> >

> > If you are not going to write a commit message with an explanation,
> 
> I will, sorry for being lazy.
> 
> > 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.

> > > ---
> > >  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.

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