Hello,

On 26/04/2018 00:57:34 CEST, Dmitry Torokhov wrote:

On Wed, Apr 25, 2018 at 03:26:50PM -0700, Dmitry Torokhov wrote:

...

On Wed, Apr 25, 2018 at 02:32:58PM +0200, Vittorio Gambaletta (VittGam) wrote:

...

This patch adds the correct platform data information for the Caroline Chromebook, so that the mouse button does not get stuck in pressed state after the first click.

The Samus button keymap and platform data definition are the correct ones for Caroline, so they have been reused here.

v2: updated patch offset after 20180409 changes.

Cc: stable@vger.kernel.org Signed-off-by: Vittorio Gambaletta linuxbugs@vittgam.net Signed-off-by: Salvatore Bellizzi lkml@seppia.net

Applied, thank you.

...

---

--- a/drivers/input/touchscreen/atmel_mxt_ts.c +++ b/drivers/input/touchscreen/atmel_mxt_ts.c @@ -3035,6 +3035,15 @@ .driver_data = samus_platform_data, }, {

• /* Samsung Chromebook Pro */
  

• .ident = "Samsung Chromebook Pro",
  

• .matches = {
  

• 	DMI_MATCH(DMI_SYS_VENDOR, "GOOGLE"),
  

I've been alerted that shipping BIOS has vendor not capitalized; where did you get the firmware that has vendor all capitals?

Upstream coreboot and mrchromebox's coreboot, which do support loading Linux natively or with UEFI, both use capitalized GOOGLE as DMI System Vendor as far as I know.

By the way, the shipping BIOS (coreboot customized by Google) does not support mainline Linux natively (eg. without seabios and nasty tricks), but only Chromium OS, which for Caroline is an old 3.something fork of Linux, so this patch does not really apply to Chromium OS or to Linux running on shipping BIOS.

Anyway if I recall correctly, the DMI System Vendor should be GOOGLE in Google's coreboot for Caroline too, but I'm not 100% sure and cannot check right now.

By the way please note that this is the DMI System Vendor and not the DMI BIOS version string, which indeed starts with "Google_" in the shipping BIOS (but not in coreboot, see f56db262e46d3368ee4e5c9e19797853cab382cd).

Thanks.

Cheers, Vittorio G

...

...

• 	DMI_MATCH(DMI_PRODUCT_NAME, "Caroline"),
  

• },
  

• .driver_data = samus_platform_data,
  

• },
• { /* Other Google Chromebooks */ .ident = "Chromebook", .matches = {

-- Dmitry