public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Leif Lindholm" <leif@nuviainc.com>
To: Rebecca Cran <rebecca@bsdio.com>,
	Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: devel@edk2.groups.io
Subject: Re: [edk2-platforms PATCH 1/4] BeagleBoardPkg: Remove the configuration and image headers from flash
Date: Tue, 14 Sep 2021 15:39:03 +0100	[thread overview]
Message-ID: <20210914143903.ors7du74byx2zirx@leviathan> (raw)
In-Reply-To: <20210911025714.26263-2-rebecca@bsdio.com>

Ard, I think you were the one who converted the old crazy header stuff
to what we have now. Do you remember how this all fits together?

For the *other* 3 patches, but not this one:
Reviewed-by: Leif Lindholm <leif@nuviainc.com>


On Fri, Sep 10, 2021 at 20:57:11 -0600, Rebecca Cran wrote:
> Remove the configuration and image headers from the flash image.
> This was likely intended for the UEFI firmware to be loaded by the ROM
> code, but the BeagleBoard only has 64KB SRAM and so EDK2 needs to be
> executed as a second stage loader.
> 
> Signed-off-by: Rebecca Cran <rebecca@bsdio.com>
> ---
>  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf | 13 ++-----------
>  1 file changed, 2 insertions(+), 11 deletions(-)
> 
> diff --git a/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf b/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf
> index a2cfeb3bc27b..dbae015ff382 100644
> --- a/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf
> +++ b/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf
> @@ -23,7 +23,7 @@
>  
>  
>  [FD.BeagleBoard_EFI]
> -BaseAddress   = 0x80007DF8|gArmTokenSpaceGuid.PcdFdBaseAddress  #The base address of the FLASH Device.
> +BaseAddress   = 0x80008000|gArmTokenSpaceGuid.PcdFdBaseAddress  #The base address of the FLASH Device.
>  Size          = 0x000B0000|gArmTokenSpaceGuid.PcdFdSize         #The size in bytes of the FLASH Device
>  ErasePolarity = 1
>  BlockSize     = 0x1
> @@ -44,16 +44,7 @@ NumBlocks     = 0xB0000
>  # RegionType <FV, DATA, or FILE>
>  #
>  ################################################################################
> -0x00000000|0x00000200
> -FILE = Platform/BeagleBoard/BeagleBoardPkg/ConfigurationHeader.bin
> -
> -0x00000200|0x00000008
> -DATA = {
> -  0xF8, 0xFD, 0x0A, 0x00,   # image size:   0xB0000 - 0x208 == 0xAFDF8
> -  0x00, 0x80, 0x00, 0x80    # entry point:  0x80008000
> -}
> -
> -0x00000208|0x000AFDF8
> +0x00000000|0x000B0000
>  gArmTokenSpaceGuid.PcdFvBaseAddress|gArmTokenSpaceGuid.PcdFvSize
>  FV = FVMAIN_COMPACT
>  
> -- 
> 2.31.1
> 
> 

  reply	other threads:[~2021-09-14 14:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-11  2:57 [edk2-platforms PATCH 0/4] Some BeagleBoardPkg fixes Rebecca Cran
2021-09-11  2:57 ` [edk2-platforms PATCH 1/4] BeagleBoardPkg: Remove the configuration and image headers from flash Rebecca Cran
2021-09-14 14:39   ` Leif Lindholm [this message]
2021-09-21 21:14     ` Rebecca Cran
2021-09-22 11:20     ` Ard Biesheuvel
2021-09-22 11:20       ` Ard Biesheuvel
2021-09-11  2:57 ` [edk2-platforms PATCH 2/4] BeagleBoardPkg: Remove PerformanceLib from build Rebecca Cran
2021-09-11  2:57 ` [edk2-platforms PATCH 3/4] BeagleBoardPkg: Align PE sections to 4K Rebecca Cran
2021-09-11  2:57 ` [edk2-platforms PATCH 4/4] Omap35xxPkg: InterruptDxe: add gEfiCpuArchProtocolGuid runtime dependency Rebecca Cran

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210914143903.ors7du74byx2zirx@leviathan \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox