From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Andi Shyti" <andi.shyti@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Nicolas Ferre" <nicolas.ferre@microchip.com>,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Claudiu Beznea" <claudiu.beznea@tuxon.dev>,
"Sergiu Moga" <sergiu.moga@microchip.com>,
"Benson Leung" <bleung@chromium.org>,
"Guenter Roeck" <groeck@chromium.org>,
"Doug Anderson" <dianders@chromium.org>,
"Enric Balletbo i Serra" <eballetbo@kernel.org>,
"Ricardo Cañuelo" <ricardo.canuelo@collabora.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Vignesh R" <vigneshr@ti.com>,
"Kamal Dasu" <kamal.dasu@broadcom.com>,
"Broadcom internal kernel review list"
<bcm-kernel-feedback-list@broadcom.com>,
"Florian Fainelli" <florian.fainelli@broadcom.com>,
"Wolfram Sang" <wsa+renesas@sang-engineering.com>,
"Chris Brandt" <chris.brandt@renesas.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
"Geert Uytterhoeven" <geert+renesas@glider.be>,
"Magnus Damm" <magnus.damm@gmail.com>,
"Pierre-Yves MORDRET" <pierre-yves.mordret@foss.st.com>
Cc: linux-i2c@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, chrome-platform@lists.linux.dev,
linux-tegra@vger.kernel.org,
Krzysztof Kozlowski <krzk@kernel.org>,
linux-omap@vger.kernel.org, Kamal Dasu <kdasu.kdev@gmail.com>,
linux-renesas-soc@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Subject: [PATCH 6/7] dt-bindings: i2c: ti,omap4: reference i2c-controller.yaml schema
Date: Thu, 20 Jun 2024 13:34:54 +0200 [thread overview]
Message-ID: <20240620-dt-bindings-i2c-clean-v1-6-3a1016a95f9d@linaro.org> (raw)
In-Reply-To: <20240620-dt-bindings-i2c-clean-v1-0-3a1016a95f9d@linaro.org>
Reference the core I2C controller schema to properly define common
properties. This allows to drop several (now redundant) properties.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/i2c/ti,omap4-i2c.yaml | 52 +++++++++-------------
1 file changed, 20 insertions(+), 32 deletions(-)
diff --git a/Documentation/devicetree/bindings/i2c/ti,omap4-i2c.yaml b/Documentation/devicetree/bindings/i2c/ti,omap4-i2c.yaml
index 781108ae1ce3..c9d1030e9857 100644
--- a/Documentation/devicetree/bindings/i2c/ti,omap4-i2c.yaml
+++ b/Documentation/devicetree/bindings/i2c/ti,omap4-i2c.yaml
@@ -37,16 +37,8 @@ properties:
clock-names:
const: fck
- clock-frequency: true
-
power-domains: true
- "#address-cells":
- const: 1
-
- "#size-cells":
- const: 0
-
ti,hwmods:
description:
Must be "i2c<n>", n being the instance number (1-based).
@@ -55,38 +47,34 @@ properties:
$ref: /schemas/types.yaml#/definitions/string
deprecated: true
-# subnode's properties
-patternProperties:
- "@[0-9a-f]+$":
- type: object
- description:
- Flash device uses the below defined properties in the subnode.
-
required:
- compatible
- reg
- interrupts
-additionalProperties: false
+allOf:
+ - $ref: /schemas/i2c/i2c-controller.yaml#
-if:
- properties:
- compatible:
- enum:
- - ti,omap2420-i2c
- - ti,omap2430-i2c
- - ti,omap3-i2c
- - ti,omap4-i2c
+ - if:
+ properties:
+ compatible:
+ enum:
+ - ti,omap2420-i2c
+ - ti,omap2430-i2c
+ - ti,omap3-i2c
+ - ti,omap4-i2c
-then:
- properties:
- ti,hwmods:
- items:
- - pattern: "^i2c([1-9])$"
+ then:
+ properties:
+ ti,hwmods:
+ items:
+ - pattern: "^i2c([1-9])$"
-else:
- properties:
- ti,hwmods: false
+ else:
+ properties:
+ ti,hwmods: false
+
+unevaluatedProperties: false
examples:
- |
--
2.43.0
next prev parent reply other threads:[~2024-06-20 11:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-20 11:34 [PATCH 0/7] dt-bindings: i2c: few fixes and cleanups Krzysztof Kozlowski
2024-06-20 11:34 ` [PATCH 1/7] dt-bindings: i2c: atmel,at91sam: correct path to i2c-controller schema Krzysztof Kozlowski
2024-06-20 11:34 ` [PATCH 2/7] dt-bindings: i2c: google,cros-ec-i2c-tunnel: " Krzysztof Kozlowski
2024-06-20 16:06 ` Conor Dooley
2024-06-20 11:34 ` [PATCH 3/7] dt-bindings: i2c: atmel,at91sam: drop unneeded address/size-cells Krzysztof Kozlowski
2024-06-20 11:34 ` [PATCH 4/7] dt-bindings: i2c: nvidia,tegra20: " Krzysztof Kozlowski
2024-06-20 11:34 ` [PATCH 5/7] dt-bindings: i2c: samsung,s3c2410: " Krzysztof Kozlowski
2024-06-20 11:34 ` Krzysztof Kozlowski [this message]
2024-06-20 11:34 ` [PATCH 7/7] dt-bindings: i2c: adjust indentation in DTS example to coding style Krzysztof Kozlowski
2024-06-20 16:06 ` [PATCH 0/7] dt-bindings: i2c: few fixes and cleanups Conor Dooley
2024-06-20 23:05 ` Andi Shyti
2024-06-21 6:02 ` Krzysztof Kozlowski
2024-06-21 9:27 ` Andi Shyti
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240620-dt-bindings-i2c-clean-v1-6-3a1016a95f9d@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=alexandre.belloni@bootlin.com \
--cc=alexandre.torgue@foss.st.com \
--cc=andi.shyti@kernel.org \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=bleung@chromium.org \
--cc=chris.brandt@renesas.com \
--cc=chrome-platform@lists.linux.dev \
--cc=claudiu.beznea@tuxon.dev \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=eballetbo@kernel.org \
--cc=florian.fainelli@broadcom.com \
--cc=geert+renesas@glider.be \
--cc=groeck@chromium.org \
--cc=jonathanh@nvidia.com \
--cc=kamal.dasu@broadcom.com \
--cc=kdasu.kdev@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux-tegra@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=nicolas.ferre@microchip.com \
--cc=pierre-yves.mordret@foss.st.com \
--cc=ricardo.canuelo@collabora.com \
--cc=robh@kernel.org \
--cc=sergiu.moga@microchip.com \
--cc=thierry.reding@gmail.com \
--cc=vigneshr@ti.com \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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 NNTP newsgroup(s).