Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/60622 )
Change subject: soc/intel/common: Remove unnecessary new lines throughout the crashlog.c ......................................................................
soc/intel/common: Remove unnecessary new lines throughout the crashlog.c
Signed-off-by: Subrata Banik subratabanik@google.com Change-Id: I0920f563d6fdf9414eab86796cedcac83173dba3 --- M src/soc/intel/common/block/crashlog/crashlog.c 1 file changed, 0 insertions(+), 35 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/60622/1
diff --git a/src/soc/intel/common/block/crashlog/crashlog.c b/src/soc/intel/common/block/crashlog/crashlog.c index 8c31d03..4081182 100644 --- a/src/soc/intel/common/block/crashlog/crashlog.c +++ b/src/soc/intel/common/block/crashlog/crashlog.c @@ -6,82 +6,67 @@ #include <intelblocks/crashlog.h> #include <string.h>
- int __weak cl_get_cpu_record_size(void) { return 0; }
- int __weak cl_get_pmc_record_size(void) { return 0; }
- u32 __weak cl_get_cpu_bar_addr(void) { return 0; }
- u32 __weak cl_get_cpu_tmp_bar(void) { return 0; }
- u32 __weak cl_get_cpu_mb_int_addr(void) { return 0; }
- int __weak cl_get_total_data_size(void) { return cl_get_cpu_record_size() + cl_get_pmc_record_size(); }
- bool __weak cl_pmc_sram_has_mmio_access(void) { return false; }
- bool __weak cpu_crashlog_support(void) { return false; }
- bool __weak pmc_crashlog_support(void) { return false; }
- bool __weak cl_cpu_data_present(void) { return false; }
- bool __weak cl_pmc_data_present(void) { return false; }
- __weak void reset_discovery_buffers(void) {}
- __weak void update_new_pmc_crashlog_size(u32 *pmc_crash_size) {}
- __weak void update_new_cpu_crashlog_size(u32 *cpu_crash_size) {}
- pmc_ipc_discovery_buf_t __weak cl_get_pmc_discovery_buf(void) { pmc_ipc_discovery_buf_t discov_buf; @@ -89,7 +74,6 @@ return discov_buf; }
- pmc_crashlog_desc_table_t __weak cl_get_pmc_descriptor_table(void) { pmc_crashlog_desc_table_t desc_tab; @@ -97,7 +81,6 @@ return desc_tab; }
- cpu_crashlog_discovery_table_t __weak cl_get_cpu_discovery_table(void) { cpu_crashlog_discovery_table_t cpu_disc_tab; @@ -105,7 +88,6 @@ return cpu_disc_tab; }
- int cpu_cl_poll_mailbox_ready(u32 cl_mailbox_addr) { cpu_crashlog_mailbox_t cl_mailbox_interface; @@ -127,7 +109,6 @@ return 1; }
- int cpu_cl_mailbox_cmd(u8 cmd, u8 param) { cpu_crashlog_mailbox_t cl_mailbox_intf; @@ -149,13 +130,11 @@ return 1; }
- int cpu_cl_clear_data(void) { return cpu_cl_mailbox_cmd(CPU_CRASHLOG_CMD_CLEAR, 0); }
- int pmc_cl_gen_descriptor_table(u32 desc_table_addr, pmc_crashlog_desc_table_t *descriptor_table) { @@ -178,19 +157,16 @@ return total_data_size; }
- bool __weak pmc_cl_discovery(void) { return false; }
- bool __weak cpu_cl_discovery(void) { return false; }
- int cl_pmc_re_arm_after_reset(void) {
@@ -213,7 +189,6 @@ return r; }
- /* Sends PMC IPC to clear CrashLog from PMC SSRAM area */ int cl_pmc_clear(void) { @@ -236,12 +211,10 @@ return r; }
- /* * Sends PMC IPC to populate CrashLog on all reboot. * The SSRAM area will be cleared on G3 by PMC automatically */ - int cl_pmc_en_gen_on_all_reboot(void) { const struct pmc_ipc_buffer req = { 0 }; @@ -263,7 +236,6 @@ return r; }
- bool discover_crashlog(void) { bool cpu_cl_discovered = false, pmc_cl_discovered = false; @@ -279,7 +251,6 @@ return (cpu_cl_discovered || pmc_cl_discovered); }
- bool cl_copy_data_from_sram(u32 src_bar, u32 offset, u32 size, @@ -320,7 +291,6 @@ return true; }
- void cl_get_pmc_sram_data(void) { u32 *dest = NULL; @@ -413,7 +383,6 @@
}
- void cl_get_cpu_sram_data(void) { u32 tmp_bar_addr = 0; @@ -471,10 +440,8 @@
/* clear telemetry SRAM region */ cpu_cl_clear_data(); - }
- void collect_pmc_and_cpu_crashlog_from_srams(void) { if (pmc_crashlog_support() && cl_pmc_data_present() @@ -499,7 +466,6 @@ } }
- bool cl_fill_cpu_records(void *cl_record) { void *cl_src_addr = NULL; @@ -518,7 +484,6 @@ return true; }
- bool cl_fill_pmc_records(void *cl_record) { void *cl_src_addr = NULL;