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

drivers: ethernet: stm32: Remove support for deprecated API #73208

Merged
merged 1 commit into from
May 28, 2024

Conversation

erwango
Copy link
Member

@erwango erwango commented May 23, 2024

STM32Cube Ethernet API V2 support has been introduced in Zephyr release V3.3.

Fully remove support for the deprecated API version on impacted series (F4, F7, H7)

@zephyrbot
Copy link
Collaborator

zephyrbot commented May 23, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-hal_stm32 DNM This PR should not be merged (Do Not Merge) labels May 23, 2024
@erwango erwango force-pushed the dev/eth_v2_only branch from 7a6f7a4 to 292d83f Compare May 23, 2024 10:05
decsny
decsny previously approved these changes May 24, 2024
@erwango erwango force-pushed the dev/eth_v2_only branch from 292d83f to 44575b8 Compare May 27, 2024 07:54
@erwango erwango removed the DNM This PR should not be merged (Do Not Merge) label May 27, 2024
@zephyrbot zephyrbot added the DNM This PR should not be merged (Do Not Merge) label May 27, 2024
@erwango erwango force-pushed the dev/eth_v2_only branch from 44575b8 to fa343af Compare May 27, 2024 09:10
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label May 27, 2024
gautierg-st
gautierg-st previously approved these changes May 27, 2024
Comment on lines 36 to 37
Driver version based on legacy HAL version as the current official API version
is not available only for STM32F1 or STM32F2 series.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The wording is not clear

Suggested change
Driver version based on legacy HAL version as the current official API version
is not available only for STM32F1 or STM32F2 series.
Driver version based on legacy HAL version as the current official API version.
Available only for STM32F1 and STM32F2 SoC series.

@erwango erwango force-pushed the dev/eth_v2_only branch from 86f4c73 to 6a3d93c Compare May 27, 2024 15:33
…eries

In release V3.3 new STM32Cube HAL ethernet API was added to STM32 ethernet
driver for STM32F4, STM32F7 and STM32H7 series. At the same time, the
legacy API was deprecated for these series.
I'm now fully removing the legacy API support for these series.

Signed-off-by: Erwan Gouriou <erwan.gouriou@st.com>
@erwango erwango force-pushed the dev/eth_v2_only branch from 6a3d93c to 068639c Compare May 27, 2024 15:33
@aescolar aescolar merged commit 8cd8e53 into zephyrproject-rtos:main May 28, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants