2018-03-07 14:05:00

by Salvatore Bellizzi

[permalink] [raw]
Subject: [RESEND][PATCH] platform/chrome: cros_ec_lpc: Add support for Google devices using custom coreboot firmware

This patch adds generic device information to the DMI table of
the cros_ec_lpc driver, needed for Chromebooks/boxes using a
custom coreboot firmware.

The DMI info would not contain "Google_*" as BIOS version string,
instead the system vendor string would still be "GOOGLE", so this
seems to be a reasonable match for every Chromebook/box running
a custom firmware.

Signed-off-by: Salvatore Bellizzi <[email protected]>
Signed-off-by: Vittorio Gambaletta <[email protected]>
---
drivers/platform/chrome/cros_ec_lpc.c | 12 ++++++++++++
1 file changed, 12 insertions(+)

diff --git a/drivers/platform/chrome/cros_ec_lpc.c b/drivers/platform/chrome/cros_ec_lpc.c
index af89e82eecd2..c8450ae3cd60 100644
--- a/drivers/platform/chrome/cros_ec_lpc.c
+++ b/drivers/platform/chrome/cros_ec_lpc.c
@@ -341,6 +341,18 @@ static const struct dmi_system_id cros_ec_lpc_dmi_table[] __initconst = {
DMI_MATCH(DMI_BIOS_VERSION, "Google_"),
},
},
+ {
+ /*
+ * If the box is running custom coreboot firmware then the
+ * DMI BIOS version string will not be matched by "Google_",
+ * but the system vendor string will still be matched by
+ * "GOOGLE".
+ */
+ .matches = {
+ DMI_MATCH(DMI_BIOS_VENDOR, "coreboot"),
+ DMI_MATCH(DMI_SYS_VENDOR, "GOOGLE"),
+ },
+ },
{
/* x86-link, the Chromebook Pixel. */
.matches = {
--
2.16.2



2018-03-20 02:04:43

by Benson Leung

[permalink] [raw]
Subject: Re: [RESEND][PATCH] platform/chrome: cros_ec_lpc: Add support for Google devices using custom coreboot firmware

Hi Salvatore,

On Wed, Mar 07, 2018 at 02:56:43PM +0100, Salvatore Bellizzi wrote:
> This patch adds generic device information to the DMI table of
> the cros_ec_lpc driver, needed for Chromebooks/boxes using a
> custom coreboot firmware.
>
> The DMI info would not contain "Google_*" as BIOS version string,
> instead the system vendor string would still be "GOOGLE", so this
> seems to be a reasonable match for every Chromebook/box running
> a custom firmware.
>
> Signed-off-by: Salvatore Bellizzi <[email protected]>
> Signed-off-by: Vittorio Gambaletta <[email protected]>

Applied for v4.17.

Thanks!
Benson

--
Benson Leung
Staff Software Engineer
Chrome OS Kernel
Google Inc.
[email protected]
Chromium OS Project
[email protected]


Attachments:
(No filename) (776.00 B)
signature.asc (849.00 B)
Download all attachments