Page MenuHomeFreeBSD

powerpc ps3bus: Use bus_generic_rman_*
ClosedPublic

Authored by jhb on Jan 12 2024, 10:32 PM.
Tags
None
Referenced Files
F83252094: D43432.diff
Wed, May 8, 2:11 AM
Unknown Object (File)
Wed, Apr 24, 4:01 AM
Unknown Object (File)
Mon, Apr 22, 6:32 PM
Unknown Object (File)
Sun, Apr 21, 11:11 AM
Unknown Object (File)
Sun, Apr 21, 11:11 AM
Unknown Object (File)
Sun, Apr 21, 11:07 AM
Unknown Object (File)
Sun, Apr 21, 12:15 AM
Unknown Object (File)
Sat, Apr 20, 4:51 AM
Subscribers

Details

Summary
  • Split out part of ps3bus_activate_resource into a ps3bus_map_resource method.
  • Implement ps3bus_unmap_resource via pmap_unmapdev.
  • Use bus_generic_rman_* to add bus_adjust_resource, bus_deactivate_resource, and bus_release_resource methods.

Diff Detail

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

Event Timeline

jhb requested review of this revision.Jan 12 2024, 10:32 PM

Looks fine. I have no way to test, though.

This revision is now accepted and ready to land.Jan 19 2024, 2:34 AM

I wonder if the PS3 option might be past its prime and time to retire.
I couldn't figure out how to run it for my kboot work, for example.

In D43432#991924, @imp wrote:

I wonder if the PS3 option might be past its prime and time to retire.
I couldn't figure out how to run it for my kboot work, for example.

Agreed, and our resident maintainer, Nathan, is no longer available.

In D43432#991924, @imp wrote:

I wonder if the PS3 option might be past its prime and time to retire.
I couldn't figure out how to run it for my kboot work, for example.

Agreed, and our resident maintainer, Nathan, is no longer available.

I've had two offers to test PS/3 based on people having PS/3 sitting in their closet, not based on people still running it day to day.

This revision was automatically updated to reflect the committed changes.