* [PATCH] xen/arm: Fix comments coding style in assembler files
@ 2017-07-27 14:55 Andrii Anisov
2017-07-31 20:39 ` Stefano Stabellini
0 siblings, 1 reply; 2+ messages in thread
From: Andrii Anisov @ 2017-07-27 14:55 UTC (permalink / raw)
To: xen-devel; +Cc: julien.grall, sstabellini, Andrii Anisov
From: Andrii Anisov <andrii_anisov@epam.com>
Signed-off-by: Andrii Anisov <andrii_anisov@epam.com>
---
xen/arch/arm/arm32/debug-8250.inc | 12 +++--
xen/arch/arm/arm32/debug-exynos4210.inc | 12 +++--
xen/arch/arm/arm32/debug-pl011.inc | 18 +++++---
xen/arch/arm/arm32/debug-scif.inc | 6 ++-
xen/arch/arm/arm32/debug.S | 6 ++-
xen/arch/arm/arm32/head.S | 81 ++++++++++++++++++++++-----------
xen/arch/arm/arm64/debug-8250.inc | 12 +++--
xen/arch/arm/arm64/debug-cadence.inc | 12 +++--
xen/arch/arm/arm64/debug-pl011.inc | 18 +++++---
xen/arch/arm/arm64/debug.S | 6 ++-
xen/arch/arm/arm64/entry.S | 50 ++++++++++----------
11 files changed, 147 insertions(+), 86 deletions(-)
diff --git a/xen/arch/arm/arm32/debug-8250.inc b/xen/arch/arm/arm32/debug-8250.inc
index 757ffd8..0759a27 100644
--- a/xen/arch/arm/arm32/debug-8250.inc
+++ b/xen/arch/arm/arm32/debug-8250.inc
@@ -16,9 +16,11 @@
#include <xen/8250-uart.h>
-/* 8250 UART wait UART to be ready to transmit
+/*
+ * 8250 UART wait UART to be ready to transmit
* rb: register which contains the UART base address
- * rc: scratch register */
+ * rc: scratch register
+ */
.macro early_uart_ready rb rc
1:
ldr \rc, [\rb, #(UART_LSR << EARLY_UART_REG_SHIFT)] /* Read LSR */
@@ -26,9 +28,11 @@
beq 1b /* Wait for the UART to be ready */
.endm
-/* 8250 UART transmit character
+/*
+ * 8250 UART transmit character
* rb: register which contains the UART base address
- * rt: register which contains the character to transmit */
+ * rt: register which contains the character to transmit
+ */
.macro early_uart_transmit rb rt
str \rt, [\rb, #UART_THR] /* Write Transmit buffer */
.endm
diff --git a/xen/arch/arm/arm32/debug-exynos4210.inc b/xen/arch/arm/arm32/debug-exynos4210.inc
index 752942d..4e80a21 100644
--- a/xen/arch/arm/arm32/debug-exynos4210.inc
+++ b/xen/arch/arm/arm32/debug-exynos4210.inc
@@ -18,9 +18,11 @@
#include <asm/exynos4210-uart.h>
-/* Exynos 5 UART wait UART to be ready to transmit
+/*
+ * Exynos 5 UART wait UART to be ready to transmit
* rb: register which contains the UART base address
- * rc: scratch register */
+ * rc: scratch register
+ */
.macro early_uart_ready rb rc
1:
ldr \rc, [\rb, #UTRSTAT] /* <- UTRSTAT (Flag register) */
@@ -28,9 +30,11 @@
beq 1b /* Wait for the UART to be ready */
.endm
-/* Exynos 5 UART transmit character
+/*
+ * Exynos 5 UART transmit character
* rb: register which contains the UART base address
- * rt: register which contains the character to transmit */
+ * rt: register which contains the character to transmit
+ */
.macro early_uart_transmit rb rt
str \rt, [\rb, #UTXH] /* -> UTXH (Data Register) */
.endm
diff --git a/xen/arch/arm/arm32/debug-pl011.inc b/xen/arch/arm/arm32/debug-pl011.inc
index 6a64dbf..ec462ea 100644
--- a/xen/arch/arm/arm32/debug-pl011.inc
+++ b/xen/arch/arm/arm32/debug-pl011.inc
@@ -18,10 +18,12 @@
#include <asm/pl011-uart.h>
-/* PL011 UART initialization
+/*
+ * PL011 UART initialization
* rb: register which contains the UART base address
* rc: scratch register 1
- * rd: scratch register 2 (unused here) */
+ * rd: scratch register 2 (unused here)
+ */
.macro early_uart_init rb, rc, rd
mov \rc, #(7372800 / EARLY_PRINTK_BAUD % 16)
str \rc, [\rb, #FBRD] /* -> UARTFBRD (Baud divisor fraction) */
@@ -33,9 +35,11 @@
str \rc, [\rb, #CR] /* -> UARTCR (Control Register) */
.endm
-/* PL011 UART wait UART to be ready to transmit
+/*
+ * PL011 UART wait UART to be ready to transmit
* rb: register which contains the UART base address
- * rc: scratch register */
+ * rc: scratch register
+ */
.macro early_uart_ready rb, rc
1:
ldr \rc, [\rb, #FR] /* <- UARTFR (Flag register) */
@@ -43,9 +47,11 @@
bne 1b /* Wait for the UART to be ready */
.endm
-/* PL011 UART transmit character
+/*
+ * PL011 UART transmit character
* rb: register which contains the UART base address
- * rt: register which contains the character to transmit */
+ * rt: register which contains the character to transmit
+ */
.macro early_uart_transmit rb, rt
str \rt, [\rb, #DR] /* -> UARTDR (Data Register) */
.endm
diff --git a/xen/arch/arm/arm32/debug-scif.inc b/xen/arch/arm/arm32/debug-scif.inc
index ce85752..143f05d 100644
--- a/xen/arch/arm/arm32/debug-scif.inc
+++ b/xen/arch/arm/arm32/debug-scif.inc
@@ -19,7 +19,8 @@
#include <asm/scif-uart.h>
-/* SCIF UART wait UART to be ready to transmit
+/*
+ * SCIF UART wait UART to be ready to transmit
* rb: register which contains the UART base address
* rc: scratch register
*/
@@ -30,7 +31,8 @@
beq 1b /* Wait for the UART to be ready */
.endm
-/* SCIF UART transmit character
+/*
+ * SCIF UART transmit character
* rb: register which contains the UART base address
* rt: register which contains the character to transmit
*/
diff --git a/xen/arch/arm/arm32/debug.S b/xen/arch/arm/arm32/debug.S
index 1538090..1829b29 100644
--- a/xen/arch/arm/arm32/debug.S
+++ b/xen/arch/arm/arm32/debug.S
@@ -23,8 +23,10 @@
#include EARLY_PRINTK_INC
#endif
-/* Print a character on the UART - this function is called by C
- * r0: character to print */
+/*
+ * Print a character on the UART - this function is called by C
+ * r0: character to print
+ */
GLOBAL(early_putch)
ldr r1, =EARLY_UART_VIRTUAL_ADDRESS /* r1 := VA UART base address */
early_uart_ready r1, r2
diff --git a/xen/arch/arm/arm32/head.S b/xen/arch/arm/arm32/head.S
index 4090f4a..43374e7 100644
--- a/xen/arch/arm/arm32/head.S
+++ b/xen/arch/arm/arm32/head.S
@@ -71,12 +71,15 @@
.arm
- /* This must be the very first address in the loaded image.
+ /*
+ * This must be the very first address in the loaded image.
* It should be linked at XEN_VIRT_START, and loaded at any
* 4K-aligned address. All of text+data+bss must fit in 2MB,
- * or the initial pagetable code below will need adjustment. */
+ * or the initial pagetable code below will need adjustment.
+ */
GLOBAL(start)
- /* zImage magic header, see:
+ /*
+ * zImage magic header, see:
* http://www.simtec.co.uk/products/SWLINUX/files/booting_article.html#d0e309
*/
.rept 8
@@ -222,33 +225,40 @@ cpu_init_done:
mcr CP32(r0, HMAIR0)
mcr CP32(r1, HMAIR1)
- /* Set up the HTCR:
+ /*
+ * Set up the HTCR:
* PT walks use Inner-Shareable accesses,
* PT walks are write-back, write-allocate in both cache levels,
- * Full 32-bit address space goes through this table. */
+ * Full 32-bit address space goes through this table.
+ */
ldr r0, =(TCR_RES1|TCR_SH0_IS|TCR_ORGN0_WBWA|TCR_IRGN0_WBWA|TCR_T0SZ(0))
mcr CP32(r0, HTCR)
- /* Set up the HSCTLR:
+ /*
+ * Set up the HSCTLR:
* Exceptions in LE ARM,
* Low-latency IRQs disabled,
* Write-implies-XN disabled (for now),
* D-cache disabled (for now),
* I-cache enabled,
* Alignment checking enabled,
- * MMU translation disabled (for now). */
+ * MMU translation disabled (for now).
+ */
ldr r0, =(HSCTLR_BASE|SCTLR_A)
mcr CP32(r0, HSCTLR)
- /* Rebuild the boot pagetable's first-level entries. The structure
+ /*
+ * Rebuild the boot pagetable's first-level entries. The structure
* is described in mm.c.
*
* After the CPU enables paging it will add the fixmap mapping
* to these page tables, however this may clash with the 1:1
* mapping. So each CPU must rebuild the page tables here with
- * the 1:1 in place. */
+ * the 1:1 in place.
+ */
- /* If Xen is loaded at exactly XEN_VIRT_START then we don't
+ /*
+ * If Xen is loaded at exactly XEN_VIRT_START then we don't
* need an additional 1:1 mapping, the virtual mapping will
* suffice.
*/
@@ -331,8 +341,10 @@ cpu_init_done:
cmp r1, #(LPAE_ENTRIES<<3) /* 512*8-byte entries per page */
blo 1b
- /* Defer fixmap and dtb mapping until after paging enabled, to
- * avoid them clashing with the 1:1 mapping. */
+ /*
+ * Defer fixmap and dtb mapping until after paging enabled, to
+ * avoid them clashing with the 1:1 mapping.
+ */
/* boot pagetable setup complete */
@@ -357,12 +369,16 @@ virtphys_clash:
mov pc, r1 /* Get a proper vaddr into PC */
paging:
- /* Now we can install the fixmap and dtb mappings, since we
- * don't need the 1:1 map any more */
+ /*
+ * Now we can install the fixmap and dtb mappings, since we
+ * don't need the 1:1 map any more
+ */
dsb
#if defined(CONFIG_EARLY_PRINTK) /* Fixmap is only used by early printk */
- /* Non-boot CPUs don't need to rebuild the fixmap itself, just
- * the mapping from boot_second to xen_fixmap */
+ /*
+ * Non-boot CPUs don't need to rebuild the fixmap itself, just
+ * the mapping from boot_second to xen_fixmap
+ */
teq r12, #0
bne 1f
@@ -408,8 +424,10 @@ paging:
teq r12, #0
beq launch
- /* Non-boot CPUs need to move on to the proper pagetables, which were
- * setup in init_secondary_pagetables. */
+ /*
+ * Non-boot CPUs need to move on to the proper pagetables, which were
+ * setup in init_secondary_pagetables.
+ */
ldr r4, =init_ttbr /* VA of HTTBR value stashed by CPU 0 */
ldrd r4, r5, [r4] /* Actual value */
@@ -443,7 +461,8 @@ fail: PRINT("- Boot failed -\r\n")
GLOBAL(_end_boot)
-/* Copy Xen to new location and switch TTBR
+/*
+ * Copy Xen to new location and switch TTBR
* r1:r0 ttbr
* r2 source address
* r3 destination address
@@ -452,7 +471,8 @@ GLOBAL(_end_boot)
* Source and destination must be word aligned, length is rounded up
* to a 16 byte boundary.
*
- * MUST BE VERY CAREFUL when saving things to RAM over the copy */
+ * MUST BE VERY CAREFUL when saving things to RAM over the copy
+ */
ENTRY(relocate_xen)
push {r4,r5,r6,r7,r8,r9,r10,r11}
@@ -521,9 +541,11 @@ ENTRY(relocate_xen)
mov pc, lr
#ifdef CONFIG_EARLY_PRINTK
-/* Bring up the UART.
+/*
+ * Bring up the UART.
* r11: Early UART base address
- * Clobbers r0-r2 */
+ * Clobbers r0-r2
+ */
init_uart:
#ifdef EARLY_PRINTK_INIT_UART
early_uart_init r11, r1, r2
@@ -533,10 +555,12 @@ init_uart:
1: .asciz "- UART enabled -\r\n"
.align 4
-/* Print early debug messages.
+/*
+ * Print early debug messages.
* r0: Nul-terminated string to print.
* r11: Early UART base address
- * Clobbers r0-r1 */
+ * Clobbers r0-r1
+ */
puts:
early_uart_ready r11, r1
ldrb r1, [r0], #1 /* Load next char */
@@ -545,10 +569,12 @@ puts:
early_uart_transmit r11, r1
b puts
-/* Print a 32-bit number in hex. Specific to the PL011 UART.
+/*
+ * Print a 32-bit number in hex. Specific to the PL011 UART.
* r0: Number to print.
* r11: Early UART base address
- * Clobbers r0-r3 */
+ * Clobbers r0-r3
+ */
putn:
adr r1, hex
mov r3, #8
@@ -582,7 +608,8 @@ ENTRY(lookup_processor_type)
mov r0, r1
ldmfd sp!, {r4, r10, pc}
-/* Read processor ID register (CP#15, CR0), and Look up in the linker-built
+/*
+ * Read processor ID register (CP#15, CR0), and Look up in the linker-built
* supported processor list. Note that we can't use the absolute addresses for
* the __proc_info lists since we aren't running with the MMU on (and therefore,
* we are not in correct address space). We have to calculate the offset.
diff --git a/xen/arch/arm/arm64/debug-8250.inc b/xen/arch/arm/arm64/debug-8250.inc
index 158549a..53d6828 100644
--- a/xen/arch/arm/arm64/debug-8250.inc
+++ b/xen/arch/arm/arm64/debug-8250.inc
@@ -18,9 +18,11 @@
#include <xen/8250-uart.h>
-/* UART wait UART to be ready to transmit
+/*
+ * UART wait UART to be ready to transmit
* xb: register which contains the UART base address
- * c: scratch register */
+ * c: scratch register
+ */
.macro early_uart_ready xb c
1:
ldrb w\c, [\xb, #UART_LSR << EARLY_UART_REG_SHIFT]
@@ -29,9 +31,11 @@
b.ne 1b
.endm
-/* UART transmit character
+/*
+ * UART transmit character
* xb: register which contains the UART base address
- * wt: register which contains the character to transmit */
+ * wt: register which contains the character to transmit
+ */
.macro early_uart_transmit xb wt
/* UART_THR transmit holding */
strb \wt, [\xb, #UART_THR << EARLY_UART_REG_SHIFT]
diff --git a/xen/arch/arm/arm64/debug-cadence.inc b/xen/arch/arm/arm64/debug-cadence.inc
index 84dee4c..7df0abe 100644
--- a/xen/arch/arm/arm64/debug-cadence.inc
+++ b/xen/arch/arm/arm64/debug-cadence.inc
@@ -20,9 +20,11 @@
#include <asm/asm_defns.h>
#include <asm/cadence-uart.h>
-/* Cadence UART wait UART to be ready to transmit
+/*
+ * Cadence UART wait UART to be ready to transmit
* xb: register which contains the UART base address
- * c: scratch register number */
+ * c: scratch register number
+ */
.macro early_uart_ready xb, c
1:
ldrh w\c, [\xb, #R_UART_SR]
@@ -30,9 +32,11 @@
b.ne 1b
.endm
-/* Cadence UART transmit character
+/*
+ * Cadence UART transmit character
* xb: register which contains the UART base address
- * wt: register which contains the character to transmit */
+ * wt: register which contains the character to transmit
+ */
.macro early_uart_transmit xb, wt
strb \wt, [\xb, #R_UART_TX]
.endm
diff --git a/xen/arch/arm/arm64/debug-pl011.inc b/xen/arch/arm/arm64/debug-pl011.inc
index b416235..569c3df 100644
--- a/xen/arch/arm/arm64/debug-pl011.inc
+++ b/xen/arch/arm/arm64/debug-pl011.inc
@@ -18,9 +18,11 @@
#include <asm/asm_defns.h>
-/* PL011 UART initialization
+/*
+ * PL011 UART initialization
* xb: register which containts the UART base address
- * c: scratch register number */
+ * c: scratch register number
+ */
.macro early_uart_init xb, c
mov x\c, #(7372800 / EARLY_PRINTK_BAUD % 16)
strh w\c, [\xb, #0x28] /* -> UARTFBRD (Baud divisor fraction) */
@@ -32,9 +34,11 @@
str w\c, [\xb, #0x30] /* -> UARTCR (Control Register) */
.endm
-/* PL011 UART wait UART to be ready to transmit
+/*
+ * PL011 UART wait UART to be ready to transmit
* xb: register which contains the UART base address
- * c: scratch register number */
+ * c: scratch register number
+ */
.macro early_uart_ready xb, c
1:
ldrh w\c, [\xb, #0x18] /* <- UARTFR (Flag register) */
@@ -42,9 +46,11 @@
b.ne 1b /* Wait for the UART to be ready */
.endm
-/* PL011 UART transmit character
+/*
+ * PL011 UART transmit character
* xb: register which contains the UART base address
- * wt: register which contains the character to transmit */
+ * wt: register which contains the character to transmit
+ */
.macro early_uart_transmit xb, wt
strb \wt, [\xb] /* -> UARTDR (Data Register) */
.endm
diff --git a/xen/arch/arm/arm64/debug.S b/xen/arch/arm/arm64/debug.S
index 87ee8dc..b7f53ac 100644
--- a/xen/arch/arm/arm64/debug.S
+++ b/xen/arch/arm/arm64/debug.S
@@ -23,8 +23,10 @@
#include EARLY_PRINTK_INC
#endif
-/* Print a character on the UART - this function is called by C
- * x0: character to print */
+/*
+ * Print a character on the UART - this function is called by C
+ * x0: character to print
+ */
GLOBAL(early_putch)
ldr x15, =EARLY_UART_VIRTUAL_ADDRESS
early_uart_ready x15, 1
diff --git a/xen/arch/arm/arm64/entry.S b/xen/arch/arm/arm64/entry.S
index 06afc8a..6d99e46 100644
--- a/xen/arch/arm/arm64/entry.S
+++ b/xen/arch/arm/arm64/entry.S
@@ -6,7 +6,7 @@
/*
* Register aliases.
*/
-lr .req x30 // link register
+lr .req x30 /* link register */
/*
* Stack pushing/popping (register pairs only). Equivalent to store decrement
@@ -293,8 +293,8 @@ ENTRY(return_to_new_vcpu64)
return_from_trap:
msr daifset, #2 /* Mask interrupts */
- ldr x21, [sp, #UREGS_PC] // load ELR
- ldr w22, [sp, #UREGS_CPSR] // load SPSR
+ ldr x21, [sp, #UREGS_PC] /* load ELR */
+ ldr w22, [sp, #UREGS_CPSR] /* load SPSR */
pop x0, x1
pop x2, x3
@@ -302,7 +302,7 @@ return_from_trap:
pop x6, x7
pop x8, x9
- msr elr_el2, x21 // set up the return data
+ msr elr_el2, x21 /* set up the return data */
msr spsr_el2, x22
pop x10, x11
@@ -388,25 +388,25 @@ abort_guest_exit_end:
.align 11
ENTRY(hyp_traps_vector)
- ventry hyp_sync_invalid // Synchronous EL2t
- ventry hyp_irq_invalid // IRQ EL2t
- ventry hyp_fiq_invalid // FIQ EL2t
- ventry hyp_error_invalid // Error EL2t
-
- ventry hyp_sync // Synchronous EL2h
- ventry hyp_irq // IRQ EL2h
- ventry hyp_fiq_invalid // FIQ EL2h
- ventry hyp_error // Error EL2h
-
- ventry guest_sync // Synchronous 64-bit EL0/EL1
- ventry guest_irq // IRQ 64-bit EL0/EL1
- ventry guest_fiq_invalid // FIQ 64-bit EL0/EL1
- ventry guest_error // Error 64-bit EL0/EL1
-
- ventry guest_sync_compat // Synchronous 32-bit EL0/EL1
- ventry guest_irq_compat // IRQ 32-bit EL0/EL1
- ventry guest_fiq_invalid_compat // FIQ 32-bit EL0/EL1
- ventry guest_error_compat // Error 32-bit EL0/EL1
+ ventry hyp_sync_invalid /* Synchronous EL2t */
+ ventry hyp_irq_invalid /* IRQ EL2t */
+ ventry hyp_fiq_invalid /* FIQ EL2t */
+ ventry hyp_error_invalid /* Error EL2t */
+
+ ventry hyp_sync /* Synchronous EL2h */
+ ventry hyp_irq /* IRQ EL2h */
+ ventry hyp_fiq_invalid /* FIQ EL2h */
+ ventry hyp_error /* Error EL2h */
+
+ ventry guest_sync /* Synchronous 64-bit EL0/EL1 */
+ ventry guest_irq /* IRQ 64-bit EL0/EL1 */
+ ventry guest_fiq_invalid /* FIQ 64-bit EL0/EL1 */
+ ventry guest_error /* Error 64-bit EL0/EL1 */
+
+ ventry guest_sync_compat /* Synchronous 32-bit EL0/EL1 */
+ ventry guest_irq_compat /* IRQ 32-bit EL0/EL1 */
+ ventry guest_fiq_invalid_compat /* FIQ 32-bit EL0/EL1 */
+ ventry guest_error_compat /* Error 32-bit EL0/EL1 */
/*
* struct vcpu *__context_switch(struct vcpu *prev, struct vcpu *next)
@@ -419,7 +419,7 @@ ENTRY(hyp_traps_vector)
ENTRY(__context_switch)
add x8, x0, #VCPU_arch_saved_context
mov x9, sp
- stp x19, x20, [x8], #16 // store callee-saved registers
+ stp x19, x20, [x8], #16 /* store callee-saved registers */
stp x21, x22, [x8], #16
stp x23, x24, [x8], #16
stp x25, x26, [x8], #16
@@ -428,7 +428,7 @@ ENTRY(__context_switch)
str lr, [x8]
add x8, x1, #VCPU_arch_saved_context
- ldp x19, x20, [x8], #16 // restore callee-saved registers
+ ldp x19, x20, [x8], #16 /* restore callee-saved registers */
ldp x21, x22, [x8], #16
ldp x23, x24, [x8], #16
ldp x25, x26, [x8], #16
--
2.7.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] xen/arm: Fix comments coding style in assembler files
2017-07-27 14:55 [PATCH] xen/arm: Fix comments coding style in assembler files Andrii Anisov
@ 2017-07-31 20:39 ` Stefano Stabellini
0 siblings, 0 replies; 2+ messages in thread
From: Stefano Stabellini @ 2017-07-31 20:39 UTC (permalink / raw)
To: Andrii Anisov; +Cc: julien.grall, sstabellini, Andrii Anisov, xen-devel
On Thu, 27 Jul 2017, Andrii Anisov wrote:
> From: Andrii Anisov <andrii_anisov@epam.com>
>
> Signed-off-by: Andrii Anisov <andrii_anisov@epam.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
> ---
> xen/arch/arm/arm32/debug-8250.inc | 12 +++--
> xen/arch/arm/arm32/debug-exynos4210.inc | 12 +++--
> xen/arch/arm/arm32/debug-pl011.inc | 18 +++++---
> xen/arch/arm/arm32/debug-scif.inc | 6 ++-
> xen/arch/arm/arm32/debug.S | 6 ++-
> xen/arch/arm/arm32/head.S | 81 ++++++++++++++++++++++-----------
> xen/arch/arm/arm64/debug-8250.inc | 12 +++--
> xen/arch/arm/arm64/debug-cadence.inc | 12 +++--
> xen/arch/arm/arm64/debug-pl011.inc | 18 +++++---
> xen/arch/arm/arm64/debug.S | 6 ++-
> xen/arch/arm/arm64/entry.S | 50 ++++++++++----------
> 11 files changed, 147 insertions(+), 86 deletions(-)
>
> diff --git a/xen/arch/arm/arm32/debug-8250.inc b/xen/arch/arm/arm32/debug-8250.inc
> index 757ffd8..0759a27 100644
> --- a/xen/arch/arm/arm32/debug-8250.inc
> +++ b/xen/arch/arm/arm32/debug-8250.inc
> @@ -16,9 +16,11 @@
>
> #include <xen/8250-uart.h>
>
> -/* 8250 UART wait UART to be ready to transmit
> +/*
> + * 8250 UART wait UART to be ready to transmit
> * rb: register which contains the UART base address
> - * rc: scratch register */
> + * rc: scratch register
> + */
> .macro early_uart_ready rb rc
> 1:
> ldr \rc, [\rb, #(UART_LSR << EARLY_UART_REG_SHIFT)] /* Read LSR */
> @@ -26,9 +28,11 @@
> beq 1b /* Wait for the UART to be ready */
> .endm
>
> -/* 8250 UART transmit character
> +/*
> + * 8250 UART transmit character
> * rb: register which contains the UART base address
> - * rt: register which contains the character to transmit */
> + * rt: register which contains the character to transmit
> + */
> .macro early_uart_transmit rb rt
> str \rt, [\rb, #UART_THR] /* Write Transmit buffer */
> .endm
> diff --git a/xen/arch/arm/arm32/debug-exynos4210.inc b/xen/arch/arm/arm32/debug-exynos4210.inc
> index 752942d..4e80a21 100644
> --- a/xen/arch/arm/arm32/debug-exynos4210.inc
> +++ b/xen/arch/arm/arm32/debug-exynos4210.inc
> @@ -18,9 +18,11 @@
>
> #include <asm/exynos4210-uart.h>
>
> -/* Exynos 5 UART wait UART to be ready to transmit
> +/*
> + * Exynos 5 UART wait UART to be ready to transmit
> * rb: register which contains the UART base address
> - * rc: scratch register */
> + * rc: scratch register
> + */
> .macro early_uart_ready rb rc
> 1:
> ldr \rc, [\rb, #UTRSTAT] /* <- UTRSTAT (Flag register) */
> @@ -28,9 +30,11 @@
> beq 1b /* Wait for the UART to be ready */
> .endm
>
> -/* Exynos 5 UART transmit character
> +/*
> + * Exynos 5 UART transmit character
> * rb: register which contains the UART base address
> - * rt: register which contains the character to transmit */
> + * rt: register which contains the character to transmit
> + */
> .macro early_uart_transmit rb rt
> str \rt, [\rb, #UTXH] /* -> UTXH (Data Register) */
> .endm
> diff --git a/xen/arch/arm/arm32/debug-pl011.inc b/xen/arch/arm/arm32/debug-pl011.inc
> index 6a64dbf..ec462ea 100644
> --- a/xen/arch/arm/arm32/debug-pl011.inc
> +++ b/xen/arch/arm/arm32/debug-pl011.inc
> @@ -18,10 +18,12 @@
>
> #include <asm/pl011-uart.h>
>
> -/* PL011 UART initialization
> +/*
> + * PL011 UART initialization
> * rb: register which contains the UART base address
> * rc: scratch register 1
> - * rd: scratch register 2 (unused here) */
> + * rd: scratch register 2 (unused here)
> + */
> .macro early_uart_init rb, rc, rd
> mov \rc, #(7372800 / EARLY_PRINTK_BAUD % 16)
> str \rc, [\rb, #FBRD] /* -> UARTFBRD (Baud divisor fraction) */
> @@ -33,9 +35,11 @@
> str \rc, [\rb, #CR] /* -> UARTCR (Control Register) */
> .endm
>
> -/* PL011 UART wait UART to be ready to transmit
> +/*
> + * PL011 UART wait UART to be ready to transmit
> * rb: register which contains the UART base address
> - * rc: scratch register */
> + * rc: scratch register
> + */
> .macro early_uart_ready rb, rc
> 1:
> ldr \rc, [\rb, #FR] /* <- UARTFR (Flag register) */
> @@ -43,9 +47,11 @@
> bne 1b /* Wait for the UART to be ready */
> .endm
>
> -/* PL011 UART transmit character
> +/*
> + * PL011 UART transmit character
> * rb: register which contains the UART base address
> - * rt: register which contains the character to transmit */
> + * rt: register which contains the character to transmit
> + */
> .macro early_uart_transmit rb, rt
> str \rt, [\rb, #DR] /* -> UARTDR (Data Register) */
> .endm
> diff --git a/xen/arch/arm/arm32/debug-scif.inc b/xen/arch/arm/arm32/debug-scif.inc
> index ce85752..143f05d 100644
> --- a/xen/arch/arm/arm32/debug-scif.inc
> +++ b/xen/arch/arm/arm32/debug-scif.inc
> @@ -19,7 +19,8 @@
>
> #include <asm/scif-uart.h>
>
> -/* SCIF UART wait UART to be ready to transmit
> +/*
> + * SCIF UART wait UART to be ready to transmit
> * rb: register which contains the UART base address
> * rc: scratch register
> */
> @@ -30,7 +31,8 @@
> beq 1b /* Wait for the UART to be ready */
> .endm
>
> -/* SCIF UART transmit character
> +/*
> + * SCIF UART transmit character
> * rb: register which contains the UART base address
> * rt: register which contains the character to transmit
> */
> diff --git a/xen/arch/arm/arm32/debug.S b/xen/arch/arm/arm32/debug.S
> index 1538090..1829b29 100644
> --- a/xen/arch/arm/arm32/debug.S
> +++ b/xen/arch/arm/arm32/debug.S
> @@ -23,8 +23,10 @@
> #include EARLY_PRINTK_INC
> #endif
>
> -/* Print a character on the UART - this function is called by C
> - * r0: character to print */
> +/*
> + * Print a character on the UART - this function is called by C
> + * r0: character to print
> + */
> GLOBAL(early_putch)
> ldr r1, =EARLY_UART_VIRTUAL_ADDRESS /* r1 := VA UART base address */
> early_uart_ready r1, r2
> diff --git a/xen/arch/arm/arm32/head.S b/xen/arch/arm/arm32/head.S
> index 4090f4a..43374e7 100644
> --- a/xen/arch/arm/arm32/head.S
> +++ b/xen/arch/arm/arm32/head.S
> @@ -71,12 +71,15 @@
>
> .arm
>
> - /* This must be the very first address in the loaded image.
> + /*
> + * This must be the very first address in the loaded image.
> * It should be linked at XEN_VIRT_START, and loaded at any
> * 4K-aligned address. All of text+data+bss must fit in 2MB,
> - * or the initial pagetable code below will need adjustment. */
> + * or the initial pagetable code below will need adjustment.
> + */
> GLOBAL(start)
> - /* zImage magic header, see:
> + /*
> + * zImage magic header, see:
> * http://www.simtec.co.uk/products/SWLINUX/files/booting_article.html#d0e309
> */
> .rept 8
> @@ -222,33 +225,40 @@ cpu_init_done:
> mcr CP32(r0, HMAIR0)
> mcr CP32(r1, HMAIR1)
>
> - /* Set up the HTCR:
> + /*
> + * Set up the HTCR:
> * PT walks use Inner-Shareable accesses,
> * PT walks are write-back, write-allocate in both cache levels,
> - * Full 32-bit address space goes through this table. */
> + * Full 32-bit address space goes through this table.
> + */
> ldr r0, =(TCR_RES1|TCR_SH0_IS|TCR_ORGN0_WBWA|TCR_IRGN0_WBWA|TCR_T0SZ(0))
> mcr CP32(r0, HTCR)
>
> - /* Set up the HSCTLR:
> + /*
> + * Set up the HSCTLR:
> * Exceptions in LE ARM,
> * Low-latency IRQs disabled,
> * Write-implies-XN disabled (for now),
> * D-cache disabled (for now),
> * I-cache enabled,
> * Alignment checking enabled,
> - * MMU translation disabled (for now). */
> + * MMU translation disabled (for now).
> + */
> ldr r0, =(HSCTLR_BASE|SCTLR_A)
> mcr CP32(r0, HSCTLR)
>
> - /* Rebuild the boot pagetable's first-level entries. The structure
> + /*
> + * Rebuild the boot pagetable's first-level entries. The structure
> * is described in mm.c.
> *
> * After the CPU enables paging it will add the fixmap mapping
> * to these page tables, however this may clash with the 1:1
> * mapping. So each CPU must rebuild the page tables here with
> - * the 1:1 in place. */
> + * the 1:1 in place.
> + */
>
> - /* If Xen is loaded at exactly XEN_VIRT_START then we don't
> + /*
> + * If Xen is loaded at exactly XEN_VIRT_START then we don't
> * need an additional 1:1 mapping, the virtual mapping will
> * suffice.
> */
> @@ -331,8 +341,10 @@ cpu_init_done:
> cmp r1, #(LPAE_ENTRIES<<3) /* 512*8-byte entries per page */
> blo 1b
>
> - /* Defer fixmap and dtb mapping until after paging enabled, to
> - * avoid them clashing with the 1:1 mapping. */
> + /*
> + * Defer fixmap and dtb mapping until after paging enabled, to
> + * avoid them clashing with the 1:1 mapping.
> + */
>
> /* boot pagetable setup complete */
>
> @@ -357,12 +369,16 @@ virtphys_clash:
> mov pc, r1 /* Get a proper vaddr into PC */
> paging:
>
> - /* Now we can install the fixmap and dtb mappings, since we
> - * don't need the 1:1 map any more */
> + /*
> + * Now we can install the fixmap and dtb mappings, since we
> + * don't need the 1:1 map any more
> + */
> dsb
> #if defined(CONFIG_EARLY_PRINTK) /* Fixmap is only used by early printk */
> - /* Non-boot CPUs don't need to rebuild the fixmap itself, just
> - * the mapping from boot_second to xen_fixmap */
> + /*
> + * Non-boot CPUs don't need to rebuild the fixmap itself, just
> + * the mapping from boot_second to xen_fixmap
> + */
> teq r12, #0
> bne 1f
>
> @@ -408,8 +424,10 @@ paging:
> teq r12, #0
> beq launch
>
> - /* Non-boot CPUs need to move on to the proper pagetables, which were
> - * setup in init_secondary_pagetables. */
> + /*
> + * Non-boot CPUs need to move on to the proper pagetables, which were
> + * setup in init_secondary_pagetables.
> + */
>
> ldr r4, =init_ttbr /* VA of HTTBR value stashed by CPU 0 */
> ldrd r4, r5, [r4] /* Actual value */
> @@ -443,7 +461,8 @@ fail: PRINT("- Boot failed -\r\n")
>
> GLOBAL(_end_boot)
>
> -/* Copy Xen to new location and switch TTBR
> +/*
> + * Copy Xen to new location and switch TTBR
> * r1:r0 ttbr
> * r2 source address
> * r3 destination address
> @@ -452,7 +471,8 @@ GLOBAL(_end_boot)
> * Source and destination must be word aligned, length is rounded up
> * to a 16 byte boundary.
> *
> - * MUST BE VERY CAREFUL when saving things to RAM over the copy */
> + * MUST BE VERY CAREFUL when saving things to RAM over the copy
> + */
> ENTRY(relocate_xen)
> push {r4,r5,r6,r7,r8,r9,r10,r11}
>
> @@ -521,9 +541,11 @@ ENTRY(relocate_xen)
> mov pc, lr
>
> #ifdef CONFIG_EARLY_PRINTK
> -/* Bring up the UART.
> +/*
> + * Bring up the UART.
> * r11: Early UART base address
> - * Clobbers r0-r2 */
> + * Clobbers r0-r2
> + */
> init_uart:
> #ifdef EARLY_PRINTK_INIT_UART
> early_uart_init r11, r1, r2
> @@ -533,10 +555,12 @@ init_uart:
> 1: .asciz "- UART enabled -\r\n"
> .align 4
>
> -/* Print early debug messages.
> +/*
> + * Print early debug messages.
> * r0: Nul-terminated string to print.
> * r11: Early UART base address
> - * Clobbers r0-r1 */
> + * Clobbers r0-r1
> + */
> puts:
> early_uart_ready r11, r1
> ldrb r1, [r0], #1 /* Load next char */
> @@ -545,10 +569,12 @@ puts:
> early_uart_transmit r11, r1
> b puts
>
> -/* Print a 32-bit number in hex. Specific to the PL011 UART.
> +/*
> + * Print a 32-bit number in hex. Specific to the PL011 UART.
> * r0: Number to print.
> * r11: Early UART base address
> - * Clobbers r0-r3 */
> + * Clobbers r0-r3
> + */
> putn:
> adr r1, hex
> mov r3, #8
> @@ -582,7 +608,8 @@ ENTRY(lookup_processor_type)
> mov r0, r1
> ldmfd sp!, {r4, r10, pc}
>
> -/* Read processor ID register (CP#15, CR0), and Look up in the linker-built
> +/*
> + * Read processor ID register (CP#15, CR0), and Look up in the linker-built
> * supported processor list. Note that we can't use the absolute addresses for
> * the __proc_info lists since we aren't running with the MMU on (and therefore,
> * we are not in correct address space). We have to calculate the offset.
> diff --git a/xen/arch/arm/arm64/debug-8250.inc b/xen/arch/arm/arm64/debug-8250.inc
> index 158549a..53d6828 100644
> --- a/xen/arch/arm/arm64/debug-8250.inc
> +++ b/xen/arch/arm/arm64/debug-8250.inc
> @@ -18,9 +18,11 @@
>
> #include <xen/8250-uart.h>
>
> -/* UART wait UART to be ready to transmit
> +/*
> + * UART wait UART to be ready to transmit
> * xb: register which contains the UART base address
> - * c: scratch register */
> + * c: scratch register
> + */
> .macro early_uart_ready xb c
> 1:
> ldrb w\c, [\xb, #UART_LSR << EARLY_UART_REG_SHIFT]
> @@ -29,9 +31,11 @@
> b.ne 1b
> .endm
>
> -/* UART transmit character
> +/*
> + * UART transmit character
> * xb: register which contains the UART base address
> - * wt: register which contains the character to transmit */
> + * wt: register which contains the character to transmit
> + */
> .macro early_uart_transmit xb wt
> /* UART_THR transmit holding */
> strb \wt, [\xb, #UART_THR << EARLY_UART_REG_SHIFT]
> diff --git a/xen/arch/arm/arm64/debug-cadence.inc b/xen/arch/arm/arm64/debug-cadence.inc
> index 84dee4c..7df0abe 100644
> --- a/xen/arch/arm/arm64/debug-cadence.inc
> +++ b/xen/arch/arm/arm64/debug-cadence.inc
> @@ -20,9 +20,11 @@
> #include <asm/asm_defns.h>
> #include <asm/cadence-uart.h>
>
> -/* Cadence UART wait UART to be ready to transmit
> +/*
> + * Cadence UART wait UART to be ready to transmit
> * xb: register which contains the UART base address
> - * c: scratch register number */
> + * c: scratch register number
> + */
> .macro early_uart_ready xb, c
> 1:
> ldrh w\c, [\xb, #R_UART_SR]
> @@ -30,9 +32,11 @@
> b.ne 1b
> .endm
>
> -/* Cadence UART transmit character
> +/*
> + * Cadence UART transmit character
> * xb: register which contains the UART base address
> - * wt: register which contains the character to transmit */
> + * wt: register which contains the character to transmit
> + */
> .macro early_uart_transmit xb, wt
> strb \wt, [\xb, #R_UART_TX]
> .endm
> diff --git a/xen/arch/arm/arm64/debug-pl011.inc b/xen/arch/arm/arm64/debug-pl011.inc
> index b416235..569c3df 100644
> --- a/xen/arch/arm/arm64/debug-pl011.inc
> +++ b/xen/arch/arm/arm64/debug-pl011.inc
> @@ -18,9 +18,11 @@
>
> #include <asm/asm_defns.h>
>
> -/* PL011 UART initialization
> +/*
> + * PL011 UART initialization
> * xb: register which containts the UART base address
> - * c: scratch register number */
> + * c: scratch register number
> + */
> .macro early_uart_init xb, c
> mov x\c, #(7372800 / EARLY_PRINTK_BAUD % 16)
> strh w\c, [\xb, #0x28] /* -> UARTFBRD (Baud divisor fraction) */
> @@ -32,9 +34,11 @@
> str w\c, [\xb, #0x30] /* -> UARTCR (Control Register) */
> .endm
>
> -/* PL011 UART wait UART to be ready to transmit
> +/*
> + * PL011 UART wait UART to be ready to transmit
> * xb: register which contains the UART base address
> - * c: scratch register number */
> + * c: scratch register number
> + */
> .macro early_uart_ready xb, c
> 1:
> ldrh w\c, [\xb, #0x18] /* <- UARTFR (Flag register) */
> @@ -42,9 +46,11 @@
> b.ne 1b /* Wait for the UART to be ready */
> .endm
>
> -/* PL011 UART transmit character
> +/*
> + * PL011 UART transmit character
> * xb: register which contains the UART base address
> - * wt: register which contains the character to transmit */
> + * wt: register which contains the character to transmit
> + */
> .macro early_uart_transmit xb, wt
> strb \wt, [\xb] /* -> UARTDR (Data Register) */
> .endm
> diff --git a/xen/arch/arm/arm64/debug.S b/xen/arch/arm/arm64/debug.S
> index 87ee8dc..b7f53ac 100644
> --- a/xen/arch/arm/arm64/debug.S
> +++ b/xen/arch/arm/arm64/debug.S
> @@ -23,8 +23,10 @@
> #include EARLY_PRINTK_INC
> #endif
>
> -/* Print a character on the UART - this function is called by C
> - * x0: character to print */
> +/*
> + * Print a character on the UART - this function is called by C
> + * x0: character to print
> + */
> GLOBAL(early_putch)
> ldr x15, =EARLY_UART_VIRTUAL_ADDRESS
> early_uart_ready x15, 1
> diff --git a/xen/arch/arm/arm64/entry.S b/xen/arch/arm/arm64/entry.S
> index 06afc8a..6d99e46 100644
> --- a/xen/arch/arm/arm64/entry.S
> +++ b/xen/arch/arm/arm64/entry.S
> @@ -6,7 +6,7 @@
> /*
> * Register aliases.
> */
> -lr .req x30 // link register
> +lr .req x30 /* link register */
>
> /*
> * Stack pushing/popping (register pairs only). Equivalent to store decrement
> @@ -293,8 +293,8 @@ ENTRY(return_to_new_vcpu64)
> return_from_trap:
> msr daifset, #2 /* Mask interrupts */
>
> - ldr x21, [sp, #UREGS_PC] // load ELR
> - ldr w22, [sp, #UREGS_CPSR] // load SPSR
> + ldr x21, [sp, #UREGS_PC] /* load ELR */
> + ldr w22, [sp, #UREGS_CPSR] /* load SPSR */
>
> pop x0, x1
> pop x2, x3
> @@ -302,7 +302,7 @@ return_from_trap:
> pop x6, x7
> pop x8, x9
>
> - msr elr_el2, x21 // set up the return data
> + msr elr_el2, x21 /* set up the return data */
> msr spsr_el2, x22
>
> pop x10, x11
> @@ -388,25 +388,25 @@ abort_guest_exit_end:
>
> .align 11
> ENTRY(hyp_traps_vector)
> - ventry hyp_sync_invalid // Synchronous EL2t
> - ventry hyp_irq_invalid // IRQ EL2t
> - ventry hyp_fiq_invalid // FIQ EL2t
> - ventry hyp_error_invalid // Error EL2t
> -
> - ventry hyp_sync // Synchronous EL2h
> - ventry hyp_irq // IRQ EL2h
> - ventry hyp_fiq_invalid // FIQ EL2h
> - ventry hyp_error // Error EL2h
> -
> - ventry guest_sync // Synchronous 64-bit EL0/EL1
> - ventry guest_irq // IRQ 64-bit EL0/EL1
> - ventry guest_fiq_invalid // FIQ 64-bit EL0/EL1
> - ventry guest_error // Error 64-bit EL0/EL1
> -
> - ventry guest_sync_compat // Synchronous 32-bit EL0/EL1
> - ventry guest_irq_compat // IRQ 32-bit EL0/EL1
> - ventry guest_fiq_invalid_compat // FIQ 32-bit EL0/EL1
> - ventry guest_error_compat // Error 32-bit EL0/EL1
> + ventry hyp_sync_invalid /* Synchronous EL2t */
> + ventry hyp_irq_invalid /* IRQ EL2t */
> + ventry hyp_fiq_invalid /* FIQ EL2t */
> + ventry hyp_error_invalid /* Error EL2t */
> +
> + ventry hyp_sync /* Synchronous EL2h */
> + ventry hyp_irq /* IRQ EL2h */
> + ventry hyp_fiq_invalid /* FIQ EL2h */
> + ventry hyp_error /* Error EL2h */
> +
> + ventry guest_sync /* Synchronous 64-bit EL0/EL1 */
> + ventry guest_irq /* IRQ 64-bit EL0/EL1 */
> + ventry guest_fiq_invalid /* FIQ 64-bit EL0/EL1 */
> + ventry guest_error /* Error 64-bit EL0/EL1 */
> +
> + ventry guest_sync_compat /* Synchronous 32-bit EL0/EL1 */
> + ventry guest_irq_compat /* IRQ 32-bit EL0/EL1 */
> + ventry guest_fiq_invalid_compat /* FIQ 32-bit EL0/EL1 */
> + ventry guest_error_compat /* Error 32-bit EL0/EL1 */
>
> /*
> * struct vcpu *__context_switch(struct vcpu *prev, struct vcpu *next)
> @@ -419,7 +419,7 @@ ENTRY(hyp_traps_vector)
> ENTRY(__context_switch)
> add x8, x0, #VCPU_arch_saved_context
> mov x9, sp
> - stp x19, x20, [x8], #16 // store callee-saved registers
> + stp x19, x20, [x8], #16 /* store callee-saved registers */
> stp x21, x22, [x8], #16
> stp x23, x24, [x8], #16
> stp x25, x26, [x8], #16
> @@ -428,7 +428,7 @@ ENTRY(__context_switch)
> str lr, [x8]
>
> add x8, x1, #VCPU_arch_saved_context
> - ldp x19, x20, [x8], #16 // restore callee-saved registers
> + ldp x19, x20, [x8], #16 /* restore callee-saved registers */
> ldp x21, x22, [x8], #16
> ldp x23, x24, [x8], #16
> ldp x25, x26, [x8], #16
> --
> 2.7.4
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-31 20:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-27 14:55 [PATCH] xen/arm: Fix comments coding style in assembler files Andrii Anisov
2017-07-31 20:39 ` Stefano Stabellini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.