Skip to content

API improvements#2

Closed
Aymane-ST wants to merge 6 commits intostm32duino:mainfrom
Aymane-ST:dev
Closed

API improvements#2
Aymane-ST wants to merge 6 commits intostm32duino:mainfrom
Aymane-ST:dev

Conversation

@Aymane-ST
Copy link
Copy Markdown

@Aymane-ST Aymane-ST commented Jan 12, 2026

This PR is part of the effort to rework ArduinoCore-API so it can be used as a git submodule in STM32duino’s Arduino_Core_STM32.

The changes in this branch:

  • align the core API with STM32duino requirements.
  • propose fixes, enhancements, and improvements discovered while integrating ArduinoCore-API into the STM32 core back to ArduinoCore-API so they can benefit other cores as well.

@Aymane-ST Aymane-ST requested a review from fpistm January 12, 2026 15:47
@Aymane-ST Aymane-ST self-assigned this Jan 12, 2026
@Aymane-ST Aymane-ST added the draft Proposed fix not ready to be merged label Jan 12, 2026
Copy link
Copy Markdown
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

pay attention to kept as mush as possible original code.

Signed-off-by: Aymane Bahssain <aymane.bahssain@st.com>
Ref: arduino#242

Signed-off-by: Aymane Bahssain <aymane.bahssain@st.com>
Signed-off-by: Aymane Bahssain <aymane.bahssain@st.com>
- Change pulseIn and pulseInLong prototypes from uint8_t pin to pin_size_t pin.

Signed-off-by: Aymane Bahssain <aymane.bahssain@st.com>
@Aymane-ST Aymane-ST force-pushed the dev branch 3 times, most recently from 4ddffe6 to 99e54cc Compare January 19, 2026 14:34
- Add float overloads to Print::print/println
- Enhance Print with printf() and vprintf() helpers

Signed-off-by: Aymane Bahssain <aymane.bahssain@st.com>
SPISettings was originally implemented with
init_AlwaysInline / init_MightInline helpers, using runtime
initialization only.

This change:
	- removes init_AlwaysInline / init_MightInline helpers,
	- replaces them with constexpr constructors that directly initialize the members,
	- keeps the existing API surface
refs: stm32duino/Arduino_Core_STM32#2204 stm32duino/Arduino_Core_STM32#2201
@fpistm fpistm self-requested a review April 7, 2026 09:25
Copy link
Copy Markdown
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

This one is missing?
#9

@Aymane-ST Aymane-ST closed this Apr 7, 2026
@github-project-automation github-project-automation bot moved this from Needs review to Done in STM32 core based on ST HAL Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

draft Proposed fix not ready to be merged

Projects

Development

Successfully merging this pull request may close these issues.

2 participants