Hi Bhupesh, On 11/10/21 12:59 PM, Bhupesh Sharma wrote:
In some configurations, the BAM DMA controller is set up by a remote processor and the local processor can simply start making use of it without setting up the BAM. This is already supported using the "qcom,controlled-remotely" property. However, for some reason another possible configuration is that the remote processor is responsible for powering up the BAM, but we are still responsible for initializing it (e.g. resetting it etc). Add a "qcom,powered-remotely" property to describe that configuration. Cc: Thara Gopinath <thara.gopinath@xxxxxxxxxx> Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> Cc: Rob Herring <robh+dt@xxxxxxxxxx> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@xxxxxxxxxx> [moved Stephan's change to the YAML dt-binding format] Signed-off-by: Stephan Gerhold <stephan@xxxxxxxxxxx> --- Documentation/devicetree/bindings/dma/qcom_bam_dma.yaml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/dma/qcom_bam_dma.yaml b/Documentation/devicetree/bindings/dma/qcom_bam_dma.yaml index cfff3a2286fb..bf0a59e8a2bf 100644 --- a/Documentation/devicetree/bindings/dma/qcom_bam_dma.yaml +++ b/Documentation/devicetree/bindings/dma/qcom_bam_dma.yaml @@ -73,6 +73,12 @@ properties: Indicates that the bam is controlled by remote proccessor i.e. execution environment.+ qcom,powered-remotely:+ $ref: /schemas/types.yaml#/definitions/flag + description: + Indicates that the bam is powered up by a remote processor + but must be initialized by the local processor. + qcom,num-ees: $ref: /schemas/types.yaml#/definitions/uint32 minimum: 0
after rebasing the change on top of master this particular patch won't be needed anymore. See my review comment to v5 03/22. -- Best wishes, Vladimir
