diff options
author | Takashi Iwai <tiwai@suse.de> | 2012-04-23 12:40:49 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-04-23 12:41:17 -0400 |
commit | 8353e6c632aeaea1470a286b83e68ca233073068 (patch) | |
tree | 8d2345f96834e4242adb773dece50cbeee1fc0ed /include/drm | |
parent | c09dedb7a50e23f0166e0bbae61c75c7ec23cf7f (diff) |
drm/edid: Add packed attribute to new gtf2 and cvt structs
The new structs added in struct detailed_data_monitor_range must be
marked with packed attribute although the outer struct itself is
already marked as packed. Otherwise these 7-bytes structs may be
aligned, and give the wrong position and size for the data.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Acked-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'include/drm')
-rw-r--r-- | include/drm/drm_edid.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h index 8cefbbee996e..0cac551c5347 100644 --- a/include/drm/drm_edid.h +++ b/include/drm/drm_edid.h | |||
@@ -99,7 +99,7 @@ struct detailed_data_monitor_range { | |||
99 | __le16 m; | 99 | __le16 m; |
100 | u8 k; | 100 | u8 k; |
101 | u8 j; /* need to divide by 2 */ | 101 | u8 j; /* need to divide by 2 */ |
102 | } gtf2; | 102 | } __attribute__((packed)) gtf2; |
103 | struct { | 103 | struct { |
104 | u8 version; | 104 | u8 version; |
105 | u8 data1; /* high 6 bits: extra clock resolution */ | 105 | u8 data1; /* high 6 bits: extra clock resolution */ |
@@ -108,7 +108,7 @@ struct detailed_data_monitor_range { | |||
108 | u8 flags; /* preferred aspect and blanking support */ | 108 | u8 flags; /* preferred aspect and blanking support */ |
109 | u8 supported_scalings; | 109 | u8 supported_scalings; |
110 | u8 preferred_refresh; | 110 | u8 preferred_refresh; |
111 | } cvt; | 111 | } __attribute__((packed)) cvt; |
112 | } formula; | 112 | } formula; |
113 | } __attribute__((packed)); | 113 | } __attribute__((packed)); |
114 | 114 | ||