[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 3/7] arm: link a device tree blob into the xen image

At 19:15 +0000 on 03 Feb (1328296511), David Vrabel wrote:
> diff --git a/xen/arch/arm/head.S b/xen/arch/arm/head.S
> index b98c921..9951f37 100644
> --- a/xen/arch/arm/head.S
> +++ b/xen/arch/arm/head.S
> @@ -55,6 +55,11 @@ start:
>       adr   r9, start              /* r9  := paddr (start) */
>       sub   r10, r9, r0            /* r10 := phys-offset */
> +        /* FIXME: using the DTB in the .dtb section. */

The rest of this file uses leading tabs for indentation (not for any
particular reason except emascs did it by default).  I'd be happy with a
patch to change to spaces throughout but I'd rather not mix.

> +        mov   r7, #0


> +        ldr   r8, =_sdtb
> +        add   r8, r10

This should have a comment like 'r8  := paddr (DTB)', to make it clear
what you're overriding the previous r8 value with.  (Yes, I'm being picky
but I really believe you can't over-comment asm, specially wierd start
of day stuff.)



Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.