Messages in this thread | | | From | Ulf Hansson <> | Date | Fri, 15 Sep 2023 00:24:55 +0200 | Subject | Re: [PATCH 13/17] pmdomain: starfive: Move Kconfig file to the pmdomain subsystem |
| |
On Thu, 14 Sept 2023 at 14:00, Conor Dooley <conor.dooley@microchip.com> wrote: > > Hey, > > > On Thu, Sep 14, 2023 at 01:19:04PM +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. > > > > Cc: Walker Chen <walker.chen@starfivetech.com> > > Cc: Conor Dooley <conor@kernel.org> > > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> > > Thanks for doing this, saved me having to! > The series appears to have broken threading though, which can be seen > pretty easily on lore: > https://lore.kernel.org/all/20230914111904.586744-1-ulf.hansson@linaro.org/ > Dunno if something is borked with your process while sending patches to > only a subset of the total recipient list.
Yes, I screwed up when sending. Anyway, it looks like it works for people to review it anyway.
> > > --- > > MAINTAINERS | 2 +- > > drivers/pmdomain/Kconfig | 1 + > > drivers/{soc => pmdomain}/starfive/Kconfig | 0 > > drivers/soc/Kconfig | 1 - > > 4 files changed, 2 insertions(+), 2 deletions(-) > > rename drivers/{soc => pmdomain}/starfive/Kconfig (100%) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 6b491ebcf790..b8eae18f0fb1 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -20523,7 +20523,7 @@ 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/ > > +F: drivers/pmdomain/starfive/ > > This isn't the correct change, just remove the drivers/soc/starfive > bit entirely from this entry. > I've just been doing some review & sending PRs to Arnd for this stuff > (or failing to send the PRs as you saw), I don't have particular > interest in this driver other than helping out the starfive people with > actually getting the code merged. > Instead I think you should generalise the existing entry for the driver > to cover the whole directory, so that it includes the Kconfig bits too: > > diff --git a/MAINTAINERS b/MAINTAINERS > index fa7487b7729b..22c1a5e5b130 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -20564,7 +20564,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
Thanks for clarifying this. I will update (either amending when applying or when re-submitting).
> > STARFIVE SOC DRIVERS > > Thanks, > Conor.
I guess I consider this as an ack, assuming the above gets changed?
[...]
Kind regards Uffe
| |