Skip to content

[MKC-1854] GIGA CAN / ADC fixes #2184

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

Merged
merged 6 commits into from
Oct 3, 2024
Merged

Conversation

karlsoderby
Copy link
Contributor

The GIGA documentation had incorrectly labelled CANTX/CANRX pins (D93/D94) as ADC pins. These pins are not connected to the ADC.

  • Update CANTX / CANRX mentions in CS / Audio guide.

Contribution Guidelines

the CANRX and CANTX pins are not attached to the ADC, and they were incorrectly documented, likely due to previous boards with form factor having these pins as A14/15
@karlsoderby karlsoderby changed the title Karlsoderby/giga can adc fixes [MKC-1854] GIGA CAN / ADC fixes Sep 23, 2024
@Hannes7eicher Hannes7eicher added documentation Improvements or additions to documentation fix/update A small fix or update arduino Bugs and fixes added by the Arduino Team maker preview labels Sep 23, 2024
Copy link

github-actions bot commented Sep 23, 2024

Preview Deployment

🚀 Preview this PR: https://66fa658020084c4566dad8e9--docs-content.netlify.app
📍 Commit SHA: d34a503

Copy link
Contributor

@Hannes7eicher Hannes7eicher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the auto formatter got a little too excited and broke a few note banners :))

Although it also includes many fixes I would suggest we only change the pins and if we want to clean up articles we make a seperate PR.

What do you think?

@karlsoderby
Copy link
Contributor Author

@Hannes7eicher totally agree. Changes fixed!

Copy link
Contributor

@Hannes7eicher Hannes7eicher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two super small fixes I missed during the last review, otherwisee looks good!

karlsoderby and others added 2 commits September 30, 2024 10:27
…heet/cheat-sheet.md

Co-authored-by: Hannes Siebeneicher <60609597+Hannes7eicher@users.noreply.github.com>
…heet/cheat-sheet.md

Co-authored-by: Hannes Siebeneicher <60609597+Hannes7eicher@users.noreply.github.com>
@karlsoderby karlsoderby merged commit f729e2a into main Oct 3, 2024
4 checks passed
@karlsoderby karlsoderby deleted the karlsoderby/giga-can-adc-fixes branch October 3, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arduino Bugs and fixes added by the Arduino Team documentation Improvements or additions to documentation fix/update A small fix or update maker preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants