[Mellanox] Add support for new platform SN2201 #19
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
Includes commits ``` d7900dd 2021-09-01 | [config][generic-update] Implementing patch sorting (2nd attempt) (sonic-net#1794) [Mohamed Ghoneim] 7ed9a91 2021-09-01 | [show] Allow system with no ports in config db run without errors (sonic-net#1754) [Lior Avramov] ```
…onic-net#7965)" (sonic-net#8705) cause regression on the kvm testing This reverts commit 76d5223.
Commits on Sep 01, 2021 hw-mgmt: attributes: Add PSU power sensor attributes d8fce39 Commits on Sep 02, 2021 Remove MFT package flint tool from hw-management dump generation. 53d06b2 hw-mgmt: debug: Add timeout to generate-dump.sh b661fa3 Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
…8684) Catch up on fixes from BRCM SUG repo to pick up fixes after 5.0.0.6 all the way up to 5.0.0.8 Fixes include the following: ``` CS00012201827: Warmreboot causes syncd crash with SAI_API_UNSPECIFIED:syncdb_data_file_read:2230 Failed to parse JSON: error -2 DNX: Fix for ACL table create with v6 next hdr attr and many unspecified changes that also went into 5.0.0.8 ``` #### How to verify it Preliminary tests looks fine on both XGS (gechiang) and DNX (judyjoseph) On XGS testing done as following: BGP neighbors were all up with proper routes programmed interfaces are all up Manually ran the following test cases on 7260CX3 T0 DUT and all passed: ``` ipfwd/test_dir_bcast.py fdb/test_fdb.py fib/test_fib.py vlan/test_valn.py ``` Also validated for for CS00012201827 (sonic-net#8300)
Why I did it Power cycle test case fails for Z9332f in sonic-mgmt framework(sonic-net#8605). How I did it Modified the platform API to return expected strings. How to verify it Power cycle the device and verify the reboot reason. Run sonic-mgmt test_reboot script.
To include sonic_installer swap setup changes 171eb4f [sonic_installer] Add swap setup support (sonic-net#1787) 6483b0b QOS fieldvalue refernce ABNF format to string (sonic-net#1626) 8d16eb5 [Fast-reboot] Set flex counters delay indicator to prevent flex counters enablement after fast-reboot (sonic-net#1768) 1bc24ca [portstat pfcstat] Unify the packet number format in the output of portstat and pfcstat in all cases (sonic-net#1755) d7900dd [config][generic-update] Implementing patch sorting (2nd attempt) (sonic-net#1794) 7ed9a91 [show] Allow system with no ports in config db run without errors (sonic-net#1754) Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
What I did it Add new platform x86_64-ragile_ra-b6910-64c-r0 (Tomahawk 3) ASIC Vendor: Broadcom Switch ASIC: Tomahawk 3 Port Config: 64x100G -How I did it Provide device and platform related files. -How to verify it show platform fan show platform ssdhealth show platform psustatus show platform summary show platform syseeprom show platform temperature show interface status
…provided (sonic-net#8271) Why I did it Fix an issue on the Clearwater2 linecard. When the linecard is started with a fresh image without configuration, phys would not be initialized. How I did it Added default_sku for Clearwater2 which prevents config-setup from failing to create a default config_db.json. Added some extra logic in the phy-credo-init script to run the phy_config.sh of the hwsku pointed by default_sku if the DEVICE_METADATA.localhost.hwsku information is not populated in CONFIG_DB. How to verify it Booting an image with this change and without configuration will lead to the phys being initialized using the phy_config.sh from default_sku.
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
- Why I did it Advance to Mellanox SAI ver 1.19.2 to pick up dynamic Policy Based Hashing support. For this version above the static Policy Based Hashing is no longer supported. For detailed release notes check https://github.com/Mellanox/SAI-Implementation/blob/sonic2111/release%20notes.txt - How I did it Updated SAI-Implementation submodule - How to verify it 1. make configure PLATFORM=mellanox 2. make target/sonic-mellanox.bin Run full regression as well as new dynamic PBH tests Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Fix sonic-net#8722 retreat two commits which cause warm reboot regression * eb79ca4 2021-09-01 | [pbh]: Add PBH OA (sonic-net#1782) [Nazarii Hnydyn] * 3d6b1f0 2021-08-31 | [buffer orch] Bugfix: Don't query counter SAI_BUFFER_POOL_STAT_XOFF_ROOM_WATERMARK_BYTES on a pool where it is not supported (sonic-net#1857) [Stephen Sun] Signed-off-by: Guohan Lu <lguohan@gmail.com>
* [Nokia ixs7215] Miscellaneous platform API fixes This commit delivers the following fixes for the Nokia ixs7215 platform - Fix bug in a fan API error path - Add support for setting the fan drawer led - Add support for getting/setting the front panel PSU status led - Add support for getting the min/max observed temperature value * [Nokia ixs7215] code review changes: temperature min/max values
- Why I did it Removed 2x40G for SN3800. This mode is not supported by hardware. - How I did it Removing it from hwsku.json and platform.json - How to verify it Load it in the device and check supported modes
Why I did it fstrim has dependency on pmon docker. How I did it start fstrim timer after sonic.target. How to verify it local test and PR test. Signed-off-by: Ying Xie ying.xie@microsoft.com
…oading minigraph (sonic-net#8711) To Fix sonic-net#8697 . The config load_minigraph initializes 'admin_status' to up when platform.json has DPB configs. This doesn't happen when using port_config.ini The update minigraph has logic to initialize only the ports whose neighbors are defined or those belonging to portchannel However, a change was introduced to have default admin status to be 'up' in portconfig.py when the minigraph was using platform.json This will lead to sanity check failure in sonic-mgmt and thus no test cases could be run
*Removed execute permissions from the systemd copp-config.service file. Without this we will get a warning: "Configuration file /lib/systemd/system/copp-config.service is marked executable. Please remove executable permission bits. Proceeding anyway."
… incomprehensible firmware Error print (sonic-net#8690) What I did: add platform components How I did it: In platform_components.json add chassis and empty component How to verify it: Run show platform firmware updates
…ot time (sonic-net#7965)" (sonic-net#8705)" (sonic-net#8724) This reverts commit f4dea87.
… commands are executed from linecards (sonic-net#8065) * [multi-asic][cli][chassis-db] Avoiding connecting to chassis db Currently, for all the cli commands, we connect to all databases mentioned in the database_config.json. The database_config.json also includes the databases from chassis redis server from supervisor card. It is unneccessary to connect to databases from chassis redis server when cli commands are executed form linecard. But we need to allow connection to chassis databases when the cli commands are executed from supervisor card. The changes in this PR fixes this problem. This PR requires that asic.conf in supervisor card includes VOQ_SUPERVISOR with value 1 to indentify the supervisor card. The connect_to_all_dbs_for_ns() is changed to skip chassis databases form the list of collected databases if the card is not supervisor card.
2cd6236 [voq][chassis] Fix for issue in chassis redis server connection check (sonic-net#110) 9da577d [port_util] Fix issue in function get_interface_oid_map (sonic-net#114) 740a44c [port_util] Allow system without ports in config db run without errors (sonic-net#109) d07682e Merge pull request sonic-net#111 from xumia/azp-coverage 482cac2 [Ethernet-IB][index] Modifed the port_util to support the VoQ Inband Port (sonic-net#113) 69eac92 Support azp diff coverage Signed-off-by: liora <liora@nvidia.com>
Added logrotate file for wtmp and btmp to override default conf and set size cap as 100K as done in PR: sonic-net#865. For buster this is control by separate file wtmp and btmp. Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
* DellEMC: Z9332f fix platform bugs * update sfp.py
* [Nokia ixs7215] Support show system-health * [Nokia ixs7215] Fix LGTM alert
* Add pipeline to build vhdx image
… metadata part of the version (sonic-net#8920) - Why I did it In case an app.ext requires a dependency syncd^1.0.0, the RPC version of syncd will not satisfy this constraint, since 1.0.0-rpc < 1.0.0. This is not correct to put 'rpc' as a prerelease identifier. Instead put 'rpc' as build metadata in the version: 1.0.0+rpc which satisfies the constraint ^1.0.0. - How I did it Changed the way how to version in RPC and DBG images are constructed. - How to verify it Install app.ext with syncd^1.0.0 dependency on a switch with RPC syncd docker. Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
* Enable flex counters for Vxlan tunnel
…er-sonic-vs (sonic-net#9044) Added support for Mellanox-SN2700 based SKU for docker-sonic-vs and to differentiate platform based on hw-sku rather than on fake_platform in VS. Currently SAI VS library uses hwsku based SAI profile to differentiate and mock different platform implementations. The same functionality in swss is achieved using a fake_platform env variable. Using a fake_platform has some drawbacks that the vs container appears to still use a different vendor hw-sku env PLATFORM=x86_64-kvm_x86_64-r0 HOSTNAME=dd21a1637723 PWD=/ HOME=/root TERM=xterm HWSKU=Force10-S6000 SHLVL=1 fake_platform=mellanox PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin DEBIAN_FRONTEND=noninteractive _=/usr/bin/env In order to unify the approach at both swss and vs SAI and to be uniform throughout this PR introduces the approach of using hw-sku to differentiate different platforms. This requires support for Mellanox-SN2700 HWSKU for Mellanox platform which is also addressed by this PR. root@23c9ba83b0aa:/# show platform summary /bin/sh: 1: sudo: not found Platform: x86_64-kvm_x86_64-r0 HwSKU: Mellanox-SN2700 ASIC: vs ASIC Count: 1 Serial Number: N/A Model Number: N/A Hardware Revision: N/A root@23c9ba83b0aa:/# env PLATFORM=x86_64-kvm_x86_64-r0 HOSTNAME=23c9ba83b0aa PWD=/ HOME=/root TERM=xterm HWSKU=Mellanox-SN2700 SHLVL=1 PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin DEBIAN_FRONTEND=noninteractive _=/usr/bin/env root@23c9ba83b0aa:/# Signed-off-by: Sudharsan Dhamal Gopalarathnam <sudharsand@nvidia.com>
``` ca728b8 [config] fix interface IPv6 address removal. (sonic-net#1819) 0665d6f VxLAN Tunnel Counters and Rates implementation (sonic-net#1748) 80a10dc Fix log_ssd_health hang issue (sonic-net#1904) ea4a730 [config][cbf] Added config commands for CBF (sonic-net#1799) 02ce8d6 [sonic-package-manager] update FEATURE entries on upgrade (sonic-net#1803) 9f123c0 [generate_dump] remove secrets from dump files (sonic-net#1886) 3a8ab73 [fwutil] Add `fwutil update all` to support the automatic platform component fw updates (sonic-net#1242) 776fddf [sonic-package-manager] code style fixes and enhancements (sonic-net#1802) f53baac [watermarkstat] Fix for error in processing empty array from couters db (sonic-net#1810) 0b2536b Generic_upater: Apply JSON change (sonic-net#1856) ``` Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Updated SDK packages to SAI 1.9.1 Signed-off-by: Roman Savchuk <romanx.savchuk@intel.com>
Missing comment change which is supposed to arrive with sonic-net#8861 is added here Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
- update DHCP_PACKET_MARK schema in state_db - this is an update over PR: Add service mark_dhcp_packet to mux container sonic-net#9015
Why I did it During swss container startup, if ndppd starts up before/with vlanmgrd, ndppd will be pinned at nearly 100% CPU usage. How I did it Only start ndppd after vlanmgrd is running. Also, call ndppd directly instead of through bash for improved logging and to prevent orphaned processes. Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
…card. (sonic-net#9139) The Nokia IXR7250E supervisor card asic_type should be broadcom-dnx instead of broadcom. Fixes sonic-net#9140
Signed-off-by: Rajkumar Pennadam Ramamoorthy <rpennadamram@marvell.com>
Added get_system_mac support for cisco-8000 device Why I did it This is required to assign unique MACs to namespaces on Backend and Frontend asics for cisco-8000 platforms How I did it Add vendor specific hook in get_system_mac. The MAC address are read from /profile.ini file How to verify it config load_minigraph on a multi-asic system config save check mac addresses in DEVICE_METADATA section in each config_db json file
SONiC Yang model for DHCP-Relay parameters What I did Added DHCPv6 Servers. Tables: VLAN, DHCP_RELAY. How I did it Defined Yang model for Relay based on Guideline doc: https://github.com/Azure/SONiC/blob/master/doc/mgmt/SONiC_YANG_Model_Guidelines.md How to verify it Added test cases to verify it. Signed-off-by: Akhilesh Samineni akhilesh.samineni@broadcom.com
…-net#9071) * DellEMC: Z9332f - SFP API get_error_description implementation * Update error message
… TACACS+ per-command authorization&accounting. (sonic-net#9029) [TACACS+] Add Config DB schema and HostCfg Enforcer plugin to support TACACS+ per-command authorization&accounting. (sonic-net#9029) #### Why I did it Support TACACS per-command authorization&accounting. #### How I did it Change ConfigDB schema and HostCfg enforcer. Add UT to cover changed code. #### How to verify it Build following project and pass all UTs: make target/python-wheels/sonic_host_services-1.0-py3-none-any.whl #### Which release branch to backport (provide reason below if selected) N/A #### Description for the changelog Add Config DB schema and HostCfg Enforcer plugin to support TACACS+ per-command authorization&accounting. #### A picture of a cute animal (not mandatory but encouraged)
d23cf88 - 2021-10-28 : [schema] Add ACL_COUNTER_ATTR_ID_LIST (sonic-net#533) [Stepan Blyshchak] 53b84c7 - 2021-10-26 : Add default route table name to State DB (sonic-net#543) [Prince Sunny]
Including the below commits to update swss submodule 8448a60 [vs tests]Migrating sonic-swss tests to use hwsku instead of fakeplatform (sonic-net#1978) faa26db Fix random failure in PR/CI build. (sonic-net#2006) e03edb6 Allow interface type value none (sonic-net#1991) 71b9650 [orchagent] Fix group name of port-buffer-drop in flexcounterorch.cpp (sonic-net#1967) facdef5 [VS test] Skip flaky virtual chassis test (sonic-net#2004) 8261c1f [pytest]: Increase timeout when checking services (sonic-net#2000) 67278be [teammgrd]: Handle LAGs cleanup gracefully on Warm/Fast reboot. (sonic-net#1934) e92c1df Enable FEC statistics collection for Ethernet ports (sonic-net#1994) 9f30ca1 VxLAN Tunnel Counters and Rates implementation (sonic-net#1859) Signed-off-by: Sudharsan Dhamal Gopalarathnam <sudharsand@nvidia.com>
… used (sonic-net#9175) *Removing fake_platform environment variable. Following the merge of sonic-net#9044 and sonic-net/sonic-swss#1978 the fake_platform environment variable is not used in any place and removing the stale references. Signed-off-by: Sudharsan Dhamal Gopalarathnam <sudharsand@nvidia.com>
This PR allow user to set none value to interface type. So there is a way to achieve the goal via CLI: config interface type XXX none config interface speed XXX 10000 config interface type XXX CR
…-net#9179) The dhcp6relay rules file had a line overwriting a variable for docker-dhcp-relay. Remove that line. This line caused a limited impact where if some (many?) of the docker containers were already built, except for dhcp-relay, and the build failed or was interrupted, then dhcp-relay container would fail to build because this variable was overwritten and the python3-swsscommon wouldn't get installed into the slave container. Most builds would be fine, though. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
setup sonic-slave template and use the template for amd64, arm64 and armhf sonic-slave Signed-off-by: Guohan Lu <lguohan@gmail.com>
Update the repos for amd64, and add support for at least arm64. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: liora <liora@nvidia.com>
liorghub
pushed a commit
that referenced
this pull request
Jan 26, 2022
* [BFN] Updated platform APIs impl Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * Extended BFN platform SFP APIs implementation * Update sfp.py * [BFN] Extended SFP platform plugin implementation Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * [BFN] Extended Fans platform plugin implementation * [BFN] divided classes Fan and FanDrawer into 2 files * Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com> What I did Add get_model() function Add get_low_critical_threshold() function Change __get(...) function. How I did it Differnece from previous implementation of __get(...) function is return real value or -9999.9 if value is not provided by thrift API * Add get_presence() function and revised __get() function Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com> * [BFN] Updated PSU platform APIs impl Signed-off-by: Dmytro Lytvynenko <dmytrox.lytvynenko@intel.com> * Added BFN PSU cache (#9) Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * [BFN] Fans and Fantray platform APIs update (#7) * [BFN] Updated SFP platform APIs (#10) Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com> * [BFN] Updated platform API for thermal (#8) * Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com> * Revert "[BFN] Fans and Fantray platform APIs update (#7)" (#11) This reverts commit c62a733. * Add support health monitor system (#15) Signed-off-by: Petro Bratash <petrox.bratash@intel.com> * Update chassis.py * [BFN] Updated FANs and FAN Tray platform API (#14) * Fix fix_alignment (#17) Signed-off-by: Petro Bratash <petrox.bratash@intel.com> * [BFN] Improvement show environment (#16) * Added PSU temperature skip into platform.json (#18) Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * Do not skip psud on Newport Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * [BFN] fix fan status from Not OK to Ok (#19) * [BFN] Updated SFP platform plugin (#13) Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com> * [DPB] Fix typo for Ethernet0 2x200G[100G,40G] breakout mode (#21) Signed-off-by: Mykola Gerasymenko <mykolax.gerasymenko@intel.com> * [barefoot] Tmp fix vendor_rev (#22) Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com> * Fixed python issues in sonic_platform/fan_drawer.py Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * Updated fan_drawer.py * Fixing trailing white spaces in fan_drawer.py * [BFN] Fix thrift for SFPs API Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com> * In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * [Newport] Thermal manager (#23) * Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com> * Revert "In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue" This reverts commit 1e73127. * Removed 'controllable' options from platform.json to fix factory default config generation Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> * Update thermal_manager.py * Migrated SFP plugin to sonic_xcvr API (#30) Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com> Co-authored-by: KostiantynYarovyiBf <kostiantynx.yarovyi@intel.com> Co-authored-by: Vadym Yashchenko <vadymx.yashchenko@intel.com> Co-authored-by: Dmytro Lytvynenko <dmytrox.lytvynenko@intel.com> Co-authored-by: Volodymyr Boiko <volodymyrx.boiko@intel.com> Co-authored-by: Petro Bratash <petrox.bratash@intel.com> Co-authored-by: Mykola Gerasymenko <mykolax.gerasymenko@intel.com>
liorghub
pushed a commit
that referenced
this pull request
Nov 27, 2022
sonic-host-services submodule update with following changes 6eac2d3 Merge pull request #19 from judyjoseph/macsec_feature_enable 31c6108 Sync has_per_asic_scope attribute to config_db in all namespaces for multi-asic 185547f Add UT to improve coverage, for namespace config update 7c3aca0 macsec_supported info is part of DEVICE_RUNTIME_METADATA itself.
liorghub
pushed a commit
that referenced
this pull request
Jan 3, 2023
Added below commits: 9b30690 jcaiMR Fri Dec 16 fix handleSwssNotification crash in dhcp6relay (#28) 047afb7 jcaiMR Wed Dec 14 14:08:58 2022 +0800 Fix multiple vlan issue (#27) ff6bec3 Vivek Thu Dec 8 09:44:15 2022 -0800 Made the Error log informative (#22) 2fbe729 jcaiMR Wed Nov 30 14:41:53 2022 +0800 disable cfg dynamic change (#25) 13d0805 Liu Shilong Wed Nov 30 10:54:11 2022 +0800 Use github code scanning instead of LGTM (#26) 1e846f6 kellyyeh Wed Nov 23 14:36:02 2022 -0800 Fix packet range check for relay-reply packets (#21) 4d19e13 kellyyeh Thu Nov 17 10:04:53 2022 -0800 Add unittest infrastructure (#5) 7f4fdab jcaiMR Fri Nov 11 14:47:51 2022 +0800 fix packet range check issue (#20) 257ecdf kellyyeh Thu Nov 3 11:34:11 2022 -0700 Add client packet UDP header length check (#19)
liorghub
pushed a commit
that referenced
this pull request
Mar 15, 2023
advance dhcp relay for 202211 4bf1868 - (HEAD, origin/master, origin/HEAD, master) fix relay-reply dhcpv6 packet counter issue (add support for a7050 qx32 platform #29) (2 weeks ago) [jcaiMR] 9b30690 - fix handleSwssNotification crash in dhcp6relay (Add libnl-nf-3-200 to docker-team #28) (4 weeks ago) [jcaiMR] 047afb7 - Fix multiple vlan issue (Failure trying to run: chroot /sonic-buildimage/fsroot mount -t proc proc /proc #27) (4 weeks ago) [jcaiMR] ff6bec3 - Made the Error log informative (add python-tenjin as build dependency for p4-switch #22) (5 weeks ago) [Vivek] 2fbe729 - disable cfg dynamic change (p4: fix build dependency for python-p4c-bm #25) (6 weeks ago) [jcaiMR] 13d0805 - Use github code scanning instead of LGTM (Removed sx-libnl from Mellanox containers dependencies. #26) (6 weeks ago) [Liu Shilong] 1e846f6 - Fix packet range check for relay-reply packets (update sonic-swss and p4-switch submodule to fix docker sonic p4 bug #21) (7 weeks ago) [kellyyeh] 4d19e13 - Add unittest infrastructure (Cavium customization for docker containers #5) (8 weeks ago) [kellyyeh] 7f4fdab - fix packet range check issue (Makefile: add build dependency for python-p4c-bm #20) (9 weeks ago) [jcaiMR] 257ecdf - Add client packet UDP header length check (change port_config.ini directory for s6000 #19) (2 months ago) [kellyyeh]
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.
Why I did it
Add support for new platform SN2201.
How I did it
Add required content for new platform support.
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)