Re: HKEY version 0x200

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

 



On Wed, Apr 13, 2016, at 09:28, Dennis Wassenberg wrote:
> I would like to contribute to this topic as well.

Thanks!

> I believe (don't know, just guessing) that we should not make the
> feature "has_adaptive_kbd" depend on hkeyv 0x200. Instead we should use
> the "MHKA" method to evaluate if there is a flags mask set for the
> adaptive keyboard. I assume that if we pass "1" to the MHKA method we
> will get the hotkey_all_mask. If we pass "2" to the MHKA mask we will
> get the hotkey_all_adaptive_mask. Thats my assumption which is based on
> the observation of current Lenovo Skylake devices and the Thinkpad X1
> Carbon (2014).

It might need to depend on both.  Are there any adaptive keyboards with
HKEY 0x100?

If so, do they have the full ACPI interface for the adaptative keyboard,
or do they like something, e.g. MHKA(0x2) ?

Because HKEY returning 0x200 should actually be the marker that the MHK*
methods have changed to support the adaptive keyboard better...

> I checked it with T460, T460p, T460s, T560, X260 and X1 Tablet (2016).
> All of these devices return 0 in that case. Only the X1 Carbon (2014)
> will return 0x1FFFFFFF. The Thinkpads L460 and L560 are not covered by
> this because they have hkeyv 0x100.

Seems like it would also work if we detect by HKEY 0x200 + MHKA(2) !=
0...

> I prepared a patch which does what I described above (attachment). This
> works fine in my case with kernel >= 4.3 until 4.6-rc3 for T460, T460p,
> T460s, T560, X260. All hkeyv 0x100 Thinkpads should not be affected by
> this change. Whats your opinion about this patch?

I will likely be able to only look at it in detail during the weekend.

> Testing this with Lenovo Thinkpad X1 Tablet (2016) will make the hotkeys
> generate an event but currently there seems to be a mapping issue
> because the most events are "KEY_UNKNOWN".

Ok.  This needs a fix, but I will only know which one after I have time
to look over the current adaptive code and your patch in detail.

We likely need to extend the keymap.

> Additionally I observed that the new Skylake Thinkpads have no HKEY
> methods GDFR, STRW, GTRW, SBMI, SVOI, GVOI, GCGI, GHOV, NALL, SAPI,
> GAPI, SMNC, GMNC, SCMI, GCMI, SCMD, GCMD, GMMS, UAND, DGBL, SDHK, GDHK,
> SDFL, GDFL, SDAM, GDAM in contrast to X1 Carbon (2014) with HKEY 0x200.

Can you send to me in private mail the full acpidump (i.e. every table)
and dmidecode output for all these thinkpads you tested with?  Please
cross-out any UUIDs and serial numbers...

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
ibm-acpi-devel mailing list
ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel



[Index of Archives]     [Linux ACPI]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Photo]     [Yosemite Photos]     [Yosemite Advice]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]

  Powered by Linux