Commit d72c3911 authored by Hon Ching \(Vicky\) Lo's avatar Hon Ching \(Vicky\) Lo Committed by Peter Huewe

TPM: remove unnecessary little endian conversion

The base pointer for the event log is allocated in the local
kernel (in prom_instantiate_sml()), therefore it is already in
the host's endian byte order and requires no conversion.

The content of the 'basep' pointer in read_log() stores the
base address of the log. This patch ensures that it is correctly
implemented.
Signed-off-by: default avatarHon Ching(Vicky) Lo <honclo@linux.vnet.ibm.com>
Signed-off-by: default avatarJoy Latten <jmlatten@linux.vnet.ibm.com>
Reviewed-by: default avatarAshley Lai <ashley@ahsleylai.com>
Signed-off-by: default avatarPeter Huewe <peterhuewe@gmx.de>
parent 0cc698af
...@@ -24,7 +24,7 @@ int read_log(struct tpm_bios_log *log) ...@@ -24,7 +24,7 @@ int read_log(struct tpm_bios_log *log)
{ {
struct device_node *np; struct device_node *np;
const u32 *sizep; const u32 *sizep;
const __be64 *basep; const u64 *basep;
if (log->bios_event_log != NULL) { if (log->bios_event_log != NULL) {
pr_err("%s: ERROR - Eventlog already initialized\n", __func__); pr_err("%s: ERROR - Eventlog already initialized\n", __func__);
...@@ -63,7 +63,7 @@ int read_log(struct tpm_bios_log *log) ...@@ -63,7 +63,7 @@ int read_log(struct tpm_bios_log *log)
log->bios_event_log_end = log->bios_event_log + *sizep; log->bios_event_log_end = log->bios_event_log + *sizep;
memcpy(log->bios_event_log, __va(be64_to_cpup(basep)), *sizep); memcpy(log->bios_event_log, __va(*basep), *sizep);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment