Paths

Table of Contentst

Differential D4215

Fix GIC interrupts decoding in FDT
ClosedPublic
Actions

Authored by bsz_semihalf.com on Nov 19 2015, 12:26 PM.

Tags

Referenced Files

	Unknown Object (File)
	Tue, Apr 23, 7:06 AM

	Unknown Object (File)
	Mon, Apr 22, 11:01 PM

	Unknown Object (File)
	Mon, Apr 22, 10:55 PM

	Unknown Object (File)
	Sun, Apr 7, 2:10 AM

	Unknown Object (File)
	Sat, Apr 6, 11:51 PM

	Unknown Object (File)
	Feb 24 2024, 7:02 AM

	Unknown Object (File)
	Jan 2 2024, 4:00 PM

	Unknown Object (File)
	Dec 20 2023, 2:34 AM

View All 41 Files

Subscribers

Details

Reviewers

imp
• ian

Commits

rS294422: Fix GIC FDT interrupts decoding

Summary

Interrupt type in FDT was interpreted incorrectly.
Patch taken from freebsd-arm thread 'GIC - interrupts interpretation in
DTS/FDT':
https://lists.freebsd.org/pipermail/freebsd-arm/2015-August/012145.html

Obtained from: Semihalf
Sponsored by: Stormshield
Submitted by: Michal Stanek <mst@semihalf.com>

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bsz_semihalf.com updated this revision to Diff 10343.Nov 19 2015, 12:26 PM

bsz_semihalf.com retitled this revision from to Fix GIC interrupts decoding in FDT.

bsz_semihalf.com updated this object.

bsz_semihalf.com edited the test plan for this revision. (Show Details)

bsz_semihalf.com added reviewers: • ian, imp.

bsz_semihalf.com set the repository for this revision to rS FreeBSD src repository - subversion.

bsz_semihalf.com added a project: ARM.

bsz_semihalf.com added a subscriber: zbb.

Herald added a subscriber: andrew. · View Herald TranscriptNov 19 2015, 12:26 PM

bsz_semihalf.com added a parent revision: D4214: Use GIC-specific decoding function in mv_common.c.Nov 19 2015, 12:26 PM

bsz_semihalf.com added a child revision: D4216: Add compatibility string for dw-apb-uart in ns8250 driver.

imp accepted this revision.Nov 19 2015, 2:48 PM

imp edited edge metadata.

This revision is now accepted and ready to land.Nov 19 2015, 2:48 PM

andrew added inline comments.Nov 19 2015, 2:54 PM

sys/arm/arm/gic.c
339–340 ↗	(On Diff #10343)	You should follow style(9) here, new line agter the `&&`

Style corrected.

This revision now requires review to proceed.Nov 20 2015, 1:45 PM

• ian added inline comments.Dec 9 2015, 11:08 PM

sys/arm/arm/gic.c
118 ↗	(On Diff #10376)	This is a horrible name, all 4 bits are trigger bits, the distinction is between edge and level triggering. How about something like GIC_FDT_EDGE_TRIGGERED? (or _EDGE_BITS).

bsz_semihalf.com added inline comments.Dec 10 2015, 10:11 AM

sys/arm/arm/gic.c
118 ↗	(On Diff #10376)	Somebody fixed that already in tree, so I will remove it.

bsz_semihalf.com updated this revision to Diff 11104.Dec 11 2015, 12:47 PM

bsz_semihalf.com edited edge metadata.

bsz_semihalf.com marked 2 inline comments as done.

• ian accepted this revision.Dec 22 2015, 3:49 AM

• ian edited edge metadata.

This revision is now accepted and ready to land.Dec 22 2015, 3:49 AM

Closed by commit rS294422: Fix GIC FDT interrupts decoding (authored by zbb). · Explain WhyJan 20 2016, 1:46 PM

This revision was automatically updated to reflect the committed changes.

bsz_semihalf.com mentioned this in D5029: Fix GIC interrupt decoding in INTRNG code.Jan 22 2016, 10:16 AM

Revision Contents
Changeset List

Path

Size

head/

sys/

arm/

arm/

6 lines

Diff 12500

head/sys/arm/arm/gic.c

Loading...