Skip to content

Update ESP32SJA1000.cpp #155

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ZillKhan
Copy link

Resolving CAN driver issue for ESP32S3 variant as it doesn't have dport register available.

Resolving CAN driver issue for ESP32S3 variant as it doesn't have dport register available.
@HenokTZA
Copy link

HenokTZA commented May 8, 2025

Hello, Thank you for the fix

were trying it and the below error throws, would be glad if you help

   10 |     #include "soc/system_reg.h"
      |              ^~~~~~~~~~~~~~~~~~
compilation terminated.

exit status 1

Compilation error: exit status 1```


@ZillKhan
Copy link
Author

ZillKhan commented May 9, 2025

@HenokTZA Hello, Would be glad to help you.
To get better understanding which esp32 vairant are you using?

@HenokTZA
Copy link

HenokTZA commented May 9, 2025

i'm using esp32 s3

@ZillKhan
Copy link
Author

@HenokTZA I have rechecked its working on my side as I am able to compile the code using Arduino IDE.
Which version of Arduino are you using also are you using the patch-1 branch that I have fixed?
Please recheck once again on your side.

Please try using this branch to verify if its working (able to compile any example)
https://github.com/ZillKhan/arduino-CAN/tree/patch-1

@ZillKhan
Copy link
Author

@HenokTZA The compilation error you have shown
10 | #include "soc/system_reg.h"
Shows that the branch you are using doesn't include my fix.
Please use the branch-1 from here and try to compile. (https://github.com/ZillKhan/arduino-CAN/tree/patch-1)
Let me know if it works.

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

Successfully merging this pull request may close these issues.

2 participants