Page MenuHomeFreeBSD
Differential D49497

arm/mv: Remove pre-armv7 support
ClosedPublic
Actions

Authored by andrew on Mar 25 2025, 3:34 PM.
Tags
None
Referenced Files
F133340829: D49497.diff
Sat, Oct 25, 1:24 AM
F133320518: D49497.id152650.diff
Fri, Oct 24, 10:09 PM
F133308071: D49497.id.diff
Fri, Oct 24, 8:09 PM
F133307946: D49497.id152742.diff
Fri, Oct 24, 8:07 PM
F133221905: D49497.id152646.diff
Fri, Oct 24, 2:35 AM
Unknown Object (File)
Sat, Oct 18, 11:21 PM
Unknown Object (File)
Sat, Oct 18, 4:50 AM
Unknown Object (File)
Thu, Oct 16, 5:28 PM
View All 49 Files
Subscribers
None

Details

Reviewers
cognet
imp
Group Reviewers
ARM
Commits
rGd25a708ba773: arm/mv: Remove pre-armv7 support
Summary

Armv4, Armv5, and Armv6 support has been removed. Remove the Marvell
SoCs that used these cores.

Sponsored by: Arm Ltd

Diff Detail

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

Event Timeline

andrew requested review of this revision.Mar 25 2025, 3:34 PM
andrew created this revision.
andrew added a parent revision: D49496: arm/mv: Remove ic.c, it's unused.
Harbormaster completed remote builds in B63107: Diff 152646.Mar 25 2025, 3:34 PM
andrew added a child revision: D49498: arm: Remove FLASHADDR and PHYSADDR.Mar 25 2025, 3:34 PM
andrew updated this revision to Diff 152650.Mar 25 2025, 3:49 PM

Remove cpu_extra_feat, it's unused

Harbormaster completed remote builds in B63110: Diff 152650.Mar 25 2025, 3:50 PM
cognet accepted this revision.Mar 25 2025, 3:51 PM
cognet added inline comments.
sys/arm/mv/mv_common.c
2050

Any reason to keep those dummy functions at all?

This revision is now accepted and ready to land.Mar 25 2025, 3:51 PM
andrew added inline comments.Mar 25 2025, 3:54 PM
sys/arm/mv/mv_common.c
2050

I have a later change to remove them. They are referenced by the soc_nodes array, however the compat strings use the old FreeBSD specific names, and don't appear in any dts in the tree.

cognet added inline comments.Mar 25 2025, 3:58 PM
sys/arm/mv/mv_common.c
2050

Yeah given the code implies that those dummys where only there to get it to compile if the device were not there, I assumed it could be just removed from soc_nodes.

imp accepted this revision.Mar 25 2025, 10:07 PM
Closed by commit rGd25a708ba773: arm/mv: Remove pre-armv7 support (authored by andrew). · Explain WhyMar 27 2025, 1:48 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rGd25a708ba773: arm/mv: Remove pre-armv7 support.

Revision Contents
Changeset List

PathSize
sys/
arm/
mv/
822 lines
99 lines
4 lines
27 lines
57 lines
conf/
3 lines
dev/
cesa/
20 lines
mge/
37 lines
mvs/
5 lines

Diff 152742

View Options

sys/arm/mv/mv_common.c

Loading...
View Options

sys/arm/mv/mvreg.h

Loading...
View Options

sys/arm/mv/mvvar.h

Loading...
View Options

sys/arm/mv/mvwin.h

Loading...
View Options

sys/arm/mv/timer.c

Loading...
View Options

sys/conf/options.arm

Loading...
View Options

sys/dev/cesa/cesa.c

Loading...
View Options

sys/dev/mge/if_mge.c

Loading...
View Options

sys/dev/mvs/mvs_soc.c

Loading...