summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2021-01-20 22:36:20 +0100
committerAngel Pons <th3fanbus@gmail.com>2021-01-24 12:06:39 +0000
commitac1c9bb5cdca24312c669eaaa908b13202e3bb35 (patch)
treeb26cf14dc049b07744c18d8213b2339f6e96c4f8
parentd626e554aa6f2a15c2ac4c971d1e5d3a9405b5fc (diff)
downloadcoreboot-ac1c9bb5cdca24312c669eaaa908b13202e3bb35.tar.xz
broadwell: Clean up `mainboard_post_raminit`
Make it optional and change its signature. Change-Id: I4b5f3fb08e8954514ebf39e72c95aa62d66856d7 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/49775 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
-rw-r--r--src/mainboard/google/auron/romstage.c4
-rw-r--r--src/mainboard/google/auron/variants/gandof/variant.c4
-rw-r--r--src/mainboard/google/auron/variants/lulu/variant.c4
-rw-r--r--src/mainboard/google/auron/variants/samus/variant.c4
-rw-r--r--src/mainboard/google/jecht/romstage.c2
-rw-r--r--src/mainboard/intel/wtm2/romstage.c4
-rw-r--r--src/mainboard/purism/librem_bdw/romstage.c4
-rw-r--r--src/soc/intel/broadwell/include/soc/romstage.h2
-rw-r--r--src/soc/intel/broadwell/romstage.c6
9 files changed, 13 insertions, 21 deletions
diff --git a/src/mainboard/google/auron/romstage.c b/src/mainboard/google/auron/romstage.c
index 9de58a86d2..a3705bc1b8 100644
--- a/src/mainboard/google/auron/romstage.c
+++ b/src/mainboard/google/auron/romstage.c
@@ -14,7 +14,3 @@ void mainboard_pre_raminit(struct romstage_params *rp)
mainboard_fill_spd_data(&rp->pei_data);
}
-
-__weak void mainboard_post_raminit(struct romstage_params *rp)
-{
-}
diff --git a/src/mainboard/google/auron/variants/gandof/variant.c b/src/mainboard/google/auron/variants/gandof/variant.c
index 0d2f47ecf5..b93e2c1472 100644
--- a/src/mainboard/google/auron/variants/gandof/variant.c
+++ b/src/mainboard/google/auron/variants/gandof/variant.c
@@ -24,8 +24,8 @@ int variant_smbios_data(struct device *dev, int *handle, unsigned long *current)
return len;
}
-void mainboard_post_raminit(struct romstage_params *rp)
+void mainboard_post_raminit(const int s3resume)
{
- if (rp->power_state->prev_sleep_state != ACPI_S3)
+ if (!s3resume)
google_chromeec_kbbacklight(75);
}
diff --git a/src/mainboard/google/auron/variants/lulu/variant.c b/src/mainboard/google/auron/variants/lulu/variant.c
index b38807bfcd..8349acaeff 100644
--- a/src/mainboard/google/auron/variants/lulu/variant.c
+++ b/src/mainboard/google/auron/variants/lulu/variant.c
@@ -34,8 +34,8 @@ int variant_smbios_data(struct device *dev, int *handle, unsigned long *current)
return len;
}
-void mainboard_post_raminit(struct romstage_params *rp)
+void mainboard_post_raminit(const int s3resume)
{
- if (rp->power_state->prev_sleep_state != ACPI_S3)
+ if (!s3resume)
google_chromeec_kbbacklight(75);
}
diff --git a/src/mainboard/google/auron/variants/samus/variant.c b/src/mainboard/google/auron/variants/samus/variant.c
index c69e66298c..1e4a258749 100644
--- a/src/mainboard/google/auron/variants/samus/variant.c
+++ b/src/mainboard/google/auron/variants/samus/variant.c
@@ -21,9 +21,9 @@ int variant_smbios_data(struct device *dev, int *handle, unsigned long *current)
return 0;
}
-void mainboard_post_raminit(struct romstage_params *rp)
+void mainboard_post_raminit(const int s3resume)
{
- if (rp->power_state->prev_sleep_state != ACPI_S3)
+ if (!s3resume)
google_chromeec_kbbacklight(100);
printk(BIOS_INFO, "MLB: board version %s\n", samus_board_version());
diff --git a/src/mainboard/google/jecht/romstage.c b/src/mainboard/google/jecht/romstage.c
index 612901123e..58db3cf522 100644
--- a/src/mainboard/google/jecht/romstage.c
+++ b/src/mainboard/google/jecht/romstage.c
@@ -18,7 +18,7 @@ void mainboard_pre_raminit(struct romstage_params *rp)
mainboard_fill_spd_data(&rp->pei_data);
}
-void mainboard_post_raminit(struct romstage_params *rp)
+void mainboard_post_raminit(const int s3resume)
{
if (CONFIG(CHROMEOS))
init_bootmode_straps();
diff --git a/src/mainboard/intel/wtm2/romstage.c b/src/mainboard/intel/wtm2/romstage.c
index 9e22250935..1b6539b55d 100644
--- a/src/mainboard/intel/wtm2/romstage.c
+++ b/src/mainboard/intel/wtm2/romstage.c
@@ -11,7 +11,3 @@ void mainboard_pre_raminit(struct romstage_params *rp)
/* Fill out PEI DATA */
mainboard_fill_pei_data(&rp->pei_data);
}
-
-void mainboard_post_raminit(struct romstage_params *rp)
-{
-}
diff --git a/src/mainboard/purism/librem_bdw/romstage.c b/src/mainboard/purism/librem_bdw/romstage.c
index 8fc2f9eaf7..8ffd078335 100644
--- a/src/mainboard/purism/librem_bdw/romstage.c
+++ b/src/mainboard/purism/librem_bdw/romstage.c
@@ -9,7 +9,3 @@ void mainboard_pre_raminit(struct romstage_params *rp)
/* Fill out PEI DATA */
mainboard_fill_pei_data(&rp->pei_data);
}
-
-void mainboard_post_raminit(struct romstage_params *rp)
-{
-}
diff --git a/src/soc/intel/broadwell/include/soc/romstage.h b/src/soc/intel/broadwell/include/soc/romstage.h
index 646ad0e5dc..eb51ea151e 100644
--- a/src/soc/intel/broadwell/include/soc/romstage.h
+++ b/src/soc/intel/broadwell/include/soc/romstage.h
@@ -12,7 +12,7 @@ struct romstage_params {
};
void mainboard_pre_raminit(struct romstage_params *params);
-void mainboard_post_raminit(struct romstage_params *params);
+void mainboard_post_raminit(const int s3resume);
void raminit(struct pei_data *pei_data);
diff --git a/src/soc/intel/broadwell/romstage.c b/src/soc/intel/broadwell/romstage.c
index 8e884d5d35..084c3e6e99 100644
--- a/src/soc/intel/broadwell/romstage.c
+++ b/src/soc/intel/broadwell/romstage.c
@@ -14,6 +14,10 @@
#include <stdint.h>
#include <timestamp.h>
+__weak void mainboard_post_raminit(const int s3resume)
+{
+}
+
/* Entry from cpu/intel/car/romstage.c. */
void mainboard_romstage_entry(void)
{
@@ -64,5 +68,5 @@ void mainboard_romstage_entry(void)
romstage_handoff_init(rp.power_state->prev_sleep_state == ACPI_S3);
- mainboard_post_raminit(&rp);
+ mainboard_post_raminit(rp.power_state->prev_sleep_state == ACPI_S3);
}