-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[Inventec][d7032]: Update config.bcm and platform modules to latest. #3522
base: master
Are you sure you want to change the base?
Conversation
@lguohan : Please review it. |
@jleveque : Please review and merge it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I would like @lguohan to take a look, also.
@lguohan : Please review it. |
@jleveque : Please merge it. Its pending from a long time. |
Retest vsimage please |
Hi @jleveque, |
@lguohan: This looks good to me. Can you please review as well? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than using sff8436 SFP driver, can you please utilize the optoe driver which is already installed in SONiC?
* Update config.bcm * Update platform modules Signed-off-by: Ashwini Udupa udupa.ashwini@inventec.com
* Add new platform APIs * Add pmon_daemon_control.json Signed-off-by: Ashwini Udupa udupa.ashwini@inventec.com
@jleveque : Yes, I have updated inventec_d7032_util.py to use optoe2 driver. |
@jleveque : Please review my changes. I have used optoe2 driver. |
Retest mellanox please |
Any updates on getting this merged in ? |
@lguohan to provide final review. |
Might need input on this from @ashwiniudupa1 having issues with i2c on this build still. Built from his branch with master rebased on top of it.
|
…lly (#21774) #### Why I did it src/sonic-swss ``` * 9875694a - (HEAD -> 202411, origin/202411) Added change not to create ECMP Group in SAI and program the route if none of ECMP members are active/link-up (#3522) (19 hours ago) [mssonicbld] * 60fd7355 - [BufferOrch] Use SAI bulk API to configure port, PG and queue (#3523) (19 hours ago) [mssonicbld] * 5071becf - Merge pull request #3519 from stepanblyschak/202411-fc-after-apply-view (33 hours ago) [Kumaresh Perumal] * c3437bd2 - [202411][FC] process FC after apply view (3 days ago) [Stepan Blyschak] ``` #### How I did it #### How to verify it #### Description for the changelog
Signed-off-by: Ashwini Udupa udupa.ashwini@inventec.com
- What I did
Upgraded platform modules for D7032 platform.
Commented out the loopback and management ports in bcm config files
- How I did it
Added updated files to device/inventec/xx86_64-inventec_d7032q28b-r0 and
platform/broadcom/sonic-platform-modules-inventec/d7032q28b/modules
- How to verify it
Built SONiC image and tested on switch.
"show interfaces status" shows correct output of port status and reboot works fine.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)