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

[Backport v3.5-branch] drivers: can: be consistent in filter_id checks when removing rx filters #64416

Merged
merged 1 commit into from Oct 27, 2023
Merged

[Backport v3.5-branch] drivers: can: be consistent in filter_id checks when removing rx filters #64416

merged 1 commit into from Oct 27, 2023

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Oct 26, 2023 •
edited by henrikbrixandersen

Backport 6c5400d from #64399.

Fixes: #64398

@henrikbrixandersen @github-actions
drivers: can: be consistent in filter_id checks when removing rx filters
10d4a36 
Change the CAN controller driver implementations for the
can_remove_rx_filter() API call to be consistent in their validation of the
supplied filter_id.

Fixes: #64398

Signed-off-by: Henrik Brix Andersen <hebad@vestas.com>
(cherry picked from commit 6c5400d)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues platform: STM32 ST Micro STM32 area: native port Host native arch port (native_posix) area: CAN platform: Renesas R-Car Renesas R-Car platform: NXP NXP labels Oct 26, 2023
@henrikbrixandersen henrikbrixandersen requested a review from a team October 26, 2023 09:14
@fabiobaltieri fabiobaltieri merged commit 3a67f3d into v3.5-branch Oct 27, 2023
24 checks passed
@nashif nashif deleted the backport-64399-to-v3.5-branch branch February 13, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabiobaltieri fabiobaltieri fabiobaltieri approved these changes

@jhedberg jhedberg jhedberg approved these changes

@henrikbrixandersen henrikbrixandersen henrikbrixandersen approved these changes

@dkalowsk dkalowsk dkalowsk approved these changes

@martinjaeger martinjaeger martinjaeger approved these changes

@aaillet aaillet aaillet approved these changes

@karstenkoenig karstenkoenig Awaiting requested review from karstenkoenig karstenkoenig is a code owner

@alexanderwachter alexanderwachter Awaiting requested review from alexanderwachter alexanderwachter is a code owner

@danieldegrasse danieldegrasse Awaiting requested review from danieldegrasse

@decsny decsny Awaiting requested review from decsny

@DerekSnell DerekSnell Awaiting requested review from DerekSnell

@dleach02 dleach02 Awaiting requested review from dleach02

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@manuargue manuargue Awaiting requested review from manuargue

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@PetervdPerk-NXP PetervdPerk-NXP Awaiting requested review from PetervdPerk-NXP

@str4t0m str4t0m Awaiting requested review from str4t0m

@yvanderv yvanderv Awaiting requested review from yvanderv

Assignees

@henrikbrixandersen henrikbrixandersen

Labels
area: CAN area: native port Host native arch port (native_posix) Backport Backport PR and backport failure issues platform: NXP NXP platform: Renesas R-Car Renesas R-Car platform: STM32 ST Micro STM32
Projects
Backports
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@zephyrbot @fabiobaltieri @jhedberg @henrikbrixandersen @dkalowsk @martinjaeger @aaillet