Skip to content
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

[202012][Dell] S6100 - Add iTCO watchdog support #16726

Merged

Conversation

ArunSaravananBalachandran
Copy link
Contributor

@ArunSaravananBalachandran ArunSaravananBalachandran commented Sep 27, 2023

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Update installer.conf for initializing the required drivers.
  • Update the reboot cause determination logic to check for iTCO watchdog reset if the BIOS version supports iTCO watchdog.

How to verify it

Verified that the watchdog APIs' return values are as expected.
UT logs: 202012_UT_logs.txt

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

[202012][Dell] S6100 - Add iTCO watchdog support

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@prgeor
Copy link
Contributor

prgeor commented Sep 27, 2023

@lguohan @qiluo-msft MSFT ADO# 25153914

@lguohan
Copy link
Collaborator

lguohan commented Sep 27, 2023

please have pr description to be more clear and precise.

@ArunSaravananBalachandran ArunSaravananBalachandran changed the title [202012][Dell] S6100 - iTCO watchdog support and reboot cause determination changes [202012][Dell] S6100 - Add iTCO watchdog support Sep 28, 2023
@lguohan lguohan merged commit 6f2e2f3 into sonic-net:202012 Sep 28, 2023
12 checks passed
@ArunSaravananBalachandran ArunSaravananBalachandran deleted the 202012_S6100_watchdog branch December 8, 2023 03:51
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