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

Remove Reset pin from Digital Pin mapping XMC1100 Boot Kit #200

Merged
merged 1 commit into from
Aug 29, 2022
Merged

Remove Reset pin from Digital Pin mapping XMC1100 Boot Kit #200

merged 1 commit into from
Aug 29, 2022

Conversation

techpaul
Copy link
Contributor

XMC1100 Boot Kit

Remove Reset pin from digital I/O mapping as cannot be used.
New XMC1100 Boot Kit pinout image in resources folder

Created this way after comments in #196

Remove Reset pin from digital I/O mapping as cannot be used.
New XMC1100 Boot Kit pinout image in resources folder
Copy link
Contributor

@djaumann djaumann left a comment

Choose a reason for hiding this comment

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

Looks good :) Just one thing from my side - I wouldn't have removed the comments (pin numbers) behind the LED definitions. I think that makes it easier for the user to quickly see which LED is on which pin. But just a minor thing.

But they can check it of course in the pin mapping array.

@techpaul
Copy link
Contributor Author

Looks good :) Just one thing from my side - I wouldn't have removed the comments (pin numbers) behind the LED definitions. I think that makes it easier for the user to quickly see which LED is on which pin. But just a minor thing.

But they can check it of course in the pin mapping array.

A few things

Firstly in Arduino land you rarely talk P0.5 you talk abstracted digital pin 'n' which is already in the define. The port definitions are for lower levels after pin mapping talking to XMCLib etc.

Secondly if ANOTHER compiler upgrade causes problems like previously where sbrk got lost, comments on a #define can cause compilation errors if compiler stops doing substitute till a comment, as the substitution is often everything to end of line. Then you end up with lots of errors that need fixing due to ccompiler change.

General rule on defines in ANY flaviour of C, C++ C#...... do NOT put comments on same line as define, it can have hidden effects especially if a module is being included in a C and also C++ module, and different rules may apply.

You may have GOT AWAY with the practice so far as many madules exist as C or C++

@jaenrig-ifx jaenrig-ifx merged commit 6d73e57 into Infineon:develop-2.x Aug 29, 2022
@techpaul techpaul deleted the develop-2.x branch March 4, 2023 15:54
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.

None yet

3 participants