| Message ID | 20240708194808.1819185-4-gnstark@salutedevices.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | amlogic SoC's power-domains fixes | expand |
On 08/07/2024 21:48, George Stark wrote: > On newer SoCs, the thermal hardware can require a power domain to > operate so add corresponding property as optional by default and as > required for Meson A1 due to it's temperature sensor has dedicated > power domain. > That's again ABI break with a reason "there is power domain", so not really sufficient. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml b/Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml index 01fccdfc4178..5c9147e3b734 100644 --- a/Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +++ b/Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml @@ -30,6 +30,9 @@ properties: clocks: maxItems: 1 + power-domains: + maxItems: 1 + amlogic,ao-secure: description: phandle to the ao-secure syscon $ref: /schemas/types.yaml#/definitions/phandle @@ -44,6 +47,17 @@ required: - clocks - amlogic,ao-secure +allOf: + - if: + properties: + compatible: + contains: + enum: + - amlogic,a1-cpu-thermal + then: + required: + - power-domains + additionalProperties: false examples:
On newer SoCs, the thermal hardware can require a power domain to operate so add corresponding property as optional by default and as required for Meson A1 due to it's temperature sensor has dedicated power domain. Signed-off-by: George Stark <gnstark@salutedevices.com> --- .../bindings/thermal/amlogic,thermal.yaml | 14 ++++++++++++++ 1 file changed, 14 insertions(+)