Skip to content

[component]: Introduce new firmware management API#82

Merged
liat-grozovik merged 4 commits intosonic-net:masterfrom
nazariig:master-component-firmware-api
Apr 30, 2020
Merged

[component]: Introduce new firmware management API#82
liat-grozovik merged 4 commits intosonic-net:masterfrom
nazariig:master-component-firmware-api

Conversation

@nazariig
Copy link
Copy Markdown
Collaborator

@nazariig nazariig commented Apr 13, 2020

Signed-off-by: Nazarii Hnydyn nazariig@mellanox.com

Summary:
The purpose of this PR is to propagate some new component firmware management APIs which are intended to make the process more versatile. These new API's will be used mostly to extend fwutil functionality in order to support new platform components (e.g, SSD).

Motivation:

  1. Inform user in advance about required component firmware update actions when operating in manual mode
  2. Allow fwutil to read component firmware version directly from image instead of configuration file if not provided

Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
@liat-grozovik liat-grozovik merged commit 28c39c5 into sonic-net:master Apr 30, 2020
@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented May 1, 2020

@nazariig create PR for 201911 . Cherry-pick is not happening as there is some issue with merge commit

@nazariig
Copy link
Copy Markdown
Collaborator Author

@abdosi Done: #87

oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…asses (sonic-net#82)

Previous commit to transition to importing sonic-py-common accidentally replaced all `logger` instances in xcvrd with `self`, although not all of these instances resided in the `DaemonXcvrd` class, breaking them. Here we add a global Logger instance for these references to use. Eventually we should refactor so that there is only one Logger instance created by xcvrd, not two.

- Also rename class names using PascalCase for consistency across the codebase.
- Also add Logger instance to ThermalMonitor class in thermalctld
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-platform-common that referenced this pull request Jun 10, 2025
…et#82)

<!-- Provide a general summary of your changes in the Title above -->

#### Description
<!--
 Describe your changes in detail
-->
The DOM and Status related APIs need to be modified to ensure that the fields which have latched registers are moved to the flag specific API.
Also, all the VDM specific fields have now been removed from DOM and Status related APIs since separate APIs pertaining to processing VDM data have now been created.

This change is in line to the requirements of [SONiC/doc/platform_api/CMIS_Diagnostic_Monitoring_Overview_in_SONiC.md at master · sonic-net/SONiC](https://github.com/sonic-net/SONiC/blob/master/doc/platform_api/CMIS_Diagnostic_Monitoring_Overview_in_SONiC.md)

#### Motivation and Context
<!--
 Why is this change required? What problem does it solve?
 If this pull request closes/resolves an open Issue, make sure you
 include the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here
-->
N/A

#### How Has This Been Tested?
<!--
 Please describe in detail how you tested your changes.
 Include details of your testing environment, and the tests you ran to
 see how your change affects other areas of the code, etc.
-->
Please refer to sonic-net/sonic-utilities#3844 for the test details

#### Additional Information (Optional)
MSFT ADO - 30090754

Related PRs
sonic-net/SONiC#1954
sonic-net/sonic-platform-daemons#604
sonic-net/sonic-utilities#3844
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.

5 participants