diff options
| author | Felix Held <felix.held@amd.corp-partner.google.com> | 2020-04-15 17:05:38 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2020-04-16 23:17:41 +0000 |
| commit | 7d304186057cc6c66e5469fadd003e3a62b8eb63 (patch) | |
| tree | 168d0e7939c8fd45fa41cba0906f82a93a2db802 /src/soc/amd/common/block/psp/psp_def.h | |
| parent | 90b8339b8d4338c14cfd791bf4d5058aa515da34 (diff) | |
soc/amd/common/psp: refactor psp_print_cmd_status parameters
psp_print_cmd_status only needs data from the mbox buffer header and not
the whole buffer. This avoids type casts when the buffer type isn't
mbox_default_buffer.
BUG=b:153677737
Change-Id: I8688b66fefe89fc4f3ce2207d4360ceb2dbaef12
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40412
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/common/block/psp/psp_def.h')
| -rw-r--r-- | src/soc/amd/common/block/psp/psp_def.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/common/block/psp/psp_def.h b/src/soc/amd/common/block/psp/psp_def.h index 7772ca6bda..7bdec21943 100644 --- a/src/soc/amd/common/block/psp/psp_def.h +++ b/src/soc/amd/common/block/psp/psp_def.h @@ -109,7 +109,7 @@ struct mbox_cmd_sx_info_buffer { #define PSP_INIT_TIMEOUT 10000 /* 10 seconds */ #define PSP_CMD_TIMEOUT 1000 /* 1 second */ -void psp_print_cmd_status(int cmd_status, struct mbox_default_buffer *buffer); +void psp_print_cmd_status(int cmd_status, struct mbox_buffer_header *header); /* This command needs to be implemented by the generation specific code. */ int send_psp_command(u32 command, void *buffer); |