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

[Wei-Ning Huang <wnhuang@google.com>]

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