Differential D16892

add pci support at rt1310
AbandonedPublic
Actions

Authored by yamori813_yahoo.co.jp on Aug 24 2018, 10:27 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Aug 21, 3:37 AM

	Unknown Object (File)
	Sun, Aug 3, 12:13 PM

	Unknown Object (File)
	Jun 25 2025, 7:48 PM

	Unknown Object (File)
	Jun 21 2025, 6:41 AM

	Unknown Object (File)
	Jun 18 2025, 10:01 PM

	Unknown Object (File)
	Jun 17 2025, 10:03 AM

	Unknown Object (File)
	Jun 9 2025, 7:35 PM

	Unknown Object (File)
	May 1 2025, 1:31 AM

View All 55 Files

Subscribers

Details

Reviewers

imp
adrian
mizhka
manu

Test Plan

WZR2-G300N

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19160
Build 18783: arc lint + arc unit

Event Timeline

yamori813_yahoo.co.jp created this revision.Aug 24 2018, 10:27 PM

Herald added a reviewer: manu. · View Herald TranscriptAug 24 2018, 10:27 PM

Harbormaster completed remote builds in B19160: Diff 47269.Aug 24 2018, 10:27 PM

manu removed a reviewer: manu.Mar 21 2019, 7:31 AM

Herald added a reviewer: manu. · View Herald TranscriptMar 21 2019, 7:31 AM

Herald added a subscriber: andrew. · View Herald Transcript

Remove myself from blocking reviewer.

manu removed a reviewer: manu.Mar 21 2019, 7:31 AM

Herald added a reviewer: manu. · View Herald TranscriptMar 21 2019, 7:31 AM

Herald doesn't like this ...
Consider it ok from me for dts changes.

Generally looks good. Please fix easy remarks.

sys/arm/ralink/rt1310_machdep.c
107	Why so big window? May be better to add additional entry?
sys/arm/ralink/rt1310_pci.c
256	Please move to beginning of function
274	Please split to 2 lines: declaration and assignment.

andrew added inline comments.Mar 21 2019, 10:20 AM

sys/arm/ralink/rt1310_machdep.c
127	This should be in a common header.
sys/arm/ralink/rt1310_pci.c
35	Is this needed?
237	What is this delay for?
239–243	What are these magic numbers?
244	And this delay. It's also held while we hold a mutex.
249–254	More magic numbers.

yamori813_yahoo.co.jp abandoned this revision.Feb 2 2020, 12:46 PM

Revision Contents
Changeset List

Path

Size

sys/

arm/

ralink/

1 line

rt1310_machdep.c

19 lines

573 lines

dts/

arm/

21 lines

Diff 47269

sys/arm/ralink/files.ralink

Loading...

sys/arm/ralink/rt1310_machdep.c

Loading...

sys/arm/ralink/rt1310_pci.c

Loading...

sys/dts/arm/rt1310a.dtsi

Loading...