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)
Sun, Jul 5, 8:10 AM
Unknown Object (File)
Wed, Jul 1, 11:26 PM
Unknown Object (File)
Sat, Jun 27, 4:23 PM
Unknown Object (File)
Jun 5 2026, 9:33 PM
Unknown Object (File)
Jun 5 2026, 8:35 PM
Unknown Object (File)
May 28 2026, 12:02 AM
Unknown Object (File)
May 14 2026, 2:56 PM
Unknown Object (File)
May 14 2026, 6:43 AM
View All 74 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...