Skip to content

[platform] update ingrasys platform change#2860

Closed
linyutsung wants to merge 6 commits intosonic-net:masterfrom
linyutsung:master
Closed

[platform] update ingrasys platform change#2860
linyutsung wants to merge 6 commits intosonic-net:masterfrom
linyutsung:master

Conversation

@linyutsung
Copy link
Copy Markdown
Contributor

@linyutsung linyutsung commented May 3, 2019

- 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

  1. remove gpio_ich when init
  2. enable barefoot ingrasys platform deb installation
  3. use sff8436 for sfp eeprom driver
  4. use optoe for qsfp eeprom driver for nephose platform
  5. fix gpio sysfs init not pull low as default
  6. fix driver kernel module not load when first boot up after installation

@linyutsung linyutsung closed this May 3, 2019
@linyutsung linyutsung reopened this May 3, 2019
@@ -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
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why doing this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

  1. Barefoot SDE already update to ver 8.8.1, the platform deb need to sync with SDE
  2. Currently the new version only support stretch

Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
Contributor Author

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 ?

@linyutsung
Copy link
Copy Markdown
Contributor Author

retest this please

@linyutsung
Copy link
Copy Markdown
Contributor Author

@lguohan
please comment

@linyutsung
Copy link
Copy Markdown
Contributor Author

retest this please

@linyutsung linyutsung closed this Nov 28, 2019
mssonicbld added a commit that referenced this pull request Aug 18, 2023
…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
mssonicbld added a commit that referenced this pull request Sep 3, 2023
…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
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