Skip to content

Adding support for the LSM303DLHC e-Compass #720

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

Closed
wants to merge 13 commits into from

Conversation

BryanSouza91
Copy link

I have added support for the LSM303DLHC e-Compass using the existing LSM303AGR driver as the starting point. I updated the registers as necessary from the information provided by the datasheet. I also fixed spelling in the connection error message in the LSM303AGR driver.

@deadprogram
Copy link
Member

Thanks for working on this @BryanSouza91

Can you please add an example, and also then add that example to the smoke tests? https://github.com/tinygo-org/drivers/blob/dev/smoketest.sh

Thank you!

ysoldak and others added 6 commits June 25, 2025 10:03
* lsm6ds3tr: avoid unnecessary heap allocations
* lsm6ds3tr: use helper functions, for readability
* lsm6ds3tr: return slice of the internal buffer on readBytes
Adafruit's Mini GPS PA1010D Module works with this device driver, but requires 0x10 as the address, rather than 0x42.

This change allows the device to be initialised with whatever i2c address is needed, while maintaining backward compatibility.

Adds new constants to allow easy configuration of both the ublox device and the PA1010D.
* ssd1306: avoid unnecessary heap allocations

* ssd1306: extract i2c and spi bus implementations

* ssd1306: refactor tests -- show fps and heap usage

* ssd1306: bring back the lost exported methods

* Adjust examples

* Fix smoketests for ssd1306
Adding 150MHz support for the RP2350
@BryanSouza91
Copy link
Author

@deadprogram I finally got around to adding the example and then adding that to the smoketest.

@deadprogram
Copy link
Member

@BryanSouza91 due to having added the merge commits to this PR, it is not really possible to merge in its current state. Perhaps you can cherry-pick your commits related to the LSM303DLHC into a separate branch, squash them, and then either overwrite the branch in this PR, or create a new PR.

What do you think?

@BryanSouza91
Copy link
Author

I shouldn't have an issue squashing then overwriting this branch, if I do I will create a new PR.

@BryanSouza91
Copy link
Author

BryanSouza91 commented Aug 10, 2025

@deadprogram I don't think that last commit will work... I have dev-PR branch ready to go.

@deadprogram
Copy link
Member

Closing since was replaced by #783 thank you!

@BryanSouza91 BryanSouza91 deleted the dev branch August 11, 2025 07:11
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

Successfully merging this pull request may close these issues.

6 participants