Differential D50869

gpiobus: add gpio_pin_acquire
ClosedPublic
Actions

Authored by vexeduxr on Jun 16 2025, 7:46 AM.

Tags

None

Referenced Files

	F125048247: D50869.id157313.diff
	Sat, Aug 2, 8:46 PM

	F125015348: D50869.diff
	Sat, Aug 2, 10:30 AM

	Unknown Object (File)
	Tue, Jul 29, 11:40 AM

	Unknown Object (File)
	Tue, Jul 29, 7:52 AM

	Unknown Object (File)
	Sun, Jul 27, 3:21 PM

	Unknown Object (File)
	Wed, Jul 23, 10:22 PM

	Unknown Object (File)
	Wed, Jul 23, 9:56 AM

	Unknown Object (File)
	Tue, Jul 22, 4:18 PM

View All 29 Files

Subscribers

Details

Reviewers

imp
wulf
mmel

Commits

rG4a21713278a8: gpiobus: add gpio_pin_acquire

Summary

In some cases, drivers may need to acquire an existing gpio_pin_t. With
the functions gpiobus currently exposes, this isn't possible as they
allocate a new pin then acquire that. Add a new gpio_pin_acquire
function which accepts an existing gpiobus_pin structure.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64896
Build 61779: arc lint + arc unit

Event Timeline

vexeduxr created this revision.Jun 16 2025, 7:46 AM

Herald added a subscriber: loos. · View Herald TranscriptJun 16 2025, 7:46 AM

vexeduxr requested review of this revision.Jun 16 2025, 7:46 AM

Harbormaster completed remote builds in B64896: Diff 157090.Jun 16 2025, 7:46 AM

vexeduxr added a parent revision: D50868: gpiobus: gpio_pin_release: convert checks to KASSERTs.Jun 16 2025, 7:46 AM

vexeduxr added a child revision: D50870: regulator: don't use internal gpiobus function.Jun 16 2025, 7:53 AM

Otherwise, it looks fine to me.

sys/dev/gpio/gpiobusvar.h
160	Wrong place, see the comment a line above. And, please, add short comment all other functions here have it.

This revision now requires changes to proceed.Jun 17 2025, 1:14 PM

vexeduxr added inline comments.Jun 17 2025, 5:01 PM

sys/dev/gpio/gpiobusvar.h
160	Ah, not sure how I missed that. Thanks.

fixup function declaration

Harbormaster completed remote builds in B64945: Diff 157202.Jun 17 2025, 5:07 PM

mmel accepted this revision.Jun 17 2025, 5:57 PM

This revision is now accepted and ready to land.Jun 17 2025, 5:57 PM

Convert null check to KASSERT

This revision now requires review to proceed.Jun 19 2025, 2:38 PM

Harbormaster completed remote builds in B64984: Diff 157313.Jun 19 2025, 2:38 PM

love the extra context for the function's use.

This revision is now accepted and ready to land.Jul 3 2025, 8:00 PM

Closed by commit rG4a21713278a8: gpiobus: add gpio_pin_acquire (authored by vexeduxr). · Explain WhyFri, Jul 4, 8:26 PM

This revision was automatically updated to reflect the committed changes.

vexeduxr added a commit: rG4a21713278a8: gpiobus: add gpio_pin_acquire.

Revision Contents
Changeset List

Path

Size

sys/

dev/

gpio/

24 lines

1 line

Diff 157090

sys/dev/gpio/gpiobus.c

Loading...

sys/dev/gpio/gpiobusvar.h

Loading...