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

Groups > linux.kernel > #1329619

Re: [PATCH v3 1/2] bluetooth: btmrvl: register platform specific driver

Show all headers | View raw

On Mon, Feb 08, 2016 at 02:27:10AM -0800, Amitkumar Karwar wrote:
> From: Xinming Hu <huxm@marvell.com>
> 
> Platform device and driver provide easy way to
> interact with device-tree-enabled system.
> 
> This patch registers platform driver and reorganise
> existing device tree specific code.
> 
> Signed-off-by: Xinming Hu <huxm@marvell.com>
> Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> ---
> v3: Add NULL check for 'btmrvl_plt_dev'
> ---
>  Documentation/devicetree/bindings/btmrvl.txt |  8 ++--

This needs to move somewhere else.

>  drivers/bluetooth/Makefile                   |  1 +
>  drivers/bluetooth/btmrvl_drv.h               | 16 ++++++++
>  drivers/bluetooth/btmrvl_main.c              |  8 +++-
>  drivers/bluetooth/btmrvl_platform.c          | 60 ++++++++++++++++++++++++++++
>  5 files changed, 88 insertions(+), 5 deletions(-)
>  create mode 100644 drivers/bluetooth/btmrvl_platform.c
> 
> diff --git a/Documentation/devicetree/bindings/btmrvl.txt b/Documentation/devicetree/bindings/btmrvl.txt
> index 58f964b..25b804b 100644
> --- a/Documentation/devicetree/bindings/btmrvl.txt
> +++ b/Documentation/devicetree/bindings/btmrvl.txt
> @@ -3,7 +3,7 @@ btmrvl
>  
>  Required properties:
>  
> -  - compatible : must be "btmrvl,cfgdata"
> +  - compatible : must be "marvell,btmrvl"

While the new name is less worse than the old one, I'm not inclined to 
take any changes for this binding because it is problematic. Given you 
don't care this is an incompatible change, then the binding can be 
completely redone.

It should reflect the actual chip name (e.g. sd8???) and be a child node 
of its connection to the host (a UART). See UART slave discussions.

>  
>  Optional properties:
>  
> @@ -15,11 +15,11 @@ Optional properties:
>  
>  Example:
>  
> -GPIO pin 13 is configured as a wakeup source and GAP is set to 100 msecs
> -in below example.
> +GPIO pin 13 and gap 100ms are configured for host wakeup, calibration
> +data is also available in below example.

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

Thread

[PATCH v3 1/2] bluetooth: btmrvl: register platform specific driver Amitkumar Karwar <akarwar@marvell.com> - 2016-02-08 11:40 +0100
  [PATCH v3 2/2] bluetooth: btmrvl: parse host wakeup configuration from device tree Amitkumar Karwar <akarwar@marvell.com> - 2016-02-08 11:40 +0100
  Re: [PATCH v3 1/2] bluetooth: btmrvl: register platform specific  driver Rob Herring <robh@kernel.org> - 2016-02-08 23:10 +0100
    RE: [PATCH v3 1/2] bluetooth: btmrvl: register platform specific  driver Amitkumar Karwar <akarwar@marvell.com> - 2016-02-16 11:10 +0100

csiph-web