Return-Path: MIME-Version: 1.0 In-Reply-To: <1464750564-21027-1-git-send-email-wnhuang@chromium.org> References: <1464750564-21027-1-git-send-email-wnhuang@chromium.org> From: Wei-Ning Huang Date: Wed, 1 Jun 2016 14:58:28 +0800 Message-ID: Subject: Re: [PATCH] dt: bindings: fix documentation for MARVELL's bt-sd8xxx wireless device To: Linux Bluetooth Cc: LKML , Amitkumar Karwar , Wei-Ning Huang , robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, Hu Xinming , marcel@holtmann.org, devicetree@vger.kernel.org Content-Type: multipart/alternative; boundary=94eb2c034a544bbeba053432045e List-ID: --94eb2c034a544bbeba053432045e Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Please ignore this patch. I think we should make the driver parse 32 bit value instead (to be consistent with mwifiex driver). On Wed, Jun 1, 2016 at 11:09 AM, Wei-Ning Huang wrote: > The property marvell,wakeup-pin and marvell,wakeup-gap-ms are read as > u16 in the driver. Fix documentation and example accordingly. > > Signed-off-by: Wei-Ning Huang > --- > Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt > b/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt > index 14aa6cf..6a9a63c 100644 > --- a/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt > +++ b/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt > @@ -13,10 +13,10 @@ Optional properties: > initialization. This is an array of 28 values(u8). > > - marvell,wakeup-pin: It represents wakeup pin number of the bluetooth > chip. > - firmware will use the pin to wakeup host system. > + firmware will use the pin to wakeup host system > (u16). > - marvell,wakeup-gap-ms: wakeup gap represents wakeup latency of the > host > platform. The value will be configured to firmware. > This > - is needed to work chip's sleep feature as expected. > + is needed to work chip's sleep feature as expected > (u16). > - interrupt-parent: phandle of the parent interrupt controller > - interrupts : interrupt pin number to the cpu. Driver will request an > irq based > on this interrupt number. During system suspend, the irq > will be > @@ -50,7 +50,7 @@ calibration data is also available in below example. > 0x37 0x01 0x1c 0x00 0xff 0xff 0xff 0xff 0x01 0x7f > 0x04 0x02 > 0x00 0x00 0xba 0xce 0xc0 0xc6 0x2d 0x00 0x00 0x00 > 0x00 0x00 > 0x00 0x00 0xf0 0x00>; > - marvell,wakeup-pin =3D <0x0d>; > - marvell,wakeup-gap-ms =3D <0x64>; > + marvell,wakeup-pin =3D /bits/ 16 <0x0d>; > + marvell,wakeup-gap-ms =3D /bits/ 16 <0x64>; > }; > }; > -- > 2.1.2 > > --=20 Wei-Ning Huang, =E9=BB=83=E5=81=89=E5=AF=A7 | Software Engineer, Google Inc= ., Taiwan | wnhuang@google.com | Cell: +886 910-380678 --94eb2c034a544bbeba053432045e Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Please ignore this patch. I think we should make the drive= r parse 32 bit value instead (to be consistent with mwifiex driver).
<= div class=3D"gmail_extra">
On Wed, Jun 1, 201= 6 at 11:09 AM, Wei-Ning Huang <wnhuang@chromium.org> wrot= e:
The property marvell,wakeup-pin and ma= rvell,wakeup-gap-ms are read as
u16 in the driver. Fix documentation and example accordingly.

Signed-off-by: Wei-Ning Huang <w= nhuang@chromium.org>
---
=C2=A0Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt | 8 ++++-= ---
=C2=A01 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt b/= Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt
index 14aa6cf..6a9a63c 100644
--- a/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt
+++ b/Documentation/devicetree/bindings/net/marvell-bt-sd8xxx.txt
@@ -13,10 +13,10 @@ Optional properties:
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 initialization. This is an array of 28 values(u8).

=C2=A0 =C2=A0- marvell,wakeup-pin: It represents wakeup pin number of the b= luetooth chip.
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0firmware will use the pin to wakeup host system.
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0firmware will use the pin to wakeup host system (u16).
=C2=A0 =C2=A0- marvell,wakeup-gap-ms: wakeup gap represents wakeup latency = of the host
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 platform. The value will be configured to firmware. This
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0is needed to work chip's sleep feature as expected.
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0is needed to work chip's sleep feature as expected (u16).
=C2=A0 =C2=A0- interrupt-parent: phandle of the parent interrupt controller=
=C2=A0 =C2=A0- interrupts : interrupt pin number to the cpu. Driver will re= quest an irq based
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0on this inter= rupt number. During system suspend, the irq will be
@@ -50,7 +50,7 @@ calibration data is also available in below example.
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 0x37 0x01 0x1c 0x00 0xff 0xff 0xff 0xff 0x01 0x7f 0x04 0x02
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 0x00 0x00 0xba 0xce 0xc0 0xc6 0x2d 0x00 0x00 0x00 0x00 0x00
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 0x00 0x00 0xf0 0x00>;
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0marvell,wakeup-pin = =3D <0x0d>;
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0marvell,wakeup-gap-= ms =3D <0x64>;
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0marvell,wakeup-pin = =3D /bits/ 16 <0x0d>;
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0marvell,wakeup-gap-= ms =3D /bits/ 16 <0x64>;
=C2=A0 =C2=A0 =C2=A0 =C2=A0 };
=C2=A0};
--
2.1.2




--
Wei-Ning Huang, =E9=BB=83=E5=81=89=E5=AF=A7 |=C2=A0Sof= tware Engineer, Google Inc., Taiwan |=C2=A0wnhuang@google.com=C2=A0|=C2=A0Cell: +886 910-380678=C2=A0
--94eb2c034a544bbeba053432045e--