new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCE
ClosedPublic
Actions

Authored by jhb on Feb 27 2024, 7:47 PM.

Details

Reviewers

imp
andrew
manu

Commits

rGd77f2092ceeb: new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCE

Summary

The public bus_map/unmap_resource() API still accepts both forms, but
the internal kobj methods no longer pass the argument.
Implementations which need the type now use rman_get_type() to fetch
the value from the allocated resource.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 56302
Build 53190: arc lint + arc unit

Event Timeline

jhb created this revision.Feb 27 2024, 7:47 PM

Herald added a reviewer: andrew. · View Herald TranscriptFeb 27 2024, 7:47 PM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

Herald added subscribers: riscv, olce, jhibbits, emaste. · View Herald Transcript

jhb requested review of this revision.Feb 27 2024, 7:47 PM

Harbormaster completed remote builds in B56302: Diff 135118.Feb 27 2024, 7:47 PM

jhb added a parent revision: D44128: new-bus: Remove the 'type' argument from BUS_ADJUST_RESOURCE.Feb 27 2024, 7:47 PM

jhb added a child revision: D44130: new-bus: Remove the 'rid' and 'type' arguments from BUS_*ACTIVATE_RESOURCE.

imp accepted this revision.Mar 8 2024, 6:45 PM

This revision is now accepted and ready to land.Mar 8 2024, 6:45 PM

Closed by commit rGd77f2092ceeb: new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCE (authored by jhb). · Explain WhyMar 13 2024, 10:14 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGd77f2092ceeb: new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCE.

Revision Contents
Changeset List

Path

Size

share/

man/

man9/

bus_map_resource.9

14 lines

sys/

arm/

nexus.c

8 lines

mv/

mv_pci.c

14 lines

arm64/

nexus.c

11 lines

cavium/

thunder_pcie_pem.c

19 lines

dev/

acpica/

acpi.c

19 lines

fdt/

simplebus.c

30 lines

ofw/

ofw_pcib.c

19 lines

pci/

pci_host_generic.c

22 lines

pci_pci.c

15 lines

vmd/

vmd.c

8 lines

kern/

bus_if.m

4 lines

subr_bus.c

35 lines

powerpc/

mpc85xx/

lbc.c

12 lines

powermac/

macio.c

17 lines

uninorth.c

17 lines

powerpc/

nexus.c

10 lines

ps3/

ps3bus.c

14 lines

psim/

iobus.c

12 lines

riscv/

nexus.c

8 lines

sys/

bus.h

25 lines

x86/

nexus.c

9 lines

Diff 135118

View Options

share/man/man9/bus_map_resource.9

View Options

sys/arm/arm/nexus.c

View Options

sys/arm/mv/mv_pci.c

View Options

sys/arm64/arm64/nexus.c

View Options

sys/arm64/cavium/thunder_pcie_pem.c

View Options

sys/dev/acpica/acpi.c

View Options

sys/dev/fdt/simplebus.c

View Options

sys/dev/ofw/ofw_pcib.c

View Options

sys/dev/pci/pci_host_generic.c

View Options

sys/dev/pci/pci_pci.c

View Options

sys/dev/vmd/vmd.c

View Options

sys/kern/bus_if.m

View Options

sys/kern/subr_bus.c

View Options

sys/powerpc/mpc85xx/lbc.c

View Options

sys/powerpc/powermac/macio.c

View Options

sys/powerpc/powermac/uninorth.c

View Options

sys/powerpc/powerpc/nexus.c

View Options

sys/powerpc/ps3/ps3bus.c

View Options

sys/powerpc/psim/iobus.c

View Options

sys/riscv/riscv/nexus.c

View Options

sys/sys/bus.h

View Options

new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCEClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 135118

share/man/man9/bus_map_resource.9

sys/arm/arm/nexus.c

sys/arm/mv/mv_pci.c

sys/arm64/arm64/nexus.c

sys/arm64/cavium/thunder_pcie_pem.c

sys/dev/acpica/acpi.c

sys/dev/fdt/simplebus.c

sys/dev/ofw/ofw_pcib.c

sys/dev/pci/pci_host_generic.c

sys/dev/pci/pci_pci.c

sys/dev/vmd/vmd.c

sys/kern/bus_if.m

sys/kern/subr_bus.c

sys/powerpc/mpc85xx/lbc.c

sys/powerpc/powermac/macio.c

sys/powerpc/powermac/uninorth.c

sys/powerpc/powerpc/nexus.c

sys/powerpc/ps3/ps3bus.c

sys/powerpc/psim/iobus.c

sys/riscv/riscv/nexus.c

sys/sys/bus.h

sys/x86/x86/nexus.c

new-bus: Remove the 'type' argument from BUS_MAP/UNMAP_RESOURCE
ClosedPublic
Actions

Revision Contents
Changeset List