| Message ID | 20200422004507.2025-2-aiden.park@intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | Support 64-bit U-Boot for Slim Bootloader | expand |
Hi Aiden, On Tue, 21 Apr 2020 at 18:45, <aiden.park at intel.com> wrote: > > From: Aiden Park <aiden.park at intel.com> > > This will build U-Boot as a pure 64-bit binary with no SPL. > It can be used with a pre-stage boot firmware > which has already done 16-bit, 32-bit and 64-bit init. > > Signed-off-by: Aiden Park <aiden.park at intel.com> > --- > arch/x86/Kconfig | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index c8eae24c07..89add17e69 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -40,6 +40,14 @@ config X86_RUN_64BIT > runs through the 16-bit and 32-bit init, then switches to 64-bit > mode and jumps to U-Boot proper. > > +config X86_RUN_64BIT_ONLY Perhaps X86_RUN_64BIT_NO_SPL? Bin might hav a better idea. > + bool "64-bit only" > + select X86_64 > + help > + Build U-Boot as a pure 64-bit binary with no 32-bit SPL. This can > + be used with a pre-stage boot firmware which has already done > + 16-bit, 32-bit and 64-bit init. > + > endchoice > > config X86_64 > -- > 2.20.1 > Regards, Simon
Hi Simon, > -----Original Message----- > From: Simon Glass <sjg at chromium.org> > Sent: Sunday, April 26, 2020 1:16 PM > To: Park, Aiden <aiden.park at intel.com> > Cc: Bin Meng <bmeng.cn at gmail.com>; U-Boot Mailing List <u- > boot at lists.denx.de> > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to Kconfig > > Hi Aiden, > > On Tue, 21 Apr 2020 at 18:45, <aiden.park at intel.com> wrote: > > > > From: Aiden Park <aiden.park at intel.com> > > > > This will build U-Boot as a pure 64-bit binary with no SPL. > > It can be used with a pre-stage boot firmware which has already done > > 16-bit, 32-bit and 64-bit init. > > > > Signed-off-by: Aiden Park <aiden.park at intel.com> > > --- > > arch/x86/Kconfig | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index > > c8eae24c07..89add17e69 100644 > > --- a/arch/x86/Kconfig > > +++ b/arch/x86/Kconfig > > @@ -40,6 +40,14 @@ config X86_RUN_64BIT > > runs through the 16-bit and 32-bit init, then switches to 64-bit > > mode and jumps to U-Boot proper. > > > > +config X86_RUN_64BIT_ONLY > > Perhaps X86_RUN_64BIT_NO_SPL? Bin might hav a better idea. > I considered X64_RUN_64BIT_NO_SPL, X64_RUN_64BIT_PURE and X64_RUN_64BIT_NATIVE. Hi Bin, can you recommend better naming? > > + bool "64-bit only" > > + select X86_64 > > + help > > + Build U-Boot as a pure 64-bit binary with no 32-bit SPL. This can > > + be used with a pre-stage boot firmware which has already done > > + 16-bit, 32-bit and 64-bit init. > > + > > endchoice > > > > config X86_64 > > -- > > 2.20.1 > > > Regards, > Simon Best Regards, Aiden
Hi Aiden, On Wed, Apr 29, 2020 at 1:44 PM Park, Aiden <aiden.park at intel.com> wrote: > > Hi Simon, > > > -----Original Message----- > > From: Simon Glass <sjg at chromium.org> > > Sent: Sunday, April 26, 2020 1:16 PM > > To: Park, Aiden <aiden.park at intel.com> > > Cc: Bin Meng <bmeng.cn at gmail.com>; U-Boot Mailing List <u- > > boot at lists.denx.de> > > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to Kconfig > > > > Hi Aiden, > > > > On Tue, 21 Apr 2020 at 18:45, <aiden.park at intel.com> wrote: > > > > > > From: Aiden Park <aiden.park at intel.com> > > > > > > This will build U-Boot as a pure 64-bit binary with no SPL. > > > It can be used with a pre-stage boot firmware which has already done > > > 16-bit, 32-bit and 64-bit init. > > > > > > Signed-off-by: Aiden Park <aiden.park at intel.com> > > > --- > > > arch/x86/Kconfig | 8 ++++++++ > > > 1 file changed, 8 insertions(+) > > > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index > > > c8eae24c07..89add17e69 100644 > > > --- a/arch/x86/Kconfig > > > +++ b/arch/x86/Kconfig > > > @@ -40,6 +40,14 @@ config X86_RUN_64BIT > > > runs through the 16-bit and 32-bit init, then switches to 64-bit > > > mode and jumps to U-Boot proper. > > > > > > +config X86_RUN_64BIT_ONLY > > > > Perhaps X86_RUN_64BIT_NO_SPL? Bin might hav a better idea. > > > I considered X64_RUN_64BIT_NO_SPL, X64_RUN_64BIT_PURE and X64_RUN_64BIT_NATIVE. > Hi Bin, can you recommend better naming? Cannot figure out a better name :) Let's use it. Reviewed-by: Bin Meng <bmeng.cn at gmail.com> Regards, Bin
Hi Bin, > -----Original Message----- > From: Bin Meng <bmeng.cn at gmail.com> > Sent: Thursday, April 30, 2020 2:14 AM > To: Park, Aiden <aiden.park at intel.com> > Cc: Simon Glass <sjg at chromium.org>; U-Boot Mailing List <u- > boot at lists.denx.de> > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to Kconfig > > Hi Aiden, > > On Wed, Apr 29, 2020 at 1:44 PM Park, Aiden <aiden.park at intel.com> wrote: > > > > Hi Simon, > > > > > -----Original Message----- > > > From: Simon Glass <sjg at chromium.org> > > > Sent: Sunday, April 26, 2020 1:16 PM > > > To: Park, Aiden <aiden.park at intel.com> > > > Cc: Bin Meng <bmeng.cn at gmail.com>; U-Boot Mailing List <u- > > > boot at lists.denx.de> > > > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to > > > Kconfig > > > > > > Hi Aiden, > > > > > > On Tue, 21 Apr 2020 at 18:45, <aiden.park at intel.com> wrote: > > > > > > > > From: Aiden Park <aiden.park at intel.com> > > > > > > > > This will build U-Boot as a pure 64-bit binary with no SPL. > > > > It can be used with a pre-stage boot firmware which has already > > > > done 16-bit, 32-bit and 64-bit init. > > > > > > > > Signed-off-by: Aiden Park <aiden.park at intel.com> > > > > --- > > > > arch/x86/Kconfig | 8 ++++++++ > > > > 1 file changed, 8 insertions(+) > > > > > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index > > > > c8eae24c07..89add17e69 100644 > > > > --- a/arch/x86/Kconfig > > > > +++ b/arch/x86/Kconfig > > > > @@ -40,6 +40,14 @@ config X86_RUN_64BIT > > > > runs through the 16-bit and 32-bit init, then switches to 64-bit > > > > mode and jumps to U-Boot proper. > > > > > > > > +config X86_RUN_64BIT_ONLY > > > > > > Perhaps X86_RUN_64BIT_NO_SPL? Bin might hav a better idea. > > > > > I considered X64_RUN_64BIT_NO_SPL, X64_RUN_64BIT_PURE and > X64_RUN_64BIT_NATIVE. > > Hi Bin, can you recommend better naming? > > Cannot figure out a better name :) Let's use it. Thanks Bin. Let me use X86_RUN_64BIT_ONLY for now. Hi Simon, is this naming okay for you? > > Reviewed-by: Bin Meng <bmeng.cn at gmail.com> > > Regards, > Bin Best Regards, Aiden
Hi Aiden, On Fri, 1 May 2020 at 12:36, Park, Aiden <aiden.park at intel.com> wrote: > > Hi Bin, > > > -----Original Message----- > > From: Bin Meng <bmeng.cn at gmail.com> > > Sent: Thursday, April 30, 2020 2:14 AM > > To: Park, Aiden <aiden.park at intel.com> > > Cc: Simon Glass <sjg at chromium.org>; U-Boot Mailing List <u- > > boot at lists.denx.de> > > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to Kconfig > > > > Hi Aiden, > > > > On Wed, Apr 29, 2020 at 1:44 PM Park, Aiden <aiden.park at intel.com> wrote: > > > > > > Hi Simon, > > > > > > > -----Original Message----- > > > > From: Simon Glass <sjg at chromium.org> > > > > Sent: Sunday, April 26, 2020 1:16 PM > > > > To: Park, Aiden <aiden.park at intel.com> > > > > Cc: Bin Meng <bmeng.cn at gmail.com>; U-Boot Mailing List <u- > > > > boot at lists.denx.de> > > > > Subject: Re: [PATCH 1/8] x86: Add a new X86_RUN_64BIT_ONLY to > > > > Kconfig > > > > > > > > Hi Aiden, > > > > > > > > On Tue, 21 Apr 2020 at 18:45, <aiden.park at intel.com> wrote: > > > > > > > > > > From: Aiden Park <aiden.park at intel.com> > > > > > > > > > > This will build U-Boot as a pure 64-bit binary with no SPL. > > > > > It can be used with a pre-stage boot firmware which has already > > > > > done 16-bit, 32-bit and 64-bit init. > > > > > > > > > > Signed-off-by: Aiden Park <aiden.park at intel.com> > > > > > --- > > > > > arch/x86/Kconfig | 8 ++++++++ > > > > > 1 file changed, 8 insertions(+) > > > > > > > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index > > > > > c8eae24c07..89add17e69 100644 > > > > > --- a/arch/x86/Kconfig > > > > > +++ b/arch/x86/Kconfig > > > > > @@ -40,6 +40,14 @@ config X86_RUN_64BIT > > > > > runs through the 16-bit and 32-bit init, then switches to 64-bit > > > > > mode and jumps to U-Boot proper. > > > > > > > > > > +config X86_RUN_64BIT_ONLY > > > > > > > > Perhaps X86_RUN_64BIT_NO_SPL? Bin might hav a better idea. > > > > > > > I considered X64_RUN_64BIT_NO_SPL, X64_RUN_64BIT_PURE and > > X64_RUN_64BIT_NATIVE. > > > Hi Bin, can you recommend better naming? > > > > Cannot figure out a better name :) Let's use it. > Thanks Bin. Let me use X86_RUN_64BIT_ONLY for now. > > Hi Simon, is this naming okay for you? Yes that's fine thanks. Regards, Simon
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index c8eae24c07..89add17e69 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -40,6 +40,14 @@ config X86_RUN_64BIT runs through the 16-bit and 32-bit init, then switches to 64-bit mode and jumps to U-Boot proper. +config X86_RUN_64BIT_ONLY + bool "64-bit only" + select X86_64 + help + Build U-Boot as a pure 64-bit binary with no 32-bit SPL. This can + be used with a pre-stage boot firmware which has already done + 16-bit, 32-bit and 64-bit init. + endchoice config X86_64