From nobody Thu May 7 17:21:54 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E1242C433EF for ; Mon, 23 May 2022 09:08:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232678AbiEWJI6 (ORCPT ); Mon, 23 May 2022 05:08:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47612 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232655AbiEWJIy (ORCPT ); Mon, 23 May 2022 05:08:54 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D91784505F; Mon, 23 May 2022 02:08:53 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 8A6E0B80F2B; Mon, 23 May 2022 09:08:52 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A9593C34117; Mon, 23 May 2022 09:08:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653296931; bh=sPsMNa6xJuOBGkIQpMmbCnLS4e+pUl2hEV+vBFGxYGs=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=es5fHf3IfoXFFBf4QCe+ASCKKFn9V0es2k6FoFrvTDA98CI05H6v0eKiQEqSaMlXq yIEU8EXC814Ill8dlmZrFRwYHYgZG6ohpnnukzz1R17oKTTjR02WsJCVsLVUzexFjR 46eBPtWQxgRgqSQVeijmJCzEWdQxqj3sJChpt1GXpJdqGiQOykkxgjwgEIJ6Oo6UPj MKaf8bg72ahSSo4FfPrTtJ2eJplLj2gOq8yz0fa8lcpRR26ZCbQ7PDNC0Kg+NJCrqU XyTfKiiCJH0PbfCPh5C4b5C46nSqd8O1CExABwR0kcUJt9uyZSuEXqpIsimpWyQ0w4 lp2qbtBeUrxWw== From: Tzung-Bi Shih To: bleung@chromium.org, groeck@chromium.org, robh+dt@kernel.org Cc: chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, mka@chromium.org, devicetree@vger.kernel.org, tzungbi@kernel.org, Guenter Roeck Subject: [PATCH v4 1/5] platform/chrome: cros_kbd_led_backlight: sort headers alphabetically Date: Mon, 23 May 2022 17:08:18 +0800 Message-Id: <20220523090822.3035189-2-tzungbi@kernel.org> X-Mailer: git-send-email 2.36.1.124.g0e6072fb45-goog In-Reply-To: <20220523090822.3035189-1-tzungbi@kernel.org> References: <20220523090822.3035189-1-tzungbi@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" To be neat and reduce conflict possibility, sort the headers alphabetically. Reviewed-by: Guenter Roeck Signed-off-by: Tzung-Bi Shih --- No changes from v3. Changes from v2: - Add commit message. - Add R-b tag. Changes from v1: - Update email address accordingly. drivers/platform/chrome/cros_kbd_led_backlight.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/pla= tform/chrome/cros_kbd_led_backlight.c index aa409f0201fb..f9587a562bb7 100644 --- a/drivers/platform/chrome/cros_kbd_led_backlight.c +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c @@ -4,12 +4,12 @@ // Copyright (C) 2012 Google, Inc. =20 #include -#include #include #include -#include #include #include +#include +#include #include #include =20 --=20 2.36.1.124.g0e6072fb45-goog From nobody Thu May 7 17:21:54 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 36FA1C433EF for ; Mon, 23 May 2022 09:09:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232712AbiEWJJB (ORCPT ); Mon, 23 May 2022 05:09:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47694 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232665AbiEWJI5 (ORCPT ); Mon, 23 May 2022 05:08:57 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F34A945058; Mon, 23 May 2022 02:08:55 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 6D58AB80FA6; Mon, 23 May 2022 09:08:54 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B158BC3411A; Mon, 23 May 2022 09:08:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653296933; bh=j0siKCr2t3cIHVL6rBUrteD0xCpUE62iGsH1eC3VLAI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=jqCaTQ73ncMw9xmIs8VYv2wtzlXx5fSyGOHnUJ5Hpij46D05mDLnC2zukvvOjkMCh c7xHhFz4T1lZJQaNDxfafUQ2Y9+AFox4GlGtIpbvYFqgL0Kz6gpWj8DCQaoRZevw1d XZz/IxelExgPj32+gyjdSlZirrHN56oRPBJPJAE1+1iXqvC4Sfo0hgH+kOwVoOpZ2p QBiZgfNWfTU1NMLXupfHHIVd5ugRlLS+jGMgain4TzZLpIO9J91mzqAoeBypMtcTnQ YdfVoR2Kz8FFk20Vwmgm9SwMpkgxaw99nEmZO/xLOquyKGVHlE/Sb7oBvJeQftHAYW BfmF72QwCVVCw== From: Tzung-Bi Shih To: bleung@chromium.org, groeck@chromium.org, robh+dt@kernel.org Cc: chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, mka@chromium.org, devicetree@vger.kernel.org, tzungbi@kernel.org Subject: [PATCH v4 2/5] platform/chrome: cros_kbd_led_backlight: separate ACPI backend Date: Mon, 23 May 2022 17:08:19 +0800 Message-Id: <20220523090822.3035189-3-tzungbi@kernel.org> X-Mailer: git-send-email 2.36.1.124.g0e6072fb45-goog In-Reply-To: <20220523090822.3035189-1-tzungbi@kernel.org> References: <20220523090822.3035189-1-tzungbi@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" cros_kbd_led_backlight uses ACPI_KEYBOARD_BACKLIGHT_WRITE and ACPI_KEYBOARD_BACKLIGHT_READ for setting and getting the brightness respectively. Separate ACPI operations for preparing the driver to support other backends. Signed-off-by: Tzung-Bi Shih Reviewed-by: Guenter Roeck --- Changes from v3: - Remove CROS_KBD_LED_BACKLIGHT_ACPI Kconfig. - Remove stub function. Changes from v2: - Use #ifdef for boolean CONFIG_CROS_KBD_LED_BACKLIGHT_ACPI. Changes from v1: - Update email address accordingly. - Use CONFIG_ACPI guard per "kernel test robot " reported an unused variable issue. .../platform/chrome/cros_kbd_led_backlight.c | 82 ++++++++++++++++--- 1 file changed, 69 insertions(+), 13 deletions(-) diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/pla= tform/chrome/cros_kbd_led_backlight.c index f9587a562bb7..a86d664854ae 100644 --- a/drivers/platform/chrome/cros_kbd_led_backlight.c +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c @@ -13,6 +13,33 @@ #include #include =20 +/** + * struct keyboard_led_drvdata - keyboard LED driver data. + * @init: Init function. + * @brightness_get: Get LED brightness level. + * @brightness_set: Set LED brightness level. Must not sleep. + * @brightness_set_blocking: Set LED brightness level. It can block the + * caller for the time required for accessing a + * LED device register + * @max_brightness: Maximum brightness. + * + * See struct led_classdev in include/linux/leds.h for more details. + */ +struct keyboard_led_drvdata { + int (*init)(struct platform_device *pdev); + + enum led_brightness (*brightness_get)(struct led_classdev *led_cdev); + + void (*brightness_set)(struct led_classdev *led_cdev, + enum led_brightness brightness); + int (*brightness_set_blocking)(struct led_classdev *led_cdev, + enum led_brightness brightness); + + enum led_brightness max_brightness; +}; + +#ifdef CONFIG_ACPI + /* Keyboard LED ACPI Device must be defined in firmware */ #define ACPI_KEYBOARD_BACKLIGHT_DEVICE "\\_SB.KBLT" #define ACPI_KEYBOARD_BACKLIGHT_READ ACPI_KEYBOARD_BACKLIGHT_DEVICE ".KBQC" @@ -20,8 +47,8 @@ =20 #define ACPI_KEYBOARD_BACKLIGHT_MAX 100 =20 -static void keyboard_led_set_brightness(struct led_classdev *cdev, - enum led_brightness brightness) +static void keyboard_led_set_brightness_acpi(struct led_classdev *cdev, + enum led_brightness brightness) { union acpi_object param; struct acpi_object_list input; @@ -40,7 +67,7 @@ static void keyboard_led_set_brightness(struct led_classd= ev *cdev, } =20 static enum led_brightness -keyboard_led_get_brightness(struct led_classdev *cdev) +keyboard_led_get_brightness_acpi(struct led_classdev *cdev) { unsigned long long brightness; acpi_status status; @@ -56,12 +83,10 @@ keyboard_led_get_brightness(struct led_classdev *cdev) return brightness; } =20 -static int keyboard_led_probe(struct platform_device *pdev) +static int keyboard_led_init_acpi(struct platform_device *pdev) { - struct led_classdev *cdev; acpi_handle handle; acpi_status status; - int error; =20 /* Look for the keyboard LED ACPI Device */ status =3D acpi_get_handle(ACPI_ROOT_OBJECT, @@ -73,15 +98,44 @@ static int keyboard_led_probe(struct platform_device *p= dev) return -ENXIO; } =20 + return 0; +} + +static const struct keyboard_led_drvdata keyboard_led_drvdata_acpi =3D { + .init =3D keyboard_led_init_acpi, + .brightness_set =3D keyboard_led_set_brightness_acpi, + .brightness_get =3D keyboard_led_get_brightness_acpi, + .max_brightness =3D ACPI_KEYBOARD_BACKLIGHT_MAX, +}; + +#endif /* CONFIG_ACPI */ + +static int keyboard_led_probe(struct platform_device *pdev) +{ + struct led_classdev *cdev; + const struct keyboard_led_drvdata *drvdata; + int error; + + drvdata =3D acpi_device_get_match_data(&pdev->dev); + if (!drvdata) + return -EINVAL; + + if (drvdata->init) { + error =3D drvdata->init(pdev); + if (error) + return error; + } + cdev =3D devm_kzalloc(&pdev->dev, sizeof(*cdev), GFP_KERNEL); if (!cdev) return -ENOMEM; =20 cdev->name =3D "chromeos::kbd_backlight"; - cdev->max_brightness =3D ACPI_KEYBOARD_BACKLIGHT_MAX; cdev->flags |=3D LED_CORE_SUSPENDRESUME; - cdev->brightness_set =3D keyboard_led_set_brightness; - cdev->brightness_get =3D keyboard_led_get_brightness; + cdev->max_brightness =3D drvdata->max_brightness; + cdev->brightness_set =3D drvdata->brightness_set; + cdev->brightness_set_blocking =3D drvdata->brightness_set_blocking; + cdev->brightness_get =3D drvdata->brightness_get; =20 error =3D devm_led_classdev_register(&pdev->dev, cdev); if (error) @@ -90,16 +144,18 @@ static int keyboard_led_probe(struct platform_device *= pdev) return 0; } =20 -static const struct acpi_device_id keyboard_led_id[] =3D { - { "GOOG0002", 0 }, +#ifdef CONFIG_ACPI +static const struct acpi_device_id keyboard_led_acpi_match[] =3D { + { "GOOG0002", (kernel_ulong_t)&keyboard_led_drvdata_acpi }, { } }; -MODULE_DEVICE_TABLE(acpi, keyboard_led_id); +MODULE_DEVICE_TABLE(acpi, keyboard_led_acpi_match); +#endif =20 static struct platform_driver keyboard_led_driver =3D { .driver =3D { .name =3D "chromeos-keyboard-leds", - .acpi_match_table =3D ACPI_PTR(keyboard_led_id), + .acpi_match_table =3D ACPI_PTR(keyboard_led_acpi_match), }, .probe =3D keyboard_led_probe, }; --=20 2.36.1.124.g0e6072fb45-goog From nobody Thu May 7 17:21:54 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F3CB7C433F5 for ; Mon, 23 May 2022 09:09:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232724AbiEWJJF (ORCPT ); Mon, 23 May 2022 05:09:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47698 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232676AbiEWJI5 (ORCPT ); Mon, 23 May 2022 05:08:57 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 379BE4507C; Mon, 23 May 2022 02:08:56 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id C9B2F60FC7; Mon, 23 May 2022 09:08:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 92E38C34119; Mon, 23 May 2022 09:08:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653296935; bh=8566TNAIYuMPUSqB2VKigugwmemkLIms0TpE+9DCQW8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Fm+XiAo8zTvPBLqbwfdmXIgNP8eJyeTPyGy4wvfZ7wrzE4YL16cwW1xi2cm+k/2H2 fRcgwiRuHSKVEoi3rMvsNIEVAlazq7gu8t9TAxlm/vIld2SQuBHJQMgZNhkK9DeDV9 sRozIlXrL7j7rcb+/Pky9QemCwAv8WIWJjct6Cdnhibi/caz24ojdCgJKmD22fbMjL h8ezV21F3ZVhpRXVWCic1OwgovX9sqycm5NF3abi0I4yXnBq/W5YXbtuyMJuDMOXNc BVl7Nnqj0sUZkA/AP+kJokjk3yjrtSZfIWDm1gF7CIuSfZCVF4VC4NPqzhdnMIVyQo EB6ycSEc80c/g== From: Tzung-Bi Shih To: bleung@chromium.org, groeck@chromium.org, robh+dt@kernel.org Cc: chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, mka@chromium.org, devicetree@vger.kernel.org, tzungbi@kernel.org, Rob Herring Subject: [PATCH v4 3/5] dt-bindings: add google,cros-kbd-led-backlight Date: Mon, 23 May 2022 17:08:20 +0800 Message-Id: <20220523090822.3035189-4-tzungbi@kernel.org> X-Mailer: git-send-email 2.36.1.124.g0e6072fb45-goog In-Reply-To: <20220523090822.3035189-1-tzungbi@kernel.org> References: <20220523090822.3035189-1-tzungbi@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Acked-by: Rob Herring Signed-off-by: Tzung-Bi Shih --- No content changes from v3 but rebase to the latest Documentation/devicetree/bindings/mfd/google,cros-ec.yaml. No changes from v2. Changes from v1: (https://patchwork.kernel.org/project/chrome-platform/patch/20220214053646.= 3088298-4-tzungbi@google.com/) - Update email address accordingly. - Add A-b tag. .../chrome/google,cros-kbd-led-backlight.yaml | 35 +++++++++++++++++++ .../bindings/mfd/google,cros-ec.yaml | 3 ++ 2 files changed, 38 insertions(+) create mode 100644 Documentation/devicetree/bindings/chrome/google,cros-kb= d-led-backlight.yaml diff --git a/Documentation/devicetree/bindings/chrome/google,cros-kbd-led-b= acklight.yaml b/Documentation/devicetree/bindings/chrome/google,cros-kbd-le= d-backlight.yaml new file mode 100644 index 000000000000..5b875af6a95a --- /dev/null +++ b/Documentation/devicetree/bindings/chrome/google,cros-kbd-led-backligh= t.yaml @@ -0,0 +1,35 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/chrome/google,cros-kbd-led-backlight.ya= ml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: ChromeOS keyboard backlight LED driver. + +maintainers: + - Tzung-Bi Shih + +properties: + compatible: + const: google,cros-kbd-led-backlight + +required: + - compatible + +additionalProperties: false + +examples: + - | + spi0 { + #address-cells =3D <1>; + #size-cells =3D <0>; + + cros_ec: ec@0 { + compatible =3D "google,cros-ec-spi"; + reg =3D <0>; + + kbd-led-backlight { + compatible =3D "google,cros-kbd-led-backlight"; + }; + }; + }; diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Do= cumentation/devicetree/bindings/mfd/google,cros-ec.yaml index afec0bd2f1de..09e00adf445e 100644 --- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml +++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml @@ -90,6 +90,9 @@ properties: pwm: $ref: "/schemas/pwm/google,cros-ec-pwm.yaml#" =20 + kbd-led-backlight: + $ref: "/schemas/chrome/google,cros-kbd-led-backlight.yaml#" + keyboard-controller: $ref: "/schemas/input/google,cros-ec-keyb.yaml#" =20 --=20 2.36.1.124.g0e6072fb45-goog From nobody Thu May 7 17:21:54 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7E08DC433F5 for ; Mon, 23 May 2022 09:09:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232663AbiEWJJK (ORCPT ); Mon, 23 May 2022 05:09:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47762 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232695AbiEWJI7 (ORCPT ); Mon, 23 May 2022 05:08:59 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3C15445062; Mon, 23 May 2022 02:08:58 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id CBF8560F84; Mon, 23 May 2022 09:08:57 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 99B52C385A9; Mon, 23 May 2022 09:08:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653296937; bh=Q+d3XF3B7LVnN1nYfZWpSlRiPE0fKGy1w0hTJi3wqlg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=SYg6xRayoZlRZAxXhNbLc52kS1I/F99HpDreqKFZ9LDgacRePlG8Is+xcnpqxAhMd 6Q61KsXgRPA96C6PCWapoXO8lW5Lo1YMla00HbbFnl2y655w5Xu5Ab2oqVQA8h9yHn CksedTBYu8EAIlmgJQVGet+9Da6V9Fq+wc+Tfo5S56pbM0jrIPDjNe1LAR+8Lk3xoS ADfyvZ7qdRZ968ArSRrMlw3xEIjIYrBwypnxaZ1xL9hkuQ3OtaTr4RcByeV3QUhhjn CLlzcKx4hDcoJ2vNlCULO8mi2G3HltJ8JEBcQ/WTjHlSfjC+r5vuxYkbqzuKa0d9fT 2ZIuXznbWC/qQ== From: Tzung-Bi Shih To: bleung@chromium.org, groeck@chromium.org, robh+dt@kernel.org Cc: chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, mka@chromium.org, devicetree@vger.kernel.org, tzungbi@kernel.org, Guenter Roeck Subject: [PATCH v4 4/5] platform/chrome: cros_kbd_led_backlight: support OF match Date: Mon, 23 May 2022 17:08:21 +0800 Message-Id: <20220523090822.3035189-5-tzungbi@kernel.org> X-Mailer: git-send-email 2.36.1.124.g0e6072fb45-goog In-Reply-To: <20220523090822.3035189-1-tzungbi@kernel.org> References: <20220523090822.3035189-1-tzungbi@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" For letting device tree based machines to use the driver, support OF match. Reviewed-by: Guenter Roeck Signed-off-by: Tzung-Bi Shih Reviewed-by: Matthias Kaehlcke --- No changes from v3. Changes from v2: - Add commit message. - Add R-b tag. Changes from v1: (https://patchwork.kernel.org/project/chrome-platform/patch/20220214053646.= 3088298-5-tzungbi@google.com/) - Update email address accordingly. - Use device_get_match_data() per review comment in v1. drivers/platform/chrome/cros_kbd_led_backlight.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/pla= tform/chrome/cros_kbd_led_backlight.c index a86d664854ae..4bca880d7721 100644 --- a/drivers/platform/chrome/cros_kbd_led_backlight.c +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c @@ -10,7 +10,9 @@ #include #include #include +#include #include +#include #include =20 /** @@ -116,7 +118,7 @@ static int keyboard_led_probe(struct platform_device *p= dev) const struct keyboard_led_drvdata *drvdata; int error; =20 - drvdata =3D acpi_device_get_match_data(&pdev->dev); + drvdata =3D device_get_match_data(&pdev->dev); if (!drvdata) return -EINVAL; =20 @@ -152,10 +154,21 @@ static const struct acpi_device_id keyboard_led_acpi_= match[] =3D { MODULE_DEVICE_TABLE(acpi, keyboard_led_acpi_match); #endif =20 +#ifdef CONFIG_OF +static const struct of_device_id keyboard_led_of_match[] =3D { + { + .compatible =3D "google,cros-kbd-led-backlight", + }, + {} +}; +MODULE_DEVICE_TABLE(of, keyboard_led_of_match); +#endif + static struct platform_driver keyboard_led_driver =3D { .driver =3D { .name =3D "chromeos-keyboard-leds", .acpi_match_table =3D ACPI_PTR(keyboard_led_acpi_match), + .of_match_table =3D of_match_ptr(keyboard_led_of_match), }, .probe =3D keyboard_led_probe, }; --=20 2.36.1.124.g0e6072fb45-goog From nobody Thu May 7 17:21:54 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 790D6C433F5 for ; Mon, 23 May 2022 09:09:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232744AbiEWJJP (ORCPT ); Mon, 23 May 2022 05:09:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47916 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232714AbiEWJJC (ORCPT ); Mon, 23 May 2022 05:09:02 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5FD1A4507C; Mon, 23 May 2022 02:09:00 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id B0B7860F84; Mon, 23 May 2022 09:08:59 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9E960C36AE5; Mon, 23 May 2022 09:08:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653296939; bh=rIFpUgFbYYJCuRyZO6yCCt9+l07s4JFTgK0X7jWKLI4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=MdspXEm0IsQ7B80hm50yfU69NDLgB3p9iEgoabB4MgMTng5sAnqG4ZsBmD9HCK2tz cDSunYbb+w5e7/TYDKKiHYg9L/pOlzIGVja0dBjN5isALVtD0N69bLK2toyKascN/z OzCPGiDNvhy6/JNPEwIZQCyisR3c/MGqXAECE5drFzmG2MOost9fmCmSTvUjkodnsH zPC+Wtq/GYsGO0JMfq+7AhqfHlROBjY450T3PmIvmCLf1NGBFm+mRiFmN5e4c40HRf s+zYwSs1SIn50Q+LURSLVzXF50PTnSSll3R15oBIdobimRZxuYa81A/l0gYq9kqcnG Pyv+g8ZU9IUeg== From: Tzung-Bi Shih To: bleung@chromium.org, groeck@chromium.org, robh+dt@kernel.org Cc: chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, mka@chromium.org, devicetree@vger.kernel.org, tzungbi@kernel.org Subject: [PATCH v4 5/5] platform/chrome: cros_kbd_led_backlight: support EC PWM backend Date: Mon, 23 May 2022 17:08:22 +0800 Message-Id: <20220523090822.3035189-6-tzungbi@kernel.org> X-Mailer: git-send-email 2.36.1.124.g0e6072fb45-goog In-Reply-To: <20220523090822.3035189-1-tzungbi@kernel.org> References: <20220523090822.3035189-1-tzungbi@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" EC PWM backend uses EC_CMD_PWM_SET_KEYBOARD_BACKLIGHT and EC_CMD_PWM_GET_KEYBOARD_BACKLIGHT for setting and getting the brightness respectively. Signed-off-by: Tzung-Bi Shih Reviewed-by: Matthias Kaehlcke Tested-by: Matthias Kaehlcke --- Changes from v3: - Remove CROS_KBD_LED_BACKLIGHT_EC_PWM Kconfig. - Remove stub function. - Remove redundant variable assignments. - Remove the "private" term. Changes from v2: - Turn CROS_KBD_LED_BACKLIGHT_EC_PWM to boolean. - Use #ifdef for boolean CROS_KBD_LED_BACKLIGHT_EC_PWM. Changes from v1: - Update email address accordingly. drivers/platform/chrome/Kconfig | 2 +- .../platform/chrome/cros_kbd_led_backlight.c | 113 +++++++++++++++--- 2 files changed, 99 insertions(+), 16 deletions(-) diff --git a/drivers/platform/chrome/Kconfig b/drivers/platform/chrome/Kcon= fig index 717299cbccac..3caf3194e3f5 100644 --- a/drivers/platform/chrome/Kconfig +++ b/drivers/platform/chrome/Kconfig @@ -139,7 +139,7 @@ config CROS_EC_PROTO =20 config CROS_KBD_LED_BACKLIGHT tristate "Backlight LED support for Chrome OS keyboards" - depends on LEDS_CLASS && ACPI + depends on LEDS_CLASS && (ACPI || CROS_EC) help This option enables support for the keyboard backlight LEDs on select Chrome OS systems. diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/pla= tform/chrome/cros_kbd_led_backlight.c index 4bca880d7721..5ad41c10412d 100644 --- a/drivers/platform/chrome/cros_kbd_led_backlight.c +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c @@ -11,10 +11,17 @@ #include #include #include +#include +#include #include #include #include =20 +struct keyboard_led { + struct led_classdev cdev; + struct cros_ec_device *ec; +}; + /** * struct keyboard_led_drvdata - keyboard LED driver data. * @init: Init function. @@ -40,6 +47,8 @@ struct keyboard_led_drvdata { enum led_brightness max_brightness; }; =20 +#define KEYBOARD_BACKLIGHT_MAX 100 + #ifdef CONFIG_ACPI =20 /* Keyboard LED ACPI Device must be defined in firmware */ @@ -47,8 +56,6 @@ struct keyboard_led_drvdata { #define ACPI_KEYBOARD_BACKLIGHT_READ ACPI_KEYBOARD_BACKLIGHT_DEVICE ".KBQC" #define ACPI_KEYBOARD_BACKLIGHT_WRITE ACPI_KEYBOARD_BACKLIGHT_DEVICE ".KBC= M" =20 -#define ACPI_KEYBOARD_BACKLIGHT_MAX 100 - static void keyboard_led_set_brightness_acpi(struct led_classdev *cdev, enum led_brightness brightness) { @@ -107,39 +114,114 @@ static const struct keyboard_led_drvdata keyboard_le= d_drvdata_acpi =3D { .init =3D keyboard_led_init_acpi, .brightness_set =3D keyboard_led_set_brightness_acpi, .brightness_get =3D keyboard_led_get_brightness_acpi, - .max_brightness =3D ACPI_KEYBOARD_BACKLIGHT_MAX, + .max_brightness =3D KEYBOARD_BACKLIGHT_MAX, }; =20 #endif /* CONFIG_ACPI */ =20 +#ifdef CONFIG_CROS_EC + +static int +keyboard_led_set_brightness_ec_pwm(struct led_classdev *cdev, + enum led_brightness brightness) +{ + struct { + struct cros_ec_command msg; + struct ec_params_pwm_set_keyboard_backlight params; + } __packed buf; + struct ec_params_pwm_set_keyboard_backlight *params =3D &buf.params; + struct cros_ec_command *msg =3D &buf.msg; + struct keyboard_led *keyboard_led =3D container_of(cdev, struct keyboard_= led, cdev); + + memset(&buf, 0, sizeof(buf)); + + msg->command =3D EC_CMD_PWM_SET_KEYBOARD_BACKLIGHT; + msg->outsize =3D sizeof(*params); + + params->percent =3D brightness; + + return cros_ec_cmd_xfer_status(keyboard_led->ec, msg); +} + +static enum led_brightness +keyboard_led_get_brightness_ec_pwm(struct led_classdev *cdev) +{ + struct { + struct cros_ec_command msg; + struct ec_response_pwm_get_keyboard_backlight resp; + } __packed buf; + struct ec_response_pwm_get_keyboard_backlight *resp =3D &buf.resp; + struct cros_ec_command *msg =3D &buf.msg; + struct keyboard_led *keyboard_led =3D container_of(cdev, struct keyboard_= led, cdev); + int ret; + + memset(&buf, 0, sizeof(buf)); + + msg->command =3D EC_CMD_PWM_GET_KEYBOARD_BACKLIGHT; + msg->insize =3D sizeof(*resp); + + ret =3D cros_ec_cmd_xfer_status(keyboard_led->ec, msg); + if (ret < 0) + return ret; + + return resp->percent; +} + +static int keyboard_led_init_ec_pwm(struct platform_device *pdev) +{ + struct keyboard_led *keyboard_led =3D platform_get_drvdata(pdev); + + keyboard_led->ec =3D dev_get_drvdata(pdev->dev.parent); + if (!keyboard_led->ec) { + dev_err(&pdev->dev, "no parent EC device\n"); + return -EINVAL; + } + + return 0; +} + +static const struct keyboard_led_drvdata keyboard_led_drvdata_ec_pwm =3D { + .init =3D keyboard_led_init_ec_pwm, + .brightness_set_blocking =3D keyboard_led_set_brightness_ec_pwm, + .brightness_get =3D keyboard_led_get_brightness_ec_pwm, + .max_brightness =3D KEYBOARD_BACKLIGHT_MAX, +}; + +#else /* CONFIG_CROS_EC */ + +static const struct keyboard_led_drvdata keyboard_led_drvdata_ec_pwm =3D {= }; + +#endif /* CONFIG_CROS_EC */ + static int keyboard_led_probe(struct platform_device *pdev) { - struct led_classdev *cdev; const struct keyboard_led_drvdata *drvdata; + struct keyboard_led *keyboard_led; int error; =20 drvdata =3D device_get_match_data(&pdev->dev); if (!drvdata) return -EINVAL; =20 + keyboard_led =3D devm_kzalloc(&pdev->dev, sizeof(*keyboard_led), GFP_KERN= EL); + if (!keyboard_led) + return -ENOMEM; + platform_set_drvdata(pdev, keyboard_led); + if (drvdata->init) { error =3D drvdata->init(pdev); if (error) return error; } =20 - cdev =3D devm_kzalloc(&pdev->dev, sizeof(*cdev), GFP_KERNEL); - if (!cdev) - return -ENOMEM; - - cdev->name =3D "chromeos::kbd_backlight"; - cdev->flags |=3D LED_CORE_SUSPENDRESUME; - cdev->max_brightness =3D drvdata->max_brightness; - cdev->brightness_set =3D drvdata->brightness_set; - cdev->brightness_set_blocking =3D drvdata->brightness_set_blocking; - cdev->brightness_get =3D drvdata->brightness_get; + keyboard_led->cdev.name =3D "chromeos::kbd_backlight"; + keyboard_led->cdev.flags |=3D LED_CORE_SUSPENDRESUME; + keyboard_led->cdev.max_brightness =3D drvdata->max_brightness; + keyboard_led->cdev.brightness_set =3D drvdata->brightness_set; + keyboard_led->cdev.brightness_set_blocking =3D drvdata->brightness_set_bl= ocking; + keyboard_led->cdev.brightness_get =3D drvdata->brightness_get; =20 - error =3D devm_led_classdev_register(&pdev->dev, cdev); + error =3D devm_led_classdev_register(&pdev->dev, &keyboard_led->cdev); if (error) return error; =20 @@ -158,6 +240,7 @@ MODULE_DEVICE_TABLE(acpi, keyboard_led_acpi_match); static const struct of_device_id keyboard_led_of_match[] =3D { { .compatible =3D "google,cros-kbd-led-backlight", + .data =3D &keyboard_led_drvdata_ec_pwm, }, {} }; --=20 2.36.1.124.g0e6072fb45-goog