Commit f0cda7e6 authored by Luc Van Oostenryck's avatar Luc Van Oostenryck Committed by Will Deacon

arm64: fix endianness annotation in acpi_parking_protocol.c

Here both variables 'cpu_id' and 'entry_point' are read via
read[lq]_relaxed(), from a little-endian annotated pointer
and then used as a native endian value.

This is correct since the read[lq]() family of function
internally do a little-to-native endian conversion.

But in this case, it is wrong to declare these variable as
little-endian since there are native ones.

Fix this by changing the declaration of these variables
as 'u32' or 'u64' instead of '__le32' / '__le64'.
Signed-off-by: default avatarLuc Van Oostenryck <luc.vanoostenryck@gmail.com>
Signed-off-by: default avatarWill Deacon <will.deacon@arm.com>
parent c0d109de
...@@ -71,7 +71,7 @@ static int acpi_parking_protocol_cpu_boot(unsigned int cpu) ...@@ -71,7 +71,7 @@ static int acpi_parking_protocol_cpu_boot(unsigned int cpu)
{ {
struct cpu_mailbox_entry *cpu_entry = &cpu_mailbox_entries[cpu]; struct cpu_mailbox_entry *cpu_entry = &cpu_mailbox_entries[cpu];
struct parking_protocol_mailbox __iomem *mailbox; struct parking_protocol_mailbox __iomem *mailbox;
__le32 cpu_id; u32 cpu_id;
/* /*
* Map mailbox memory with attribute device nGnRE (ie ioremap - * Map mailbox memory with attribute device nGnRE (ie ioremap -
...@@ -123,7 +123,7 @@ static void acpi_parking_protocol_cpu_postboot(void) ...@@ -123,7 +123,7 @@ static void acpi_parking_protocol_cpu_postboot(void)
int cpu = smp_processor_id(); int cpu = smp_processor_id();
struct cpu_mailbox_entry *cpu_entry = &cpu_mailbox_entries[cpu]; struct cpu_mailbox_entry *cpu_entry = &cpu_mailbox_entries[cpu];
struct parking_protocol_mailbox __iomem *mailbox = cpu_entry->mailbox; struct parking_protocol_mailbox __iomem *mailbox = cpu_entry->mailbox;
__le64 entry_point; u64 entry_point;
entry_point = readq_relaxed(&mailbox->entry_point); entry_point = readq_relaxed(&mailbox->entry_point);
/* /*
......
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