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

samples: usb: dfu: Exclude all Atmel SAM0 #84534

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nandojve
Copy link
Member

The dfu sample filter seems not to work. It should select only boards when slot0_partition and slot1_partition are define. This is just a continuation of #84195. The issue is documented on #84196.

This add all the sam0 boards in the exclusion list until the issue is fixed.

The dfu sample filter seems not to work. It should select only boards
when slot0_partition and slot1_partition are define. This is just a
continuation of zephyrproject-rtos#84195.

Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Jan 24, 2025
@zephyrbot zephyrbot added area: Samples Samples area: USB Universal Serial Bus labels Jan 24, 2025
@nandojve nandojve added this to the v4.1.0 milestone Jan 24, 2025
Copy link
Member

@nashif nashif left a comment

Choose a reason for hiding this comment

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

need a proper fix, the issue is based on this commit 1440f5c.

@jfischer-no jfischer-no removed the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Jan 25, 2025
@nandojve
Copy link
Member Author

need a proper fix, the issue is based on this commit 1440f5c.

Hi @nashif , @kartben

I agreed that a fix is necessary and I open an issue to proper report but this is blocking a DMA fix #84118 which blocks a full RTC driver on #81819. I would like not to be penalize in this case. I don't mind to revert this changes as soon we have a fix.
There are already boards in the exclusion list that need to be rollback anyway.

@nashif
Copy link
Member

nashif commented Jan 25, 2025

I would like not to be penalize in this case.

Nobody is getting penalized. If there is no immediate fix for the issue introduced by 1440f5c, we should revert it now. What this PR is doing is not correct and it does not scale in any way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Samples Samples area: USB Universal Serial Bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants