Page MenuHomeFreeBSD
Differential D50940

gpiobus: gpiobus_acquire_pin: panic on invalid pin
ClosedPublic
Actions

Authored by vexeduxr on Jun 19 2025, 2:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 7, 6:43 PM
Unknown Object (File)
Fri, Feb 6, 3:27 PM
Unknown Object (File)
Sat, Jan 31, 12:01 PM
Unknown Object (File)
Sat, Jan 31, 12:14 AM
Unknown Object (File)
Wed, Jan 28, 11:42 PM
Unknown Object (File)
Mon, Jan 26, 1:40 AM
Unknown Object (File)
Jan 12 2026, 11:25 AM
Unknown Object (File)
Dec 29 2025, 7:30 PM
View All 50 Files
Subscribers
None

Details

Reviewers
imp
wulf
mmel
Commits
rGed50879c4a95: gpiobus: gpiobus_acquire_pin: panic on invalid pin
Summary

It is a programming error to call this function with an invalid pin.
Also return proper error value on failure.

Suggested by: mmel

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 64982
Build 61865: arc lint + arc unit

Event Timeline

vexeduxr requested review of this revision.Jun 19 2025, 2:33 PM
vexeduxr created this revision.
Harbormaster completed remote builds in B64982: Diff 157311.Jun 19 2025, 2:33 PM
vexeduxr added a child revision: D50939: gpiobus: gpiobus_release_pin: convert errors to panic.Jun 19 2025, 2:34 PM
mmel accepted this revision.Jun 19 2025, 2:38 PM
This revision is now accepted and ready to land.Jun 19 2025, 2:38 PM
imp accepted this revision.Jul 3 2025, 7:57 PM

Go ahead and commit this. I saw the discussion, and this represents what was talked about elsewhere.

Closed by commit rGed50879c4a95: gpiobus: gpiobus_acquire_pin: panic on invalid pin (authored by vexeduxr). · Explain WhyJul 4 2025, 8:26 PM
This revision was automatically updated to reflect the committed changes.
vexeduxr added a commit: rGed50879c4a95: gpiobus: gpiobus_acquire_pin: panic on invalid pin.

Revision Contents
Changeset List

PathSize
sys/
dev/
gpio/
7 lines

Diff 157311

View Options

sys/dev/gpio/gpiobus.c

Loading...