devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: power: mediatek: Refine multiple level power domain nodes
@ 2024-10-04  7:49 Macpaul Lin
  2024-10-04  8:24 ` Fei Shao
  0 siblings, 1 reply; 3+ messages in thread
From: Macpaul Lin @ 2024-10-04  7:49 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
	AngeloGioacchino Del Regno, devicetree, linux-kernel,
	linux-arm-kernel, linux-mediatek, Alexandre Mergnat
  Cc: Bear Wang, Pablo Sun, Macpaul Lin, Macpaul Lin, Sen Chu,
	Chris-qj chen, MediaTek Chromebook Upstream, Chen-Yu Tsai

Extract duplicated properties and support more levels of power
domain nodes.

This change fix following error when do dtbs_check,
  mt8390-genio-700-evk.dtb:
    power-controller: power-domain@15:power-domain@16:power-domain@29:power-domain@30:
    Unevaluated properties are not allowed ('power-domain@31', 'power-domain@32'
    were unexpected)

Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
 .../devicetree/bindings/power/mediatek,power-controller.yaml  | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
index 8985e2df8a56..a7df4041b745 100644
--- a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
+++ b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
@@ -54,6 +54,10 @@ patternProperties:
             patternProperties:
               "^power-domain@[0-9a-f]+$":
                 $ref: "#/$defs/power-domain-node"
+                patternProperties:
+                  "^power-domain@[0-9a-f]+$":
+                    $ref: "#/$defs/power-domain-node"
+                    unevaluatedProperties: false
                 unevaluatedProperties: false
             unevaluatedProperties: false
         unevaluatedProperties: false
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: power: mediatek: Refine multiple level power domain nodes
  2024-10-04  7:49 [PATCH] dt-bindings: power: mediatek: Refine multiple level power domain nodes Macpaul Lin
@ 2024-10-04  8:24 ` Fei Shao
  2024-10-04  8:40   ` Macpaul Lin
  0 siblings, 1 reply; 3+ messages in thread
From: Fei Shao @ 2024-10-04  8:24 UTC (permalink / raw)
  To: Macpaul Lin
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
	AngeloGioacchino Del Regno, devicetree, linux-kernel,
	linux-arm-kernel, linux-mediatek, Alexandre Mergnat, Bear Wang,
	Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
	MediaTek Chromebook Upstream, Chen-Yu Tsai

On Fri, Oct 4, 2024 at 3:53 PM Macpaul Lin <macpaul.lin@mediatek.com> wrote:
>
> Extract duplicated properties and support more levels of power
> domain nodes.
>
> This change fix following error when do dtbs_check,
>   mt8390-genio-700-evk.dtb:
>     power-controller: power-domain@15:power-domain@16:power-domain@29:power-domain@30:
>     Unevaluated properties are not allowed ('power-domain@31', 'power-domain@32'
>     were unexpected)
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
>  .../devicetree/bindings/power/mediatek,power-controller.yaml  | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
> index 8985e2df8a56..a7df4041b745 100644
> --- a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
> +++ b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
> @@ -54,6 +54,10 @@ patternProperties:
>              patternProperties:
>                "^power-domain@[0-9a-f]+$":
>                  $ref: "#/$defs/power-domain-node"
> +                patternProperties:
> +                  "^power-domain@[0-9a-f]+$":
> +                    $ref: "#/$defs/power-domain-node"
> +                    unevaluatedProperties: false
>                  unevaluatedProperties: false
>              unevaluatedProperties: false
>          unevaluatedProperties: false

For reference, I sent the exact same patch and it's been acked:
https://lore.kernel.org/all/20241001113052.3124869-2-fshao@chromium.org/

Regards,
Fei

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: power: mediatek: Refine multiple level power domain nodes
  2024-10-04  8:24 ` Fei Shao
@ 2024-10-04  8:40   ` Macpaul Lin
  0 siblings, 0 replies; 3+ messages in thread
From: Macpaul Lin @ 2024-10-04  8:40 UTC (permalink / raw)
  To: Fei Shao
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
	AngeloGioacchino Del Regno, devicetree, linux-kernel,
	linux-arm-kernel, linux-mediatek, Alexandre Mergnat, Bear Wang,
	Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
	MediaTek Chromebook Upstream, Chen-Yu Tsai



On 10/4/24 16:24, Fei Shao wrote:
> 	
> 
> External email : Please do not click links or open attachments until you 
> have verified the sender or the content.
> 
> On Fri, Oct 4, 2024 at 3:53 PM Macpaul Lin <macpaul.lin@mediatek.com> wrote:
>>
>> Extract duplicated properties and support more levels of power
>> domain nodes.
>>
>> This change fix following error when do dtbs_check,
>>   mt8390-genio-700-evk.dtb:
>>     power-controller: power-domain@15:power-domain@16:power-domain@29:power-domain@30:
>>     Unevaluated properties are not allowed ('power-domain@31', 'power-domain@32'
>>     were unexpected)
>>
>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>> ---
>>  .../devicetree/bindings/power/mediatek,power-controller.yaml  | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
>> index 8985e2df8a56..a7df4041b745 100644
>> --- a/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
>> +++ b/Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
>> @@ -54,6 +54,10 @@ patternProperties:
>>              patternProperties:
>>                "^power-domain@[0-9a-f]+$":
>>                  $ref: "#/$defs/power-domain-node"
>> +                patternProperties:
>> +                  "^power-domain@[0-9a-f]+$":
>> +                    $ref: "#/$defs/power-domain-node"
>> +                    unevaluatedProperties: false
>>                  unevaluatedProperties: false
>>              unevaluatedProperties: false
>>          unevaluatedProperties: false
> 
> For reference, I sent the exact same patch and it's been acked:
> https://lore.kernel.org/all/20241001113052.3124869-2-fshao@chromium.org/
> 
> Regards,
> Fei
> 

Thanks for the reminder.
This patch could be dropped by duplicate works.

Regards,
Macpaul Lin

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-10-04  8:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-04  7:49 [PATCH] dt-bindings: power: mediatek: Refine multiple level power domain nodes Macpaul Lin
2024-10-04  8:24 ` Fei Shao
2024-10-04  8:40   ` Macpaul Lin

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).