Skip to content

[Backport v4.0-branch] test: arm: irq: Add overlays files for Infineon boards #81620

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
Merged
merged 2 commits into from
Feb 11, 2025
Merged

[Backport v4.0-branch] test: arm: irq: Add overlays files for Infineon boards #81620

merged 2 commits into from
Feb 11, 2025

Conversation

@zephyrbot
Copy link

@zephyrbot zephyrbot commented Nov 19, 2024 •
edited by kartben
Loading

Backport 6172092~2..6172092 from #81464.

Fixes #85059

@npal-cy @github-actions
Infineon: board: Add CONFIG_GPIO to defconfigs
d9f0658 
Add CONFIG_GPIO from defconfigs for Infineon boards.

Revert pull/81377, which affect some ble samples which
used GPIO.

Signed-off-by: Nazar Palamar <[email protected]>
(cherry picked from commit 697efe8)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues platform: Infineon Infineon Technologies AG area: ARM ARM (32-bit) Architecture labels Nov 19, 2024
@npal-cy
Copy link
Contributor

npal-cy commented Nov 20, 2024 •
edited
Loading

Hi @ithinuel, Not sure if need addition updates to fix Backport Issue Check.

@kartben
Copy link
Contributor

kartben commented Nov 29, 2024

Hi @ithinuel, Not sure if need addition updates to fix Backport Issue Check.

Yes, it needs the issue it's fixing mentioned in the PR description ("Fixes #xxxxx"), since backports are meant to be only for bugfixes.

@ithinuel ithinuel assigned wearyzen and unassigned ithinuel Jan 17, 2025
@wearyzen
Copy link
Contributor

wearyzen commented Jan 22, 2025

Hi @npal-cy, I see there is no update since last message from Benjamin. Is this PR still needed?

@npal-cy
test: arm: irq: Add overlays files for Infineon boards
60b6295 
Changed interrupt priority for GPIO, default 6 is not suitable for
for the ZERO_LATENCY_IRQS function used in this test.
used in this test.

Fixes for problem with arm_irq_zero_latency_levels
refer to pull/81377

Signed-off-by: Nazar Palamar <[email protected]>
(cherry picked from commit 6172092)
@npal-cy npal-cy force-pushed the backport-81464-to-v4.0-branch branch from 560cce1 to 60b6295 Compare January 22, 2025 12:30
@npal-cy
Copy link
Contributor

npal-cy commented Jan 22, 2025

looks like i missed this...I thoughts that i do not have an access to that branch, but i have.

We do not have a issue ticket and we fixed the changes from pull/81377

I updated text to:

Fixes for problem with arm_irq_zero_latency_levels refer to pull/81377

but looks kike it does't help.

@wearyzen
Copy link
Contributor

wearyzen commented Jan 22, 2025 •
edited
Loading

@npal-cy, I think you need to create an issue explaining what the problem was with the PR and update the PR descriptions as mentioned here https://github.com/zephyrproject-rtos/zephyr/actions/runs/12908240388/job/35993350107?pr=81620
Please ensure the body of each PR to a release branch contains "Fixes #1234"

@wearyzen
Copy link
Contributor

wearyzen commented Feb 3, 2025

@npal-cy, I think you need to create an issue explaining what the problem was with the PR and update the PR descriptions as mentioned here https://github.com/zephyrproject-rtos/zephyr/actions/runs/12908240388/job/35993350107?pr=81620 Please ensure the body of each PR to a release branch contains "Fixes #1234"

Hi @npal-cy, gentle reminder on this :)

@npal-cy
Copy link
Contributor

npal-cy commented Feb 3, 2025

Hey @wearyzen, I'm so sorry I didn't get a chance to reply earlier. I was busy..., but I'm on it!

I've filed a bug report here: #85059. I'll update the commit today.

@kartben
Copy link
Contributor

kartben commented Feb 3, 2025

Hey @wearyzen, I'm so sorry I didn't get a chance to reply earlier. I was busy..., but I'm on it!

I've filed a bug report here: #85059. I'll update the commit today.

No need to update the commit, just the description of this PR. I just did it

@dkalowsk dkalowsk added this to the v4.0.1 milestone Feb 11, 2025
@dkalowsk dkalowsk linked an issue Feb 11, 2025 that may be closed by this pull request
Add overlays files for Infineon boards changed interrupt priority for GPIO for ZERO_LATENCY_IRQS test #85059
Closed
@dkalowsk dkalowsk merged commit da7ec06 into v4.0-branch Feb 11, 2025
21 of 22 checks passed
@dkalowsk dkalowsk deleted the backport-81464-to-v4.0-branch branch February 11, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ifyall ifyall ifyall approved these changes

@wearyzen wearyzen wearyzen approved these changes

@bbolen bbolen Awaiting requested review from bbolen

@carlocaione carlocaione Awaiting requested review from carlocaione

@galak galak Awaiting requested review from galak

@ithinuel ithinuel Awaiting requested review from ithinuel

@MaureenHelm MaureenHelm Awaiting requested review from MaureenHelm

@microbuilder microbuilder Awaiting requested review from microbuilder

@stephanosio stephanosio Awaiting requested review from stephanosio

@talih0 talih0 Awaiting requested review from talih0

+1 more reviewer

@npal-cy npal-cy npal-cy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@wearyzen wearyzen

Labels

area: ARM ARM (32-bit) Architecture Backport Backport PR and backport failure issues platform: Infineon Infineon Technologies AG

Projects

Backports
Status: Done

Milestone

v4.0.1

Development

Successfully merging this pull request may close these issues.

Add overlays files for Infineon boards changed interrupt priority for GPIO for ZERO_LATENCY_IRQS test

8 participants

@zephyrbot @npal-cy @kartben @wearyzen @ifyall @ithinuel @dkalowsk