Hi Adam,
Am Sonntag, dem 03.10.2021 um 05:43 -0500 schrieb Adam Ford:
> On Fri, Oct 1, 2021 at 8:00 PM Lucas Stach <l.stach@xxxxxxxxxxxxxx> wrote:
> >
> > This reverts commit a77ebdd9f553. It turns out that the VPU domain has no
> > different requirements, even though the downstream ATF implementation seems
> > to suggest otherwise. Powering on the domain with the reset asserted works
> > fine. As the changed sequence has caused sporadic issues with the GPU
> > domains, just revert the change to go back to the working sequence.
> >
> > Cc: <stable@xxxxxxxxxxxxxxx> # 5.14
> > Signed-off-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>
> > Acked-by: Peng Fan <peng.fan@xxxxxxx>
> > ---
>
> Lucas,
>
> I applied your series to the 5.14.y kernel to test with the
> imx8mm-beacon board, but I found that it doesn't wake from sleep.
> I'll experiment with other versions of ATF. If nobody else has this
> problem, I'll assume, it's an error on my part.
I've tested this series on the i.MX8MM-EVK and a custom (not yet
public) i.MX8MM board and both did work as expected with both system
suspend/resume and runtime power management for the display parts. I've
used the upstream TF-A release v2.5.
Regards,
Lucas
>
> adam
>
> > drivers/soc/imx/gpcv2.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/soc/imx/gpcv2.c b/drivers/soc/imx/gpcv2.c
> > index 34a9ac1f2b9b..8b7a01773aec 100644
> > --- a/drivers/soc/imx/gpcv2.c
> > +++ b/drivers/soc/imx/gpcv2.c
> > @@ -244,6 +244,8 @@ static int imx_pgc_power_up(struct generic_pm_domain *genpd)
> > goto out_regulator_disable;
> > }
> >
> > + reset_control_assert(domain->reset);
> > +
> > if (domain->bits.pxx) {
> > /* request the domain to power up */
> > regmap_update_bits(domain->regmap, GPC_PU_PGC_SW_PUP_REQ,
> > @@ -266,8 +268,6 @@ static int imx_pgc_power_up(struct generic_pm_domain *genpd)
> > GPC_PGC_CTRL_PCR);
> > }
> >
> > - reset_control_assert(domain->reset);
> > -
> > /* delay for reset to propagate */
> > udelay(5);
> >
> > --
> > 2.30.2
> >
