Patch "platform/x86: ideapad-laptop: Stop writing VPCCMD_W_TOUCHPAD at probe time" has been added to the 6.1-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    platform/x86: ideapad-laptop: Stop writing VPCCMD_W_TOUCHPAD at probe time

to the 6.1-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-ideapad-laptop-stop-writing-vpccmd_w_to.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 4d9a7638a44e4768da2ba06180db0488c2bbdbe5
Author: Hans de Goede <hdegoede@xxxxxxxxxx>
Date:   Thu Nov 17 12:02:43 2022 +0100

    platform/x86: ideapad-laptop: Stop writing VPCCMD_W_TOUCHPAD at probe time
    
    [ Upstream commit a10ba160d427e78ffa2ab15a86cacaec291fa58a ]
    
    Commit d69cd7eea93e ("platform/x86: ideapad-laptop: Disable touchpad_switch
    for ELAN0634") from Janary 2021 added a flag hiding the touchpad sysfs-attr
    and disabling ideapad_sync_touchpad_state() because some devices
    "do not use EC to switch touchpad".
    
    At the same time this added a write(VPCCMD_W_TOUCHPAD, 1) call at probe
    time on these same devices. This seems to be copied from the rfkill code
    which does something similar when hw rfkill support is disabled.
    
    But for the rfkill code this is known to be necessary on some models,
    where as for the touchpad control no motivation is given for doing this
    and prior to this patch there were no reports of needing to do this.
    
    So this seems unnecessary; and it is best to avoid poking the hardware
    unnecessary to avoid unwanted side effects, so remove this.
    
    Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
    Reviewed-by: Jiaxun Yang <jiaxun.yang@xxxxxxxxxxx>
    Tested-by: Jiaxun Yang <jiaxun.yang@xxxxxxxxxxx>
    Tested-by: Maxim Mikityanskiy <maxtram95@xxxxxxxxx>
    Link: https://lore.kernel.org/r/20221117110244.67811-6-hdegoede@xxxxxxxxxx
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/platform/x86/ideapad-laptop.c
index 9b36cfddd36f..fc3d47a75944 100644
--- a/drivers/platform/x86/ideapad-laptop.c
+++ b/drivers/platform/x86/ideapad-laptop.c
@@ -1875,10 +1875,6 @@ static int ideapad_acpi_add(struct platform_device *pdev)
 	if (!priv->features.hw_rfkill_switch)
 		write_ec_cmd(priv->adev->handle, VPCCMD_W_RF, 1);
 
-	/* The same for Touchpad */
-	if (!priv->features.touchpad_ctrl_via_ec)
-		write_ec_cmd(priv->adev->handle, VPCCMD_W_TOUCHPAD, 1);
-
 	for (i = 0; i < IDEAPAD_RFKILL_DEV_NUM; i++)
 		if (test_bit(ideapad_rfk_data[i].cfgbit, &priv->cfg))
 			ideapad_register_rfkill(priv, i);



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux