Commit 26658868 authored by Stephan Gerhold's avatar Stephan Gerhold Committed by Bjorn Andersson

dt-bindings: soc: qcom: apr: Make qcom,protection-domain optional again

The protection domain functionality exists only in SoCs starting from
MSM8998 [1], while the APR bindings are also used on older platforms.

Commit 41288c30 ("ASoC: dt-bindings: qcom,apr: Split services to
shared schema") made the "qcom,protection-domain" required but it
should remain optional to avoid dtbs_check warnings on older platforms,
e.g.:

arch/arm64/boot/dts/qcom/apq8096-db820c.dtb:
  apr: service@3: 'qcom,protection-domain' is a required property
  From schema: Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml

[1]: https://lore.kernel.org/all/20200312120842.21991-1-sibis@codeaurora.org/

Fixes: 41288c30 ("ASoC: dt-bindings: qcom,apr: Split services to shared schema")
Signed-off-by: default avatarStephan Gerhold <stephan@gerhold.net>
Reviewed-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: default avatarBjorn Andersson <andersson@kernel.org>
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20221229151648.19839-2-stephan@gerhold.net
parent 1b929c02
...@@ -39,8 +39,8 @@ properties: ...@@ -39,8 +39,8 @@ properties:
qcom,protection-domain: qcom,protection-domain:
$ref: /schemas/types.yaml#/definitions/string-array $ref: /schemas/types.yaml#/definitions/string-array
description: | description: |
Protection domain service name and path for APR service Protection domain service name and path for APR service (if supported).
possible values are:: Possible values are::
"avs/audio", "msm/adsp/audio_pd". "avs/audio", "msm/adsp/audio_pd".
"kernel/elf_loader", "msm/modem/wlan_pd". "kernel/elf_loader", "msm/modem/wlan_pd".
"tms/servreg", "msm/adsp/audio_pd". "tms/servreg", "msm/adsp/audio_pd".
...@@ -49,6 +49,5 @@ properties: ...@@ -49,6 +49,5 @@ properties:
required: required:
- reg - reg
- qcom,protection-domain
additionalProperties: true additionalProperties: true
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment