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.
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
>
