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

upgrade platform drivers for D7054, add pmon control & add bcm config… #3423

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

habeebmohammed
Copy link
Contributor

… file for dynamicPortBreakout

- What I did
upgraded platform drivers for D7054,
added pmon control &
added bcm config for Dynamic Port Breakout
Also, comment out the loopback and management ports in bcm config files
- How I did it
Added updated files to device/inventec/x86_64-inventec_d7054q28b-r0/
platform/broadcom/sonic-platform-modules-inventec/d7054q28b/modules

- How to verify it
show interfaces status
reboot

- Description for the changelog

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

@msftclas
Copy link

msftclas commented Sep 9, 2019

CLA assistant check
All CLA requirements met.

@wendani
Copy link
Contributor

wendani commented Sep 10, 2019

retest this please

@habeebmohammed
Copy link
Contributor Author

retested..
In previous version of drivers rmmod was having kernel hang during reboot. Now it works fine.
cypress_log_test.txt

{
"skip_ledd": true,
"skip_xcvrd": true,
"skip_psud": true
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why skip psud?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

currently deleted the file we no more use it.
the file pmon.json would not take effect until it matches the name as pmon_daemon_control.json.

@@ -5,37 +5,152 @@
#include "io_expander.h"
#include "transceiver.h"

/* For build single module using (Ex: ONL platform) */
#include <linux/module.h>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use optoe driver.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

optoe2 is used to get "eeprom" info. Updated inventec_d7054_util.py.
Inventec driver is used to get "lpmode", "presence" info that optoe does not support.

@habeebmohammed habeebmohammed requested a review from a team as a code owner June 10, 2022 02:01
mssonicbld added a commit that referenced this pull request Jul 18, 2024
…atically (#19586)

#### Why I did it
src/sonic-utilities
```
* b9a6049a - (HEAD -> master, origin/master, origin/HEAD) [Bug Fix] Fix disk check test and drops group test (#3424) (28 hours ago) [Changrong Wu]
* c03c9c84 - Revert "fix: fix show bgp summary output typo" (#3423) (2 days ago) [Jianquan Ye]
```
#### How I did it
#### How to verify it
#### Description for the changelog
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
…atically (sonic-net#19586)

#### Why I did it
src/sonic-utilities
```
* b9a6049a - (HEAD -> master, origin/master, origin/HEAD) [Bug Fix] Fix disk check test and drops group test (sonic-net#3424) (28 hours ago) [Changrong Wu]
* c03c9c84 - Revert "fix: fix show bgp summary output typo" (sonic-net#3423) (2 days ago) [Jianquan Ye]
```
#### How I did it
#### How to verify it
#### Description for the changelog
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Aug 1, 2024
…atically (sonic-net#19586)

#### Why I did it
src/sonic-utilities
```
* b9a6049a - (HEAD -> master, origin/master, origin/HEAD) [Bug Fix] Fix disk check test and drops group test (sonic-net#3424) (28 hours ago) [Changrong Wu]
* c03c9c84 - Revert "fix: fix show bgp summary output typo" (sonic-net#3423) (2 days ago) [Jianquan Ye]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

6 participants