Re: [PATCH 2/3] ARM: at91: pm: group constants and addresses loading

From: Alexandre Belloni
Date: Tue Sep 28 2021 - 06:01:10 EST


Hello,

On 28/09/2021 11:42:56+0300, Claudiu Beznea wrote:
> Group constants and addresses loading. This commit prepares the field for
> the next one. Added fixes tag on this commit as the next one will fail
> to apply if backported alone to older kernels.

The proper notation is to use Depends-on: on the next patch.

>
> Fixes: f0bbf17958e8 ("ARM: at91: pm: add self-refresh support for sama7g5")
> Signed-off-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
> ---
> arch/arm/mach-at91/pm_suspend.S | 17 +++++++++--------
> 1 file changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
> index cbd61a3bcab1..34f251fdb743 100644
> --- a/arch/arm/mach-at91/pm_suspend.S
> +++ b/arch/arm/mach-at91/pm_suspend.S
> @@ -1014,6 +1014,15 @@ ENTRY(at91_pm_suspend_in_sram)
> mov tmp1, #0
> mcr p15, 0, tmp1, c7, c10, 4
>
> + ldr tmp1, [r0, #PM_DATA_PMC_MCKR_OFFSET]
> + str tmp1, .mckr_offset
> + ldr tmp1, [r0, #PM_DATA_PMC_VERSION]
> + str tmp1, .pmc_version
> + ldr tmp1, [r0, #PM_DATA_MEMCTRL]
> + str tmp1, .memtype
> + ldr tmp1, [r0, #PM_DATA_MODE]
> + str tmp1, .pm_mode
> +
> ldr tmp1, [r0, #PM_DATA_PMC]
> str tmp1, .pmc_base
> ldr tmp1, [r0, #PM_DATA_RAMC0]
> @@ -1022,14 +1031,6 @@ ENTRY(at91_pm_suspend_in_sram)
> str tmp1, .sramc1_base
> ldr tmp1, [r0, #PM_DATA_RAMC_PHY]
> str tmp1, .sramc_phy_base
> - ldr tmp1, [r0, #PM_DATA_MEMCTRL]
> - str tmp1, .memtype
> - ldr tmp1, [r0, #PM_DATA_MODE]
> - str tmp1, .pm_mode
> - ldr tmp1, [r0, #PM_DATA_PMC_MCKR_OFFSET]
> - str tmp1, .mckr_offset
> - ldr tmp1, [r0, #PM_DATA_PMC_VERSION]
> - str tmp1, .pmc_version
> /* Both ldrne below are here to preload their address in the TLB */
> ldr tmp1, [r0, #PM_DATA_SHDWC]
> str tmp1, .shdwc
> --
> 2.25.1
>

--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com