linux-kernel - Re: [PATCH 15/18] soc: ti: Mover power-domain drivers to the genpd dir

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAPDyKFoc0hr=9LEtwwwe3R6rMn0b7TB1MCZN0ArUq+h9Pud08Q@mail.gmail.com>
Date:   Tue, 11 Jul 2023 11:16:03 +0200
From:   Ulf Hansson <ulf.hansson@...aro.org>
To:     Tony Lindgren <tony@...mide.com>, Nishanth Menon <nm@...com>
Cc:     Arnd Bergmann <arnd@...db.de>, Olof Johansson <olof@...om.net>,
        linux-pm@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org,
        Santosh Shilimkar <ssantosh@...nel.org>,
        Tero Kristo <kristo@...nel.org>
Subject: Re: [PATCH 15/18] soc: ti: Mover power-domain drivers to the genpd dir

On Fri, 7 Jul 2023 at 19:50, Nishanth Menon <nm@...com> wrote:
>
> On 16:04-20230707, Ulf Hansson wrote:
> > Cc: Nishanth Menon <nm@...com>
> > Cc: Santosh Shilimkar <ssantosh@...nel.org>
> > Cc: Tero Kristo <kristo@...nel.org>
> > Signed-off-by: Ulf Hansson <ulf.hansson@...aro.org>
> > ---
> >  MAINTAINERS                                   | 3 ++-
> >  drivers/genpd/Makefile                        | 1 +
> >  drivers/genpd/ti/Makefile                     | 3 +++
> >  drivers/{soc => genpd}/ti/omap_prm.c          | 0
> >  drivers/{soc => genpd}/ti/ti_sci_pm_domains.c | 0
> >  drivers/soc/ti/Makefile                       | 2 --
> >  6 files changed, 6 insertions(+), 3 deletions(-)
> >  create mode 100644 drivers/genpd/ti/Makefile
> >  rename drivers/{soc => genpd}/ti/omap_prm.c (100%)
> >  rename drivers/{soc => genpd}/ti/ti_sci_pm_domains.c (100%)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 9e580df3e5db..3cf16ffac892 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -21101,7 +21101,7 @@ F:    drivers/irqchip/irq-ti-sci-inta.c
> >  F:   drivers/irqchip/irq-ti-sci-intr.c
> >  F:   drivers/reset/reset-ti-sci.c
> >  F:   drivers/soc/ti/ti_sci_inta_msi.c
> > -F:   drivers/soc/ti/ti_sci_pm_domains.c
> > +F:   drivers/genpd/ti/ti_sci_pm_domains.c
> >  F:   include/dt-bindings/soc/ti,sci_pm_domain.h
> >  F:   include/linux/soc/ti/ti_sci_inta_msi.h
> >  F:   include/linux/soc/ti/ti_sci_protocol.h
> > @@ -21335,6 +21335,7 @@ L:    linux-kernel@...r.kernel.org
> >  L:   linux-arm-kernel@...ts.infradead.org (moderated for non-subscribers)
> >  S:   Maintained
> >  T:   git git://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
> > +F:   drivers/genpd/ti/omap_prm.c
>
> Probably the wrong place for this as omap_prm is'nt a keystone navigator
> driver set. Maybe Tony has a suggestion?

I guess we could add it to the OMAP2+ section then?

In any case, I suggest we consider that as a separate patch on top, as
I am just obeying to the existing pattern that the get_maintainers
script provides.

Kind regards
Uffe

>
> >  F:   drivers/soc/ti/*
> >
> >  TI LM49xxx FAMILY ASoC CODEC DRIVERS
> > diff --git a/drivers/genpd/Makefile b/drivers/genpd/Makefile
> > index e6f34d82e6a8..193892189f0d 100644
> > --- a/drivers/genpd/Makefile
> > +++ b/drivers/genpd/Makefile
> > @@ -11,3 +11,4 @@ obj-y                                       += samsung/
> >  obj-y                                        += starfive/
> >  obj-y                                        += sunxi/
> >  obj-y                                        += tegra/
> > +obj-y                                        += ti/
> > diff --git a/drivers/genpd/ti/Makefile b/drivers/genpd/ti/Makefile
> > new file mode 100644
> > index 000000000000..69580afbb436
> > --- /dev/null
> > +++ b/drivers/genpd/ti/Makefile
> > @@ -0,0 +1,3 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +obj-$(CONFIG_ARCH_OMAP2PLUS)         += omap_prm.o
> > +obj-$(CONFIG_TI_SCI_PM_DOMAINS)              += ti_sci_pm_domains.o
> > diff --git a/drivers/soc/ti/omap_prm.c b/drivers/genpd/ti/omap_prm.c
> > similarity index 100%
> > rename from drivers/soc/ti/omap_prm.c
> > rename to drivers/genpd/ti/omap_prm.c
> > diff --git a/drivers/soc/ti/ti_sci_pm_domains.c b/drivers/genpd/ti/ti_sci_pm_domains.c
> > similarity index 100%
> > rename from drivers/soc/ti/ti_sci_pm_domains.c
> > rename to drivers/genpd/ti/ti_sci_pm_domains.c
> > diff --git a/drivers/soc/ti/Makefile b/drivers/soc/ti/Makefile
> > index cc3c972fad2e..cb800a745e66 100644
> > --- a/drivers/soc/ti/Makefile
> > +++ b/drivers/soc/ti/Makefile
> > @@ -6,9 +6,7 @@ obj-$(CONFIG_KEYSTONE_NAVIGATOR_QMSS) += knav_qmss.o
> >  knav_qmss-y := knav_qmss_queue.o knav_qmss_acc.o
> >  obj-$(CONFIG_KEYSTONE_NAVIGATOR_DMA) += knav_dma.o
> >  obj-$(CONFIG_AMX3_PM)                        += pm33xx.o
> > -obj-$(CONFIG_ARCH_OMAP2PLUS)         += omap_prm.o
> >  obj-$(CONFIG_WKUP_M3_IPC)            += wkup_m3_ipc.o
> > -obj-$(CONFIG_TI_SCI_PM_DOMAINS)              += ti_sci_pm_domains.o
> >  obj-$(CONFIG_TI_SCI_INTA_MSI_DOMAIN) += ti_sci_inta_msi.o
> >  obj-$(CONFIG_TI_K3_RINGACC)          += k3-ringacc.o
> >  obj-$(CONFIG_TI_K3_SOCINFO)          += k3-socinfo.o
> > --
> > 2.34.1
> >
>
> --
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D