On Thu, 27 Oct 2016, Lyude Paul wrote:
> as well, can someone confirm this patch made it to the ibm-acpi-devel
> list? When I originally sent this I realized I wasn't subscribed to the
> list, so I'm guessing I might need to resend.
I received it. I just didn't have the time to go over it in detail.
At first look, it seems correct...
> > + /* For X1 Yoga (2016) */
> > + if (!res && acpi_evalf(ec_handle, &status, "CMMD", "qd")) {
> > + tp_features.hotkey_tablet = 1;
> > + tp_features.hotkey_tablet_cmmd = 1;
> > + tabletsw_state = (status == TP_EC_CMMD_TABLET_MODE);
> > +
> > + 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.a
> > ttr);
> > + }
Ehh... the code doesn't act as if it were a "possible" switch, but
rather that it _did_ find a x1_yoga-style switch, so the debug message
should be assertive to match the code:
pr_info("Tablet mode switch found (X1 Yoga style), Thinkpad is in...")
or something to that effect.
If the code is wrong, and it has false possitives, we enhance the
detection (possibly by using a quirk table if we must).
Care to respin with that change?
--
Henrique Holschuh
------------------------------------------------------------------------------
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive.
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
_______________________________________________
ibm-acpi-devel mailing list
ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel
