Page MenuHomeFreeBSD
Differential D45319

devmap: eliminate unused arguments
ClosedPublic
Actions

Authored by mhorne on Thu, May 23, 7:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 11, 7:01 AM
Unknown Object (File)
Wed, Jun 5, 1:07 AM
Unknown Object (File)
Tue, Jun 4, 7:34 PM
Unknown Object (File)
Mon, Jun 3, 6:46 AM
Unknown Object (File)
Sat, May 25, 5:14 PM
Unknown Object (File)
Fri, May 24, 2:47 PM
Unknown Object (File)
Thu, May 23, 9:57 PM
Subscribers
emaste
imp
olce
riscv

Details

Reviewers
andrew
markj
manu
Commits
rG5df74441b35f: devmap: eliminate unused arguments
Summary

The optional 'table' pointer is a legacy part of the interface, which
has been replaced by devmap_register_table()/devmap_add_entry(). The few
in-tree callers have already adapted to this, so it can be removed.

The 'l1pt' argument is already entirely unused within the function.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mhorne created this revision.Thu, May 23, 7:15 PM
Herald added a reviewer: manu. · View Herald TranscriptThu, May 23, 7:15 PM
Herald added subscribers: riscv, olce, emaste, imp. · View Herald Transcript
mhorne requested review of this revision.Thu, May 23, 7:15 PM
Harbormaster completed remote builds in B57872: Diff 139001.Thu, May 23, 7:15 PM
markj accepted this revision.Thu, May 23, 7:45 PM
This revision is now accepted and ready to land.Thu, May 23, 7:45 PM
mhorne updated this revision to Diff 139445.Tue, Jun 4, 5:17 PM

Remove the table argument too; this is a legacy interface that is now unused.

Current code is either provides a table explicitly via devmap_register_table(), or implicitly by the first call to devmap_add_entry().

This revision now requires review to proceed.Tue, Jun 4, 5:17 PM
Harbormaster completed remote builds in B58054: Diff 139445.Tue, Jun 4, 5:17 PM
mhorne retitled this revision from devmap: eliminate useless argument to devmap: eliminate unused arguments.Tue, Jun 4, 5:27 PM
mhorne edited the summary of this revision. (Show Details)
andrew accepted this revision.Tue, Jun 4, 5:33 PM
This revision is now accepted and ready to land.Tue, Jun 4, 5:33 PM
markj accepted this revision.Tue, Jun 4, 8:59 PM
Closed by commit rG5df74441b35f: devmap: eliminate unused arguments (authored by mhorne). · Explain WhyTue, Jun 4, 11:20 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rG5df74441b35f: devmap: eliminate unused arguments.

Revision Contents
Changeset List

PathSize
sys/
arm/
arm/
2 lines
arm64/
arm64/
2 lines
kern/
17 lines
riscv/
riscv/
2 lines
sys/
7 lines

Diff 139474

View Options

sys/arm/arm/machdep.c

Loading...
View Options

sys/arm64/arm64/machdep.c

Loading...
View Options

sys/kern/subr_devmap.c

Loading...
View Options

sys/riscv/riscv/machdep.c

Loading...
View Options

sys/sys/devmap.h

Loading...