Page MenuHomeFreeBSD

powerpc mpc85xx: Use bus_generic_rman_*
ClosedPublic

Authored by jhb on Jan 12 2024, 10:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 5, 11:55 PM
Unknown Object (File)
Fri, Oct 3, 7:21 AM
Unknown Object (File)
Tue, Sep 30, 8:24 PM
Unknown Object (File)
Mon, Sep 29, 12:05 PM
Unknown Object (File)
Mon, Sep 29, 12:04 PM
Unknown Object (File)
Sat, Sep 27, 10:51 PM
Unknown Object (File)
Sat, Sep 27, 10:50 PM
Unknown Object (File)
Wed, Sep 17, 6:17 AM
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.