Page MenuHomeFreeBSD

powerpc ps3bus: Use bus_generic_rman_*
ClosedPublic

Authored by jhb on Jan 12 2024, 10:32 PM.
Tags
None
Referenced Files
F174425202: D43432.diff
Sat, Oct 3, 2:49 AM
Unknown Object (File)
Sun, Sep 20, 6:56 PM
Unknown Object (File)
Fri, Sep 11, 3:18 AM
Unknown Object (File)
Sun, Sep 6, 5:36 AM
Unknown Object (File)
Sun, Sep 6, 1:05 AM
Unknown Object (File)
Sat, Sep 5, 5:52 AM
Unknown Object (File)
Sat, Sep 5, 3:47 AM
Unknown Object (File)
Fri, Sep 4, 2:20 PM
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.