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

ESP32-S3 Series #100

Closed
gin66 opened this issue Nov 14, 2021 · 8 comments
Closed

ESP32-S3 Series #100

gin66 opened this issue Nov 14, 2021 · 8 comments
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@gin66
Copy link
Owner

gin66 commented Nov 14, 2021

This device contains mcpwm module, but only 4 instead of 8 pulse counters.
Thus 4 steppers should be possible to be supported.

Anyone available for test, so an update of the driver makes sense ?

@gin66 gin66 added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Nov 14, 2021
@doudar
Copy link

doudar commented Apr 8, 2022

Sure, I'm available to test if you like.

Just tried a quick compile for S3 from master on an existing large project (SmartSpin2k). Relevant platformio.ini:

platform = https://github.com/tasmota/platform-espressif32/archive/refs/heads/IDF44/ESP32-S3.zip
board = esp32-s3-devkitc-1
framework = arduino

errors (Probably because you haven't implemented anything yet):

.pio/libdeps/release/FastAccelStepper/src/common.h:150:2: error: #error "Unsupported derivate" #error "Unsupported derivate" ^~~~~ In file included from .pio/libdeps/release/FastAccelStepper/src/common.h:162, from .pio/libdeps/release/FastAccelStepper/src/FastAccelStepper.h:5,

.pio/libdeps/release/FastAccelStepper/src/common.h:168:21: error: 'QUEUES_MCPWM_PCNT' was not declared in this scope #define NUM_QUEUES (QUEUES_MCPWM_PCNT + QUEUES_RMT)

.pio/libdeps/release/FastAccelStepper/src/common.h:168:41: error: 'QUEUES_RMT' was not declared in this scope #define NUM_QUEUES (QUEUES_MCPWM_PCNT + QUEUES_RMT)

@gin66
Copy link
Owner Author

gin66 commented Apr 11, 2022

Thanks for your willingness to test on esp32-s3. Please be patient, because these days I am quite busy. Hope to start on the adaptation within the next two weeks.

The driver already supports rmt in addition to mcpwm+pcnt. This means, two possible paths to follow. During the adaptation will see, which path generates the least effort.

@martinmay67
Copy link

martinmay67 commented Nov 11, 2022

Hi guys, any progress on this? At least idea about which direction to go?

@martinmay67
Copy link

Okay, I dis some changes and have a version that works with ESP32S3. Should I create PR?

@gin66
Copy link
Owner Author

gin66 commented Nov 16, 2022

sure. Looking forward to this

@HeldeReis
Copy link
Contributor

Okay, I dis some changes and have a version that works with ESP32S3. Should I create PR?

Hello Martin,

Can you share your modifications?
I will be migrating a project soon to the esp32-s3 and it would be of great help.

@HeldeReis
Copy link
Contributor

Hello gin66,

I made some changes to allow support to ESP32 S3 and made a pullrequest (ESP32 S3 MCPWM/PCNT Support #162).
It is the first time that i make something on gihub so i hope not to have missed something.
I am available if you need some specific tests to be done.

@gin66
Copy link
Owner Author

gin66 commented Mar 2, 2023

Thanks for the patch. I have merged in and the github action to build the sw has worked.
So I release it with 0.28.4

@gin66 gin66 closed this as completed Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants