Page MenuHomeFreeBSD

powerpc mpc85xx: Use bus_generic_rman_*
ClosedPublic

Authored by jhb on Jan 12 2024, 10:32 PM.
Tags
None
Referenced Files
F102050446: D43434.diff
Thu, Nov 7, 12:05 AM
Unknown Object (File)
Wed, Oct 16, 10:14 PM
Unknown Object (File)
Tue, Oct 15, 6:53 AM
Unknown Object (File)
Mon, Oct 14, 9:30 AM
Unknown Object (File)
Sat, Oct 12, 9:28 PM
Unknown Object (File)
Fri, Oct 11, 12:50 PM
Unknown Object (File)
Thu, Oct 10, 8:08 AM
Unknown Object (File)
Oct 7 2024, 12:30 PM
Subscribers

Details

Summary
  • Consistently map SYS_RES_IOPORT to SYS_RES_MEMORY for bus_activate/deactivate/adjust/release_resource.
  • Implement bus_map/unmap_resource.
  • Add an implementation of bus_adjust_resource.

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

I like the approach of the RF_LITTLEENDIAN, but fsl_sata hangs off simple-bus, not the lbc, so it's stuck with manually setting the bus tag. I can test the lbc changes later.

This revision now requires changes to proceed.Jan 19 2024, 2:39 AM

I like the approach of the RF_LITTLEENDIAN, but fsl_sata hangs off simple-bus, not the lbc, so it's stuck with manually setting the bus tag. I can test the lbc changes later.

Hmm, so if it is off of simplebus, does this mean it that the powerpc nexus devices need to honor the flag?

I like the concept of RF_LITTLEENDIAN, but will leave this one alone and let you and Justin hash it out.

I've moved RF_LITTLEENDIAN to a separate change (D43553).

jhibbits added inline comments.
sys/powerpc/mpc85xx/lbc.c
726

Heh, very wrong, and wrong since the beginning. Should be *rid = di->di_bank, from the look of it.

This revision is now accepted and ready to land.Jan 23 2024, 4:56 PM
sys/powerpc/mpc85xx/lbc.c
726

Yes, that's my guess too, but wasn't sure.

This revision was automatically updated to reflect the committed changes.