Differential D50871

gpiopps: don't use internal gpiobus function
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 6, 11:54 PM

	Unknown Object (File)
	Sat, Nov 1, 1:37 AM

	Unknown Object (File)
	Tue, Oct 28, 8:35 AM

	Unknown Object (File)
	Sun, Oct 26, 6:48 AM

	Unknown Object (File)
	Wed, Oct 22, 8:37 AM

	Unknown Object (File)
	Mon, Oct 20, 4:03 AM

	Unknown Object (File)
	Oct 6 2025, 8:45 AM

	Unknown Object (File)
	Oct 1 2025, 7:02 AM

View All 37 Files

Subscribers

Details

Reviewers

imp
wulf

Commits

rGff826f8ed434: gpiopps: don't use internal gpiobus function

Summary

gpiobus_release_pin is only meant to be used internally by gpiobus. Use
gpio_pin_release instead.

This also fixes a memory leak as gpio_pin_get_by_ofw_idx returns a
malloc'd pointer, which gpio_pin_release now frees.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

Harbormaster completed remote builds in B64898: Diff 157092.Jun 16 2025, 7:51 AM

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

vexeduxr added a child revision: D50872: gpio: hide internal gpiobus symbols.

LGTM

This revision is now accepted and ready to land.Jun 25 2025, 8:32 AM

imp accepted this revision.Jul 3 2025, 7:58 PM

Closed by commit rGff826f8ed434: gpiopps: don't use internal gpiobus function (authored by vexeduxr). · Explain WhyJul 4 2025, 8:26 PM

This revision was automatically updated to reflect the committed changes.

vexeduxr added a commit: rGff826f8ed434: gpiopps: don't use internal gpiobus function.

Revision Contents
Changeset List

Path

Size

sys/

dev/

gpio/

2 lines

Diff 157988

sys/dev/gpio/gpiopps.c

Loading...