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

tests: drivers: build_all: ethernet: Test handling plural "compatible"s #82693

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

soburi
Copy link
Member

@soburi soburi commented Dec 7, 2024

Add device definitions in dt to test drivers that handle
multiple "compatible"s by a single driver.

phy_adin2111 failed to compile in such a state.
I fixed this isssue.

@soburi soburi force-pushed the ethernet_build_all_adin2111 branch from 9b03cc4 to 9339c02 Compare December 7, 2024 23:49
@soburi soburi changed the title Ethernet build all adin2111 tests: drivers: build_all: ethernet: Test handling plural "compatible"s Dec 8, 2024
Compilation will fail if both adin2111 and adin1100 are used
at the same time.
Changing to define different unique names for the symbols
to avoid conflicts.

Signed-off-by: TOKITA Hiroshi <[email protected]>
Add device definitions in dt to test drivers that handle
multiple "compatible"s by a single driver.

Signed-off-by: TOKITA Hiroshi <[email protected]>
@soburi soburi force-pushed the ethernet_build_all_adin2111 branch from 9339c02 to a5f1f4a Compare December 8, 2024 14:15
@soburi soburi marked this pull request as ready for review December 11, 2024 22:19
@soburi soburi self-assigned this Dec 13, 2024
@kartben kartben merged commit 1022a89 into zephyrproject-rtos:main Dec 14, 2024
29 checks passed
@soburi soburi deleted the ethernet_build_all_adin2111 branch December 14, 2024 00:08
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.

4 participants