Hi,
On 10/05/21 22:45, Rob Herring wrote:
Users of common properties shouldn't have a type definition as the common schemas already have one. A few new ones slipped in and *-names was missed in the last clean-up pass. Drop all the unnecessary type references in the tree.
A meta-schema update to catch these is pending.
Cc: Luca Ceresoli luca@lucaceresoli.net Cc: Stephen Boyd sboyd@kernel.org Cc: Olivier Moysan olivier.moysan@foss.st.com Cc: Arnaud Pouliquen arnaud.pouliquen@foss.st.com Cc: Jonathan Cameron jic23@kernel.org Cc: Lars-Peter Clausen lars@metafoo.de Cc: Dmitry Torokhov dmitry.torokhov@gmail.com Cc: Bjorn Andersson bjorn.andersson@linaro.org Cc: Georgi Djakov djakov@kernel.org Cc: "David S. Miller" davem@davemloft.net Cc: Jakub Kicinski kuba@kernel.org Cc: Sebastian Reichel sre@kernel.org Cc: Orson Zhai orsonzhai@gmail.com Cc: Baolin Wang baolin.wang7@gmail.com Cc: Chunyan Zhang zhang.lyra@gmail.com Cc: Liam Girdwood lgirdwood@gmail.com Cc: Mark Brown broonie@kernel.org Cc: Fabrice Gasnier fabrice.gasnier@st.com Cc: Odelu Kukatla okukatla@codeaurora.org Cc: Alex Elder elder@kernel.org Cc: Shengjiu Wang shengjiu.wang@nxp.com Cc: linux-clk@vger.kernel.org Cc: alsa-devel@alsa-project.org Cc: linux-iio@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-input@vger.kernel.org Cc: linux-pm@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Rob Herring robh@kernel.org
Documentation/devicetree/bindings/clock/idt,versaclock5.yaml | 2 -- .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 1 - Documentation/devicetree/bindings/input/input.yaml | 1 - Documentation/devicetree/bindings/interconnect/qcom,rpmh.yaml | 1 - Documentation/devicetree/bindings/net/qcom,ipa.yaml | 1 - .../devicetree/bindings/power/supply/sc2731-charger.yaml | 2 +- Documentation/devicetree/bindings/sound/fsl,rpmsg.yaml | 2 +- 7 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml index c268debe5b8d..28675b0b80f1 100644 --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml @@ -60,7 +60,6 @@ properties: maxItems: 2
idt,xtal-load-femtofarads:
- $ref: /schemas/types.yaml#/definitions/uint32 minimum: 9000 maximum: 22760 description: Optional load capacitor for XTAL1 and XTAL2
@@ -84,7 +83,6 @@ patternProperties: enum: [ 1800000, 2500000, 3300000 ] idt,slew-percent: description: The Slew rate control for CMOS single-ended.
$ref: /schemas/types.yaml#/definitions/uint32 enum: [ 80, 85, 90, 100 ]
Ok, but shouldn't "percent" be listed in Documentation/devicetree/bindings/property-units.txt?
Reviewed-by: Luca Ceresoli luca@lucaceresoli.net