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, Apr 4, 11:29 PM

	Unknown Object (File)
	Feb 14 2024, 4:55 AM

	Unknown Object (File)
	Dec 20 2023, 5:17 AM

	Unknown Object (File)
	Dec 12 2023, 9:25 PM

	Unknown Object (File)
	Sep 6 2023, 6:46 AM

	Unknown Object (File)
	Aug 14 2023, 10:03 PM

	Unknown Object (File)
	Aug 14 2023, 11:51 AM

	Unknown Object (File)
	Jun 27 2023, 7:50 AM

View All 17 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...