From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754160AbcHWSc1 (ORCPT ); Tue, 23 Aug 2016 14:32:27 -0400 Received: from mail-oi0-f67.google.com ([209.85.218.67]:34137 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750879AbcHWScX (ORCPT ); Tue, 23 Aug 2016 14:32:23 -0400 Date: Tue, 23 Aug 2016 13:31:21 -0500 From: Rob Herring To: Bjorn Andersson Cc: Ohad Ben-Cohen , Andy Gross , Mark Rutland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-soc@vger.kernel.org Subject: Re: [PATCH v2 1/4] dt-binding: remoteproc: Introduce ADSP loader binding Message-ID: <20160823183121.GA13010@rob-hp-laptop> References: <1471931866-3107-1-git-send-email-bjorn.andersson@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1471931866-3107-1-git-send-email-bjorn.andersson@linaro.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 22, 2016 at 10:57:43PM -0700, Bjorn Andersson wrote: > This document defines the binding for a component that loads firmware > and control the life cycle of the Qualcomm ADSP core. > > Signed-off-by: Bjorn Andersson > --- > > Changes since v1: > - Added platform names to compatibility > > .../devicetree/bindings/remoteproc/qcom,adsp.txt | 70 ++++++++++++++++++++++ > 1 file changed, 70 insertions(+) > create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,adsp.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.txt b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.txt > new file mode 100644 > index 000000000000..3820151ce3e9 > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.txt > @@ -0,0 +1,70 @@ > +Qualcomm ADSP Peripheral Image Loader > + > +This document defines the binding for a component that loads and boots firmware > +on the Qualcomm ADSP core. ADSP is for Audio DSP? So there is another driver for the audio functions? Why doesn't it do the firmware loading? I'm still confused why this binding is separate? If you had one common interface (a rproc) to load and boot various other blocks like ADSP and Venus, then this would make sense. Or does every accel block have some separate control uC associated with it? > + > +- compatible: > + Usage: required > + Value type: > + Definition: must be one of: > + "qcom,msm8974-adsp-pil" > + "qcom,msm8996-adsp-pil" > + > +- 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" > + > +- cx-supply: > + Usage: required > + Value type: > + Definition: reference to the regulator to be held on behalf of the > + booting of the Hexagon core > + > +- 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" > + > += EXAMPLE > +The following example describes the resources needed to boot control the > +ADSP, as it is found on MSM8974 boards. > + > + adsp { > + compatible = "qcom,adsp-pil"; > + > + interrupts-extended = <&intc 0 162 IRQ_TYPE_EDGE_RISING>, > + <&adsp_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <&adsp_smp2p_in 1 IRQ_TYPE_EDGE_RISING>, > + <&adsp_smp2p_in 2 IRQ_TYPE_EDGE_RISING>, > + <&adsp_smp2p_in 3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "wdog", > + "fatal", > + "ready", > + "handover", > + "stop-ack"; > + > + cx-supply = <&pm8841_s2>; > + > + memory-region = <&adsp_region>; > + > + qcom,smem-states = <&modem_smp2p_out 0>; > + qcom,smem-state-names = "stop"; > + }; > -- > 2.5.0 >