Page MenuHomeFreeBSD
Differential D57886

iichid(4): Always release interrupt resource
ClosedPublic
Actions

Authored by obiwac on Jun 26 2026, 2:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Aug 15, 12:51 PM
Unknown Object (File)
Wed, Aug 12, 6:28 AM
Unknown Object (File)
Wed, Aug 12, 5:43 AM
Unknown Object (File)
Wed, Aug 12, 12:00 AM
Unknown Object (File)
Tue, Aug 11, 6:08 PM
Unknown Object (File)
Tue, Aug 11, 1:39 PM
Unknown Object (File)
Tue, Aug 11, 12:48 PM
Unknown Object (File)
Mon, Aug 10, 5:56 PM
View All 41 Files
Subscribers
imp

Details

Reviewers
wulf
aokblast
Commits
rG0cedb626959c: iichid(4): Always release interrupt resource
Summary

We shouldn't just be doing this when IICHID_SAMPLING is not defined. Also I think it's still worth logging that the interrupt setup failed.

Event: Halifax Hackathon 202606
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

obiwac created this revision.Jun 26 2026, 2:29 PM
Herald added a subscriber: imp. · View Herald TranscriptJun 26 2026, 2:29 PM
obiwac requested review of this revision.Jun 26 2026, 2:29 PM
Harbormaster completed remote builds in B74303: Diff 180716.Jun 26 2026, 2:29 PM
obiwac edited the summary of this revision. (Show Details)Jun 26 2026, 2:29 PM
aokblast accepted this revision.Jun 26 2026, 6:49 PM

LGTM! irq_res = bus_alloc_resource_any is not guarded with IICHID_SAMPLING. It should be paired.

This revision is now accepted and ready to land.Jun 26 2026, 6:49 PM
Closed by commit rG0cedb626959c: iichid(4): Always release interrupt resource (authored by obiwac). · Explain WhyJun 26 2026, 7:00 PM
This revision was automatically updated to reflect the committed changes.
obiwac added a commit: rG0cedb626959c: iichid(4): Always release interrupt resource.

Revision Contents
Changeset List

PathSize
sys/
dev/
iicbus/
8 lines

Diff 180777

View Options

sys/dev/iicbus/iichid.c

Loading...