Re: Bluetooth and 6310

[Marcel Holtmann]

Hi Pawel,

> So here is the patch I use. It's not yet fully finished, but
> should work for all the phones supported by nk6510 driver.
> Use not changed config, ie. use bluetooth specific options:
>  - rfcomm_cn
>  - bt_address
> It doesn't matter what you'll give as rfcomm channell, it
> will be overwritten with 14 value.

nice work. The patch is working fine with my Nokia 6310 and it runs now
for over 2 hours in the monitor mode without any problems.

> You need(?) to run hcid and sdpd before running gnokii.

You only need hcid, because it handles the PIN and link key management.
The sdpd is not needed, but in general it is a good idea to start it.

But I also have some comments about the patch. It is not a good idea to
use the string BLUEZ in the code. The complete stack is named BlueZ, but
we try to avoid naming constants or functions like this. And Nokia also
names the protocol "Phonet", which they run over Bluetooth. So I think
you should use the following names

        FBUS_PHONET_IRDA_FRAME_ID       etc.
        FBUS_PHONET_BLUETOOTH_FRAME_ID  etc.

And don't forget the Nokia 6210, which needs the AT*NOKIAFBUS stuff ;)

Regards

Marcel