lkml.org 
[lkml]   [2022]   [Dec]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 3/4] dt-bindings: arm: mediatek: Add missing power-domains property
From
On 15/12/2022 13:00, Allen-KH Cheng wrote:
> The "mediatek,mt8192-scp_adsp" binding requires a power domain to be
> specified.
>
> Fixes: 4a803990aeb1 ("dt-bindings: ARM: Mediatek: Add new document bindings of MT8192 clock")
> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com>
> ---
> .../arm/mediatek/mediatek,mt8192-clock.yaml | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
> index b57cc2e69efb..cbedef114103 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
> @@ -40,6 +40,9 @@ properties:
> reg:
> maxItems: 1
>
> + power-domains:
> + maxItems: 1
> +
> '#clock-cells':
> const: 1
>
> @@ -49,11 +52,25 @@ required:
>
> additionalProperties: false
>
> +allOf:

allOf goes just before additionalProperties.

> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - mediatek,mt8192-scp_adsp

For the future: please don't use underscores in compatibles.

> + then:
> + required:
> + - power-domains
> +
> examples:
> - |
> + #include <dt-bindings/power/mt8192-power.h>
> +
> scp_adsp: clock-controller@10720000 {
> compatible = "mediatek,mt8192-scp_adsp";
> reg = <0x10720000 0x1000>;
> + power-domains = <&spm MT8192_POWER_DOMAIN_ADSP>;
> #clock-cells = <1>;
> };
>

Best regards,
Krzysztof

\
 
 \ /
  Last update: 2022-12-16 11:59    [W:0.080 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site