Re: [PATCH 1/3] dt-bindings: i2c: nuvoton,ma35d1-i2c: Add MA35D1 I2C controller
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-03-02 07:20:08
Also in:
linux-devicetree, linux-i2c
On Mon, Mar 02, 2026 at 02:08:20AM +0000, Zi-Yu Chen wrote:
quoted hunk ↗ jump to hunk
Add device tree binding documentation for the I2C controller found in the Nuvoton MA35D1 SoC. Signed-off-by: Zi-Yu Chen <redacted> --- .../bindings/i2c/nuvoton,ma35d1-i2c.yaml | 65 +++++++++++++++++++ 1 file changed, 65 insertions(+) create mode 100644 Documentation/devicetree/bindings/i2c/nuvoton,ma35d1-i2c.yamldiff --git a/Documentation/devicetree/bindings/i2c/nuvoton,ma35d1-i2c.yaml b/Documentation/devicetree/bindings/i2c/nuvoton,ma35d1-i2c.yaml new file mode 100644 index 000000000000..fa8b01e2c5b1 --- /dev/null +++ b/Documentation/devicetree/bindings/i2c/nuvoton,ma35d1-i2c.yaml@@ -0,0 +1,65 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/i2c/nuvoton,ma35d1-i2c.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Nuvoton MA35D1 I2C Controller + +maintainers: + - Zi-Yu Chen <zychennvt@gmail.com> + +description: |
Do not need '|' unless you need to preserve formatting.
+ The Nuvoton MA35D1 I2C controller supports master mode and optional + slave mode operation. The controller is configured via Device Tree
Use modern naming, not master/slave.
+ and supports interrupt-driven I2C transfers.
Drop "The controller is configured via Device Tree", because it is completely irrelevant. Why telling in DT binding that you use DT? Can you use ACPI here? And with dropping this it could be one simple sentence. With these changes: Reviewed-by: Krzysztof Kozlowski <redacted> Best regards, Krzysztof