Skip to content

Conversation

@bcostm
Copy link
Contributor

@bcostm bcostm commented Feb 18, 2014

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

hello, can you explain why there are lines commented out without any comment in this commit?

@emilmont
Copy link
Contributor

I talked with bcostm: the PA_5 is the second DAC channel on this device but unfortunately there is also the LED on this pin. He has kept it in the code as a reference for future boards, without this constraint.

In the future, we probably want to keep this code under conditional compilation, depending on the target board.

I am going to accept the pull request, we will improve the code in a second step.

emilmont added a commit that referenced this pull request Feb 18, 2014
[NUCLEO_xxx] Fix us_ticker and analog_out issues when -O3/-Otime options are used
@emilmont emilmont merged commit 10297b3 into ARMmbed:master Feb 18, 2014
yossi2le pushed a commit to yossi2le/mbed-os that referenced this pull request Mar 27, 2019
* Updated to Mbed OS 5.11.5
* Enabled powercut test for armcc
pan- added a commit to pan-/mbed that referenced this pull request May 29, 2020
Updating mbed-os to mbed-os-5.9.6
ccli8 pushed a commit to ccli8/mbed-os that referenced this pull request Aug 13, 2024
* Add HALs to MbedCE

* HALs for i3c like an option

* Update stm32h5xx.h because of UTF-8 check
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.

3 participants