Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230619094705.51337-1-alexghiti@rivosinc.com> <20230619094705.51337-2-alexghiti@rivosinc.com>
In-Reply-To: <20230619094705.51337-2-alexghiti@rivosinc.com>
From:   Atish Patra <atishp@atishpatra.org>
Date:   Mon, 19 Jun 2023 11:58:28 -0700
Message-ID: <CAOnJCUJ5BgApWz3=F7f_CwATF0CV8dnZNw6hPYL0=b7-p63TiA@mail.gmail.com>
Subject: Re: [PATCH 2/2] Documentation: riscv: Update boot image header since
 EFI stub is supported
To:     Alexandre Ghiti <alexghiti@rivosinc.com>
Cc:     Jonathan Corbet <corbet@lwn.net>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>, linux-doc@vger.kernel.org,
        linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, Jun 19, 2023 at 2:48=E2=80=AFAM Alexandre Ghiti <alexghiti@rivosinc=
.com> wrote:
>
> The EFI stub is supported on RISC-V so update the documentation that
> explains how the boot image header was reused to support it.
>
> Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
> ---
>  Documentation/riscv/boot-image-header.rst | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/riscv/boot-image-header.rst b/Documentation/ri=
scv/boot-image-header.rst
> index a4a45310c4c4..df2ffc173e80 100644
> --- a/Documentation/riscv/boot-image-header.rst
> +++ b/Documentation/riscv/boot-image-header.rst
> @@ -28,11 +28,11 @@ header in future.
>  Notes
>  =3D=3D=3D=3D=3D
>
> -- This header can also be reused to support EFI stub for RISC-V in futur=
e. EFI
> -  specification needs PE/COFF image header in the beginning of the kerne=
l image
> -  in order to load it as an EFI application. In order to support EFI stu=
b,
> -  code0 should be replaced with "MZ" magic string and res3(at offset 0x3=
c) should
> -  point to the rest of the PE/COFF header.
> +- This header is also reused to support EFI stub for RISC-V. EFI specifi=
cation
> +  needs PE/COFF image header in the beginning of the kernel image in ord=
er to
> +  load it as an EFI application. In order to support EFI stub, code0 is =
replaced
> +  with "MZ" magic string and res3(at offset 0x3c) points to the rest of =
the
> +  PE/COFF header.
>
>  - version field indicate header version number
>
> --
> 2.39.2
>
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv


Reviewed-by: Atish Patra <atishp@rivosinc.com>

--=20
Regards,
Atish