Add 36 port 100g sku for x86_64-arista_7800r3a_36d series of linecards. #11813
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arlakshm
reviewed
Aug 30, 2022
| @@ -0,0 +1,21 @@ | |||
| # name lanes alias index role speed asic_port_name coreId corePortId numVoq | |||
| Ethernet0 72,73,74,75 Ethernet1/1 1 Ext 100000 Eth0-ASIC0 1 1 8 | |||
| Ethernet8 80,81,82,83 Ethernet2/1 2 Ext 100000 Eth8-ASIC0 1 3 8 | |||
There was a problem hiding this comment.
Why is the corePortId not continous here ?
arlakshm
reviewed
Aug 30, 2022
| @@ -0,0 +1,984 @@ | |||
| soc_family=BCM8885X | |||
|
|
|||
There was a problem hiding this comment.
system_ref_core_clock_khz=1600000 is missing ?
There was a problem hiding this comment.
This is default for j2c+, we do not need to specify it explicitly in bcm conf file.
| @@ -0,0 +1,41 @@ | |||
| # name lanes alias index role speed asic_port_name coreId corePortId numVoq | |||
There was a problem hiding this comment.
Can we remove this file, it is not needed
| Ethernet264 16,17,18,19 Ethernet34/1 34 Ext 100000 Eth264-ASIC1 0 31 8 | ||
| Ethernet272 8,9,10,11 Ethernet35/1 35 Ext 100000 Eth272-ASIC1 0 33 8 | ||
| Ethernet280 0,1,2,3 Ethernet36/1 36 Ext 100000 Eth280-ASIC1 0 35 8 | ||
| Ethernet-Rec1 221 Recirc0/0 19 Rec 400000 Rcy0-ASIC1 0 221 8 |
There was a problem hiding this comment.
The index for Ethernet-Rec1 and Ethernet-IB1 is overlapping with the index for Ethernet144 and Ethernet-152
| Ethernet120 16,17,18,19 Ethernet16/1 16 Ext 100000 Eth120-ASIC0 0 31 8 | ||
| Ethernet128 8,9,10,11 Ethernet17/1 17 Ext 100000 Eth128-ASIC0 0 33 8 | ||
| Ethernet136 0,1,2,3 Ethernet18/1 18 Ext 100000 Eth136-ASIC0 0 35 8 | ||
| Ethernet-Rec0 221 Recirc0/0 19 Rec 400000 Rcy0-ASIC0 0 221 8 |
There was a problem hiding this comment.
The index for Ethernet-Rec0 and Ethernet-IB0 is overlapping with the index for Ethernet144 and Ethernet-152
arlakshm
reviewed
Sep 30, 2022
| @@ -0,0 +1,21 @@ | |||
| # name lanes alias index role speed asic_port_name coreId corePortId numVoq | |||
| Ethernet144 72,73,74,75 Ethernet19/1 19 Ext 100000 Eth144-ASIC1 1 1 8 | |||
There was a problem hiding this comment.
The 'asic_port_name' should start from Eth0 again.
|
/Azp Azure.sonic-buildimage (BuildVS vs) |
|
Command 'Azure.sonic-buildimage' is not supported by Azure Pipelines. Supported commands
See additional documentation. |
|
/Azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
arlakshm
approved these changes
Oct 17, 2022
yxieca
pushed a commit
that referenced
this pull request
Oct 25, 2022
…s. (#11813) Add 36 port 100g sku for x86_64-arista_7800r3a_36d series of linecards.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add 36 port 100g sku for x86_64-arista_7800r3a_36d series of linecards.