lkml.org 
[lkml]   [2016]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] dt: bindings: fix documentation for MARVELL's bt-sd8xxx wireless device
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 <wnhuang@chromium.org> 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 <wnhuang@chromium.org>
> ---
> 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 = <0x0d>;
> - marvell,wakeup-gap-ms = <0x64>;
> + marvell,wakeup-pin = /bits/ 16 <0x0d>;
> + marvell,wakeup-gap-ms = /bits/ 16 <0x64>;
> };
> };
> --
> 2.1.2
>



--
Wei-Ning Huang, 黃偉寧 | Software Engineer, Google Inc., Taiwan |
wnhuang@google.com | Cell: +886 910-380678

\
 
 \ /
  Last update: 2016-06-01 09:41    [W:0.043 / U:0.508 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site