Skip to content

[2/3]OOB_eeprom support: common: fw_utils:Added operations to update oob_eeprom #362

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jaganathK-celestica
Copy link

Description:
This pull request implements the missing firmware operations for oob_eeprom within the fw_util utility. This operation is crucial for supporting hardware interactions during firmware upgrades those involving I2C devices and dynamic device mapping. The implemented operations includes the following key changes:

  • Implemented performI2cEepromOperation to handle firmware upgrade for I2C EEPROM devices.
  • Implemented driver binding and unbinding through pre-existing functions from I2cExplorer.
  • Executes dd commands for firmware writing and reading.

Motivation:
The motivation for this change is to extend the firmware utility's capabilities to support firmware upgrades on I2C EEPROM devices such as OOB eeprom. This breakage was caused during the recent changes done during the commit 'optimizing configs' here at: 47b49e9

Testing:
The following testing procedures were performed to ensure the correctness and stability of the changes:

  • Verified that the correct I2C driver is binded and unbinded.
  • Confirmed that the dd commands are executed with the appropriate parameters.
  • Validated that the firmware image is correctly written to the I2C EEPROM.

Uint test:
Logs after merging PR's [1/3]common: I2cExplorer, [2/3]common: fw_utils and [3/3]tahan: config
upgrade_log.txt
downgrade_log.txt

@facebook-github-bot
Copy link
Contributor

@joancaneus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@jaganathK-celestica jaganathK-celestica changed the title [2/3]common: fw_utils: Added I2cExplorer operations to update oob_eeprom [2/3]common: fw_utils:Added operations to update oob_eeprom Apr 9, 2025
@jaganathK-celestica jaganathK-celestica changed the title [2/3]common: fw_utils:Added operations to update oob_eeprom OOB_eeprom support: common: fw_utils:Added operations to update oob_eeprom Apr 15, 2025
@jaganathK-celestica jaganathK-celestica changed the title OOB_eeprom support: common: fw_utils:Added operations to update oob_eeprom [2/3]OOB_eeprom support: common: fw_utils:Added operations to update oob_eeprom Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants