soc/intel: Remove unused pointer argument in mca_configure()

Change-Id: Iad3982d9db07a1f17ac39e87ff9c37956e40c258
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32616
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: David Guckian
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Subrata Banik 2019-05-06 19:23:26 +05:30
parent 7196be433e
commit f91344cd07
7 changed files with 7 additions and 7 deletions

View File

@ -76,7 +76,7 @@ void soc_core_init(struct device *cpu)
scope. For now every CPU clears every bank. */ scope. For now every CPU clears every bank. */
if (CONFIG(SOC_INTEL_COMMON_BLOCK_SGX) || if (CONFIG(SOC_INTEL_COMMON_BLOCK_SGX) ||
acpi_get_sleep_type() == ACPI_S5) acpi_get_sleep_type() == ACPI_S5)
mca_configure(NULL); mca_configure();
/* Set core MSRs */ /* Set core MSRs */
reg_script_run(core_msr_script); reg_script_run(core_msr_script);

View File

@ -413,7 +413,7 @@ void soc_core_init(struct device *cpu)
/* TODO(adurbin): This should only be done on a cold boot. Also, some /* TODO(adurbin): This should only be done on a cold boot. Also, some
* of these banks are core vs package scope. For now every CPU clears * of these banks are core vs package scope. For now every CPU clears
* every bank. */ * every bank. */
mca_configure(NULL); mca_configure();
/* Enable the local CPU apics */ /* Enable the local CPU apics */
enable_lapic_tpr(); enable_lapic_tpr();

View File

@ -300,7 +300,7 @@ uint32_t cpu_get_max_turbo_ratio(void)
return msr.lo & 0xff; return msr.lo & 0xff;
} }
void mca_configure(void *unused) void mca_configure(void)
{ {
msr_t msr; msr_t msr;
int i; int i;

View File

@ -159,6 +159,6 @@ uint32_t cpu_get_power_max(void);
uint32_t cpu_get_max_turbo_ratio(void); uint32_t cpu_get_max_turbo_ratio(void);
/* Configure Machine Check Architecture support */ /* Configure Machine Check Architecture support */
void mca_configure(void *unused); void mca_configure(void);
#endif /* SOC_INTEL_COMMON_BLOCK_CPULIB_H */ #endif /* SOC_INTEL_COMMON_BLOCK_CPULIB_H */

View File

@ -58,7 +58,7 @@ static void dnv_configure_mca(void)
/* TODO(adurbin): This should only be done on a cold boot. Also, some /* TODO(adurbin): This should only be done on a cold boot. Also, some
of these banks are core vs package scope. For now every CPU clears of these banks are core vs package scope. For now every CPU clears
every bank. */ every bank. */
mca_configure(NULL); mca_configure();
} }
static void denverton_core_init(struct device *cpu) static void denverton_core_init(struct device *cpu)

View File

@ -192,7 +192,7 @@ void soc_core_init(struct device *cpu)
/* TODO(adurbin): This should only be done on a cold boot. Also, some /* TODO(adurbin): This should only be done on a cold boot. Also, some
* of these banks are core vs package scope. For now every CPU clears * of these banks are core vs package scope. For now every CPU clears
* every bank. */ * every bank. */
mca_configure(NULL); mca_configure();
/* Enable the local CPU apics */ /* Enable the local CPU apics */
enable_lapic_tpr(); enable_lapic_tpr();

View File

@ -428,7 +428,7 @@ void soc_core_init(struct device *cpu)
/* TODO(adurbin): This should only be done on a cold boot. Also, some /* TODO(adurbin): This should only be done on a cold boot. Also, some
* of these banks are core vs package scope. For now every CPU clears * of these banks are core vs package scope. For now every CPU clears
* every bank. */ * every bank. */
mca_configure(NULL); mca_configure();
/* Enable the local CPU apics */ /* Enable the local CPU apics */
enable_lapic_tpr(); enable_lapic_tpr();