Page MenuHomeFreeBSD

[nvram2env] split implementation into generic & MIPS-based code
ClosedPublic

Authored by mizhka on May 23 2016, 4:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 20 2024, 11:36 AM
Unknown Object (File)
Oct 18 2024, 1:00 PM
Unknown Object (File)
Oct 16 2024, 5:12 PM
Unknown Object (File)
Oct 1 2024, 12:36 PM
Unknown Object (File)
Oct 1 2024, 8:12 AM
Unknown Object (File)
Sep 30 2024, 1:33 PM
Unknown Object (File)
Sep 30 2024, 7:34 AM
Unknown Object (File)
Sep 23 2024, 9:04 PM
Subscribers

Details

Summary

This is simple change to split implementation of nvram2env to generic (MI) & MIPS-based code:

  • removed includes like "*siba*", because they are unused
  • added nvram2env_mips.c file with MIPS-specific code, code moved from nvram2env.c
  • added header file to shared defines/structures/function prototypes between MI and MIPS code

Also this fix allows to implement own nvram2env drivers.

Test Plan

Tested on BroadCom router RT-N53. Additional testing are welcome.

Diff Detail

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

Event Timeline

mizhka retitled this revision from to [nvram2env] split implementation into generic & MIPS-based code.
mizhka updated this object.
mizhka edited the test plan for this revision. (Show Details)
mizhka added reviewers: ray, adrian, landon_landonf.org.
mizhka set the repository for this revision to rS FreeBSD src repository - subversion.
  • remove directory path from header protector
  • rename NVRAM_SIGNATURE to CFE_NVRAM_SIGNATURE

Only one comment left.

sys/dev/nvram2env/nvram2env.c
238 ↗(On Diff #16744)

Think, it's time to remove #if-s.

mizhka edited edge metadata.
  • removed "#if 1"s
ray edited edge metadata.

Now, looks fine.

This revision is now accepted and ready to land.May 27 2016, 8:50 AM
adrian edited edge metadata.

approved!

This revision was automatically updated to reflect the committed changes.