* [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
@ 2023-07-31 2:42 Tzung-Bi Shih
2023-07-31 7:24 ` Muhammad Usama Anjum
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Tzung-Bi Shih @ 2023-07-31 2:42 UTC (permalink / raw
To: bleung, groeck, rafael, lenb
Cc: guillaume.tucker, denys.f, ricardo.canuelo, usama.anjum,
chrome-platform, tzungbi, linux-acpi
Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
[1]: https://crrev.com/c/2266713
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
Changes from v2[2]:
- Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
Changes from v1[3]:
- Preserve the PNP ID GGL0001 for backward compatible to older firmwares.
- Modify commit title and message accordingly.
[2]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728065438.4026629-1-tzungbi@kernel.org/
[3]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728014256.3836298-1-tzungbi@kernel.org/
.../ABI/testing/sysfs-driver-chromeos-acpi | 15 +++++++++++++++
.../firmware-guide/acpi/chromeos-acpi-device.rst | 5 ++---
drivers/platform/chrome/chromeos_acpi.c | 2 +-
3 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
index c308926e1568..22082f9a7922 100644
--- a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
+++ b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
@@ -1,4 +1,5 @@
What: /sys/bus/platform/devices/GGL0001:*/BINF.2
+ /sys/bus/platform/devices/GOOG0016:*/BINF.2
Date: May 2022
KernelVersion: 5.19
Description:
@@ -10,6 +11,7 @@ Description:
== ===============================
What: /sys/bus/platform/devices/GGL0001:*/BINF.3
+ /sys/bus/platform/devices/GOOG0016:*/BINF.3
Date: May 2022
KernelVersion: 5.19
Description:
@@ -23,6 +25,7 @@ Description:
== =====================================
What: /sys/bus/platform/devices/GGL0001:*/CHSW
+ /sys/bus/platform/devices/GOOG0016:*/CHSW
Date: May 2022
KernelVersion: 5.19
Description:
@@ -38,6 +41,7 @@ Description:
==== ===========================================
What: /sys/bus/platform/devices/GGL0001:*/FMAP
+ /sys/bus/platform/devices/GOOG0016:*/FMAP
Date: May 2022
KernelVersion: 5.19
Description:
@@ -45,6 +49,7 @@ Description:
processor firmware flashmap.
What: /sys/bus/platform/devices/GGL0001:*/FRID
+ /sys/bus/platform/devices/GOOG0016:*/FRID
Date: May 2022
KernelVersion: 5.19
Description:
@@ -52,6 +57,7 @@ Description:
main processor firmware.
What: /sys/bus/platform/devices/GGL0001:*/FWID
+ /sys/bus/platform/devices/GOOG0016:*/FWID
Date: May 2022
KernelVersion: 5.19
Description:
@@ -59,6 +65,7 @@ Description:
main processor firmware.
What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.0
+ /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.0
Date: May 2022
KernelVersion: 5.19
Description:
@@ -73,6 +80,7 @@ Description:
=========== ==================================
What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.1
+ /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.1
Date: May 2022
KernelVersion: 5.19
Description:
@@ -84,6 +92,7 @@ Description:
== =======================
What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.2
+ /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.2
Date: May 2022
KernelVersion: 5.19
Description:
@@ -91,18 +100,21 @@ Description:
controller.
What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.3
+ /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.3
Date: May 2022
KernelVersion: 5.19
Description:
Returns name of the GPIO controller.
What: /sys/bus/platform/devices/GGL0001:*/HWID
+ /sys/bus/platform/devices/GOOG0016:*/HWID
Date: May 2022
KernelVersion: 5.19
Description:
Returns hardware ID for the Chromebook.
What: /sys/bus/platform/devices/GGL0001:*/MECK
+ /sys/bus/platform/devices/GOOG0016:*/MECK
Date: May 2022
KernelVersion: 5.19
Description:
@@ -113,6 +125,7 @@ Description:
present, or if the firmware was unable to read the extended registers, this buffer size can be zero.
What: /sys/bus/platform/devices/GGL0001:*/VBNV.0
+ /sys/bus/platform/devices/GOOG0016:*/VBNV.0
Date: May 2022
KernelVersion: 5.19
Description:
@@ -122,6 +135,7 @@ Description:
clock data).
What: /sys/bus/platform/devices/GGL0001:*/VBNV.1
+ /sys/bus/platform/devices/GOOG0016:*/VBNV.1
Date: May 2022
KernelVersion: 5.19
Description:
@@ -129,6 +143,7 @@ Description:
storage block.
What: /sys/bus/platform/devices/GGL0001:*/VDAT
+ /sys/bus/platform/devices/GOOG0016:*/VDAT
Date: May 2022
KernelVersion: 5.19
Description:
diff --git a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
index f37fc90ce340..89419e116413 100644
--- a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
+++ b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
@@ -5,9 +5,8 @@ Chrome OS ACPI Device
=====================
Hardware functionality specific to Chrome OS is exposed through a Chrome OS ACPI device.
-The plug and play ID of a Chrome OS ACPI device is GGL0001. GGL is a valid PNP ID of Google.
-PNP ID can be used with the ACPI devices according to the guidelines. The following ACPI
-objects are supported:
+The plug and play ID of a Chrome OS ACPI device is GGL0001 and the hardware ID is
+GOOG0016. The following ACPI objects are supported:
.. flat-table:: Supported ACPI Objects
:widths: 1 2
diff --git a/drivers/platform/chrome/chromeos_acpi.c b/drivers/platform/chrome/chromeos_acpi.c
index 50d8a4d4352d..1cc01d893ada 100644
--- a/drivers/platform/chrome/chromeos_acpi.c
+++ b/drivers/platform/chrome/chromeos_acpi.c
@@ -235,9 +235,9 @@ static int chromeos_acpi_device_probe(struct platform_device *pdev)
return 0;
}
-/* GGL is valid PNP ID of Google. PNP ID can be used with the ACPI devices. */
static const struct acpi_device_id chromeos_device_ids[] = {
{ "GGL0001", 0 },
+ { "GOOG0016", 0 },
{}
};
MODULE_DEVICE_TABLE(acpi, chromeos_device_ids);
--
2.41.0.487.g6d72f3e995-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
2023-07-31 2:42 [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016 Tzung-Bi Shih
@ 2023-07-31 7:24 ` Muhammad Usama Anjum
2023-07-31 17:07 ` Simon Glass
2023-07-31 16:53 ` Guenter Roeck
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Muhammad Usama Anjum @ 2023-07-31 7:24 UTC (permalink / raw
To: Tzung-Bi Shih, bleung, groeck, rafael, lenb
Cc: Muhammad Usama Anjum, guillaume.tucker, denys.f, ricardo.canuelo,
chrome-platform, linux-acpi
On 7/31/23 7:42 AM, Tzung-Bi Shih wrote:
> Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
>
> [1]: https://crrev.com/c/2266713
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Muhammad Usama Anjum <usama.anjum@collabora.com>
> ---
> Changes from v2[2]:
> - Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
>
> Changes from v1[3]:
> - Preserve the PNP ID GGL0001 for backward compatible to older firmwares.
> - Modify commit title and message accordingly.
>
> [2]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728065438.4026629-1-tzungbi@kernel.org/
> [3]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728014256.3836298-1-tzungbi@kernel.org/
>
> .../ABI/testing/sysfs-driver-chromeos-acpi | 15 +++++++++++++++
> .../firmware-guide/acpi/chromeos-acpi-device.rst | 5 ++---
> drivers/platform/chrome/chromeos_acpi.c | 2 +-
> 3 files changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> index c308926e1568..22082f9a7922 100644
> --- a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> +++ b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> @@ -1,4 +1,5 @@
> What: /sys/bus/platform/devices/GGL0001:*/BINF.2
> + /sys/bus/platform/devices/GOOG0016:*/BINF.2
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -10,6 +11,7 @@ Description:
> == ===============================
>
> What: /sys/bus/platform/devices/GGL0001:*/BINF.3
> + /sys/bus/platform/devices/GOOG0016:*/BINF.3
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -23,6 +25,7 @@ Description:
> == =====================================
>
> What: /sys/bus/platform/devices/GGL0001:*/CHSW
> + /sys/bus/platform/devices/GOOG0016:*/CHSW
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -38,6 +41,7 @@ Description:
> ==== ===========================================
>
> What: /sys/bus/platform/devices/GGL0001:*/FMAP
> + /sys/bus/platform/devices/GOOG0016:*/FMAP
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -45,6 +49,7 @@ Description:
> processor firmware flashmap.
>
> What: /sys/bus/platform/devices/GGL0001:*/FRID
> + /sys/bus/platform/devices/GOOG0016:*/FRID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -52,6 +57,7 @@ Description:
> main processor firmware.
>
> What: /sys/bus/platform/devices/GGL0001:*/FWID
> + /sys/bus/platform/devices/GOOG0016:*/FWID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -59,6 +65,7 @@ Description:
> main processor firmware.
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.0
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.0
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -73,6 +80,7 @@ Description:
> =========== ==================================
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.1
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.1
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -84,6 +92,7 @@ Description:
> == =======================
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.2
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.2
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -91,18 +100,21 @@ Description:
> controller.
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.3
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.3
> Date: May 2022
> KernelVersion: 5.19
> Description:
> Returns name of the GPIO controller.
>
> What: /sys/bus/platform/devices/GGL0001:*/HWID
> + /sys/bus/platform/devices/GOOG0016:*/HWID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> Returns hardware ID for the Chromebook.
>
> What: /sys/bus/platform/devices/GGL0001:*/MECK
> + /sys/bus/platform/devices/GOOG0016:*/MECK
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -113,6 +125,7 @@ Description:
> present, or if the firmware was unable to read the extended registers, this buffer size can be zero.
>
> What: /sys/bus/platform/devices/GGL0001:*/VBNV.0
> + /sys/bus/platform/devices/GOOG0016:*/VBNV.0
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -122,6 +135,7 @@ Description:
> clock data).
>
> What: /sys/bus/platform/devices/GGL0001:*/VBNV.1
> + /sys/bus/platform/devices/GOOG0016:*/VBNV.1
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -129,6 +143,7 @@ Description:
> storage block.
>
> What: /sys/bus/platform/devices/GGL0001:*/VDAT
> + /sys/bus/platform/devices/GOOG0016:*/VDAT
> Date: May 2022
> KernelVersion: 5.19
> Description:
> diff --git a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> index f37fc90ce340..89419e116413 100644
> --- a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> +++ b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> @@ -5,9 +5,8 @@ Chrome OS ACPI Device
> =====================
>
> Hardware functionality specific to Chrome OS is exposed through a Chrome OS ACPI device.
> -The plug and play ID of a Chrome OS ACPI device is GGL0001. GGL is a valid PNP ID of Google.
> -PNP ID can be used with the ACPI devices according to the guidelines. The following ACPI
> -objects are supported:
> +The plug and play ID of a Chrome OS ACPI device is GGL0001 and the hardware ID is
> +GOOG0016. The following ACPI objects are supported:
>
> .. flat-table:: Supported ACPI Objects
> :widths: 1 2
> diff --git a/drivers/platform/chrome/chromeos_acpi.c b/drivers/platform/chrome/chromeos_acpi.c
> index 50d8a4d4352d..1cc01d893ada 100644
> --- a/drivers/platform/chrome/chromeos_acpi.c
> +++ b/drivers/platform/chrome/chromeos_acpi.c
> @@ -235,9 +235,9 @@ static int chromeos_acpi_device_probe(struct platform_device *pdev)
> return 0;
> }
>
> -/* GGL is valid PNP ID of Google. PNP ID can be used with the ACPI devices. */
> static const struct acpi_device_id chromeos_device_ids[] = {
> { "GGL0001", 0 },
> + { "GOOG0016", 0 },
> {}
> };
> MODULE_DEVICE_TABLE(acpi, chromeos_device_ids);
--
BR,
Muhammad Usama Anjum
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
2023-07-31 2:42 [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016 Tzung-Bi Shih
2023-07-31 7:24 ` Muhammad Usama Anjum
@ 2023-07-31 16:53 ` Guenter Roeck
2023-08-07 3:10 ` patchwork-bot+chrome-platform
2023-08-08 2:40 ` patchwork-bot+chrome-platform
3 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2023-07-31 16:53 UTC (permalink / raw
To: Tzung-Bi Shih
Cc: bleung, groeck, rafael, lenb, guillaume.tucker, denys.f,
ricardo.canuelo, usama.anjum, chrome-platform, linux-acpi
On Sun, Jul 30, 2023 at 7:43 PM Tzung-Bi Shih <tzungbi@kernel.org> wrote:
>
> Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
>
> [1]: https://crrev.com/c/2266713
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Guenter Roeck <groeck@chromium.org>
> ---
> Changes from v2[2]:
> - Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
>
> Changes from v1[3]:
> - Preserve the PNP ID GGL0001 for backward compatible to older firmwares.
> - Modify commit title and message accordingly.
>
> [2]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728065438.4026629-1-tzungbi@kernel.org/
> [3]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728014256.3836298-1-tzungbi@kernel.org/
>
> .../ABI/testing/sysfs-driver-chromeos-acpi | 15 +++++++++++++++
> .../firmware-guide/acpi/chromeos-acpi-device.rst | 5 ++---
> drivers/platform/chrome/chromeos_acpi.c | 2 +-
> 3 files changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> index c308926e1568..22082f9a7922 100644
> --- a/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> +++ b/Documentation/ABI/testing/sysfs-driver-chromeos-acpi
> @@ -1,4 +1,5 @@
> What: /sys/bus/platform/devices/GGL0001:*/BINF.2
> + /sys/bus/platform/devices/GOOG0016:*/BINF.2
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -10,6 +11,7 @@ Description:
> == ===============================
>
> What: /sys/bus/platform/devices/GGL0001:*/BINF.3
> + /sys/bus/platform/devices/GOOG0016:*/BINF.3
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -23,6 +25,7 @@ Description:
> == =====================================
>
> What: /sys/bus/platform/devices/GGL0001:*/CHSW
> + /sys/bus/platform/devices/GOOG0016:*/CHSW
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -38,6 +41,7 @@ Description:
> ==== ===========================================
>
> What: /sys/bus/platform/devices/GGL0001:*/FMAP
> + /sys/bus/platform/devices/GOOG0016:*/FMAP
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -45,6 +49,7 @@ Description:
> processor firmware flashmap.
>
> What: /sys/bus/platform/devices/GGL0001:*/FRID
> + /sys/bus/platform/devices/GOOG0016:*/FRID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -52,6 +57,7 @@ Description:
> main processor firmware.
>
> What: /sys/bus/platform/devices/GGL0001:*/FWID
> + /sys/bus/platform/devices/GOOG0016:*/FWID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -59,6 +65,7 @@ Description:
> main processor firmware.
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.0
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.0
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -73,6 +80,7 @@ Description:
> =========== ==================================
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.1
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.1
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -84,6 +92,7 @@ Description:
> == =======================
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.2
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.2
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -91,18 +100,21 @@ Description:
> controller.
>
> What: /sys/bus/platform/devices/GGL0001:*/GPIO.X/GPIO.3
> + /sys/bus/platform/devices/GOOG0016:*/GPIO.X/GPIO.3
> Date: May 2022
> KernelVersion: 5.19
> Description:
> Returns name of the GPIO controller.
>
> What: /sys/bus/platform/devices/GGL0001:*/HWID
> + /sys/bus/platform/devices/GOOG0016:*/HWID
> Date: May 2022
> KernelVersion: 5.19
> Description:
> Returns hardware ID for the Chromebook.
>
> What: /sys/bus/platform/devices/GGL0001:*/MECK
> + /sys/bus/platform/devices/GOOG0016:*/MECK
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -113,6 +125,7 @@ Description:
> present, or if the firmware was unable to read the extended registers, this buffer size can be zero.
>
> What: /sys/bus/platform/devices/GGL0001:*/VBNV.0
> + /sys/bus/platform/devices/GOOG0016:*/VBNV.0
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -122,6 +135,7 @@ Description:
> clock data).
>
> What: /sys/bus/platform/devices/GGL0001:*/VBNV.1
> + /sys/bus/platform/devices/GOOG0016:*/VBNV.1
> Date: May 2022
> KernelVersion: 5.19
> Description:
> @@ -129,6 +143,7 @@ Description:
> storage block.
>
> What: /sys/bus/platform/devices/GGL0001:*/VDAT
> + /sys/bus/platform/devices/GOOG0016:*/VDAT
> Date: May 2022
> KernelVersion: 5.19
> Description:
> diff --git a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> index f37fc90ce340..89419e116413 100644
> --- a/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> +++ b/Documentation/firmware-guide/acpi/chromeos-acpi-device.rst
> @@ -5,9 +5,8 @@ Chrome OS ACPI Device
> =====================
>
> Hardware functionality specific to Chrome OS is exposed through a Chrome OS ACPI device.
> -The plug and play ID of a Chrome OS ACPI device is GGL0001. GGL is a valid PNP ID of Google.
> -PNP ID can be used with the ACPI devices according to the guidelines. The following ACPI
> -objects are supported:
> +The plug and play ID of a Chrome OS ACPI device is GGL0001 and the hardware ID is
> +GOOG0016. The following ACPI objects are supported:
>
> .. flat-table:: Supported ACPI Objects
> :widths: 1 2
> diff --git a/drivers/platform/chrome/chromeos_acpi.c b/drivers/platform/chrome/chromeos_acpi.c
> index 50d8a4d4352d..1cc01d893ada 100644
> --- a/drivers/platform/chrome/chromeos_acpi.c
> +++ b/drivers/platform/chrome/chromeos_acpi.c
> @@ -235,9 +235,9 @@ static int chromeos_acpi_device_probe(struct platform_device *pdev)
> return 0;
> }
>
> -/* GGL is valid PNP ID of Google. PNP ID can be used with the ACPI devices. */
> static const struct acpi_device_id chromeos_device_ids[] = {
> { "GGL0001", 0 },
> + { "GOOG0016", 0 },
> {}
> };
> MODULE_DEVICE_TABLE(acpi, chromeos_device_ids);
> --
> 2.41.0.487.g6d72f3e995-goog
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
2023-07-31 7:24 ` Muhammad Usama Anjum
@ 2023-07-31 17:07 ` Simon Glass
0 siblings, 0 replies; 6+ messages in thread
From: Simon Glass @ 2023-07-31 17:07 UTC (permalink / raw
To: Muhammad Usama Anjum
Cc: Tzung-Bi Shih, bleung, groeck, rafael, lenb, guillaume.tucker,
denys.f, ricardo.canuelo, chrome-platform, linux-acpi
On Mon, 31 Jul 2023 at 01:29, Muhammad Usama Anjum
<usama.anjum@collabora.com> wrote:
>
> On 7/31/23 7:42 AM, Tzung-Bi Shih wrote:
> > Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
> >
> > [1]: https://crrev.com/c/2266713
> >
> > Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> Reviewed-by: Muhammad Usama Anjum <usama.anjum@collabora.com>
>
> > ---
> > Changes from v2[2]:
> > - Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
> >
> > Changes from v1[3]:
> > - Preserve the PNP ID GGL0001 for backward compatible to older firmwares.
> > - Modify commit title and message accordingly.
> >
> > [2]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728065438.4026629-1-tzungbi@kernel.org/
> > [3]: https://patchwork.kernel.org/project/chrome-platform/patch/20230728014256.3836298-1-tzungbi@kernel.org/
> >
> > .../ABI/testing/sysfs-driver-chromeos-acpi | 15 +++++++++++++++
> > .../firmware-guide/acpi/chromeos-acpi-device.rst | 5 ++---
> > drivers/platform/chrome/chromeos_acpi.c | 2 +-
> > 3 files changed, 18 insertions(+), 4 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
2023-07-31 2:42 [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016 Tzung-Bi Shih
2023-07-31 7:24 ` Muhammad Usama Anjum
2023-07-31 16:53 ` Guenter Roeck
@ 2023-08-07 3:10 ` patchwork-bot+chrome-platform
2023-08-08 2:40 ` patchwork-bot+chrome-platform
3 siblings, 0 replies; 6+ messages in thread
From: patchwork-bot+chrome-platform @ 2023-08-07 3:10 UTC (permalink / raw
To: Tzung-Bi Shih
Cc: bleung, groeck, rafael, lenb, guillaume.tucker, denys.f,
ricardo.canuelo, usama.anjum, chrome-platform, linux-acpi
Hello:
This patch was applied to chrome-platform/linux.git (for-kernelci)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Mon, 31 Jul 2023 10:42:14 +0800 you wrote:
> Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
>
> [1]: https://crrev.com/c/2266713
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
> Changes from v2[2]:
> - Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
>
> [...]
Here is the summary with links:
- [v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
https://git.kernel.org/chrome-platform/c/703e77134ebf
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
2023-07-31 2:42 [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016 Tzung-Bi Shih
` (2 preceding siblings ...)
2023-08-07 3:10 ` patchwork-bot+chrome-platform
@ 2023-08-08 2:40 ` patchwork-bot+chrome-platform
3 siblings, 0 replies; 6+ messages in thread
From: patchwork-bot+chrome-platform @ 2023-08-08 2:40 UTC (permalink / raw
To: Tzung-Bi Shih
Cc: bleung, groeck, rafael, lenb, guillaume.tucker, denys.f,
ricardo.canuelo, usama.anjum, chrome-platform, linux-acpi
Hello:
This patch was applied to chrome-platform/linux.git (for-next)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Mon, 31 Jul 2023 10:42:14 +0800 you wrote:
> Support official HID GOOG0016 for ChromeOS ACPI (see [1]).
>
> [1]: https://crrev.com/c/2266713
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
> Changes from v2[2]:
> - Instead of replacing, appending "GOOG0016" in sysfs-driver-chromeos-acpi.
>
> [...]
Here is the summary with links:
- [v3] platform/chrome: chromeos_acpi: support official HID GOOG0016
https://git.kernel.org/chrome-platform/c/703e77134ebf
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-08-08 2:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-31 2:42 [PATCH v3] platform/chrome: chromeos_acpi: support official HID GOOG0016 Tzung-Bi Shih
2023-07-31 7:24 ` Muhammad Usama Anjum
2023-07-31 17:07 ` Simon Glass
2023-07-31 16:53 ` Guenter Roeck
2023-08-07 3:10 ` patchwork-bot+chrome-platform
2023-08-08 2:40 ` patchwork-bot+chrome-platform
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).