Thread (3 messages) 3 messages, 2 authors, 2016-06-01

Re: [PATCH] dt: bindings: fix documentation for MARVELL's bt-sd8xxx wireless device

flat view

From: Wei-Ning Huang <hidden>
Date: 2016-06-01 07:25:09
Also in: linux-devicetree, lkml

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 [off-list ref] wrot=
e:
quoted hunk ↗ jump to hunk
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 <redacted>
---
 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
quoted hunk ↗ jump to hunk
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 h=
ost
                      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
quoted hunk ↗ jump to hunk
@@ -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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help