Commit b5fe2429 authored by Ard Biesheuvel's avatar Ard Biesheuvel Committed by Will Deacon

arm64: kernel: fix style issues in sleep.S

This fixes a number of style issues in sleep.S. No functional changes are
intended:
- replace absolute literal references with relative references in
  __cpu_suspend_enter(), which executes from its virtual address
- replace explicit lr assignment plus branch with bl in cpu_resume(), which
  aligns it with stext() and secondary_startup()
- don't export _cpu_resume()
- use adr_l for mpidr_hash reference, and fix the incorrect accompanying
  comment, which has been out of date since commit cabe1c81 ("arm64:
  Change cpu_resume() to enable mmu early then access sleep_sp by va")
- replace leading spaces with tabs, and add a bit of whitespace for
  readability
Reviewed-by: default avatarMark Rutland <mark.rutland@arm.com>
Signed-off-by: default avatarArd Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: default avatarWill Deacon <will.deacon@arm.com>
parent 563cada0
...@@ -73,10 +73,9 @@ ENTRY(__cpu_suspend_enter) ...@@ -73,10 +73,9 @@ ENTRY(__cpu_suspend_enter)
str x2, [x0, #SLEEP_STACK_DATA_SYSTEM_REGS + CPU_CTX_SP] str x2, [x0, #SLEEP_STACK_DATA_SYSTEM_REGS + CPU_CTX_SP]
/* find the mpidr_hash */ /* find the mpidr_hash */
ldr x1, =sleep_save_stash ldr_l x1, sleep_save_stash
ldr x1, [x1]
mrs x7, mpidr_el1 mrs x7, mpidr_el1
ldr x9, =mpidr_hash adr_l x9, mpidr_hash
ldr x10, [x9, #MPIDR_HASH_MASK] ldr x10, [x9, #MPIDR_HASH_MASK]
/* /*
* Following code relies on the struct mpidr_hash * Following code relies on the struct mpidr_hash
...@@ -95,15 +94,14 @@ ENTRY(__cpu_suspend_enter) ...@@ -95,15 +94,14 @@ ENTRY(__cpu_suspend_enter)
mov x0, #1 mov x0, #1
ret ret
ENDPROC(__cpu_suspend_enter) ENDPROC(__cpu_suspend_enter)
.ltorg
.pushsection ".idmap.text", "ax" .pushsection ".idmap.text", "ax"
ENTRY(cpu_resume) ENTRY(cpu_resume)
bl el2_setup // if in EL2 drop to EL1 cleanly bl el2_setup // if in EL2 drop to EL1 cleanly
bl __cpu_setup
/* enable the MMU early - so we can access sleep_save_stash by va */ /* enable the MMU early - so we can access sleep_save_stash by va */
adr_l lr, __enable_mmu /* __cpu_setup will return here */
adr_l x27, _resume_switched /* __enable_mmu will branch here */ adr_l x27, _resume_switched /* __enable_mmu will branch here */
b __cpu_setup b __enable_mmu
ENDPROC(cpu_resume) ENDPROC(cpu_resume)
_resume_switched: _resume_switched:
...@@ -113,16 +111,17 @@ ENDPROC(_resume_switched) ...@@ -113,16 +111,17 @@ ENDPROC(_resume_switched)
.ltorg .ltorg
.popsection .popsection
ENTRY(_cpu_resume) _cpu_resume:
mrs x1, mpidr_el1 mrs x1, mpidr_el1
adrp x8, mpidr_hash adr_l x8, mpidr_hash // x8 = struct mpidr_hash virt address
add x8, x8, #:lo12:mpidr_hash // x8 = struct mpidr_hash phys address
/* retrieve mpidr_hash members to compute the hash */ /* retrieve mpidr_hash members to compute the hash */
ldr x2, [x8, #MPIDR_HASH_MASK] ldr x2, [x8, #MPIDR_HASH_MASK]
ldp w3, w4, [x8, #MPIDR_HASH_SHIFTS] ldp w3, w4, [x8, #MPIDR_HASH_SHIFTS]
ldp w5, w6, [x8, #(MPIDR_HASH_SHIFTS + 8)] ldp w5, w6, [x8, #(MPIDR_HASH_SHIFTS + 8)]
compute_mpidr_hash x7, x3, x4, x5, x6, x1, x2 compute_mpidr_hash x7, x3, x4, x5, x6, x1, x2
/* x7 contains hash index, let's use it to grab context pointer */
/* x7 contains hash index, let's use it to grab context pointer */
ldr_l x0, sleep_save_stash ldr_l x0, sleep_save_stash
ldr x0, [x0, x7, lsl #3] ldr x0, [x0, x7, lsl #3]
add x29, x0, #SLEEP_STACK_DATA_CALLEE_REGS add x29, x0, #SLEEP_STACK_DATA_CALLEE_REGS
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment