Paths

Table of Contentst

rk_gpio: improve EOI handling
AbandonedPublic
Actions

Authored by mhorne on Jul 14 2026, 3:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 6, 3:11 AM

	Unknown Object (File)
	Sat, Sep 5, 1:38 AM

	Unknown Object (File)
	Wed, Sep 2, 10:16 AM

	Unknown Object (File)
	Sat, Aug 22, 9:47 AM

	Unknown Object (File)
	Thu, Aug 20, 3:11 AM

	Unknown Object (File)
	Thu, Aug 13, 12:59 AM

	Unknown Object (File)
	Thu, Aug 13, 12:27 AM

	Unknown Object (File)
	Thu, Aug 13, 12:27 AM

Subscribers

Details

Reviewers

mmel
manu
andrew

Summary

Defer End-Of-Interrupt signaling for level-triggered interrupts to the
pic_post_filter / pic_pre_ithread methods.

This is a reapplication of 8ffb400bfd64, with changes:

Follow correct EOI handling scheme as reported by mmel@
Simplified commentary

Co-authored-by: mhorne
Fixes: ccda002ca10f ("rk_gpio: implement PIC masking methods and mask unhandled IRQs")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 74798
Build 71681: arc lint + arc unit

Event Timeline

mhorne created this revision.Jul 14 2026, 3:41 PM

Herald added subscribers: loos, emaste, imp. · View Herald TranscriptJul 14 2026, 3:41 PM

mhorne requested review of this revision.Jul 14 2026, 3:41 PM

Harbormaster completed remote builds in B74798: Diff 181979.Jul 14 2026, 3:41 PM

So, this applies the EOI handling scheme preferred by many drivers, and endorsed by mmel@.

FYI, a survey of GPIO PIC drivers reveals a few that defer ACK/EOI until pic_post_ithread(), at least: imx_gpio, pl061, aw_gpio, qoriq_gpio. I am not sure if it is a good idea to "fix" these, but I don't plan to touch them.

Per the replies in https://github.com/freebsd/freebsd-src/pull/2245, this ends up having no functional impact. Closing.

Revision Contents
Changeset List

Path

Size

sys/

arm64/

rockchip/

rk_gpio.c

65 lines

Diff 181979

View Options

sys/arm64/rockchip/rk_gpio.c

rk_gpio: improve EOI handlingAbandonedPublicActions