[Letux-kernel] [PATCH letux 0/3] GTA04A3/4 Bluetooth via serdev

H. Nikolaus Schaller hns at goldelico.com
Mon Nov 12 19:00:28 CET 2018


Hi,

> Am 12.11.2018 um 17:00 schrieb Andreas Kemnade <andreas at kemnade.info>:
> 
> Hi,
> 
>> Well, the patches got a little separated, because we cut things differently (dt-gta04 and drivers).
>> Which is not a big problem since maintainers also want to separate new drivers and DT modifications
>> (different mailing lists).
> 
> So in the end we have three patch series?
> 1. bluetooth h4 code + devicetree bindings documentation
> 2. powerseq patches
> 3. dtb updates for gta04

Yes.

Sequence of submissions would probably be

1. new functionality
3. make use of new driver
2. work around reset issue

> 
> which all should be upstreamed separated?
> Who takes which series?
> I would probably try to upstream 1.

Yes, I can push 3. (after some consolidation).

I think 2. could be postponed since it is not absolutely necessary
for the other two to be useful and needs a little more thinking
(how do we name the DT property?) and improvement/testing.

I could also submit all of them directly from the letux-4.20-rc2 tree
(which is finished, but not pushed because I hope we can fix the Pyra
panel quickly).

More important is that we test them and add the missing pieces first.


One note for the bindings doc and the default baud rate of the w2cbw003
to consider:

the factory default rate is (was) 115200 for the module. AFAIR, there
was also a variant with ca. 920000. But we have reprogrammed ALL modules
installed in a GTA04 right after production during production test to
3 Mbit/s through a special command (bccmd if I remember correctly
http://projects.goldelico.com/p/gta04-kernel/page/Wireless/?rev=333#wikititle_9 ).

So ALL GTA04 that exist are using the high baudrate.

Nevertheless, the driver IMHO should default to 115200... And maybe the
bindings document should use an example with that or a different speed.
And a note in the speed property description, that the module must be
reprogrammed to accomodate the speed.

BTW: I don't know if the bccmd works on the new hci driver and most likely
can not change the baud rate on the fly.

BR,
Nikolaus




-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.goldelico.com/pipermail/letux-kernel/attachments/20181112/bb21414f/attachment.asc>


More information about the Letux-kernel mailing list