Re: [PATCH] thinkpad_acpi: Add support for X1 Yoga (2016) Tablet Mode

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

 



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
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux