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

From: Peng Fan
Date: Thu Aug 17 2023 - 05:40:22 EST


> Subject: Re: [PATCH V4 0/8] genpd: imx: relocate scu-pd and misc update
>
> On Mon, 14 Aug 2023 at 13:46, Peng Fan <peng.fan@xxxxxxx> wrote:
> >
> > > Subject: Re: [PATCH V4 0/8] genpd: imx: relocate scu-pd and misc
> > > update
> > >
> > > On Mon, 14 Aug 2023 at 12:36, Peng Fan (OSS) <peng.fan@xxxxxxxxxxx>
> > > wrote:
> > > >
> > > > From: Peng Fan <peng.fan@xxxxxxx>
> > > >
> > > > V4:
> > > > Update commit message in patch 4
> > > >
> > > > 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%)
> > > >
> > >
> > > I am fine to pick up patch 1 -> patch 6, to let them go in for v6.6.
> > > Should we do that or defer until the complete series is ready?
> >
> > Please take patch 1-6 first. I could handle patch 7 issue in a
> > separate patch, since patch 7 is orthogonal to other patches.
>
> Okay, I have now queued up patch 1-6 for v6.6 via my genpd tree.
>

Thanks for your work.

> In regards to patch4 (no_console_suspend), did you manage to have a look
> at the patch [1] I sent a few days ago?

Yeah, the patch looks good to me, but I have not find time to work
on the serial driver part following your suggestion, so still not test.

I will reply with R-b there.

Thanks,
Peng.
>
> Kind regards
> Uffe
>
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.
> kernel.org%2Fall%2F20230810162119.152589-1-
> ulf.hansson%40linaro.org%2F&data=05%7C01%7Cpeng.fan%40nxp.com%7
> C6d64514fd9b84f1d6a8108db9f054956%7C686ea1d3bc2b4c6fa92cd99c5c30
> 1635%7C0%7C0%7C638278617269111369%7CUnknown%7CTWFpbGZsb3d8
> eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3
> D%7C3000%7C%7C%7C&sdata=InU4ltSc26z2WJYFHTwLdyaoo9WqF6cZ1wyX
> 7FDpKkk%3D&reserved=0