Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
Thanks Rob for reviewing. On 9/11/2018 1:31 AM, Rob Herring wrote: On Mon, Sep 03, 2018 at 05:22:39PM +0530, Rohit kumar wrote: Add devicetree bindings documentation file for Qualcomm Technolgies Inc ADSP Peripheral Image Loader. Signed-off-by: Rohit kumar --- .../bindings/remoteproc/qcom,adsp-pil.txt | 123 + 1 file changed, 123 insertions(+) create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt new file mode 100644 index 000..f1c215a --- /dev/null +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt @@ -0,0 +1,123 @@ +Qualcomm Technology Inc. ADSP Peripheral Image Loader + +This document defines the binding for a component that loads and boots firmware +on the Qualcomm Technology Inc. ADSP Hexagon core. + +- compatible: + Usage: required + Value type: + Definition: must be one of: + "qcom,sdm845-adsp-pil" + +- reg: + Usage: required + Value type: + Definition: must specify the base address and size of the qdsp6ss register + +- interrupts-extended: + Usage: required + Value type: + Definition: must list the watchdog, fatal IRQs ready, handover and + stop-ack IRQs + +- interrupt-names: + Usage: required + Value type: + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" + +- clocks: + Usage: required + Value type: + Definition: List of phandle and clock specifier pairs How many clocks? + +- clock-names: + Usage: required + Value type: + Definition: List of clock input name strings sorted in the same + order as the clocks property. What are the names? I will update these in next spin. + +- power-domains: + Usage: required + Value type: + Definition: reference to cx power domain node. + +- resets: + Usage: required + Value type: + Definition: reference to the reset-controller for the lpass How many? + +- reset-names: +Usage: required +Value type: +Definition: must be "pdc_sync" and "cc_lpass" + +- qcom,halt-regs: + Usage: required + Value type: + Definition: a phandle reference to a syscon representing TCSR followed + by the offset within syscon for lpass halt register. + +- memory-region: + Usage: required + Value type: + Definition: reference to the reserved-memory for the ADSP + +- qcom,smem-states: + Usage: required + Value type: + Definition: reference to the smem state for requesting the ADSP to + shut down + +- qcom,smem-state-names: + Usage: required + Value type: + Definition: must be "stop" + + += SUBNODES +The adsp node may have an subnode named "glink-edge" that describes the +communication edge, channels and devices related to the ADSP. +See ../soc/qcom/qcom,glink.txt for details on how to describe these. + += EXAMPLE +The following example describes the resources needed to boot control the +ADSP, as it is found on SDM845 boards. + adsp-pil { + compatible = "qcom,sdm845-adsp-pil"; + + reg = <0x1730 0x40c>; + + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; + interrupt-names = "wdog", "fatal", "ready", + "handover", "stop-ack"; + + clocks = < RPMH_CXO_CLK>, + < GCC_LPASS_SWAY_CLK>, + < LPASS_AUDIO_WRAPPER_AON_CLK>, + < LPASS_Q6SS_AHBS_AON_CLK>, + < LPASS_Q6SS_AHBM_AON_CLK>, + < LPASS_QDSP6SS_XO_CLK>, + < LPASS_QDSP6SS_SLEEP_CLK>, + < LPASS_QDSP6SS_CORE_CLK>; + clock-names = "xo", "sway_cbcr", "lpass_aon", + "lpass_ahbs_aon_cbcr", + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", + "qdsp6ss_sleep", "qdsp6ss_core"; + + power-domains = < SDM845_CX>; + + resets = <_reset PDC_AUDIO_SYNC_RESET>, +<_reset AOSS_CC_LPASS_RESTART>; + reset-names = "pdc_sync", "cc_lpass"; + + qcom,halt-regs = <_mutex_regs 0x22000>; + + memory-region = <_adsp_mem>; + + qcom,smem-states = <_smp2p_out 0>; + qcom,smem-state-names = "stop"; + }; -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a
Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
Thanks Rob for reviewing. On 9/11/2018 1:31 AM, Rob Herring wrote: On Mon, Sep 03, 2018 at 05:22:39PM +0530, Rohit kumar wrote: Add devicetree bindings documentation file for Qualcomm Technolgies Inc ADSP Peripheral Image Loader. Signed-off-by: Rohit kumar --- .../bindings/remoteproc/qcom,adsp-pil.txt | 123 + 1 file changed, 123 insertions(+) create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt new file mode 100644 index 000..f1c215a --- /dev/null +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt @@ -0,0 +1,123 @@ +Qualcomm Technology Inc. ADSP Peripheral Image Loader + +This document defines the binding for a component that loads and boots firmware +on the Qualcomm Technology Inc. ADSP Hexagon core. + +- compatible: + Usage: required + Value type: + Definition: must be one of: + "qcom,sdm845-adsp-pil" + +- reg: + Usage: required + Value type: + Definition: must specify the base address and size of the qdsp6ss register + +- interrupts-extended: + Usage: required + Value type: + Definition: must list the watchdog, fatal IRQs ready, handover and + stop-ack IRQs + +- interrupt-names: + Usage: required + Value type: + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" + +- clocks: + Usage: required + Value type: + Definition: List of phandle and clock specifier pairs How many clocks? + +- clock-names: + Usage: required + Value type: + Definition: List of clock input name strings sorted in the same + order as the clocks property. What are the names? I will update these in next spin. + +- power-domains: + Usage: required + Value type: + Definition: reference to cx power domain node. + +- resets: + Usage: required + Value type: + Definition: reference to the reset-controller for the lpass How many? + +- reset-names: +Usage: required +Value type: +Definition: must be "pdc_sync" and "cc_lpass" + +- qcom,halt-regs: + Usage: required + Value type: + Definition: a phandle reference to a syscon representing TCSR followed + by the offset within syscon for lpass halt register. + +- memory-region: + Usage: required + Value type: + Definition: reference to the reserved-memory for the ADSP + +- qcom,smem-states: + Usage: required + Value type: + Definition: reference to the smem state for requesting the ADSP to + shut down + +- qcom,smem-state-names: + Usage: required + Value type: + Definition: must be "stop" + + += SUBNODES +The adsp node may have an subnode named "glink-edge" that describes the +communication edge, channels and devices related to the ADSP. +See ../soc/qcom/qcom,glink.txt for details on how to describe these. + += EXAMPLE +The following example describes the resources needed to boot control the +ADSP, as it is found on SDM845 boards. + adsp-pil { + compatible = "qcom,sdm845-adsp-pil"; + + reg = <0x1730 0x40c>; + + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; + interrupt-names = "wdog", "fatal", "ready", + "handover", "stop-ack"; + + clocks = < RPMH_CXO_CLK>, + < GCC_LPASS_SWAY_CLK>, + < LPASS_AUDIO_WRAPPER_AON_CLK>, + < LPASS_Q6SS_AHBS_AON_CLK>, + < LPASS_Q6SS_AHBM_AON_CLK>, + < LPASS_QDSP6SS_XO_CLK>, + < LPASS_QDSP6SS_SLEEP_CLK>, + < LPASS_QDSP6SS_CORE_CLK>; + clock-names = "xo", "sway_cbcr", "lpass_aon", + "lpass_ahbs_aon_cbcr", + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", + "qdsp6ss_sleep", "qdsp6ss_core"; + + power-domains = < SDM845_CX>; + + resets = <_reset PDC_AUDIO_SYNC_RESET>, +<_reset AOSS_CC_LPASS_RESTART>; + reset-names = "pdc_sync", "cc_lpass"; + + qcom,halt-regs = <_mutex_regs 0x22000>; + + memory-region = <_adsp_mem>; + + qcom,smem-states = <_smp2p_out 0>; + qcom,smem-state-names = "stop"; + }; -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a
Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
On Mon, Sep 03, 2018 at 05:22:39PM +0530, Rohit kumar wrote: > Add devicetree bindings documentation file for Qualcomm > Technolgies Inc ADSP Peripheral Image Loader. > > Signed-off-by: Rohit kumar > --- > .../bindings/remoteproc/qcom,adsp-pil.txt | 123 > + > 1 file changed, 123 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > new file mode 100644 > index 000..f1c215a > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > @@ -0,0 +1,123 @@ > +Qualcomm Technology Inc. ADSP Peripheral Image Loader > + > +This document defines the binding for a component that loads and boots > firmware > +on the Qualcomm Technology Inc. ADSP Hexagon core. > + > +- compatible: > + Usage: required > + Value type: > + Definition: must be one of: > + "qcom,sdm845-adsp-pil" > + > +- reg: > + Usage: required > + Value type: > + Definition: must specify the base address and size of the qdsp6ss > register > + > +- interrupts-extended: > + Usage: required > + Value type: > + Definition: must list the watchdog, fatal IRQs ready, handover and > + stop-ack IRQs > + > +- interrupt-names: > + Usage: required > + Value type: > + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" > + > +- clocks: > + Usage: required > + Value type: > + Definition: List of phandle and clock specifier pairs How many clocks? > + > +- clock-names: > + Usage: required > + Value type: > + Definition: List of clock input name strings sorted in the same > + order as the clocks property. What are the names? > + > +- power-domains: > + Usage: required > + Value type: > + Definition: reference to cx power domain node. > + > +- resets: > + Usage: required > + Value type: > + Definition: reference to the reset-controller for the lpass How many? > + > +- reset-names: > +Usage: required > +Value type: > +Definition: must be "pdc_sync" and "cc_lpass" > + > +- qcom,halt-regs: > + Usage: required > + Value type: > + Definition: a phandle reference to a syscon representing TCSR followed > + by the offset within syscon for lpass halt register. > + > +- memory-region: > + Usage: required > + Value type: > + Definition: reference to the reserved-memory for the ADSP > + > +- qcom,smem-states: > + Usage: required > + Value type: > + Definition: reference to the smem state for requesting the ADSP to > + shut down > + > +- qcom,smem-state-names: > + Usage: required > + Value type: > + Definition: must be "stop" > + > + > += SUBNODES > +The adsp node may have an subnode named "glink-edge" that describes the > +communication edge, channels and devices related to the ADSP. > +See ../soc/qcom/qcom,glink.txt for details on how to describe these. > + > += EXAMPLE > +The following example describes the resources needed to boot control the > +ADSP, as it is found on SDM845 boards. > + adsp-pil { > + compatible = "qcom,sdm845-adsp-pil"; > + > + reg = <0x1730 0x40c>; > + > + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "wdog", "fatal", "ready", > + "handover", "stop-ack"; > + > + clocks = < RPMH_CXO_CLK>, > + < GCC_LPASS_SWAY_CLK>, > + < LPASS_AUDIO_WRAPPER_AON_CLK>, > + < LPASS_Q6SS_AHBS_AON_CLK>, > + < LPASS_Q6SS_AHBM_AON_CLK>, > + < LPASS_QDSP6SS_XO_CLK>, > + < LPASS_QDSP6SS_SLEEP_CLK>, > + < LPASS_QDSP6SS_CORE_CLK>; > + clock-names = "xo", "sway_cbcr", "lpass_aon", > + "lpass_ahbs_aon_cbcr", > + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", > + "qdsp6ss_sleep", "qdsp6ss_core"; > + > + power-domains = < SDM845_CX>; > + > + resets = <_reset PDC_AUDIO_SYNC_RESET>, > + <_reset AOSS_CC_LPASS_RESTART>; > + reset-names = "pdc_sync", "cc_lpass"; > + > + qcom,halt-regs = <_mutex_regs 0x22000>; > + > + memory-region = <_adsp_mem>; > + > + qcom,smem-states = <_smp2p_out 0>; > + qcom,smem-state-names = "stop"; > + }; > -- > Qualcomm India Private Limited, on behalf of
Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
On Mon, Sep 03, 2018 at 05:22:39PM +0530, Rohit kumar wrote: > Add devicetree bindings documentation file for Qualcomm > Technolgies Inc ADSP Peripheral Image Loader. > > Signed-off-by: Rohit kumar > --- > .../bindings/remoteproc/qcom,adsp-pil.txt | 123 > + > 1 file changed, 123 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > new file mode 100644 > index 000..f1c215a > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > @@ -0,0 +1,123 @@ > +Qualcomm Technology Inc. ADSP Peripheral Image Loader > + > +This document defines the binding for a component that loads and boots > firmware > +on the Qualcomm Technology Inc. ADSP Hexagon core. > + > +- compatible: > + Usage: required > + Value type: > + Definition: must be one of: > + "qcom,sdm845-adsp-pil" > + > +- reg: > + Usage: required > + Value type: > + Definition: must specify the base address and size of the qdsp6ss > register > + > +- interrupts-extended: > + Usage: required > + Value type: > + Definition: must list the watchdog, fatal IRQs ready, handover and > + stop-ack IRQs > + > +- interrupt-names: > + Usage: required > + Value type: > + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" > + > +- clocks: > + Usage: required > + Value type: > + Definition: List of phandle and clock specifier pairs How many clocks? > + > +- clock-names: > + Usage: required > + Value type: > + Definition: List of clock input name strings sorted in the same > + order as the clocks property. What are the names? > + > +- power-domains: > + Usage: required > + Value type: > + Definition: reference to cx power domain node. > + > +- resets: > + Usage: required > + Value type: > + Definition: reference to the reset-controller for the lpass How many? > + > +- reset-names: > +Usage: required > +Value type: > +Definition: must be "pdc_sync" and "cc_lpass" > + > +- qcom,halt-regs: > + Usage: required > + Value type: > + Definition: a phandle reference to a syscon representing TCSR followed > + by the offset within syscon for lpass halt register. > + > +- memory-region: > + Usage: required > + Value type: > + Definition: reference to the reserved-memory for the ADSP > + > +- qcom,smem-states: > + Usage: required > + Value type: > + Definition: reference to the smem state for requesting the ADSP to > + shut down > + > +- qcom,smem-state-names: > + Usage: required > + Value type: > + Definition: must be "stop" > + > + > += SUBNODES > +The adsp node may have an subnode named "glink-edge" that describes the > +communication edge, channels and devices related to the ADSP. > +See ../soc/qcom/qcom,glink.txt for details on how to describe these. > + > += EXAMPLE > +The following example describes the resources needed to boot control the > +ADSP, as it is found on SDM845 boards. > + adsp-pil { > + compatible = "qcom,sdm845-adsp-pil"; > + > + reg = <0x1730 0x40c>; > + > + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "wdog", "fatal", "ready", > + "handover", "stop-ack"; > + > + clocks = < RPMH_CXO_CLK>, > + < GCC_LPASS_SWAY_CLK>, > + < LPASS_AUDIO_WRAPPER_AON_CLK>, > + < LPASS_Q6SS_AHBS_AON_CLK>, > + < LPASS_Q6SS_AHBM_AON_CLK>, > + < LPASS_QDSP6SS_XO_CLK>, > + < LPASS_QDSP6SS_SLEEP_CLK>, > + < LPASS_QDSP6SS_CORE_CLK>; > + clock-names = "xo", "sway_cbcr", "lpass_aon", > + "lpass_ahbs_aon_cbcr", > + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", > + "qdsp6ss_sleep", "qdsp6ss_core"; > + > + power-domains = < SDM845_CX>; > + > + resets = <_reset PDC_AUDIO_SYNC_RESET>, > + <_reset AOSS_CC_LPASS_RESTART>; > + reset-names = "pdc_sync", "cc_lpass"; > + > + qcom,halt-regs = <_mutex_regs 0x22000>; > + > + memory-region = <_adsp_mem>; > + > + qcom,smem-states = <_smp2p_out 0>; > + qcom,smem-state-names = "stop"; > + }; > -- > Qualcomm India Private Limited, on behalf of
Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
On Mon 03 Sep 04:52 PDT 2018, Rohit kumar wrote: > Add devicetree bindings documentation file for Qualcomm > Technolgies Inc ADSP Peripheral Image Loader. > > Signed-off-by: Rohit kumar Rob, this revision looks good to me and I would like to move ahead and merge it. Regards, Bjorn > --- > .../bindings/remoteproc/qcom,adsp-pil.txt | 123 > + > 1 file changed, 123 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > new file mode 100644 > index 000..f1c215a > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > @@ -0,0 +1,123 @@ > +Qualcomm Technology Inc. ADSP Peripheral Image Loader > + > +This document defines the binding for a component that loads and boots > firmware > +on the Qualcomm Technology Inc. ADSP Hexagon core. > + > +- compatible: > + Usage: required > + Value type: > + Definition: must be one of: > + "qcom,sdm845-adsp-pil" > + > +- reg: > + Usage: required > + Value type: > + Definition: must specify the base address and size of the qdsp6ss > register > + > +- interrupts-extended: > + Usage: required > + Value type: > + Definition: must list the watchdog, fatal IRQs ready, handover and > + stop-ack IRQs > + > +- interrupt-names: > + Usage: required > + Value type: > + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" > + > +- clocks: > + Usage: required > + Value type: > + Definition: List of phandle and clock specifier pairs > + > +- clock-names: > + Usage: required > + Value type: > + Definition: List of clock input name strings sorted in the same > + order as the clocks property. > + > +- power-domains: > + Usage: required > + Value type: > + Definition: reference to cx power domain node. > + > +- resets: > + Usage: required > + Value type: > + Definition: reference to the reset-controller for the lpass > + > +- reset-names: > +Usage: required > +Value type: > +Definition: must be "pdc_sync" and "cc_lpass" > + > +- qcom,halt-regs: > + Usage: required > + Value type: > + Definition: a phandle reference to a syscon representing TCSR followed > + by the offset within syscon for lpass halt register. > + > +- memory-region: > + Usage: required > + Value type: > + Definition: reference to the reserved-memory for the ADSP > + > +- qcom,smem-states: > + Usage: required > + Value type: > + Definition: reference to the smem state for requesting the ADSP to > + shut down > + > +- qcom,smem-state-names: > + Usage: required > + Value type: > + Definition: must be "stop" > + > + > += SUBNODES > +The adsp node may have an subnode named "glink-edge" that describes the > +communication edge, channels and devices related to the ADSP. > +See ../soc/qcom/qcom,glink.txt for details on how to describe these. > + > += EXAMPLE > +The following example describes the resources needed to boot control the > +ADSP, as it is found on SDM845 boards. > + adsp-pil { > + compatible = "qcom,sdm845-adsp-pil"; > + > + reg = <0x1730 0x40c>; > + > + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "wdog", "fatal", "ready", > + "handover", "stop-ack"; > + > + clocks = < RPMH_CXO_CLK>, > + < GCC_LPASS_SWAY_CLK>, > + < LPASS_AUDIO_WRAPPER_AON_CLK>, > + < LPASS_Q6SS_AHBS_AON_CLK>, > + < LPASS_Q6SS_AHBM_AON_CLK>, > + < LPASS_QDSP6SS_XO_CLK>, > + < LPASS_QDSP6SS_SLEEP_CLK>, > + < LPASS_QDSP6SS_CORE_CLK>; > + clock-names = "xo", "sway_cbcr", "lpass_aon", > + "lpass_ahbs_aon_cbcr", > + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", > + "qdsp6ss_sleep", "qdsp6ss_core"; > + > + power-domains = < SDM845_CX>; > + > + resets = <_reset PDC_AUDIO_SYNC_RESET>, > + <_reset AOSS_CC_LPASS_RESTART>; > + reset-names = "pdc_sync", "cc_lpass"; > + > + qcom,halt-regs = <_mutex_regs 0x22000>; > + > + memory-region = <_adsp_mem>; > + > + qcom,smem-states = <_smp2p_out 0>; > + qcom,smem-state-names = "stop"; > + }; > -- > Qualcomm India
Re: [PATCH v3 1/2] dt-binding: remoteproc: Add QTI ADSP PIL bindings
On Mon 03 Sep 04:52 PDT 2018, Rohit kumar wrote: > Add devicetree bindings documentation file for Qualcomm > Technolgies Inc ADSP Peripheral Image Loader. > > Signed-off-by: Rohit kumar Rob, this revision looks good to me and I would like to move ahead and merge it. Regards, Bjorn > --- > .../bindings/remoteproc/qcom,adsp-pil.txt | 123 > + > 1 file changed, 123 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > new file mode 100644 > index 000..f1c215a > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt > @@ -0,0 +1,123 @@ > +Qualcomm Technology Inc. ADSP Peripheral Image Loader > + > +This document defines the binding for a component that loads and boots > firmware > +on the Qualcomm Technology Inc. ADSP Hexagon core. > + > +- compatible: > + Usage: required > + Value type: > + Definition: must be one of: > + "qcom,sdm845-adsp-pil" > + > +- reg: > + Usage: required > + Value type: > + Definition: must specify the base address and size of the qdsp6ss > register > + > +- interrupts-extended: > + Usage: required > + Value type: > + Definition: must list the watchdog, fatal IRQs ready, handover and > + stop-ack IRQs > + > +- interrupt-names: > + Usage: required > + Value type: > + Definition: must be "wdog", "fatal", "ready", "handover", "stop-ack" > + > +- clocks: > + Usage: required > + Value type: > + Definition: List of phandle and clock specifier pairs > + > +- clock-names: > + Usage: required > + Value type: > + Definition: List of clock input name strings sorted in the same > + order as the clocks property. > + > +- power-domains: > + Usage: required > + Value type: > + Definition: reference to cx power domain node. > + > +- resets: > + Usage: required > + Value type: > + Definition: reference to the reset-controller for the lpass > + > +- reset-names: > +Usage: required > +Value type: > +Definition: must be "pdc_sync" and "cc_lpass" > + > +- qcom,halt-regs: > + Usage: required > + Value type: > + Definition: a phandle reference to a syscon representing TCSR followed > + by the offset within syscon for lpass halt register. > + > +- memory-region: > + Usage: required > + Value type: > + Definition: reference to the reserved-memory for the ADSP > + > +- qcom,smem-states: > + Usage: required > + Value type: > + Definition: reference to the smem state for requesting the ADSP to > + shut down > + > +- qcom,smem-state-names: > + Usage: required > + Value type: > + Definition: must be "stop" > + > + > += SUBNODES > +The adsp node may have an subnode named "glink-edge" that describes the > +communication edge, channels and devices related to the ADSP. > +See ../soc/qcom/qcom,glink.txt for details on how to describe these. > + > += EXAMPLE > +The following example describes the resources needed to boot control the > +ADSP, as it is found on SDM845 boards. > + adsp-pil { > + compatible = "qcom,sdm845-adsp-pil"; > + > + reg = <0x1730 0x40c>; > + > + interrupts-extended = < 0 162 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, > + <_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "wdog", "fatal", "ready", > + "handover", "stop-ack"; > + > + clocks = < RPMH_CXO_CLK>, > + < GCC_LPASS_SWAY_CLK>, > + < LPASS_AUDIO_WRAPPER_AON_CLK>, > + < LPASS_Q6SS_AHBS_AON_CLK>, > + < LPASS_Q6SS_AHBM_AON_CLK>, > + < LPASS_QDSP6SS_XO_CLK>, > + < LPASS_QDSP6SS_SLEEP_CLK>, > + < LPASS_QDSP6SS_CORE_CLK>; > + clock-names = "xo", "sway_cbcr", "lpass_aon", > + "lpass_ahbs_aon_cbcr", > + "lpass_ahbm_aon_cbcr", "qdsp6ss_xo", > + "qdsp6ss_sleep", "qdsp6ss_core"; > + > + power-domains = < SDM845_CX>; > + > + resets = <_reset PDC_AUDIO_SYNC_RESET>, > + <_reset AOSS_CC_LPASS_RESTART>; > + reset-names = "pdc_sync", "cc_lpass"; > + > + qcom,halt-regs = <_mutex_regs 0x22000>; > + > + memory-region = <_adsp_mem>; > + > + qcom,smem-states = <_smp2p_out 0>; > + qcom,smem-state-names = "stop"; > + }; > -- > Qualcomm India