Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1217111

Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree documentation

From Bjorn Andersson <bjorn.andersson@sonymobile.com>
Newsgroups linux.kernel
Subject Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree documentation
Date 2015-09-02 00:00 +0200
Message-ID <q41KW-2cN-5@gated-at.bofh.it> (permalink)
References <q29jz-2Rz-11@gated-at.bofh.it> <q29jz-2Rz-17@gated-at.bofh.it> <q3OXn-UC-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue 01 Sep 01:18 PDT 2015, Stanimir Varbanov wrote:

> On 08/27/2015 08:37 PM, Bjorn Andersson wrote:
> > This documents a device tree binding for exposing the Qualcomm Shared
> > Memory State Machine as a set of gpio- and interrupt-controllers.
> > 
> > Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
> > ---
> >  .../devicetree/bindings/gpio/qcom,smsm.txt         | 114 +++++++++++++++++++++
> 
> <snip>
> 
> > += EXAMPLE
> > +The following example shows the SMEM setup for controlling properties of the
> > +wireless processor, defined from the 8974 apps processor's point-of-view. It
> > +encompasses one outbound entry and the outgoing interrupt for the wireless
> > +processor.
> > +
> > +smsm {
> > +	compatible = "qcom,smsm";
> > +
> > +	#address-cells = <1>;
> > +	#size-cells = <0>;
> > +
> > +	qcom,ipc-3 = <&apcs 8 19>;
> 
> Can we use something more descriptive here, for example
> 
> qcom,ipc-rpm = <&apcs 8 19>;

Not really, because in the view of smsm we don't now which host the rpm
would represent. But most likely 3 would mean WCNSS, per the enum in
caf.

> 
> and replace these magic numbers with defines?
> 

I don't know what those defines would be named, what you have here is
bit 19 in the 8th byte - straight from the register documentation.

> > +
> > +	apps_smsm: apps@0 {
> > +		reg = <0>;
> > +
> > +		gpio-controller;
> > +		#gpio-cells = <2>;
> > +	};
> > +
> > +	wcnss_smsm: wcnss@7 {
> > +		reg = <7>;
> > +		interrupts = <0 144 1>;
> > +
> > +		interrupt-controller;
> > +		#interrupt-cells = <2>;
> > +	};
> > +};

Regards,
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree documentation Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-08-27 19:40 +0200
  Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree documentation Rob Herring <robherring2@gmail.com> - 2015-08-31 17:50 +0200
    Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree  documentation Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-09-01 07:30 +0200
  Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree documentation Stanimir Varbanov <svarbanov@mm-sol.com> - 2015-09-01 10:20 +0200
    Re: [PATCH 1/4] dt-binding: gpio: Add Qualcomm SMSM device tree  documentation Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-09-02 00:00 +0200

csiph-web