Commit 8d62af17 authored by Arvind Sankar's avatar Arvind Sankar Committed by Ingo Molnar

efi/gop: Remove bogus packed attribute from GOP structures

EFI structures are not packed, they follow natural alignment.

The packed attribute doesn't have any effect on the structure layout due
to the types and order of the members, and we only ever get these
structures as output from the EFI firmware so alignment issues have not
come up.
Signed-off-by: default avatarArvind Sankar <nivedita@alum.mit.edu>
Signed-off-by: default avatarArd Biesheuvel <ardb@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: James Morse <james.morse@arm.com>
Cc: Matt Fleming <matt@codeblueprint.co.uk>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-efi@vger.kernel.org
Link: https://lkml.kernel.org/r/20191224151025.32482-2-ardb@kernel.orgSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent 4911ee40
...@@ -1429,7 +1429,7 @@ struct efi_graphics_output_mode_info { ...@@ -1429,7 +1429,7 @@ struct efi_graphics_output_mode_info {
int pixel_format; int pixel_format;
struct efi_pixel_bitmask pixel_information; struct efi_pixel_bitmask pixel_information;
u32 pixels_per_scan_line; u32 pixels_per_scan_line;
} __packed; };
struct efi_graphics_output_protocol_mode_32 { struct efi_graphics_output_protocol_mode_32 {
u32 max_mode; u32 max_mode;
...@@ -1438,7 +1438,7 @@ struct efi_graphics_output_protocol_mode_32 { ...@@ -1438,7 +1438,7 @@ struct efi_graphics_output_protocol_mode_32 {
u32 size_of_info; u32 size_of_info;
u64 frame_buffer_base; u64 frame_buffer_base;
u32 frame_buffer_size; u32 frame_buffer_size;
} __packed; };
struct efi_graphics_output_protocol_mode_64 { struct efi_graphics_output_protocol_mode_64 {
u32 max_mode; u32 max_mode;
...@@ -1447,7 +1447,7 @@ struct efi_graphics_output_protocol_mode_64 { ...@@ -1447,7 +1447,7 @@ struct efi_graphics_output_protocol_mode_64 {
u64 size_of_info; u64 size_of_info;
u64 frame_buffer_base; u64 frame_buffer_base;
u64 frame_buffer_size; u64 frame_buffer_size;
} __packed; };
struct efi_graphics_output_protocol_mode { struct efi_graphics_output_protocol_mode {
u32 max_mode; u32 max_mode;
...@@ -1456,7 +1456,7 @@ struct efi_graphics_output_protocol_mode { ...@@ -1456,7 +1456,7 @@ struct efi_graphics_output_protocol_mode {
unsigned long size_of_info; unsigned long size_of_info;
u64 frame_buffer_base; u64 frame_buffer_base;
unsigned long frame_buffer_size; unsigned long frame_buffer_size;
} __packed; };
struct efi_graphics_output_protocol_32 { struct efi_graphics_output_protocol_32 {
u32 query_mode; u32 query_mode;
......
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