On 24/03/2022 20:26, Krzysztof Kozlowski wrote: > On 23/03/2022 12:18, Roger Quadros wrote: >> TI's AM64 SoC has the Error Locator Module. Add compatible and >> related properties to support ELM on AM64 SoC. >> >> Signed-off-by: Roger Quadros <rogerq@xxxxxxxxxx> >> --- >> .../devicetree/bindings/mtd/ti,elm.yaml | 27 ++++++++++++++++++- >> 1 file changed, 26 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/mtd/ti,elm.yaml b/Documentation/devicetree/bindings/mtd/ti,elm.yaml >> index 24ed0fb84204..bc01d35ce605 100644 >> --- a/Documentation/devicetree/bindings/mtd/ti,elm.yaml >> +++ b/Documentation/devicetree/bindings/mtd/ti,elm.yaml >> @@ -17,7 +17,9 @@ description: >> properties: >> compatible: >> items: >> - - const: ti,am3352-elm >> + - enum: >> + - ti,am3352-elm >> + - ti,am64-elm >> >> reg: >> maxItems: 1 >> @@ -25,6 +27,17 @@ properties: >> interrupts: >> maxItems: 1 >> >> + clocks: >> + maxItems: 1 >> + description: Functional clock. >> + >> + clock-names: >> + items: >> + - const: fck >> + >> + power-domains: >> + maxItems: 1 >> + >> ti,hwmods: >> description: >> Name of the HWMOD associated with ELM. This is for legacy >> @@ -37,6 +50,18 @@ required: >> - reg >> - interrupts >> >> +allOf: >> + - if: >> + properties: >> + compatible: >> + contains: >> + const: ti,am64-elm >> + then: >> + required: >> + - clocks >> + - clock-names >> + - power-domains > > Are these new properties also valid for am3352? No they are not required for am3352-elm. Only required for K3 based platforms like AM64. > >> + >> additionalProperties: false >> >> examples: > > > Best regards, > Krzysztof cheers, -roger
