Skip to content
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

[mellanox]: Align CPLD component with latest hw-mgmt #4485

Merged

Conversation

nazariig
Copy link
Collaborator

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

- Why I did it

  • To fix CPLD minor version representation

- How I did it

  • Fixed Mellanox platform API

- How to verify it

  1. fwutil show status
root@sonic:/home/admin# fwutil show status
Chassis                  Module    Component    Version                  Description
-----------------------  --------  -----------  -----------------------  ----------------------------------------
x86_64-mlnx_msn3700c-r0  N/A       BIOS         0ACLH004_02.02.007_9600  BIOS - Basic Input/Output System
                                   CPLD1        CPLD000120_REV0223       CPLD - Complex Programmable Logic Device
                                   CPLD2        CPLD000162_REV0800       CPLD - Complex Programmable Logic Device
                                   CPLD3        CPLD000106_REV0100       CPLD - Complex Programmable Logic Device

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
@liat-grozovik liat-grozovik merged commit 0409a32 into sonic-net:master Apr 28, 2020
abdosi pushed a commit that referenced this pull request May 1, 2020
Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
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