Hi Jon, On Tue, 1 Oct 2019, Jonathan Corbet wrote: > On Mon, 16 Sep 2019 13:01:40 +0000 > Chester Lin <clin@xxxxxxxx> wrote: > > > Correct the sequence of the magic number 2 since it's little endian. > > > > Signed-off-by: Chester Lin <clin@xxxxxxxx> > > --- > > Documentation/riscv/boot-image-header.txt | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > So I'm slowly digging my way through my docs folder, sorry for the delay. > > > diff --git a/Documentation/riscv/boot-image-header.txt b/Documentation/riscv/boot-image-header.txt > > index 14b1492f689b..f388805aa5c3 100644 > > --- a/Documentation/riscv/boot-image-header.txt > > +++ b/Documentation/riscv/boot-image-header.txt > > @@ -18,7 +18,7 @@ The following 64-byte header is present in decompressed Linux kernel image. > > u32 res1 = 0; /* Reserved */ > > u64 res2 = 0; /* Reserved */ > > u64 magic = 0x5643534952; /* Magic number, little endian, "RISCV" */ > > - u32 magic2 = 0x56534905; /* Magic number 2, little endian, "RSC\x05" */ > > + u32 magic2 = 0x05435352; /* Magic number 2, little endian, "RSC\x05" */ > > That's doing more than just reordering the bytes. Was the original > completely wrong? If so, the changelog should probably reflect that. Or > else what am I missing? This was simply due to idiocy on my part, due to a very last-minute patch submission, where I missed that the original documented magic number was already in little-endian format. The updated number from Chester is correct. So, Reviewed-by: Paul Walmsley <paul.walmsley@xxxxxxxxxx> - Paul
