lkml.org 
[lkml]   [2023]   [Apr]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH V3 1/3] dt-bindings: sram: qcom,imem: Add Boot Stat region within IMEM
From
On 13/04/2023 11:28, Souradeep Chowdhury wrote:

Thank you for your patch. There is something to discuss/improve.

> + "^stats@[0-9a-f]+$":
> + type: object
> + description:
> + Imem region dedicated for storing timestamps related
> + information regarding bootstats.
> +
> + properties:
> + compatible:
> + items:
> + - enum:
> + - qcom,sm8450-bootstats
> + - const: qcom,imem-bootstats
> +
> + reg:
> + maxItems: 1
> +
> + required:
> + - compatible
> + - reg
> +
> + additionalProperties: false

The feedback about additionalProperties was given in different place. On
purpose, because it is easier to read when it is placed before
properties for indented cases. Therefore move it after description, how
I originally asked.

> +
> required:
> - compatible
> - reg

Best regards,
Krzysztof

\
 
 \ /
  Last update: 2023-04-13 19:07    [W:0.047 / U:0.264 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site