diff mbox series

[1/3] dt-bindings: soc: samsung: exynos-pmu: cleanup assigned clocks

Message ID 20220706095037.78542-1-krzysztof.kozlowski@linaro.org
State New
Headers show
Series [1/3] dt-bindings: soc: samsung: exynos-pmu: cleanup assigned clocks | expand

Commit Message

Krzysztof Kozlowski July 6, 2022, 9:50 a.m. UTC
"assigned-clocks" are not needed in the schema as they come from DT
schema.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml | 3 ---
 1 file changed, 3 deletions(-)

Comments

Alim Akhtar July 6, 2022, 11:21 a.m. UTC | #1
>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>Sent: Wednesday, July 6, 2022 3:21 PM
>To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar <alim.akhtar@samsung.com>;
>devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>samsung-soc@vger.kernel.org; linux-kernel@vger.kernel.org
>Cc: Chanho Park <chanho61.park@samsung.com>; Krzysztof Kozlowski
><krzysztof.kozlowski@linaro.org>
>Subject: [PATCH 1/3] dt-bindings: soc: samsung: exynos-pmu: cleanup
assigned
>clocks
>
>"assigned-clocks" are not needed in the schema as they come from DT schema.
>
Sorry, I didn't get this, you mean "assigned-clocks" comes from DT so it is
not required to be mentioned in schema?

>Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>---
> Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
>b/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
>index c30a6437030d..a5d489acfdca 100644
>--- a/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
>+++ b/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
>@@ -49,9 +49,6 @@ properties:
>   reg:
>     maxItems: 1
>
>-  assigned-clock-parents: true
>-  assigned-clocks: true
>-
>   '#clock-cells':
>     const: 1
>
>--
>2.34.1
Krzysztof Kozlowski July 6, 2022, 3 p.m. UTC | #2
On 06/07/2022 13:21, Alim Akhtar wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>> Sent: Wednesday, July 6, 2022 3:21 PM
>> To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
>> <krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar <alim.akhtar@samsung.com>;
>> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>> samsung-soc@vger.kernel.org; linux-kernel@vger.kernel.org
>> Cc: Chanho Park <chanho61.park@samsung.com>; Krzysztof Kozlowski
>> <krzysztof.kozlowski@linaro.org>
>> Subject: [PATCH 1/3] dt-bindings: soc: samsung: exynos-pmu: cleanup
> assigned
>> clocks
>>
>> "assigned-clocks" are not needed in the schema as they come from DT schema.
>>
> Sorry, I didn't get this, you mean "assigned-clocks" comes from DT so it is
> not required to be mentioned in schema?

Maybe this is not accurate, so I can rephrase it to:

"assigned-clocks" are not needed in the schema as they come from core
schema.

Best regards,
Krzysztof
Rob Herring (Arm) July 6, 2022, 4:16 p.m. UTC | #3
On Wed, 06 Jul 2022 11:50:36 +0200, Krzysztof Kozlowski wrote:
> Preferred coding for referencing other schemas is to use absolute path.
> Quotes over path are also not needed.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml b/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
index c30a6437030d..a5d489acfdca 100644
--- a/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
+++ b/Documentation/devicetree/bindings/soc/samsung/exynos-pmu.yaml
@@ -49,9 +49,6 @@  properties:
   reg:
     maxItems: 1
 
-  assigned-clock-parents: true
-  assigned-clocks: true
-
   '#clock-cells':
     const: 1