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

Extend custom wire pin function for STM32 #45

Closed
wants to merge 2 commits into from

Conversation

BRTFTW
Copy link

@BRTFTW BRTFTW commented Sep 15, 2023

Hi Rob

I was using this project to evaluate the AS5600 with an STM32 Nucleo board. Somehow the standard wire pins did not function. What i ended up with, was setting the pins explicitly which was somehow needed for my setup.

The function is secured with a now extended #ifdef.

Happy to help further tests on the STM32 platform

@RobTillaart
Copy link
Owner

Thanks for this PR.
I will look into it later today or tomorrow.

@RobTillaart RobTillaart self-assigned this Sep 15, 2023
@RobTillaart RobTillaart added the enhancement New feature or request label Sep 15, 2023
@RobTillaart
Copy link
Owner

Will look at the breaking code later.
At least I know what should work for STM32

RobTillaart added a commit that referenced this pull request Sep 16, 2023
@RobTillaart
Copy link
Owner

@BRTFTW
Created Pull Request #46 for STM32 support (based on your code)

  • fixed the ESP32 / build-CI fail
  • increased version number
  • minor edits

Can you give the develop branch a try if it works for the STM32?
(if time permits I'll check if there are minor things to add to this release)

@RobTillaart
Copy link
Owner

Installed STM32 compilation and tested an example (adapted the begin() statement.
The defined(STM32) did not work, replaced with defined(ARDUINO_ARCH_STM32) which works for me
build is running

Can you give the develop branch a try if it works for the STM32?
(if time permits I'll check if there are minor things to add to this release)

@BRTFTW
Copy link
Author

BRTFTW commented Sep 18, 2023

Sure, I'll have a look at it on Wednesday.

@BRTFTW BRTFTW closed this Sep 20, 2023
RobTillaart added a commit that referenced this pull request Sep 20, 2023
- fix #45 support STM32 set I2C pins  ARDUINO_ARCH_STM32
- update readme badges
- minor edits
@laptophead
Copy link

Rob, it is Mitch again, (Laptophead)

Having 2 sensors on one bus works. But there is still the issue of not getting good initial readings.

Despite the
as5600_R.resetPosition();
as5600_L.resetPosition();

when there are 2 sensors I get erroneous readings such as
REV R: -1
REV L: 0
DEG R= 377487200°
DEG L= 132°

Rolling the motors forward increases the angles as expected, but rolling them back past 0 deg. is
resulting in those huge numbers . Ideally it would go to 359 , 358 etc degrees ,

Would you know what to do about that?
Thanks
mitch

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

Successfully merging this pull request may close these issues.

3 participants