Messages in this thread | | | Date | Thu, 13 Apr 2023 18:41:10 +0200 | Subject | Re: [PATCH V3 1/3] dt-bindings: sram: qcom,imem: Add Boot Stat region within IMEM | From | Krzysztof Kozlowski <> |
| |
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
| |