-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[platform] update ingrasys platform change #2860
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
Conversation
@@ -1,5 +1,7 @@ | |||
BFN_INGRASYS_PLATFORM = bfnplatform-ingrasys_8.2.0_amd64.deb | |||
$(BFN_INGRASYS_PLATFORM)_URL = "https://github.com/Ingrasys-sonic/packages/raw/master/lib/bfnplatform-ingrasys_8.4.0_8.5.0_amd64.deb" | |||
ifdef BLDENV |
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.
why doing this?
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.
- Barefoot SDE already update to ver 8.8.1, the platform deb need to sync with SDE
- Currently the new version only support stretch
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.
the master version is stretch.
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.
Do you mean the "ifdef BLDEVN" not needed here ? According to bfn-platform.mk, there are two different platform deb packages for stretch and jessie, but in our case only stretch is supported, so add the "ifdef BLDEVN" without "else".
Refer to https://github.com/Azure/sonic-buildimage/blob/master/platform/barefoot/bfn-platform.mk
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.
if the "ifdef BLDEVN" removed, the installation of the platform deb will fail due to dependency issue for jessie. Since our platform deb no longer support jessie build but only for stretch build, it should be ok not installing the deb for jessie build. Any better suggestion ?
platform/barefoot/sonic-platform-modules-ingrasys/s9180-32x/utils/i2c_utils.sh
Outdated
Show resolved
Hide resolved
platform/barefoot/sonic-platform-modules-ingrasys/s9180-32x/utils/i2c_utils.sh
Show resolved
Hide resolved
platform/barefoot/sonic-platform-modules-ingrasys/s9180-32x/utils/i2c_utils.sh
Outdated
Show resolved
Hide resolved
retest this please |
@lguohan |
retest this please |
…lly (#16181) #### Why I did it src/sonic-swss ``` * 859bd678 - (HEAD -> 202211, origin/202211) Fix error in peer response time when headroom is calculated for 800G (#2860) (2 days ago) [Stephen Sun] * 5f294cf1 - [Dynamic Buffer][Mellanox] Skip PGs in pending deleting set while checking accumulative headroom of a port (#2871) (2 days ago) [Stephen Sun] ``` #### How I did it #### How to verify it #### Description for the changelog
…lly (#16396) #### Why I did it src/sonic-swss ``` * c869c1df - (HEAD -> 202305, origin/202305) update portStatIds for cisco (#2876) (11 minutes ago) [Zhixin Zhu] * dd152288 - [Dynamic Buffer][Mellanox] Skip PGs in pending deleting set while checking accumulative headroom of a port (#2871) (12 minutes ago) [Stephen Sun] * 97068ff1 - Fix error in peer response time when headroom is calculated for 800G (#2860) (16 minutes ago) [Stephen Sun] ``` #### How I did it #### How to verify it #### Description for the changelog
- What I did
Update Ingrasys platform device and drivers
- How I did it
Update platform init and configuration files
- How to verify it
Do sanity checks on the device
- Description for the changelog