Messages in this thread | | | From | <> | Subject | RE: [PATCH v7 2/3] dt-bindings: remoteproc: qcom: Add SC7280 WPSS support | Date | Tue, 2 Nov 2021 19:15:09 +0530 |
| |
> -----Original Message----- > From: Stephen Boyd <swboyd@chromium.org> > Sent: Saturday, October 30, 2021 12:34 AM > To: agross@kernel.org; bjorn.andersson@linaro.org; > mathieu.poirier@linaro.org; ohad@wizery.com; p.zabel@pengutronix.de; > pillair@codeaurora.org; robh+dt@kernel.org > Cc: linux-arm-msm@vger.kernel.org; linux-remoteproc@vger.kernel.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; > sibis@codeaurora.org; mpubbise@codeaurora.org; kuabhs@chromium.org > Subject: RE: [PATCH v7 2/3] dt-bindings: remoteproc: qcom: Add SC7280 > WPSS support > > Quoting pillair@codeaurora.org (2021-10-29 03:46:03) > > > > > > + > > > > + glink-edge { > > > > + interrupts-extended = <&ipcc IPCC_CLIENT_WPSS > > > > + IPCC_MPROC_SIGNAL_GLINK_QMP > > > > + IRQ_TYPE_EDGE_RISING>; > > > > + mboxes = <&ipcc IPCC_CLIENT_WPSS > > > > + IPCC_MPROC_SIGNAL_GLINK_QMP>; > > > > + > > > > + label = "wpss"; > > > > + qcom,remote-pid = <13>; > > > > > > There are a few properties here that don't seem to be required. Is > > > that intentional? > > > > Hi Stephen, > > All the properties in the example are listed as required (except for status, > which will be removed in the subsequent patchset). > > Do you mean the glink-edge node properties ? > > Yes I mean all the properties in the glink-edge node. Are they required? > If so then we need to list them in the schema.
Hi Stephen, I have sent v8 with glink-edge properties also included in the dt-bindings.
Thanks, Rakesh Pillai.
| |