Page MenuHomeFreeBSD

Migrate SENTRY5 to bhnd(4) and unify with BCM
ClosedPublic

Authored by landonf on Jun 3 2016, 11:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 6, 1:51 PM
Unknown Object (File)
Feb 24 2024, 9:12 PM
Unknown Object (File)
Feb 23 2024, 2:28 PM
Unknown Object (File)
Feb 7 2024, 2:48 PM
Unknown Object (File)
Feb 7 2024, 1:03 PM
Unknown Object (File)
Jan 8 2024, 7:00 PM
Unknown Object (File)
Jan 8 2024, 7:00 PM
Unknown Object (File)
Jan 8 2024, 7:00 PM
Subscribers

Details

Summary

Migrates SENTRY5 fully over to bhnd's siba(4), unifying SENTRY5's and BCM's nexus handling.

The previous SENTRY5/siba code required a few changes to get it booting again, but I was able to confirmed that the new bhnd(4) code provides feature parity with the previous implementation.

Issues resolved:

  • Fixed enumeration of siba(4) per-core address maps, required for discovery of the memory mapped chipc flash region on siba(4) devices (e.g. SENTRY5).
  • Simplified bhnd kernel configuration (dropped 'bhndbus' option).
  • Replaced files.broadcom's direct file references with their corresponding kernel options.
  • Lifted out common bcma/siba nexus support, inheriting from the new generic bhnd_nexus driver.
  • Dropped now-unused sentry5 siba code.
  • Re-integrated BCM into the universe build now that it actually compiles.

Depends on D6250

Test Plan

Tested against:

  • Asus RT-N16 (BCM)
  • Netgear WGT634U (SENTRY5)
  • BCM4331 (if_bwn_pci)
  • BCM4312 (if_bwn_pci)

Non-build breakage confirmed via tinderbox (MAKE_JUST_KERNELS).

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

landonf retitled this revision from to Migrate SENTRY5 to bhnd(4) and unify with BCM.
landonf updated this object.
landonf edited the test plan for this revision. (Show Details)
landonf added a reviewer: adrian.
landonf added a subscriber: imp.
adrian edited edge metadata.

Yup, do it!

This revision is now accepted and ready to land.Jun 4 2016, 1:25 AM
landonf edited edge metadata.

Add necessary BCM bcma_nexus delta that went missing in review diff (and thus, prevented bhnd attachment). Caught when testing D6712 diff locally.

This revision now requires review to proceed.Jun 4 2016, 6:27 PM
adrian edited edge metadata.

Yup, do it!

This revision is now accepted and ready to land.Jun 4 2016, 7:10 PM
This revision was automatically updated to reflect the committed changes.