include: sbi: No need to pack struct sbi_scratch
authorAnup Patel <anup.patel@wdc.com>
Mon, 11 Jan 2021 11:15:14 +0000 (16:45 +0530)
committerAnup Patel <anup@brainfault.org>
Fri, 15 Jan 2021 05:10:31 +0000 (10:40 +0530)
All members in struct sbi_scrach are of machine word size so no need
to pack this structure. This also allows GCC to generate more optimized
instruction sequence when accessing members of struct sbi_scratch.

Reported-by: Paul Campbell <taniwha@gmail.com>
Signed-off-by: Anup Patel <anup.patel@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Atish Patra <atish.patra@wdc.com>
include/sbi/sbi_scratch.h

index e7079f5424a5e035d0999627d7db385ff7d3bd91..e35122bed156fe2e3900b81a2f364621e4f80771 100644 (file)
@@ -71,7 +71,7 @@ struct sbi_scratch {
        unsigned long tmp0;
        /** Options for OpenSBI library */
        unsigned long options;
-} __packed;
+};
 
 /** Possible options for OpenSBI library */
 enum sbi_scratch_options {