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)
Mon, Mar 18, 1:26 PM
Unknown Object (File)
Feb 29 2024, 6:13 PM
Unknown Object (File)
Feb 29 2024, 4:48 PM
Unknown Object (File)
Jan 29 2024, 3:43 AM
Unknown Object (File)
Jan 8 2024, 7:49 AM
Unknown Object (File)
Jan 1 2024, 7:47 AM
Unknown Object (File)
Dec 22 2023, 1:07 AM
Unknown Object (File)
Dec 20 2023, 7:46 AM
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 Skipped
Unit
Tests Skipped

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

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.