Change in ...coreboot[master]: Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK

[ron minnich (Code Review)]

ron minnich has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32697

Change subject: Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK
......................................................................

Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK

The definition of bootblock_systemagent_early_init was guarded by
!ENV_RAMSTAGE. But it's only called in the bootblock. So guard it
with ENV_BOOTBLOCK instead.

Change-Id: I143cf72e4a63b176e4772575e7a60a2a611e4ad9
Signed-off-by: Ronald G. Minnich 
---
M src/soc/intel/common/block/systemagent/systemagent_early.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/32697/1

diff --git a/src/soc/intel/common/block/systemagent/systemagent_early.c b/src/soc/intel/common/block/systemagent/systemagent_early.c
index 61f14a9..c12c64a 100644
--- a/src/soc/intel/common/block/systemagent/systemagent_early.c
+++ b/src/soc/intel/common/block/systemagent/systemagent_early.c
@@ -24,7 +24,7 @@

#include "systemagent_def.h"

-#if !ENV_RAMSTAGE
+#if ENV_BOOTBLOCK
 void bootblock_systemagent_early_init(void)
 {
    uint32_t reg;

[Lijian Zhao (Code Review)]

Lijian Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32697 )

Change subject: Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK
......................................................................

Patch Set 1: Code-Review+2

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32697 )

Change subject: Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK
......................................................................

Patch Set 1: Code-Review+2

[ron minnich (Code Review)]

ron minnich has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32697 )

Change subject: Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK
......................................................................

Change the guard for bootblock_systemagent_early_init to ENV_BOOTBLOCK

The definition of bootblock_systemagent_early_init was guarded by
!ENV_RAMSTAGE. But it's only called in the bootblock. So guard it
with ENV_BOOTBLOCK instead.

Change-Id: I143cf72e4a63b176e4772575e7a60a2a611e4ad9
Signed-off-by: Ronald G. Minnich 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32697
Tested-by: build bot (Jenkins) 
Reviewed-by: Lijian Zhao 
Reviewed-by: Subrata Banik 
---
M src/soc/intel/common/block/systemagent/systemagent_early.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Lijian Zhao: Looks good to me, approved
  Subrata Banik: Looks good to me, approved

diff --git a/src/soc/intel/common/block/systemagent/systemagent_early.c b/src/soc/intel/common/block/systemagent/systemagent_early.c
index 61f14a9..c12c64a 100644
--- a/src/soc/intel/common/block/systemagent/systemagent_early.c
+++ b/src/soc/intel/common/block/systemagent/systemagent_early.c
@@ -24,7 +24,7 @@

#include "systemagent_def.h"

-#if !ENV_RAMSTAGE
+#if ENV_BOOTBLOCK
 void bootblock_systemagent_early_init(void)
 {
    uint32_t reg;