Paths

Table of Contentst

Differential D38963

eqos: Mechanically convert to IfAPI
ClosedPublic
Actions

Authored by jhibbits on Mar 7 2023, 9:25 PM.

Tags

None

Referenced Files

	F153665054: D38963.id118498.diff
	Wed, Apr 22, 6:34 PM

	F153621064: D38963.id118498.diff
	Wed, Apr 22, 9:56 AM

	Unknown Object (File)
	Tue, Apr 21, 7:50 AM

	Unknown Object (File)
	Tue, Apr 21, 2:37 AM

	Unknown Object (File)
	Mon, Apr 20, 11:12 PM

	Unknown Object (File)
	Sun, Apr 19, 2:24 AM

	Unknown Object (File)
	Tue, Apr 7, 8:05 AM

	Unknown Object (File)
	Mar 13 2026, 5:41 AM

View All 85 Files

Subscribers

Details

Reviewers

sos
zlei

Group Reviewers

Commits

rGa472fd7a5737: eqos: Mechanically convert to IfAPI

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhibbits created this revision.Mar 7 2023, 9:25 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 7 2023, 9:25 PM

jhibbits requested review of this revision.Mar 7 2023, 9:25 PM

Harbormaster completed remote builds in B50211: Diff 118498.Mar 7 2023, 9:25 PM

Looks good to me.

sys/dev/eqos/if_eqos.c
1216	No local variable `ifp`.

This revision is now accepted and ready to land.Mar 8 2023, 7:00 AM

jhibbits marked an inline comment as done.Mar 8 2023, 2:55 PM

jhibbits added inline comments.

sys/dev/eqos/if_eqos.c
1216	Huh, I wonder how it compiled, then. Very strange.

Closed by commit rGa472fd7a5737: eqos: Mechanically convert to IfAPI (authored by jhibbits). · Explain WhyMar 8 2023, 3:16 PM

This revision was automatically updated to reflect the committed changes.

jhibbits marked an inline comment as done.

jhibbits added a commit: rGa472fd7a5737: eqos: Mechanically convert to IfAPI.

zlei added inline comments.Mar 8 2023, 4:27 PM

sys/dev/eqos/if_eqos.c
1216	Huh, I wonder how it compiled, then. Very strange. `if_eqos` is driver for RK356X SOC Ethernet QoS controller. That is in arm machines. IIUC it will not be built with `amd64/conf/GENERIC` kernel config.

jhibbits added inline comments.Mar 8 2023, 4:52 PM

sys/dev/eqos/if_eqos.c
1216	It also apparently doesn't get built with arm64 LINT and I didn't build GENERIC before submitting the review.

Revision Contents
Changeset List

Path

Size

sys/

dev/

eqos/

44 lines

Diff 118532

sys/dev/eqos/if_eqos.c

Loading...