Re: [PATCH 3/3] EFI: Update the documentation to reflect the correct value of OsIndications

2022-06-01 Thread Ilias Apalodimas
On Wed, 1 Jun 2022 at 09:15, Heinrich Schuchardt  wrote:
>
> On 5/31/22 09:15, Sughosh Ganu wrote:
> > The OsIndications is a 64 bit variable, and the current code expects
> > the value of the variable to be 64 bit. Update the documentation to
> > reflect this fact.
> >
> > Signed-off-by: Sughosh Ganu 
>
> Reviewed-by: Heinrich Schuchardt 
>
> > ---
> >   doc/develop/uefi/uefi.rst | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/doc/develop/uefi/uefi.rst b/doc/develop/uefi/uefi.rst
> > index 753a4e5e29..941e427093 100644
> > --- a/doc/develop/uefi/uefi.rst
> > +++ b/doc/develop/uefi/uefi.rst
> > @@ -326,7 +326,7 @@ bit in OsIndications variable with
> >
> >   .. code-block:: console
> >
> > -=> setenv -e -nv -bs -rt -v OsIndications =0x04
> > +=> setenv -e -nv -bs -rt -v OsIndications =0x0004
> >
> >   Since U-boot doesn't currently support SetVariable at runtime, its value
> >   won't be taken over across the reboot. If this is the case, you can skip
>

Reviewed-by: Ilias Apalodimas 


Re: [PATCH 3/3] EFI: Update the documentation to reflect the correct value of OsIndications

2022-05-31 Thread Heinrich Schuchardt

On 5/31/22 09:15, Sughosh Ganu wrote:

The OsIndications is a 64 bit variable, and the current code expects
the value of the variable to be 64 bit. Update the documentation to
reflect this fact.

Signed-off-by: Sughosh Ganu 


Reviewed-by: Heinrich Schuchardt 


---
  doc/develop/uefi/uefi.rst | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/develop/uefi/uefi.rst b/doc/develop/uefi/uefi.rst
index 753a4e5e29..941e427093 100644
--- a/doc/develop/uefi/uefi.rst
+++ b/doc/develop/uefi/uefi.rst
@@ -326,7 +326,7 @@ bit in OsIndications variable with

  .. code-block:: console

-=> setenv -e -nv -bs -rt -v OsIndications =0x04
+=> setenv -e -nv -bs -rt -v OsIndications =0x0004

  Since U-boot doesn't currently support SetVariable at runtime, its value
  won't be taken over across the reboot. If this is the case, you can skip




[PATCH 3/3] EFI: Update the documentation to reflect the correct value of OsIndications

2022-05-31 Thread Sughosh Ganu
The OsIndications is a 64 bit variable, and the current code expects
the value of the variable to be 64 bit. Update the documentation to
reflect this fact.

Signed-off-by: Sughosh Ganu 
---
 doc/develop/uefi/uefi.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/develop/uefi/uefi.rst b/doc/develop/uefi/uefi.rst
index 753a4e5e29..941e427093 100644
--- a/doc/develop/uefi/uefi.rst
+++ b/doc/develop/uefi/uefi.rst
@@ -326,7 +326,7 @@ bit in OsIndications variable with
 
 .. code-block:: console
 
-=> setenv -e -nv -bs -rt -v OsIndications =0x04
+=> setenv -e -nv -bs -rt -v OsIndications =0x0004
 
 Since U-boot doesn't currently support SetVariable at runtime, its value
 won't be taken over across the reboot. If this is the case, you can skip
-- 
2.25.1