Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-10 Thread Pavel Machek
Hi! > You have missed some things. See inline comments below. Ok, I've done this, it should fix the rest. diff --git a/drivers/bluetooth/nokia_core.c b/drivers/bluetooth/nokia_core.c index d6e9b3911..6615939 100644 --- a/drivers/bluetooth/nokia_core.c +++ b/drivers/bluetooth/nokia_core.c @@ -113

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-10 Thread Sebastian Reichel
Hi Pavel, You have missed some things. See inline comments below. On Fri, Jan 10, 2014 at 01:18:05PM +0100, Pavel Machek wrote: > diff --git a/drivers/bluetooth/nokia_core.c b/drivers/bluetooth/nokia_core.c > index 5c7acad..d6b0701 100644 > --- a/drivers/bluetooth/nokia_core.c > v+++ b/drivers/bl

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-10 Thread Pavel Machek
On Fri 2014-01-10 01:32:31, Sebastian Reichel wrote: > Hi Pavel, > > On Fri, Jan 10, 2014 at 12:38:43AM +0100, Pavel Machek wrote: > > > Here are some cleanup suggestions for probe, removal & module > > > initialization functions. > > > > ...and here's the patch implementing those suggestions. Th

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-09 Thread Sebastian Reichel
Hi Pavel, On Fri, Jan 10, 2014 at 12:38:43AM +0100, Pavel Machek wrote: > > Here are some cleanup suggestions for probe, removal & module > > initialization functions. > > ...and here's the patch implementing those suggestions. Thanks! That looks right, but you forgot to cleanup hci_h4p_remove.

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-09 Thread Pavel Machek
Hi! > Here are some cleanup suggestions for probe, removal & module > initialization functions. ...and here's the patch implementing those suggestions. Thanks! Reported-by: Sebastian Reichel Signed-off-by: Pavel Machek diff --git a/drivers/bluetooth/nokia_core.c b/drivers/bluetooth/nokia_core

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-05 Thread Pavel Machek
> > > The drivers can still be initialized the old way using pdata quirks > > > until all drivers are converted, but I think this driver can simply > > > be prepared for DT directly: > > > > Well, normally DT support means a bit of argumentation and bikeshed > > painting... I believe it would be

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-05 Thread Sebastian Reichel
On Sun, Jan 05, 2014 at 11:32:50PM +0100, Pavel Machek wrote: > > > Changes from v3: Moved platform data into > > > include/linux/platform_data/, something I missed before. > > > > As I wrote before Tony plans to remove the boardcode for all > > OMAP boards including the Nokia N900 for 3.14, so yo

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-05 Thread Pavel Machek
Hi! > > Changes from v3: Moved platform data into > > include/linux/platform_data/, something I missed before. > > As I wrote before Tony plans to remove the boardcode for all > OMAP boards including the Nokia N900 for 3.14, so you cannot > boot without DT from 3.14 onwards. Well, I believe it w

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-02 Thread Sebastian Reichel
Hi Pavel, Here are some cleanup suggestions for probe, removal & module initialization functions. On Fri, Jan 03, 2014 at 01:17:54AM +0100, Pavel Machek wrote: > +static int hci_h4p_probe(struct platform_device *pdev) > +{ > + struct hci_h4p_platform_data *bt_plat_data; > + struct hci_h4p

Re: [PATCH v4] Bluetooth: Add hci_h4p driver

2014-01-02 Thread Sebastian Reichel
Hi, On Fri, Jan 03, 2014 at 01:17:54AM +0100, Pavel Machek wrote: > Changes from v3: Moved platform data into > include/linux/platform_data/, something I missed before. As I wrote before Tony plans to remove the boardcode for all OMAP boards including the Nokia N900 for 3.14, so you cannot boot w