Differential D2345

gic_decode_fdt() fix for interrupts-extended property
ClosedPublic
Actions

Authored by skra on Apr 22 2015, 2:12 PM.

Tags

Referenced Files

	F105763366: D2345.diff
	Fri, Dec 20, 8:29 AM

	Unknown Object (File)
	Wed, Dec 11, 12:48 AM

	Unknown Object (File)
	Sep 23 2024, 6:27 AM

	Unknown Object (File)
	Sep 21 2024, 5:55 PM

	Unknown Object (File)
	Sep 12 2024, 12:00 AM

	Unknown Object (File)
	Aug 11 2024, 6:49 PM

	Unknown Object (File)
	Jul 14 2024, 1:24 PM

	Unknown Object (File)
	Jun 25 2024, 7:02 AM

View All 57 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rS289548: Only decode fdt data which belongs to the GIC controller.

Summary

An interrupt extended property looks like this:
interrupts-extended = <&gic GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH &omap4_pmx_core OMAP4_UART4_RX>;

Every xxx_decode_fdt() should check its compatible property and does not process alien tuples. This patch fixes that for GIC.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

skra updated this revision to Diff 4951.Apr 22 2015, 2:12 PM

skra retitled this revision from to gic_decode_fdt() fix for interrupts-extended property.

skra updated this object.

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

skra added a reviewer: ARM.

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

skra added a project: ARM.

Herald added subscribers: imp, andrew. · View Herald TranscriptApr 22 2015, 2:12 PM

Since I'm knee-deep in interrupt stuff today I'm going to commit this one too, with a small change to cache the phandle_t after a successful compatible lookup.

This revision is now accepted and ready to land.Oct 18 2015, 8:22 PM

Closed by commit rS289548: Only decode fdt data which belongs to the GIC controller. (authored by • ian). · Explain WhyOct 18 2015, 8:37 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

arm/

arm/

11 lines

include/

3 lines

Diff 4951

sys/arm/arm/gic.c

Loading...

sys/arm/include/intr.h

Loading...