Paths

Table of Contentst

Differential D58104

FDT: implement fdt_ether_get_addr()
ClosedPublic
Actions

Authored by mmel on Wed, Jul 8, 11:29 AM.

Tags

None

Referenced Files

	F162696356: D58104.id.diff
	Wed, Jul 15, 9:21 PM

	F162671406: D58104.id181592.diff
	Wed, Jul 15, 2:09 PM

	F162671091: D58104.id181592.diff
	Wed, Jul 15, 2:06 PM

	Unknown Object (File)
	Wed, Jul 15, 7:59 AM

	Unknown Object (File)
	Wed, Jul 15, 7:29 AM

	Unknown Object (File)
	Wed, Jul 15, 7:29 AM

	Unknown Object (File)
	Wed, Jul 15, 3:35 AM

	Unknown Object (File)
	Wed, Jul 15, 2:06 AM

View All 32 Files

Subscribers

Details

Reviewers

manu
adrian
bz
mhorne

Group Reviewers

Commits

rG920518cf2e75: FDT: implement fdt_ether_get_addr()

Summary

Introduce fdt_ether_get_addr() in fdt_common.c/h that tries standard
DT properties in the correct order and falls back to a random address
when needed. This should be used by ethernet drivers instead of open-coding
the same logic.

MFC after: 2 weeks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 74647
Build 71530: arc lint + arc unit

Event Timeline

mmel created this revision.Wed, Jul 8, 11:29 AM

Herald added a subscriber: imp. · View Herald TranscriptWed, Jul 8, 11:29 AM

mmel requested review of this revision.Wed, Jul 8, 11:29 AM

Harbormaster completed remote builds in B74614: Diff 181529.Wed, Jul 8, 11:29 AM

mmel added reviewers: manu, adrian, arm64.Wed, Jul 8, 11:30 AM

LGTM

jrtc27 added a subscriber: jrtc27.Wed, Jul 8, 6:16 PM

jrtc27 added inline comments.

sys/dev/fdt/fdt_common.c
566	OOI why not fdt_ether_get_addr given ether_gen_addr (and the MAC is kind of a given for ethernet)? Or, if keeping the mac, should probably separate with an underscore?

jrtc27 added a comment.Wed, Jul 8, 6:18 PM

This comment was removed by jrtc27.

sys/dev/fdt/fdt_common.c
571	.

I'll be happy if this lands as I'll finally be able to ditch (updated versions) of https://reviews.freebsd.org/D23145 out of my tree. Thank you!

sys/dev/fdt/fdt_common.c
589	One is eaddr, one is tmp here; that looks wrong?

This revision now requires changes to proceed.Wed, Jul 8, 10:23 PM

bz mentioned this in D23145: dwc(4) Ethernet MAC address setting/generating.Wed, Jul 8, 11:26 PM

Rename function and fix cut&paste error

Harbormaster completed remote builds in B74646: Diff 181591.Thu, Jul 9, 8:22 AM

update commit log

Harbormaster completed remote builds in B74647: Diff 181592.Thu, Jul 9, 8:24 AM

mmel retitled this revision from FDT: implement fdt_ether_get_macaddr() to FDT: implement fdt_ether_get_addr().Thu, Jul 9, 8:40 AM

mmel edited the summary of this revision. (Show Details)

remove redundant empty line

Harbormaster completed remote builds in B74648: Diff 181593.Thu, Jul 9, 8:42 AM

mmel removed a reviewer: bz.Thu, Jul 9, 8:43 AM

mmel marked 3 inline comments as done.

mmel added a reviewer: bz.Thu, Jul 9, 9:32 AM

LGTM. There was a recent addition to if_eqos_starfive to handle "local-mac-address", but it will be better to use this. Thanks.

adrian accepted this revision.Fri, Jul 10, 2:39 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mon, Jul 13, 8:37 AM

Closed by commit rG920518cf2e75: FDT: implement fdt_ether_get_addr() (authored by mmel). · Explain Why

This revision was automatically updated to reflect the committed changes.

mmel added a commit: rG920518cf2e75: FDT: implement fdt_ether_get_addr().

Revision Contents
Changeset List

Path

Size

sys/

dev/

fdt/

4 lines

41 lines

Diff 181592

sys/dev/fdt/fdt_common.h

Loading...

sys/dev/fdt/fdt_common.c

Loading...