hwmon: (dell-smm) Pack the whole smm_regs struct
authorArmin Wolf <W_Armin@gmx.de>
Tue, 21 Dec 2021 16:28:05 +0000 (17:28 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Sun, 26 Dec 2021 23:02:07 +0000 (15:02 -0800)
When desiring the whole struct to be packed, __packed
should be applied to the whole struct, not just
every struct member except the first one.

Tested on a Dell Inspiron 3505.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
Link: https://lore.kernel.org/r/20211221162805.104202-1-W_Armin@gmx.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/dell-smm-hwmon.c

index d8c6e75bb3745a82ea3b7a891b692811a3af8c3f..d401f9acf45065348a8087eaf7544096e1214b30 100644 (file)
@@ -113,12 +113,12 @@ MODULE_PARM_DESC(fan_max, "Maximum configurable fan speed (default: autodetect)"
 
 struct smm_regs {
        unsigned int eax;
-       unsigned int ebx __packed;
-       unsigned int ecx __packed;
-       unsigned int edx __packed;
-       unsigned int esi __packed;
-       unsigned int edi __packed;
-};
+       unsigned int ebx;
+       unsigned int ecx;
+       unsigned int edx;
+       unsigned int esi;
+       unsigned int edi;
+} __packed;
 
 static const char * const temp_labels[] = {
        "CPU",