Hi Lyndon, Thanks for working on this patch. Internal Use - Confidential +AD4- -----Original Message----- +AD4- From: Lyndon Sanche +ADw-lsanche+AEA-lyndeno.ca+AD4- +AD4- Sent: Thursday, May 2, 2024 5:58 AM +AD4- To: lsanche+AEA-lyndeno.ca +AD4- Cc: mario.limonciello+AEA-amd.com+ADs- pali+AEA-kernel.org+ADs- W+AF8-Armin+AEA-gmx.de+ADs- +AD4- srinivas.pandruvada+AEA-linux.intel.com+ADs- ilpo.jarvinen+AEA-linux.intel.com+ADs- +AD4- lkp+AEA-intel.com+ADs- Hans de Goede +ADw-hdegoede+AEA-redhat.com+AD4AOw- Matthew Garrett +AD4- +ADw-mjg59+AEA-srcf.ucam.org+AD4AOw- Jonathan Corbet +ADw-corbet+AEA-lwn.net+AD4AOw- Heiner Kallweit +AD4- +ADw-hkallweit1+AEA-gmail.com+AD4AOw- Vegard Nossum +ADw-vegard.nossum+AEA-oracle.com+AD4AOw- +AD4- platform-driver-x86+AEA-vger.kernel.org+ADs- linux-kernel+AEA-vger.kernel.org+ADs- Dell Client +AD4- Kernel +ADw-Dell.Client.Kernel+AEA-dell.com+AD4- +AD4- Subject: +AFs-PATCH v5+AF0- platform/x86: dell-laptop: Implement platform+AF8-profile +AD4- +AD4- +AD4- +AFs-EXTERNAL EMAIL+AF0- +AD4- +AD4- Some Dell laptops support configuration of preset fan modes through smbios +AD4- tables. +AD4- +AD4- If the platform supports these fan modes, set up platform+AF8-profile to change +AD4- these modes. If not supported, skip enabling platform+AF8-profile. +AD4- +AD4- Signed-off-by: Lyndon Sanche +ADw-lsanche+AEA-lyndeno.ca+AD4- +AD4- --- +AD4- v5: +AD4- - Fix indent in smbios-thermal-ctl comment +AD4- - Remove linux/wmi.h include +AD4- - Add 'select ACPI+AF8-PLATFORM+AF8-PROFILE' to Dell KConfig +AD4- v4: +AD4- - Make thermal+AF8-init and thermal+AF8-cleanup static +AD4- - Rearrange order of added includes, did not edit current includes +AD4- - Include bits.h +AD4- - Switch comment style +AD4- - Return error if platform+AF8-profile registering failed +AD4- - Add thermal calls to call+AF8-blacklist +AD4- - Align defines with tabs +AD4- - Correct separation of function and error handling +AD4- - Propagate error codes up +AD4- v3: +AD4- - Convert smbios-thermal-ctl docs to multiline comment and wrap +AD4- - Change thermal+AF8-mode+AF8-bits enum to directly be BIT() values +AD4- - Convert related code to use this +AD4- - Use FIELD+AF8-GET/PREP and GENNMASK for getting/setting thermal modes +AD4- - Correct offset for getting current ACC mode, setting offset +AD4- unchanged +AD4- - Check if thermal+AF8-handler is allocated before freeing and +AD4- unregistering platform+AF8-profile +AD4- v2: +AD4- - Wrap smbios-thermal-ctl comment +AD4- - Return proper error code when invalid state returned +AD4- - Simplify platform+AF8-profile+AF8-get returns +AD4- - Propogate ENOMEM error +AD4- --- Dell side has an initial testing with this patch on some laptops, it looks good. While changing the platform profile: 1. The corresponding USTT option in BIOS will be changed. 2. thermald will not be impacted. The related PSVT and ITMT will be loaded. Some Dell DTs does not have the USTT, Dell'll have a check if nothing is broken. Additional, with this patch, follow behavior is found: 1. For example, the platform profile is quiet. 2. Reboot the system and change the USTT to performance. 3. Boot to desktop, the platform profile is +ACI-quiet+ACI-, the USTT will be changed back to +ACI-quiet+ACI-. This looks like not a proper user experience. The platform profile should honor the BIOS setting, aka, the platform profile should be switched to +ACI-performance+ACI-. +AD4- drivers/platform/x86/dell/Kconfig +AHw- 1 +- +AD4- drivers/platform/x86/dell/dell-laptop.c +AHw- 238 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+- +AD4- drivers/platform/x86/dell/dell-smbios-base.c +AHw- 1 +- +AD4- drivers/platform/x86/dell/dell-smbios.h +AHw- 1 +- +AD4- 4 files changed, 241 insertions(+-) +AD4- +AD4- diff --git a/drivers/platform/x86/dell/Kconfig +AD4- b/drivers/platform/x86/dell/Kconfig +AD4- index bd9f445974cc..5195ad59b44d 100644 +AD4- --- a/drivers/platform/x86/dell/Kconfig +AD4- +-+-+- b/drivers/platform/x86/dell/Kconfig +AD4- +AEAAQA- -57,6 +-57,7 +AEAAQA- config DELL+AF8-LAPTOP +AD4- select POWER+AF8-SUPPLY +AD4- select LEDS+AF8-CLASS +AD4- select NEW+AF8-LEDS +AD4- +- select ACPI+AF8-PLATFORM+AF8-PROFILE +AD4- help +AD4- This driver adds support for rfkill and backlight control to Dell +AD4- laptops (except for some models covered by the Compal driver). +AD4- diff --git a/drivers/platform/x86/dell/dell-laptop.c +AD4- b/drivers/platform/x86/dell/dell-laptop.c +AD4- index 42f7de2b4522..dc530a4f5047 100644 +AD4- --- a/drivers/platform/x86/dell/dell-laptop.c +AD4- +-+-+- b/drivers/platform/x86/dell/dell-laptop.c +AD4- +AEAAQA- -27,6 +-27,9 +AEAAQA- +AD4- +ACM-include +ADw-linux/i8042.h+AD4- +AD4- +ACM-include +ADw-linux/debugfs.h+AD4- +AD4- +ACM-include +ADw-linux/seq+AF8-file.h+AD4- +AD4- +-+ACM-include +ADw-linux/bitfield.h+AD4- +AD4- +-+ACM-include +ADw-linux/bits.h+AD4- +AD4- +-+ACM-include +ADw-linux/platform+AF8-profile.h+AD4- +AD4- +ACM-include +ADw-acpi/video.h+AD4- +AD4- +ACM-include +ACI-dell-rbtn.h+ACI- +AD4- +ACM-include +ACI-dell-smbios.h+ACI- +AD4- +AEAAQA- -95,6 +-98,7 +AEAAQA- static struct backlight+AF8-device +ACo-dell+AF8-backlight+AF8-device+ADs- +AD4- static struct rfkill +ACo-wifi+AF8-rfkill+ADs- static struct rfkill +ACo-bluetooth+AF8-rfkill+ADs- static struct +AD4- rfkill +ACo-wwan+AF8-rfkill+ADs- +AD4- +-static struct platform+AF8-profile+AF8-handler +ACo-thermal+AF8-handler+ADs- +AD4- static bool force+AF8-rfkill+ADs- +AD4- static bool micmute+AF8-led+AF8-registered+ADs- +AD4- static bool mute+AF8-led+AF8-registered+ADs- +AD4- +AEAAQA- -2199,6 +-2203,232 +AEAAQA- static int mute+AF8-led+AF8-set(struct led+AF8-classdev +AD4- +ACo-led+AF8-cdev, +AD4- return 0+ADs- +AD4- +AH0- +AD4- +AD4- +-/+ACo- Derived from smbios-thermal-ctl +AD4- +- +ACo- +AD4- +- +ACo- cbClass 17 +AD4- +- +ACo- cbSelect 19 +AD4- +- +ACo- User Selectable Thermal Tables(USTT) +AD4- +- +ACo- cbArg1 determines the function to be performed +AD4- +- +ACo- cbArg1 0x0 +AD0- Get Thermal Information +AD4- +- +ACo- cbRES1 Standard return codes (0, -1, -2) +AD4- +- +ACo- cbRES2, byte 0 Bitmap of supported thermal modes. A mode is +AD4- supported if +AD4- +- +ACo- its bit is set to 1 +AD4- +- +ACo- Bit 0 Balanced +AD4- +- +ACo- Bit 1 Cool Bottom +AD4- +- +ACo- Bit 2 Quiet +AD4- +- +ACo- Bit 3 Performance +AD4- +- +ACo- cbRES2, byte 1 Bitmap of supported Active Acoustic Controller (AAC) +AD4- modes. +AD4- +- +ACo- Each mode corresponds to the supported thermal modes in +AD4- +- +ACo- byte 0. A mode is supported if its bit is set to 1. +AD4- +- +ACo- Bit 0 AAC (Balanced) +AD4- +- +ACo- Bit 1 AAC (Cool Bottom +AD4- +- +ACo- Bit 2 AAC (Quiet) +AD4- +- +ACo- Bit 3 AAC (Performance) +AD4- +- +ACo- cbRes3, byte 0 Current Thermal Mode +AD4- +- +ACo- Bit 0 Balanced +AD4- +- +ACo- Bit 1 Cool Bottom +AD4- +- +ACo- Bit 2 Quiet +AD4- +- +ACo- Bit 3 Performanc +AD4- +- +ACo- cbRes3, byte 1 AAC Configuration type +AD4- +- +ACo- 0 Global (AAC enable/disable applies to all supported USTT +AD4- modes) +AD4- +- +ACo- 1 USTT mode specific +AD4- +- +ACo- cbRes3, byte 2 Current Active Acoustic Controller (AAC) Mode +AD4- +- +ACo- If AAC Configuration Type is Global, +AD4- +- +ACo- 0 AAC mode disabled +AD4- +- +ACo- 1 AAC mode enabled +AD4- +- +ACo- If AAC Configuration Type is USTT mode specific (multiple bits may be +AD4- set), +AD4- +- +ACo- Bit 0 AAC (Balanced) +AD4- +- +ACo- Bit 1 AAC (Cool Bottom +AD4- +- +ACo- Bit 2 AAC (Quiet) +AD4- +- +ACo- Bit 3 AAC (Performance) +AD4- +- +ACo- cbRes3, byte 3 Current Fan Failure Mode +AD4- +- +ACo- Bit 0 Minimal Fan Failure (at least one fan has failed, one fan working) +AD4- +- +ACo- Bit 1 Catastrophic Fan Failure (all fans have failed) +AD4- +- +ACo- +AD4- +- +ACo- cbArg1 0x1 (Set Thermal Information), both desired thermal mode and +AD4- +- +ACo- desired AAC mode shall be applied +AD4- +- +ACo- cbArg2, byte 0 Desired Thermal Mode to set +AD4- +- +ACo- (only one bit may be set for this parameter) +AD4- +- +ACo- Bit 0 Balanced +AD4- +- +ACo- Bit 1 Cool Bottom +AD4- +- +ACo- Bit 2 Quiet +AD4- +- +ACo- Bit 3 Performance +AD4- +- +ACo- cbArg2, byte 1 Desired Active Acoustic Controller (AAC) Mode to set +AD4- +- +ACo- If AAC Configuration Type is Global, +AD4- +- +ACo- 0 AAC mode disabled +AD4- +- +ACo- 1 AAC mode enabled +AD4- +- +ACo- If AAC Configuration Type is USTT mode specific +AD4- +- +ACo- (multiple bits may be set for this parameter), +AD4- +- +ACo- Bit 0 AAC (Balanced) +AD4- +- +ACo- Bit 1 AAC (Cool Bottom +AD4- +- +ACo- Bit 2 AAC (Quiet) +AD4- +- +ACo- Bit 3 AAC (Performance) +AD4- +- +ACo-/ +AD4- +- +AD4- +-+ACM-define DELL+AF8-ACC+AF8-GET+AF8-FIELD GENMASK(19, 16) +AD4- +-+ACM-define DELL+AF8-ACC+AF8-SET+AF8-FIELD GENMASK(11, 8) +AD4- +-+ACM-define DELL+AF8-THERMAL+AF8-SUPPORTED GENMASK(3, 0) +AD4- +- +AD4- +-enum thermal+AF8-mode+AF8-bits +AHs- +AD4- +- DELL+AF8-BALANCED +AD0- BIT(0), +AD4- +- DELL+AF8-COOL+AF8-BOTTOM +AD0- BIT(1), +AD4- +- DELL+AF8-QUIET +AD0- BIT(2), +AD4- +- DELL+AF8-PERFORMANCE +AD0- BIT(3), +AD4- +-+AH0AOw- +AD4- +- +AD4- +-static int thermal+AF8-get+AF8-mode(void) +AD4- +-+AHs- +AD4- +- struct calling+AF8-interface+AF8-buffer buffer+ADs- +AD4- +- int state+ADs- +AD4- +- int ret+ADs- +AD4- +- +AD4- +- dell+AF8-fill+AF8-request(+ACY-buffer, 0x0, 0, 0, 0)+ADs- +AD4- +- ret +AD0- dell+AF8-send+AF8-request(+ACY-buffer, CLASS+AF8-INFO, +AD4- SELECT+AF8-THERMAL+AF8-MANAGEMENT)+ADs- +AD4- +- if (ret) +AD4- +- return ret+ADs- +AD4- +- state +AD0- buffer.output+AFs-2+AF0AOw- +AD4- +- if (state +ACY- DELL+AF8-BALANCED) +AD4- +- return DELL+AF8-BALANCED+ADs- +AD4- +- else if (state +ACY- DELL+AF8-COOL+AF8-BOTTOM) +AD4- +- return DELL+AF8-COOL+AF8-BOTTOM+ADs- +AD4- +- else if (state +ACY- DELL+AF8-QUIET) +AD4- +- return DELL+AF8-QUIET+ADs- +AD4- +- else if (state +ACY- DELL+AF8-PERFORMANCE) +AD4- +- return DELL+AF8-PERFORMANCE+ADs- +AD4- +- else +AD4- +- return -ENXIO+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-get+AF8-supported+AF8-modes(int +ACo-supported+AF8-bits) +AHs- +AD4- +- struct calling+AF8-interface+AF8-buffer buffer+ADs- +AD4- +- int ret+ADs- +AD4- +- +AD4- +- dell+AF8-fill+AF8-request(+ACY-buffer, 0x0, 0, 0, 0)+ADs- +AD4- +- ret +AD0- dell+AF8-send+AF8-request(+ACY-buffer, CLASS+AF8-INFO, +AD4- SELECT+AF8-THERMAL+AF8-MANAGEMENT)+ADs- +AD4- +- if (ret) +AD4- +- return ret+ADs- +AD4- +- +ACo-supported+AF8-bits +AD0- FIELD+AF8-GET(DELL+AF8-THERMAL+AF8-SUPPORTED, +AD4- buffer.output+AFs-1+AF0-)+ADs- +AD4- +- return 0+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-get+AF8-acc+AF8-mode(int +ACo-acc+AF8-mode) +AHs- +AD4- +- struct calling+AF8-interface+AF8-buffer buffer+ADs- +AD4- +- int ret+ADs- +AD4- +- +AD4- +- dell+AF8-fill+AF8-request(+ACY-buffer, 0x0, 0, 0, 0)+ADs- +AD4- +- ret +AD0- dell+AF8-send+AF8-request(+ACY-buffer, CLASS+AF8-INFO, +AD4- SELECT+AF8-THERMAL+AF8-MANAGEMENT)+ADs- +AD4- +- if (ret) +AD4- +- return ret+ADs- +AD4- +- +ACo-acc+AF8-mode +AD0- FIELD+AF8-GET(DELL+AF8-ACC+AF8-GET+AF8-FIELD, buffer.output+AFs-3+AF0-)+ADs- +AD4- +- return 0+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-set+AF8-mode(enum thermal+AF8-mode+AF8-bits state) +AHs- +AD4- +- struct calling+AF8-interface+AF8-buffer buffer+ADs- +AD4- +- int ret+ADs- +AD4- +- int acc+AF8-mode+ADs- +AD4- +- +AD4- +- ret +AD0- thermal+AF8-get+AF8-acc+AF8-mode(+ACY-acc+AF8-mode)+ADs- +AD4- +- if (ret) +AD4- +- return ret+ADs- +AD4- +- +AD4- +- dell+AF8-fill+AF8-request(+ACY-buffer, 0x1, FIELD+AF8-PREP(DELL+AF8-ACC+AF8-SET+AF8-FIELD, +AD4- acc+AF8-mode) +AHw- state, 0, 0)+ADs- +AD4- +- ret +AD0- dell+AF8-send+AF8-request(+ACY-buffer, CLASS+AF8-INFO, +AD4- SELECT+AF8-THERMAL+AF8-MANAGEMENT)+ADs- +AD4- +- return ret+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-platform+AF8-profile+AF8-set(struct platform+AF8-profile+AF8-handler +AD4- +ACo-pprof, +AD4- +- enum platform+AF8-profile+AF8-option +AD4- profile) +AHs- +AD4- +- switch (profile) +AHs- +AD4- +- case PLATFORM+AF8-PROFILE+AF8-BALANCED: +AD4- +- return thermal+AF8-set+AF8-mode(DELL+AF8-BALANCED)+ADs- +AD4- +- case PLATFORM+AF8-PROFILE+AF8-PERFORMANCE: +AD4- +- return thermal+AF8-set+AF8-mode(DELL+AF8-PERFORMANCE)+ADs- +AD4- +- case PLATFORM+AF8-PROFILE+AF8-QUIET: +AD4- +- return thermal+AF8-set+AF8-mode(DELL+AF8-QUIET)+ADs- +AD4- +- case PLATFORM+AF8-PROFILE+AF8-COOL: +AD4- +- return thermal+AF8-set+AF8-mode(DELL+AF8-COOL+AF8-BOTTOM)+ADs- +AD4- +- default: +AD4- +- return -EOPNOTSUPP+ADs- +AD4- +- +AH0- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-platform+AF8-profile+AF8-get(struct platform+AF8-profile+AF8-handler +AD4- +ACo-pprof, +AD4- +- enum platform+AF8-profile+AF8-option +AD4- +ACo-profile) +AHs- +AD4- +- int ret+ADs- +AD4- +- +AD4- +- ret +AD0- thermal+AF8-get+AF8-mode()+ADs- +AD4- +- if (ret +ADw- 0) +AD4- +- return ret+ADs- +AD4- +- +AD4- +- switch (ret) +AHs- +AD4- +- case DELL+AF8-BALANCED: +AD4- +- +ACo-profile +AD0- PLATFORM+AF8-PROFILE+AF8-BALANCED+ADs- +AD4- +- break+ADs- +AD4- +- case DELL+AF8-PERFORMANCE: +AD4- +- +ACo-profile +AD0- PLATFORM+AF8-PROFILE+AF8-PERFORMANCE+ADs- +AD4- +- break+ADs- +AD4- +- case DELL+AF8-COOL+AF8-BOTTOM: +AD4- +- +ACo-profile +AD0- PLATFORM+AF8-PROFILE+AF8-COOL+ADs- +AD4- +- break+ADs- +AD4- +- case DELL+AF8-QUIET: +AD4- +- +ACo-profile +AD0- PLATFORM+AF8-PROFILE+AF8-QUIET+ADs- +AD4- +- break+ADs- +AD4- +- default: +AD4- +- return -EINVAL+ADs- +AD4- +- +AH0- +AD4- +- +AD4- +- return 0+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static int thermal+AF8-init(void) +AD4- +-+AHs- +AD4- +- int ret+ADs- +AD4- +- int supported+AF8-modes+ADs- +AD4- +- +AD4- +- /+ACo- If thermal modes not supported, exit without error +ACo-/ +AD4- +- ret +AD0- thermal+AF8-get+AF8-supported+AF8-modes(+ACY-supported+AF8-modes)+ADs- +AD4- +- if (ret +ADw- 0) +AD4- +- return ret+ADs- +AD4- +- if (+ACE-supported+AF8-modes) +AD4- +- return 0+ADs- +AD4- +- +AD4- +- thermal+AF8-handler +AD0- kzalloc(sizeof(+ACo-thermal+AF8-handler), GFP+AF8-KERNEL)+ADs- +AD4- +- if (+ACE-thermal+AF8-handler) +AD4- +- return -ENOMEM+ADs- +AD4- +- thermal+AF8-handler-+AD4-profile+AF8-get +AD0- thermal+AF8-platform+AF8-profile+AF8-get+ADs- +AD4- +- thermal+AF8-handler-+AD4-profile+AF8-set +AD0- thermal+AF8-platform+AF8-profile+AF8-set+ADs- +AD4- +- +AD4- +- if (supported+AF8-modes +ACY- DELL+AF8-QUIET) +AD4- +- set+AF8-bit(PLATFORM+AF8-PROFILE+AF8-QUIET, thermal+AF8-handler- +AD4- +AD4-choices)+ADs- +AD4- +- if (supported+AF8-modes +ACY- DELL+AF8-COOL+AF8-BOTTOM) +AD4- +- set+AF8-bit(PLATFORM+AF8-PROFILE+AF8-COOL, thermal+AF8-handler- +AD4- +AD4-choices)+ADs- +AD4- +- if (supported+AF8-modes +ACY- DELL+AF8-BALANCED) +AD4- +- set+AF8-bit(PLATFORM+AF8-PROFILE+AF8-BALANCED, thermal+AF8-handler- +AD4- +AD4-choices)+ADs- +AD4- +- if (supported+AF8-modes +ACY- DELL+AF8-PERFORMANCE) +AD4- +- set+AF8-bit(PLATFORM+AF8-PROFILE+AF8-PERFORMANCE, +AD4- thermal+AF8-handler-+AD4-choices)+ADs- +AD4- +- +AD4- +- /+ACo- Clean up if failed +ACo-/ +AD4- +- ret +AD0- platform+AF8-profile+AF8-register(thermal+AF8-handler)+ADs- +AD4- +- if (ret) +AD4- +- kfree(thermal+AF8-handler)+ADs- +AD4- +- +AD4- +- return ret+ADs- +AD4- +-+AH0- +AD4- +- +AD4- +-static void thermal+AF8-cleanup(void) +AD4- +-+AHs- +AD4- +- if (thermal+AF8-handler) +AHs- +AD4- +- platform+AF8-profile+AF8-remove()+ADs- +AD4- +- kfree(thermal+AF8-handler)+ADs- +AD4- +- +AH0- +AD4- +-+AH0- +AD4- +- +AD4- static struct led+AF8-classdev mute+AF8-led+AF8-cdev +AD0- +AHs- +AD4- .name +AD0- +ACI-platform::mute+ACI-, +AD4- .max+AF8-brightness +AD0- 1, +AD4- +AEAAQA- -2238,6 +-2468,11 +AEAAQA- static int +AF8AXw-init dell+AF8-init(void) +AD4- goto fail+AF8-rfkill+ADs- +AD4- +AH0- +AD4- +AD4- +- /+ACo- Do not fail module if thermal modes not supported, just skip +ACo-/ +AD4- +- ret +AD0- thermal+AF8-init()+ADs- +AD4- +- if (ret) +AD4- +- goto fail+AF8-thermal+ADs- +AD4- +- +AD4- if (quirks +ACYAJg- quirks-+AD4-touchpad+AF8-led) +AD4- touchpad+AF8-led+AF8-init(+ACY-platform+AF8-device-+AD4-dev)+ADs- +AD4- +AD4- +AEAAQA- -2317,6 +-2552,8 +AEAAQA- static int +AF8AXw-init dell+AF8-init(void) +AD4- led+AF8-classdev+AF8-unregister(+ACY-mute+AF8-led+AF8-cdev)+ADs- +AD4- fail+AF8-led: +AD4- dell+AF8-cleanup+AF8-rfkill()+ADs- +AD4- +-fail+AF8-thermal: +AD4- +- thermal+AF8-cleanup()+ADs- +AD4- fail+AF8-rfkill: +AD4- platform+AF8-device+AF8-del(platform+AF8-device)+ADs- +AD4- fail+AF8-platform+AF8-device2: +AD4- +AEAAQA- -2344,6 +-2581,7 +AEAAQA- static void +AF8AXw-exit dell+AF8-exit(void) +AD4- platform+AF8-device+AF8-unregister(platform+AF8-device)+ADs- +AD4- platform+AF8-driver+AF8-unregister(+ACY-platform+AF8-driver)+ADs- +AD4- +AH0- +AD4- +- thermal+AF8-cleanup()+ADs- +AD4- +AH0- +AD4- +AD4- /+ACo- dell-rbtn.c driver export functions which will not work correctly (and could +AD4- diff --git a/drivers/platform/x86/dell/dell-smbios-base.c +AD4- b/drivers/platform/x86/dell/dell-smbios-base.c +AD4- index e61bfaf8b5c4..5bc2e394dd1c 100644 +AD4- --- a/drivers/platform/x86/dell/dell-smbios-base.c +AD4- +-+-+- b/drivers/platform/x86/dell/dell-smbios-base.c +AD4- +AEAAQA- -71,6 +-71,7 +AEAAQA- static struct smbios+AF8-call call+AF8-blacklist+AFsAXQ- +AD0- +AHs- +AD4- /+ACo- handled by kernel: dell-laptop +ACo-/ +AD4- +AHs-0x0000, CLASS+AF8-INFO, SELECT+AF8-RFKILL+AH0-, +AD4- +AHs-0x0000, CLASS+AF8-KBD+AF8-BACKLIGHT, SELECT+AF8-KBD+AF8-BACKLIGHT+AH0-, +AD4- +- +AHs-0x0000, CLASS+AF8-INFO, SELECT+AF8-THERMAL+AF8-MANAGEMENT+AH0-, +AD4- +AH0AOw- +AD4- +AD4- struct token+AF8-range +AHs- +AD4- diff --git a/drivers/platform/x86/dell/dell-smbios.h +AD4- b/drivers/platform/x86/dell/dell-smbios.h +AD4- index eb341bf000c6..585d042f1779 100644 +AD4- --- a/drivers/platform/x86/dell/dell-smbios.h +AD4- +-+-+- b/drivers/platform/x86/dell/dell-smbios.h +AD4- +AEAAQA- -19,6 +-19,7 +AEAAQA- +AD4- /+ACo- Classes and selects used only in kernel drivers +ACo-/ +ACM-define +AD4- CLASS+AF8-KBD+AF8-BACKLIGHT 4 +ACM-define SELECT+AF8-KBD+AF8-BACKLIGHT 11 +AD4- +-+ACM-define SELECT+AF8-THERMAL+AF8-MANAGEMENT 19 +AD4- +AD4- /+ACo- Tokens used in kernel drivers, any of these +AD4- +ACo- should be filtered from userspace access +AD4- -- +AD4- 2.42.0
