RE: [PATCH V3 0/8] genpd: imx: relocate scu-pd and misc update

From: Peng Fan
Date: Tue Aug 08 2023 - 21:32:34 EST


> Subject: Re: [PATCH V3 0/8] genpd: imx: relocate scu-pd and misc update
>
> On Mon, 31 Jul 2023 at 08:43, Peng Fan (OSS) <peng.fan@xxxxxxxxxxx>
> wrote:
> >
> > From: Peng Fan <peng.fan@xxxxxxx>
> >
> > V3:
> > return -EBUSY instead of return 0 in patch 4
> >
> > V2:
> > Move drivers/firmware/imx/scu-pd.c to drivers/genpd/imx
> >
> > This patchset is to upstream NXP downstream scu-pd driver patches.
> > patch is to relocate scu-pd to genpd
> > patch 2,3 is to support more PDs
> > patch 4 is to not power off console when no console suspend patch 5 is
> > to suppress bind patch 6 is to make genpd align with HW state patch 7
> > is to support LP mode in runtime suspend, OFF mode in system suspend.
> > patch 8 is to change init level to avoid uneccessary defer probe
> >
> > V1:
> > This patchset is to upstream NXP downstream scu-pd driver patches.
> > patch 1,2 is to support more PDs
> > patch 3 is to not power off console when no console suspend patch 4 is
> > to suppress bind patch 5 is to make genpd align with HW state patch 6
> > is to support LP mode in runtime suspend, OFF mode in system suspend.
> > patch 7 is to change init level to avoid uneccessary defer probe
> >
> > Dong Aisheng (1):
> > genpd: imx: scu-pd: change init level to subsys_initcall
> >
> > Peng Fan (7):
> > genpd: imx: relocate scu-pd under genpd
> > genpd: imx: scu-pd: enlarge PD range
> > genpd: imx: scu-pd: add more PDs
> > genpd: imx: scu-pd: do not power off console if no_console_suspend
> > genpd: imx: scu-pd: Suppress bind attrs
> > genpd: imx: scu-pd: initialize is_off according to HW state
> > genpd: imx: scu-pd: add multi states support
> >
> > drivers/firmware/imx/Makefile | 1 -
> > drivers/genpd/imx/Makefile | 1 +
> > drivers/{firmware => genpd}/imx/scu-pd.c | 193
> > +++++++++++++++++++++--
> > 3 files changed, 183 insertions(+), 12 deletions(-) rename
> > drivers/{firmware => genpd}/imx/scu-pd.c (70%)
> >
>
> Moving this to the new genpd subsystem makes sense to me.
>
> Even if we can't get the whole series ready for v6.6, we can certainly pick
> patch1. Either we can funnel this via my new genpd tree [1] or if Shawn
> picks it up. If the latter, Shawn needs to merge my immutable branch [2]
> before applying. I am fine either way.

There is no rush to catch v6.6 for this patchset. It could go via your genpd
tree for v6.7 from my view. Anyway, up to you and Shawn to decide.

Thanks,
Peng.

>
> Kind regards
> Uffe
>
> [1]
> git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/linux-pm.git next
>
> [2]
> git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/linux-pm.git
> genpd_create_dir