[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230811140259.GT151430@dragon>
Date: Fri, 11 Aug 2023 22:02:59 +0800
From: Shawn Guo <shawnguo@...nel.org>
To: Peng Fan <peng.fan@....com>
Cc: Ulf Hansson <ulf.hansson@...aro.org>,
"Peng Fan (OSS)" <peng.fan@....nxp.com>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
"kernel@...gutronix.de" <kernel@...gutronix.de>,
"festevam@...il.com" <festevam@...il.com>,
dl-linux-imx <linux-imx@....com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>
Subject: Re: [PATCH V3 0/8] genpd: imx: relocate scu-pd and misc update
On Wed, Aug 09, 2023 at 01:32:28AM +0000, Peng Fan wrote:
> > 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@....nxp.com>
> > wrote:
> > >
> > > From: Peng Fan <peng.fan@....com>
> > >
> > > 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.
I prefer to have this go via genpd tree.
Shawn
Powered by blists - more mailing lists