Sorry, I didn't had the time to look at the patches earlier. Now, I did and was wondering why hotkey_tablet_mode doesn't show up in sysfs anymore ... On 7 November 2016 at 18:10, Lyude <lyude@xxxxxxxxxx> wrote: > @@ -3464,17 +3495,6 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) > res = add_to_attr_set(hotkey_dev_attributes, > &dev_attr_hotkey_radio_sw.attr); > > - /* For X41t, X60t, X61t Tablets... */ > - if (!res && acpi_evalf(hkey_handle, &status, "MHKG", "qd")) { > - tp_features.hotkey_tablet = 1; > - tabletsw_state = !!(status & TP_HOTKEY_TABLET_MASK); > - pr_info("possible tablet mode switch found; " > - "ThinkPad in %s mode\n", > - (tabletsw_state) ? "tablet" : "laptop"); > - res = add_to_attr_set(hotkey_dev_attributes, > - &dev_attr_hotkey_tablet_mode.attr); > - } > - > if (!res) > res = register_attr_set_with_sysfs( > hotkey_dev_attributes, > @@ -3482,6 +3502,12 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) > if (res) > goto err_exit; > > + res = hotkey_init_tablet_mode(); > + if (res < 0) > + goto err_exit; > + > + tabletsw_state = res; > + > /* Set up key map */ > hotkey_keycode_map = kmalloc(TPACPI_HOTKEY_MAP_SIZE, > GFP_KERNEL); > -- > 2.7.4 > This block has to be before register_attr_set_with_sysfs(). With that fixed and a small patch (will be sent soon) on top it works on my Yoga 260, so: Tested-by: Daniel Martin <consume.noise@xxxxxxxxx> ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ ibm-acpi-devel mailing list ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel