2022-10-11 17:31:36

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH v2 23/34] dt-bindings: pinctrl: qcom,sm6115-tlmm: minor style cleanups

Drop "binding" from description (and align it with other Qualcomm
pinctrl bindings), use double quotes consistently, drop redundant
quotes and rename file to match compatible (to match coding convention).

Signed-off-by: Krzysztof Kozlowski <[email protected]>
Acked-by: Rob Herring <[email protected]>
---
...qcom,sm6115-pinctrl.yaml => qcom,sm6115-tlmm.yaml} | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
rename Documentation/devicetree/bindings/pinctrl/{qcom,sm6115-pinctrl.yaml => qcom,sm6115-tlmm.yaml} (95%)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
similarity index 95%
rename from Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
rename to Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
index 3f5f5229a86c..ed68c4ee032f 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: http://devicetree.org/schemas/pinctrl/qcom,sm6115-pinctrl.yaml#
+$id: http://devicetree.org/schemas/pinctrl/qcom,sm6115-tlmm.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Qualcomm Technologies, Inc. SM6115, SM4250 TLMM block
@@ -10,8 +10,8 @@ maintainers:
- Iskren Chernev <[email protected]>

description:
- This binding describes the Top Level Mode Multiplexer block found in the
- SM4250/6115 platforms.
+ Top Level Mode Multiplexer pin controller in Qualcomm SM4250 and SM6115
+ SoCs.

properties:
compatible:
@@ -36,9 +36,8 @@ properties:
gpio-reserved-ranges: true
wakeup-parent: true

-#PIN CONFIGURATION NODES
patternProperties:
- '-state$':
+ "-state$":
oneOf:
- $ref: "#/$defs/qcom-sm6115-tlmm-state"
- patternProperties:
@@ -46,7 +45,7 @@ patternProperties:
$ref: "#/$defs/qcom-sm6115-tlmm-state"
additionalProperties: false

-'$defs':
+$defs:
qcom-sm6115-tlmm-state:
type: object
description:
--
2.34.1


2022-10-12 16:23:02

by Iskren Chernev

[permalink] [raw]
Subject: Re: [PATCH v2 23/34] dt-bindings: pinctrl: qcom,sm6115-tlmm: minor style cleanups

On Tue, Oct 11, 2022 at 8:26 PM Krzysztof Kozlowski
<[email protected]> wrote:
>
> Drop "binding" from description (and align it with other Qualcomm
> pinctrl bindings), use double quotes consistently, drop redundant
> quotes and rename file to match compatible (to match coding convention).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> Acked-by: Rob Herring <[email protected]>
Reviewed-by: Iskren Chernev <[email protected]>

> ...qcom,sm6115-pinctrl.yaml => qcom,sm6115-tlmm.yaml} | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
> rename Documentation/devicetree/bindings/pinctrl/{qcom,sm6115-pinctrl.yaml => qcom,sm6115-tlmm.yaml} (95%)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> similarity index 95%
> rename from Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
> rename to Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> index 3f5f5229a86c..ed68c4ee032f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/pinctrl/qcom,sm6115-pinctrl.yaml#
> +$id: http://devicetree.org/schemas/pinctrl/qcom,sm6115-tlmm.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Technologies, Inc. SM6115, SM4250 TLMM block
> @@ -10,8 +10,8 @@ maintainers:
> - Iskren Chernev <[email protected]>
>
> description:
> - This binding describes the Top Level Mode Multiplexer block found in the
> - SM4250/6115 platforms.
> + Top Level Mode Multiplexer pin controller in Qualcomm SM4250 and SM6115
> + SoCs.
>
> properties:
> compatible:
> @@ -36,9 +36,8 @@ properties:
> gpio-reserved-ranges: true
> wakeup-parent: true
>
> -#PIN CONFIGURATION NODES
> patternProperties:
> - '-state$':
> + "-state$":
> oneOf:
> - $ref: "#/$defs/qcom-sm6115-tlmm-state"
> - patternProperties:
> @@ -46,7 +45,7 @@ patternProperties:
> $ref: "#/$defs/qcom-sm6115-tlmm-state"
> additionalProperties: false
>
> -'$defs':
> +$defs:
> qcom-sm6115-tlmm-state:
> type: object
> description:
> --
> 2.34.1
>