| Message ID | 20230814104127.1929-1-peng.fan@oss.nxp.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | genpd: imx: relocate scu-pd and misc update | expand |
On Mon, 14 Aug 2023 at 12:36, Peng Fan (OSS) <peng.fan@oss.nxp.com> wrote: > > From: Peng Fan <peng.fan@nxp.com> > > 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? Kind regards Uffe
> 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@oss.nxp.com> > wrote: > > > > From: Peng Fan <peng.fan@nxp.com> > > > > 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. Thanks, Peng. > > Kind regards > Uffe
On Mon, 14 Aug 2023 at 13:46, Peng Fan <peng.fan@nxp.com> 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@oss.nxp.com> > > wrote: > > > > > > From: Peng Fan <peng.fan@nxp.com> > > > > > > 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. In regards to patch4 (no_console_suspend), did you manage to have a look at the patch [1] I sent a few days ago? Kind regards Uffe https://lore.kernel.org/all/20230810162119.152589-1-ulf.hansson@linaro.org/
> 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@nxp.com> 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@oss.nxp.com> > > > wrote: > > > > > > > > From: Peng Fan <peng.fan@nxp.com> > > > > > > > > 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
From: Peng Fan <peng.fan@nxp.com> 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%)