Skip to content

Conversation

@arajeev-ARISTA
Copy link
Contributor

@arajeev-ARISTA arajeev-ARISTA commented Mar 19, 2025

Summary

Removed the check which restricted the use of fw_util in Darwin and Darwin48V systems.

Darwin firmware versions will still be printed using the special function printDarwinVersion(). This ensures that fw_util remains compliant with Darwin systems that have older BSP versions, which use a different type of firmware version sysfs entry.

Testing

Verified fw upgrade/downgrade with fw_util in a Darwin48V system. Also verified reading firmware versions in a Darwin system.

@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.

@facebook-github-bot
Copy link
Contributor

@arajeev-ARISTA has updated the pull request. You must reimport the pull request before landing.

@arajeev-ARISTA arajeev-ARISTA force-pushed the enable_darwin_fw_util branch from cae0a09 to ff7fcb1 Compare April 6, 2025 06:18
@facebook-github-bot
Copy link
Contributor

@arajeev-ARISTA has updated the pull request. You must reimport the pull request before landing.

@ezeob002
Copy link
Contributor

@arajeev-ARISTA, could you please make the necessary changes in the FwUtilVersionHandler class?. We restructured the code base.

@arajeev-ARISTA arajeev-ARISTA force-pushed the enable_darwin_fw_util branch from ff7fcb1 to 80ea2cb Compare May 12, 2025 03:58
@facebook-github-bot
Copy link
Contributor

@arajeev-ARISTA has updated the pull request. You must reimport the pull request before landing.

@arajeev-ARISTA
Copy link
Contributor Author

@arajeev-ARISTA, could you please make the necessary changes in the FwUtilVersionHandler class?. We restructured the code base.

Done!

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@ezeob002 merged this pull request in f173622.

joywu-coder pushed a commit to joywu-coder/fboss that referenced this pull request May 26, 2025
Summary:
Removed the check which restricted the use of fw_util in Darwin and Darwin48V systems.

Darwin firmware versions will still be printed using the special function `printDarwinVersion()`. This ensures that fw_util remains compliant with Darwin systems that have older BSP versions, which use a different type of firmware version sysfs entry.
### Testing
Verified fw upgrade/downgrade with fw_util in a Darwin48V system. Also verified reading firmware versions in a Darwin system.

Pull Request resolved: facebook#352

Reviewed By: joancaneus

Differential Revision: D71937785

Pulled By: ezeob002

fbshipit-source-id: 77e8452ab6d1ed29dab15607ea763aa0df3c77b7
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.

3 participants