Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bluetooth Long Range Support #259

Open
rosek86 opened this issue Aug 26, 2020 · 5 comments
Open

Bluetooth Long Range Support #259

rosek86 opened this issue Aug 26, 2020 · 5 comments

Comments

@rosek86
Copy link

rosek86 commented Aug 26, 2020

Hello,

We successfully use pc-ble-driver-js for a few years now, it works very well and till now it suited all our needs. We're now requested to incorporate Bluetooth Long Range in one of our next product.

It seems that pc-ble-driver-js has support for V5 now which is still s132 (no support for long-range). Do you have any plans to upgrade to s140 in the near future (pc-ble-driver V6)? Is there a roadmap available?

Kind regards,
Krzysztof

@bihanssen
Copy link
Contributor

Hi, @rosek86.
We have held back on v6 support, mainly due to poor scan performance when used over a serialized transport, due to a change in behavior of the scan API in SD v6.
Question, in your case, would degraded performance of scanning (and possibly loss of adv packets) be an issue?

@rosek86
Copy link
Author

rosek86 commented Aug 27, 2020

Hi @bihanssen,

Thank you very much for your speedy reply.

Everything depends on the rate of adv packets loss. I don't think we have high constraints here, though. We modify pc-ble-driver (firmware) to reject all adverts which don't belong to our products, and we keep ~200 ms adverts interval.

If there is a development version we'd be more than happy to give this a go.

@bihanssen
Copy link
Contributor

Hi,
there's no development version available currently of ble-driver-js with SD v6. However, regarding performance of scanning it would behave similarly to ble-driver (C/C++) if you would like to check it out immediately.

@ShaharHD
Copy link
Contributor

ShaharHD commented May 22, 2022

@rosek86

I would take the pc-ble-driver-v6 and compare the changes between the h files of v5 and v6, branch out this repo and implement the missing headers/functions in adapter.js and the various other files.

Follow the

#if NRF_SD_BLE_API_VERSION >= 5

as a guildlines for how they support v2 and v5 currently, you can add v6.

Shahar

@td-krzysiek
Copy link

td-krzysiek commented May 25, 2022

Thanks, we would like to make an attempt with Bluetooth Long Range but it has lower priority at the moment and we can't embark upgrade to v6 ourself. It seems that Nordic Semiconductor abandon the project so it is unlikely that v6 will be implemented by them anytime soon (I have couple pull requests hanging here for months now too).

I think there are two main approaches for our Node.js software, upgrade this repository to v6 at one point (with cooperation to yourself?) or migrate the firmware to Zephyr HCI controller and use e.g. noble with a facade layer which mimics API from this library.

For the second approach I also started working on altenative library which connects to Zephyr HCI controller but due to lack of time after work I can't support this recently too (https://github.com/rosek86/n-ble).

Kind regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants