Patch "platform/x86: hp-wmi: Do not shadow error values" has been added to the 4.4-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
platform/x86: hp-wmi: Do not shadow error values
to the 4.4-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
platform-x86-hp-wmi-do-not-shadow-error-values.patch
and it can be found in the queue-4.4 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
d313876925f3e7a480a02773fd333bcab9202d5eMon Sep 17 00:00:00 2001
From: Carlo Caione <[email protected]>
Date: Wed, 19 Apr 2017 22:36:39 +0200
Subject: platform/x86: hp-wmi: Do not shadow error values
From: Carlo Caione <[email protected]>
commit d313876925f3e7a480a02773fd333bcab9202d5e upstream.
All the helper functions (i.e. hp_wmi_dock_state, hp_wmi_tablet_state, ...) using hp_wmi_perform_query to perform an HP WMI query shadow the returned value in case of error.
We return -EINVAL only when the HP WMI query returns a positive value (the specific error code) to not mix this up with the actual value returned by the helper function.
--- drivers/platform/x86/hp-wmi.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)
drivers/platform/x86/hp-wmi.c +10 -10
@@ -249,7 +249,7 @@ static int hp_wmi_display_state(void)int ret =hp_wmi_perform_query (HPWMI_DISPLAY_QUERY ,0 , &state,sizeof (state),sizeof (state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return state; }
@@ -259,7 +259,7 @@ static int hp_wmi_hddtemp_state(void)int ret =hp_wmi_perform_query (HPWMI_HDDTEMP_QUERY ,0 , &state,sizeof (state),sizeof (state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return state; }
@@ -269,7 +269,7 @@ static int hp_wmi_als_state(void)int ret =hp_wmi_perform_query (HPWMI_ALS_QUERY ,0 , &state,sizeof (state),sizeof (state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return state; }
@@ -280,7 +280,7 @@ static int hp_wmi_dock_state(void) sizeof(state), sizeof(state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return state &0x1 ; }
@@ -291,7 +291,7 @@ static int hp_wmi_tablet_state(void)int ret =hp_wmi_perform_query (HPWMI_HARDWARE_QUERY ,0 , &state,sizeof (state),sizeof (state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return (state &0x4 ) ?1 :0 ; }
@@ -324,7 +324,7 @@ static int __init hp_wmi_enable_hotkeys(int ret =hp_wmi_perform_query (HPWMI_BIOS_QUERY ,1 , &value,sizeof (value),0 );if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return 0 ; }
@@ -337,7 +337,7 @@ static int hp_wmi_set_block(void *data, ret =hp_wmi_perform_query (HPWMI_WIRELESS_QUERY ,1 , &query,sizeof (query),0 );if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return 0 ; }
@@ -429,7 +429,7 @@ static int hp_wmi_post_code_state(void)int ret =hp_wmi_perform_query (HPWMI_POSTCODEERROR_QUERY ,0 , &state,sizeof (state),sizeof (state));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return state; }
@@ -495,7 +495,7 @@ static ssize_t set_als(struct device *deint ret =hp_wmi_perform_query (HPWMI_ALS_QUERY ,1 , &tmp,sizeof (tmp),sizeof (tmp));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return count; }
@@ -516,7 +516,7 @@ static ssize_t set_postcode(struct devic ret =hp_wmi_perform_query (HPWMI_POSTCODEERROR_QUERY ,1 , &tmp,sizeof (tmp),sizeof (tmp));if (ret) -return -EINVAL ; +return ret <0 ? ret : -EINVAL ;return count; }
Patches currently in stable-queue which might be from [email protected] are
queue-4.4/platform-x86-hp-wmi-do-not-shadow-error-values.patch queue-4.4/platform-x86-hp-wmi-fix-detection-for-dock-and-tablet-mode.patch queue-4.4/platform-x86-hp-wmi-fix-error-value-for-hp_wmi_tablet_state.patch