* [U-Boot] [PATCH] ARM: image: indent with tab instead of 4 spaces
@ 2018-02-13 2:32 Masahiro Yamada
2018-02-13 2:57 ` Bin Chen
2018-02-14 16:43 ` [U-Boot] " Tom Rini
0 siblings, 2 replies; 3+ messages in thread
From: Masahiro Yamada @ 2018-02-13 2:32 UTC (permalink / raw)
To: u-boot
Commit 6808ef9ac2a6 ("move booti_setup to arch/arm/lig/image.c")
not only moved the code, but also replaced a tab with 4 spaces
to break the Linux coding style.
Restore tab indentation.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---
arch/arm/lib/image.c | 92 ++++++++++++++++++++++++++--------------------------
1 file changed, 46 insertions(+), 46 deletions(-)
diff --git a/arch/arm/lib/image.c b/arch/arm/lib/image.c
index f72cf63..92bfb8f 100644
--- a/arch/arm/lib/image.c
+++ b/arch/arm/lib/image.c
@@ -15,63 +15,63 @@ DECLARE_GLOBAL_DATA_PTR;
/* See Documentation/arm64/booting.txt in the Linux kernel */
struct Image_header {
- uint32_t code0; /* Executable code */
- uint32_t code1; /* Executable code */
- uint64_t text_offset; /* Image load offset, LE */
- uint64_t image_size; /* Effective Image size, LE */
- uint64_t flags; /* Kernel flags, LE */
- uint64_t res2; /* reserved */
- uint64_t res3; /* reserved */
- uint64_t res4; /* reserved */
- uint32_t magic; /* Magic number */
- uint32_t res5;
+ uint32_t code0; /* Executable code */
+ uint32_t code1; /* Executable code */
+ uint64_t text_offset; /* Image load offset, LE */
+ uint64_t image_size; /* Effective Image size, LE */
+ uint64_t flags; /* Kernel flags, LE */
+ uint64_t res2; /* reserved */
+ uint64_t res3; /* reserved */
+ uint64_t res4; /* reserved */
+ uint32_t magic; /* Magic number */
+ uint32_t res5;
};
int booti_setup(ulong image, ulong *relocated_addr, ulong *size)
{
- struct Image_header *ih;
- uint64_t dst;
- uint64_t image_size, text_offset;
+ struct Image_header *ih;
+ uint64_t dst;
+ uint64_t image_size, text_offset;
- *relocated_addr = image;
+ *relocated_addr = image;
- ih = (struct Image_header *)map_sysmem(image, 0);
+ ih = (struct Image_header *)map_sysmem(image, 0);
- if (ih->magic != le32_to_cpu(LINUX_ARM64_IMAGE_MAGIC)) {
- puts("Bad Linux ARM64 Image magic!\n");
- return 1;
- }
+ if (ih->magic != le32_to_cpu(LINUX_ARM64_IMAGE_MAGIC)) {
+ puts("Bad Linux ARM64 Image magic!\n");
+ return 1;
+ }
- /*
- * Prior to Linux commit a2c1d73b94ed, the text_offset field
- * is of unknown endianness. In these cases, the image_size
- * field is zero, and we can assume a fixed value of 0x80000.
- */
- if (ih->image_size == 0) {
- puts("Image lacks image_size field, assuming 16MiB\n");
- image_size = 16 << 20;
- text_offset = 0x80000;
- } else {
- image_size = le64_to_cpu(ih->image_size);
- text_offset = le64_to_cpu(ih->text_offset);
- }
+ /*
+ * Prior to Linux commit a2c1d73b94ed, the text_offset field
+ * is of unknown endianness. In these cases, the image_size
+ * field is zero, and we can assume a fixed value of 0x80000.
+ */
+ if (ih->image_size == 0) {
+ puts("Image lacks image_size field, assuming 16MiB\n");
+ image_size = 16 << 20;
+ text_offset = 0x80000;
+ } else {
+ image_size = le64_to_cpu(ih->image_size);
+ text_offset = le64_to_cpu(ih->text_offset);
+ }
- *size = image_size;
+ *size = image_size;
- /*
- * If bit 3 of the flags field is set, the 2MB aligned base of the
- * kernel image can be anywhere in physical memory, so respect
- * images->ep. Otherwise, relocate the image to the base of RAM
- * since memory below it is not accessible via the linear mapping.
- */
- if (le64_to_cpu(ih->flags) & BIT(3))
- dst = image - text_offset;
- else
- dst = gd->bd->bi_dram[0].start;
+ /*
+ * If bit 3 of the flags field is set, the 2MB aligned base of the
+ * kernel image can be anywhere in physical memory, so respect
+ * images->ep. Otherwise, relocate the image to the base of RAM
+ * since memory below it is not accessible via the linear mapping.
+ */
+ if (le64_to_cpu(ih->flags) & BIT(3))
+ dst = image - text_offset;
+ else
+ dst = gd->bd->bi_dram[0].start;
- *relocated_addr = ALIGN(dst, SZ_2M) + text_offset;
+ *relocated_addr = ALIGN(dst, SZ_2M) + text_offset;
- unmap_sysmem(ih);
+ unmap_sysmem(ih);
- return 0;
+ return 0;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] ARM: image: indent with tab instead of 4 spaces
2018-02-13 2:32 [U-Boot] [PATCH] ARM: image: indent with tab instead of 4 spaces Masahiro Yamada
@ 2018-02-13 2:57 ` Bin Chen
2018-02-14 16:43 ` [U-Boot] " Tom Rini
1 sibling, 0 replies; 3+ messages in thread
From: Bin Chen @ 2018-02-13 2:57 UTC (permalink / raw)
To: u-boot
On 13 February 2018 at 13:32, Masahiro Yamada <yamada.masahiro@socionext.com
> wrote:
> Commit 6808ef9ac2a6 ("move booti_setup to arch/arm/lig/image.c")
> not only moved the code, but also replaced a tab with 4 spaces
> to break the Linux coding style.
>
> Restore tab indentation.
>
>
Thanks Masahiro, didn't know u-boot prefer tab!
Reviewed-by: Bin Chen <bin.chen@linaro.org>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
> arch/arm/lib/image.c | 92 ++++++++++++++++++++++++++----
> ----------------------
> 1 file changed, 46 insertions(+), 46 deletions(-)
>
> diff --git a/arch/arm/lib/image.c b/arch/arm/lib/image.c
> index f72cf63..92bfb8f 100644
> --- a/arch/arm/lib/image.c
> +++ b/arch/arm/lib/image.c
> @@ -15,63 +15,63 @@ DECLARE_GLOBAL_DATA_PTR;
>
> /* See Documentation/arm64/booting.txt in the Linux kernel */
> struct Image_header {
> - uint32_t code0; /* Executable code */
> - uint32_t code1; /* Executable code */
> - uint64_t text_offset; /* Image load offset, LE */
> - uint64_t image_size; /* Effective Image size, LE */
> - uint64_t flags; /* Kernel flags, LE */
> - uint64_t res2; /* reserved */
> - uint64_t res3; /* reserved */
> - uint64_t res4; /* reserved */
> - uint32_t magic; /* Magic number */
> - uint32_t res5;
> + uint32_t code0; /* Executable code */
> + uint32_t code1; /* Executable code */
> + uint64_t text_offset; /* Image load offset, LE */
> + uint64_t image_size; /* Effective Image size, LE */
> + uint64_t flags; /* Kernel flags, LE */
> + uint64_t res2; /* reserved */
> + uint64_t res3; /* reserved */
> + uint64_t res4; /* reserved */
> + uint32_t magic; /* Magic number */
> + uint32_t res5;
> };
>
> int booti_setup(ulong image, ulong *relocated_addr, ulong *size)
> {
> - struct Image_header *ih;
> - uint64_t dst;
> - uint64_t image_size, text_offset;
> + struct Image_header *ih;
> + uint64_t dst;
> + uint64_t image_size, text_offset;
>
> - *relocated_addr = image;
> + *relocated_addr = image;
>
> - ih = (struct Image_header *)map_sysmem(image, 0);
> + ih = (struct Image_header *)map_sysmem(image, 0);
>
> - if (ih->magic != le32_to_cpu(LINUX_ARM64_IMAGE_MAGIC)) {
> - puts("Bad Linux ARM64 Image magic!\n");
> - return 1;
> - }
> + if (ih->magic != le32_to_cpu(LINUX_ARM64_IMAGE_MAGIC)) {
> + puts("Bad Linux ARM64 Image magic!\n");
> + return 1;
> + }
>
> - /*
> - * Prior to Linux commit a2c1d73b94ed, the text_offset field
> - * is of unknown endianness. In these cases, the image_size
> - * field is zero, and we can assume a fixed value of 0x80000.
> - */
> - if (ih->image_size == 0) {
> - puts("Image lacks image_size field, assuming 16MiB\n");
> - image_size = 16 << 20;
> - text_offset = 0x80000;
> - } else {
> - image_size = le64_to_cpu(ih->image_size);
> - text_offset = le64_to_cpu(ih->text_offset);
> - }
> + /*
> + * Prior to Linux commit a2c1d73b94ed, the text_offset field
> + * is of unknown endianness. In these cases, the image_size
> + * field is zero, and we can assume a fixed value of 0x80000.
> + */
> + if (ih->image_size == 0) {
> + puts("Image lacks image_size field, assuming 16MiB\n");
> + image_size = 16 << 20;
> + text_offset = 0x80000;
> + } else {
> + image_size = le64_to_cpu(ih->image_size);
> + text_offset = le64_to_cpu(ih->text_offset);
> + }
>
> - *size = image_size;
> + *size = image_size;
>
> - /*
> - * If bit 3 of the flags field is set, the 2MB aligned base of the
> - * kernel image can be anywhere in physical memory, so respect
> - * images->ep. Otherwise, relocate the image to the base of RAM
> - * since memory below it is not accessible via the linear mapping.
> - */
> - if (le64_to_cpu(ih->flags) & BIT(3))
> - dst = image - text_offset;
> - else
> - dst = gd->bd->bi_dram[0].start;
> + /*
> + * If bit 3 of the flags field is set, the 2MB aligned base of the
> + * kernel image can be anywhere in physical memory, so respect
> + * images->ep. Otherwise, relocate the image to the base of RAM
> + * since memory below it is not accessible via the linear mapping.
> + */
> + if (le64_to_cpu(ih->flags) & BIT(3))
> + dst = image - text_offset;
> + else
> + dst = gd->bd->bi_dram[0].start;
>
> - *relocated_addr = ALIGN(dst, SZ_2M) + text_offset;
> + *relocated_addr = ALIGN(dst, SZ_2M) + text_offset;
>
> - unmap_sysmem(ih);
> + unmap_sysmem(ih);
>
> - return 0;
> + return 0;
> }
> --
> 2.7.4
>
>
--
Regards,
Bin
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] ARM: image: indent with tab instead of 4 spaces
2018-02-13 2:32 [U-Boot] [PATCH] ARM: image: indent with tab instead of 4 spaces Masahiro Yamada
2018-02-13 2:57 ` Bin Chen
@ 2018-02-14 16:43 ` Tom Rini
1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2018-02-14 16:43 UTC (permalink / raw)
To: u-boot
On Tue, Feb 13, 2018 at 11:32:15AM +0900, Masahiro Yamada wrote:
> Commit 6808ef9ac2a6 ("move booti_setup to arch/arm/lig/image.c")
> not only moved the code, but also replaced a tab with 4 spaces
> to break the Linux coding style.
>
> Restore tab indentation.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Reviewed-by: Bin Chen <bin.chen@linaro.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180214/73f46f47/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-02-14 16:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-13 2:32 [U-Boot] [PATCH] ARM: image: indent with tab instead of 4 spaces Masahiro Yamada
2018-02-13 2:57 ` Bin Chen
2018-02-14 16:43 ` [U-Boot] " Tom Rini
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.