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.

Sign up for GitHub

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

Jump to bottom

tests: drivers: : ethernet: fails to link for sam_v71_xult and sam_v71b_xult #41525

Closed
cfriedt opened this issue Jan 1, 2022 · 1 comment · Fixed by #41550
Closed

tests: drivers: : ethernet: fails to link for sam_v71_xult and sam_v71b_xult #41525

cfriedt opened this issue Jan 1, 2022 · 1 comment · Fixed by #41550
Assignees
@nandojve
Labels
area: Drivers area: Ethernet bug The issue is a bug, or the PR is fixing a bug platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) priority: low Low impact/importance bug

Comments

@cfriedt
Copy link
Member

cfriedt commented Jan 1, 2022

Describe the bug
The daily build fails to link for this test / platform combination.

To Reproduce

twister -i -p sam_v71_xult -s tests/drivers/build_all/ethernet/net.ethernet.build
twister -i -p sam_v71b_xult -s tests/drivers/build_all/ethernet/net.ethernet.build

Expected behavior
The daily build should not fail.

Impact
Showstopper for anyone using ethernet on these platforms.

Logs and console output

/opt/toolchains/zephyr-sdk-0.13.2/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/10.3.0/../../../../arm-zephyr-eabi/bin/ld.bfd: zephyr/drivers/spi/libdrivers__spi.a(spi_sam.c.obj):(.rodata.__compound_literal.0+0x0): undefined reference to `__device_dts_ord_27'
/opt/toolchains/zephyr-sdk-0.13.2/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/10.3.0/../../../../arm-zephyr-eabi/bin/ld.bfd: zephyr/drivers/spi/libdrivers__spi.a(spi_sam.c.obj):(.rodata.__compound_literal.0+0x8): undefined reference to `__device_dts_ord_27'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

Environment (please complete the following information):

  • OS: CI
  • Toolchain: Zephyr SDK 0.13.2
  • Commit SHA: 7f5eb92

Additional context
Daily build failure:
https://github.com/zephyrproject-rtos/zephyr/runs/4676614702?check_suite_focus=true
@cfriedt cfriedt added bug The issue is a bug, or the PR is fixing a bug area: Drivers platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) area: Ethernet labels Jan 1, 2022
@nandojve nandojve mentioned this issue Jan 3, 2022
Merged
@dkalowsk
Copy link
Contributor

seems similar to #41677

@dkalowsk dkalowsk added the priority: low Low impact/importance bug label Jan 11, 2022
nandojve added a commit to nandojve/zephyr that referenced this issue Jan 14, 2022
@nandojve
soc: arm: atmel_sam: samv71: Fix SPI build dependency
66ee8a2 
The SAM spi driver depends on GPIO driver to work. It seems that this
dependency chain it is not handled. This select GPIO driver when SPI
driver is enabled. It rework GPIO and SPI Kconfig to select driver by
devicetree and drop entries at Kconfig.defconfig.series file.

Fixes zephyrproject-rtos#41525

Signed-off-by: Gerson Fernando Budke <[email protected]>
nashif pushed a commit that referenced this issue Jan 18, 2022
@nandojve @nashif
soc: arm: atmel_sam: samv71: Fix SPI build dependency
58936e7 
The SAM spi driver depends on GPIO driver to work. It seems that this
dependency chain it is not handled. This select GPIO driver when SPI
driver is enabled. It rework GPIO and SPI Kconfig to select driver by
devicetree and drop entries at Kconfig.defconfig.series file.

Fixes #41525

Signed-off-by: Gerson Fernando Budke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nandojve nandojve

Labels
area: Drivers area: Ethernet bug The issue is a bug, or the PR is fixing a bug platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

soc: arm: atmel_sam: samv71: Fix SPI build dependency nandojve/zephyr
3 participants
@cfriedt @dkalowsk @nandojve