Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Fabio Baltieri <fabiobaltieri@chromium.org>
To:     Benson Leung <bleung@chromium.org>,
        Guenter Roeck <groeck@chromium.org>
Cc:     Thierry Reding <thierry.reding@gmail.com>,
        =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= 
        <u.kleine-koenig@pengutronix.de>, Lee Jones <lee.jones@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        chrome-platform@lists.linux.dev, linux-pwm@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        Fabio Baltieri <fabiobaltieri@chromium.org>
Subject: [PATCH v3 0/4] Add channel type support to pwm-cros-ec
Date:   Mon, 11 Apr 2022 15:21:10 +0000
Message-Id: <20220411152114.2165933-1-fabiobaltieri@chromium.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hi,

The ChromiumOS EC PWM host command protocol supports specifying the
requested PWM by type rather than channel. [1]

This series adds support for specifying PWM by type rather than channel
number in the pwm-cros-ec driver, which abstracts the node definitions
from the actual hardware configuration from the kernel perspective,
aligns the API with the one used by the bootloader, and allows removing
some dtsi overrides.

Tested on a sc7180-trogdor board.

Changes from v2:
(https://patchwork.kernel.org/project/chrome-platform/list/?series=627837)
- reworded patch 2 commit description
- reworked the driver and dt documentation to use a new compatible rather than
  boolean property
- dropped the comment about build test only, tested on actual hardware
  (trogdor), build test on x86 (with CONFIG_OF=n).

Changes from v1:
(https://patchwork.kernel.org/project/chrome-platform/list/?series=625182)
- fixed the dt include file license
- fixed the property name (s/_/-/)
- rebased on current linus tree (few dts files changed from a soc tree
  pull, so patch 4 needs a recent base to apply correctly)

[1] https://source.chromium.org/chromiumos/chromiumos/codesearch/+/main:src/platform/ec/common/pwm.c;l=24
[2] https://source.chromium.org/chromiumos/chromiumos/codesearch/+/main:src/platform/depthcharge/src/drivers/ec/cros/ec.c;l=1271-1273

Fabio Baltieri (4):
  dt-bindings: add mfd/cros_ec definitions
  drivers: pwm: pwm-cros-ec: add channel type support
  dt-bindings: update google,cros-ec-pwm documentation
  arm64: dts: address cros-ec-pwm channels by type

 .../bindings/pwm/google,cros-ec-pwm.yaml      |   9 +-
 .../mt8183-kukui-jacuzzi-fennel-sku1.dts      |   4 +-
 .../dts/mediatek/mt8183-kukui-jacuzzi.dtsi    |   4 +-
 .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi |   1 +
 .../boot/dts/qcom/sc7180-trogdor-coachz.dtsi  |   4 -
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi  |   9 +-
 .../qcom/sc7280-herobrine-herobrine-r0.dts    |   7 +-
 .../arm64/boot/dts/qcom/sc7280-herobrine.dtsi |   7 +-
 .../arm64/boot/dts/qcom/sc7280-idp-ec-h1.dtsi |   4 +-
 arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi    |   7 +-
 .../boot/dts/rockchip/rk3399-gru-bob.dts      |   4 -
 .../dts/rockchip/rk3399-gru-chromebook.dtsi   |   5 +-
 .../boot/dts/rockchip/rk3399-gru-kevin.dts    |   4 -
 arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi  |   1 +
 drivers/pwm/pwm-cros-ec.c                     | 109 ++++++++++++++----
 include/dt-bindings/mfd/cros_ec.h             |  18 +++
 16 files changed, 140 insertions(+), 57 deletions(-)
 create mode 100644 include/dt-bindings/mfd/cros_ec.h

-- 
2.35.1.1178.g4f1659d476-goog