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

Conor Dooley conor at kernel.org
Tue Jul 11 09:07:28 PDT 2023


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

Cheers,
Conor.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230711/b382e157/attachment.sig>


More information about the linux-arm-kernel mailing list