Skip to content

Method that should be added. #718

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

Open
Gabriel192K opened this issue Mar 24, 2025 · 2 comments
Open

Method that should be added. #718

Gabriel192K opened this issue Mar 24, 2025 · 2 comments

Comments

@Gabriel192K
Copy link

Hello, I was using this great library for quite some time but I never got around the fact it lacks a very basic method i.e. function. The method I imply should return some sort of device ID or chip version. This would be a great and simple way to check if the device is alive and/or connected. I saw you already used this but as a local check inside the be begin() function. So my whole point is to add a public method called version() or something similar to read that device ID. I'm already using this modified library and it works like a charm.

@waltje
Copy link

waltje commented Apr 26, 2025

I would name that getDeviceId() - version() would more relate to the library's version imho.

@Gabriel192K
Copy link
Author

@waltje yes, you’re right. The name is irrelevant as the point is to have a public method to access that register.

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

2 participants