diff mbox series

[v2,18/25] x86: coral: Update the memory map

Message ID 20200614165958.159716-16-sjg@chromium.org
State Accepted
Commit 4f58f73d38bb0b367042018ef62bcf6bc095cf44
Headers show
Series x86: Enhance MTRR functionality to support multiple CPUs | expand

Commit Message

Simon Glass June 14, 2020, 4:59 p.m. UTC
This currently excludes the temporary memory used to start up the APs.
Add it.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

Changes in v2:
- Add new patch to add AP_DEFAULT_BASE to coral's memory map

 doc/board/google/chromebook_coral.rst | 1 +
 1 file changed, 1 insertion(+)

Comments

Wolfgang Wallner June 16, 2020, 11:30 a.m. UTC | #1
Hi Simon,

-----"Simon Glass" <sjg at chromium.org> schrieb: -----
> Betreff: [PATCH v2 18/25] x86: coral: Update the memory map
> 
> This currently excludes the temporary memory used to start up the APs.
> Add it.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> Changes in v2:
> - Add new patch to add AP_DEFAULT_BASE to coral's memory map
> 
>  doc/board/google/chromebook_coral.rst | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/doc/board/google/chromebook_coral.rst b/doc/board/google/chromebook_coral.rst
> index 40bd9397d4..c39f1e310c 100644
> --- a/doc/board/google/chromebook_coral.rst
> +++ b/doc/board/google/chromebook_coral.rst
> @@ -188,6 +188,7 @@ Partial memory map
>      fef00000  1000 CONFIG_BOOTSTAGE_STASH_ADDR
>      fef00000       Base of CAR region
>  
> +       30000       AP_DEFAULT_BASE (used to start up additional CPUs)
>         f0000       CONFIG_ROM_TABLE_ADDR
>        120000       BSS (defined in u-boot-spl.lds)
>        200000       FSP-S (which is run after U-Boot is relocated)
> -- 
> 2.27.0.290.gba653c62da-goog

Reviewed-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
Bin Meng June 28, 2020, 8:06 a.m. UTC | #2
On Mon, Jun 15, 2020 at 1:00 AM Simon Glass <sjg at chromium.org> wrote:
>
> This currently excludes the temporary memory used to start up the APs.
> Add it.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Add new patch to add AP_DEFAULT_BASE to coral's memory map
>
>  doc/board/google/chromebook_coral.rst | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
diff mbox series

Patch

diff --git a/doc/board/google/chromebook_coral.rst b/doc/board/google/chromebook_coral.rst
index 40bd9397d4..c39f1e310c 100644
--- a/doc/board/google/chromebook_coral.rst
+++ b/doc/board/google/chromebook_coral.rst
@@ -188,6 +188,7 @@  Partial memory map
     fef00000  1000 CONFIG_BOOTSTAGE_STASH_ADDR
     fef00000       Base of CAR region
 
+       30000       AP_DEFAULT_BASE (used to start up additional CPUs)
        f0000       CONFIG_ROM_TABLE_ADDR
       120000       BSS (defined in u-boot-spl.lds)
       200000       FSP-S (which is run after U-Boot is relocated)